Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): bump influxdata/influxdb-templates from 0.11.0 to 0.12.0 #6960

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

bednar
Copy link
Contributor

@bednar bednar commented Oct 10, 2024

Related to https://github.com/influxdata/edge/issues/751

This PR updates influxdb-templates to the latest version, which includes security updates for the rollup dependency.

Checklist

Authors and Reviewer(s), please verify the following:

@bednar bednar marked this pull request as ready for review October 11, 2024 09:16
@bednar bednar requested review from a team as code owners October 11, 2024 09:16
@bednar
Copy link
Contributor Author

bednar commented Oct 11, 2024

I’ve successfully tested the new version of the influxdb-templates by importing the ‘InfluxDB Open Source (OSS) Metrics Template’ during the onboarding setup of InfluxDB.

Is there a way to test other templates listed at this component? Unfortunately, I wasn’t able to locate them in the UI.

Copy link
Contributor

@wdoconnell wdoconnell left a comment

Choose a reason for hiding this comment

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

Reviewed the related Edge ticket - thank you for linking it.

@wdoconnell
Copy link
Contributor

I’ve successfully tested the new version of the influxdb-templates by importing the ‘InfluxDB Open Source (OSS) Metrics Template’ during the onboarding setup of InfluxDB.

Is there a way to test other templates listed at this component? Unfortunately, I wasn’t able to locate them in the UI.

Not so familiar with that, but I'm happy to review it when back in-office on Tuesday.

@wdoconnell
Copy link
Contributor

@bednar Won't block this based on the Edge discussion, but can we wait until Tuesday to merge?

@bednar
Copy link
Contributor Author

bednar commented Oct 14, 2024

@wdoconnell, of course, it can wait until Tuesday. Could you merge this PR when it’s convenient for you? After that, I will proceed with PRs for InfluxDB to finish https://github.com/influxdata/edge/issues/751

@bednar
Copy link
Contributor Author

bednar commented Oct 16, 2024

@wdoconnell, is it okay to merge this PR now?

@wdoconnell
Copy link
Contributor

@bednar Yes, please feel free to merge.

@bednar bednar added this pull request to the merge queue Oct 17, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 17, 2024
@wdoconnell wdoconnell added this pull request to the merge queue Oct 17, 2024
@wdoconnell
Copy link
Contributor

Remerging, as it flaked.

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 17, 2024
@jdstrand jdstrand added this pull request to the merge queue Oct 17, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 17, 2024
@wdoconnell wdoconnell added this pull request to the merge queue Oct 17, 2024
Merged via the queue into master with commit ff83a1e Oct 17, 2024
6 checks passed
@wdoconnell wdoconnell deleted the bednar/influxdb-templates branch October 17, 2024 19:09
bednar added a commit that referenced this pull request Oct 18, 2024
bednar added a commit that referenced this pull request Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants