-
Notifications
You must be signed in to change notification settings - Fork 0
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: update to upstream v0.42.0 release #4
Conversation
0d22271
to
4eefe7d
Compare
Tests are failing due to what looks like a leaked goroutine. I've been unable to recreate the goroutine leak outside of GitHub actions. Marking the tests as skipped here for the time being, as they're tested successfully in the upstream extension. |
67814db
to
3beca85
Compare
Remaining checks are failing as they are attempting to read secrets from the upstream repo. They are all part of the |
9aa12e4
to
6acf93e
Compare
6acf93e
to
df4c9c5
Compare
2d3cbca
to
f197c07
Compare
The implicit expectation of the ConfigTest is that the underlying credentials provider will come from the mockCredentials struct. This change constructs the testdata config in such a way as to make this happen. Needs refactoring to remove this implicit contract.
f197c07
to
23ed32b
Compare
Description:
Updates main branch to v0.42.0 upstream release
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.