-
Notifications
You must be signed in to change notification settings - Fork 2
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
Added OpenCHAMI middleware and updated go deps #47
Conversation
Looks like the unit tests are failing:
|
Yep, I'm going to change the import to use the OpenCHAMI jwtauth. |
Uh oh, we might have broken some etcd stuff:
|
Huh? Did that happen from updating the packages? Maybe we'll just have to rollback the version for etcd if we can. |
Yeah I think that was it. Rolling back etcd fixes it. diff --git a/go.mod b/go.mod
index efc6510..24a6993 100644
--- a/go.mod
+++ b/go.mod
@@ -37,7 +37,7 @@ require (
github.com/Cray-HPE/hms-securestorage v1.13.0 // indirect
github.com/aws/aws-sdk-go v1.55.5 // indirect
github.com/cenkalti/backoff/v3 v3.2.2 // indirect
- github.com/coreos/etcd v3.3.27+incompatible // indirect
+ github.com/coreos/etcd v3.3.13+incompatible // indirect
github.com/coreos/go-semver v0.3.1 // indirect
github.com/coreos/go-systemd v0.0.0-20191104093116-d3cd4ed1dbcf // indirect
github.com/coreos/pkg v0.0.0-20240122114842-bbd7aa9bf6fb // indirect |
(Make sure to |
BSS builds with the downgraded EtcD version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested working. LGTM!
Addresses #46 by adding the OpenCHAMI middleware.