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: backport dependabot updates from master #6997

Merged
merged 2 commits into from
Jan 23, 2025

Conversation

jdstrand
Copy link
Contributor

Closes: https://github.com/influxdata/edge/issues/779.

This addresses two stray dependabot alerts:

@jdstrand jdstrand requested a review from a team as a code owner January 23, 2025 19:01
@jdstrand jdstrand requested a review from wdoconnell January 23, 2025 19:02
@jdstrand
Copy link
Contributor Author

jdstrand commented Jan 23, 2025

Per @wdoconnell, the monitor-ci-tests have been failing for a long time due to:

  1. Operator Page Tests.
  2. Operator Page (R/O) tests.
  3. UserAccounts test.

which aren't relevant to OSS. When the tests fail, they timeout and exit and the messages in https://app.circleci.com/pipelines/github/influxdata/ui are opaque. However, https://app.circleci.com/pipelines/github/influxdata/monitor-ci can be looked at instead and it lists everything that is needed. We want to see that oss-e2e is passing, eg
https://app.circleci.com/pipelines/github/influxdata/monitor-ci/21921/workflows/14d88717-3c15-45fd-815d-47b6282d1f2e/jobs/1491231)

Since all the OSS tests are passing, committing this. @wdoconnell said he'd create an issue for making release/OSS pass, but that work is not prioritized.

Thanks @wdoconnell for helping with this!

UPDATE: https://github.com/influxdata/granite/issues/3983 was created to track making this work better (sorry it is a non-public repo)

@jdstrand jdstrand merged commit 4c792a3 into release/OSS Jan 23, 2025
5 of 6 checks passed
@jdstrand jdstrand deleted the jdstrand/update-oss-for-depbot-pt2 branch January 23, 2025 22:09
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