Skip to content

Use ec2.assume_role #1336

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 21, 2024
Merged

Use ec2.assume_role #1336

merged 2 commits into from
Mar 21, 2024

Conversation

katcharov
Copy link
Collaborator

DEVPROD-5197

@katcharov katcharov requested a review from cbiphuk March 12, 2024 22:48
cbiphuk
cbiphuk suggested changes Mar 13, 2024
View reviewed changes
@katcharov katcharov requested review from cbiphuk and rozza March 13, 2024 19:12
jyemin
jyemin reviewed Mar 14, 2024
View reviewed changes
@rozza rozza removed their request for review March 18, 2024 13:39
@rozza
Copy link
Member

rozza commented Mar 18, 2024

I removed myself from the review as there are two other active reviewers.

@katcharov katcharov requested a review from jyemin March 18, 2024 14:12
@katcharov
Copy link
Collaborator Author

Merging. The role is now working. There is an ensuing failure, which is not new and goes back to at least Jan.

@katcharov katcharov merged commit 5180661 into master Mar 21, 2024
@katcharov katcharov deleted the DEVPROD-5197 branch March 21, 2024 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jyemin jyemin jyemin approved these changes

@cbiphuk cbiphuk Awaiting requested review from cbiphuk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@katcharov @rozza @jyemin @cbiphuk