Skip to content

snowflake disable generic tags#9854

Merged
hithwen merged 29 commits into
masterfrom
js/snowflake-disable-generic-tags
Sep 13, 2021
Merged

snowflake disable generic tags#9854
hithwen merged 29 commits into
masterfrom
js/snowflake-disable-generic-tags

Conversation

@hithwen

@hithwen hithwen commented Aug 5, 2021

Copy link
Copy Markdown
Contributor

depends on #10027
Includes disable generic tag feature enabled by default for new installations and starts emitting a non generic tag for all installations

@codecov

codecov Bot commented Aug 5, 2021

Copy link
Copy Markdown

Codecov Report

Merging #9854 (a5f1eac) into master (b7f9e13) will increase coverage by 0.04%.
The diff coverage is 86.66%.

Flag Coverage Δ
snowflake 94.01% <86.66%> (-0.11%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

djmitche
djmitche previously approved these changes Aug 20, 2021

@djmitche djmitche left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes to the base checks look good.

- template: instances/global
- name: disable_generic_tags
description: |
Generic tags such as cluster will be replaced by <integration_name>_cluster to avoid

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Generic tags such as cluster will be replaced by <integration_name>_cluster to avoid
Generic tags such as `cluster` will be replaced by <integration_name>_cluster to avoid

ruthnaebeck
ruthnaebeck previously approved these changes Sep 9, 2021

@ruthnaebeck ruthnaebeck left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 for docs

@hithwen

hithwen commented Sep 10, 2021

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants