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): resolve aws sso profile issues by updating aws-sdk-go@v1.38 #377

Merged
merged 1 commit into from
May 4, 2023

Conversation

nitrocode
Copy link
Contributor

@nitrocode nitrocode commented May 2, 2023

what

  • Use latest github.com/aws/aws-sdk-go@v1.38

why

  • Resolve AWS_PROFILE with aws sso issues

references

commands

Before update

✗ AWS_PROFILE=test chamber list-services
Error: Failed to get secret store: SharedConfigErr: only one credential type may be specified per profile: source profile, credential source, credential process, web identity token, or sso
✗ go get -u github.com/aws/aws-sdk-go@v1.38

Replace was necessary in order to run go get -u

✗ go mod edit -replace gopkg.in/segmentio/analytics-go.v3=github.com/segmentio/analytics-go/v3@v3.2.1
✗ go mod tidy

After update

✗ rm -f chamber && make build
✗ AWS_PROFILE=test ./chamber list-services
Service
hello-world/secret

@nitrocode nitrocode requested a review from a team as a code owner May 2, 2023 23:13
@nitrocode nitrocode changed the title chore(deps): bump go.mod and go.sum packages chore(deps): resolve aws sso and profile issues by updating dependencies May 2, 2023
@nitrocode
Copy link
Contributor Author

Could you folks please review when you get a chance?

cc: @mckern @asaf-erlich @rikez

I think adding something like renovatebot would help out a lot with automating these upgrades in the future :)

@nitrocode nitrocode force-pushed the update-go-packages branch 2 times, most recently from 8edb860 to 2e656b6 Compare May 2, 2023 23:19
go.mod Outdated Show resolved Hide resolved
@asaf-erlich
Copy link
Contributor

Is renovatebot preferred over github's dependabot which I'm more used to? Just found this: https://javascript.plainenglish.io/automate-dependency-updates-by-renovate-not-by-dependabot-6efddd549a3e . Happy to help the team turn dependabot on for now as I've done it for a bunch of our repos.

@nitrocode
Copy link
Contributor Author

Is renovatebot preferred over github's dependabot which I'm more used to?

dependabot is certainly nice but it's not as configurable as renovatebot so I prefer the latter.

To add renovatebot, it would be adding the bot to the repo and then merging the default PR that it generates.
To add dependabot, it's just enabling the bot on the repo (as you know).

Perhaps dependabot for now would be easier to keep dependencies up to date?

@nitrocode
Copy link
Contributor Author

cc: @alecjacobs5401 please re-review when time permits

@alecjacobs5401
Copy link
Contributor

@nitrocode I approved the workflow runs and it looks like things aren't happy for go1.16

@nitrocode nitrocode changed the title chore(deps): resolve aws sso and profile issues by updating dependencies chore(deps): resolve aws sso and profile issues by updating aws-sdk-go@v1.38 May 4, 2023
@nitrocode nitrocode changed the title chore(deps): resolve aws sso and profile issues by updating aws-sdk-go@v1.38 chore(deps): resolve aws sso profile issues by updating aws-sdk-go@v1.38 May 4, 2023
@nitrocode
Copy link
Contributor Author

Ok, I tested on go 1.16 and only updated the aws-sdk-go and that seems to have fixed the issue.

If I update to the latest aws-sdk-go, then it requires at least go 1.17 due to x/sys@v0.1.0

I was unable to test on 1.15 because it does not have an arm binary.

Please see if the new changes build across the different versions.

@nitrocode
Copy link
Contributor Author

cc: @alecjacobs5401

Copy link
Contributor

@alecjacobs5401 alecjacobs5401 left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for contributing this fix @nitrocode!

@alecjacobs5401 alecjacobs5401 merged commit 71b9ce3 into segmentio:master May 4, 2023
@nitrocode nitrocode deleted the update-go-packages branch May 4, 2023 18:29
@nitrocode
Copy link
Contributor Author

Thanks for merging @alecjacobs5401 ! What's the next step to get a new release out?

I'd imagine it would be 2.12.1 since it's a bug fix.

@nitrocode
Copy link
Contributor Author

Oh I see this PR coming through #382 so looks like soon 🙏

@alecjacobs5401
Copy link
Contributor

@nitrocode I think we have a few things currently batched up so this might end up being a minor version bump. Should have a better answer/release for you by end of week

@nitrocode
Copy link
Contributor Author

nitrocode commented May 19, 2023

Thank you for releasing 2.13.0!!!

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.

Issues with chamber and aws sso
3 participants