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.

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

[OR-1789] remove ec2 runner feature #206

Merged
merged 1 commit into from
Aug 7, 2024
Merged

[OR-1789] remove ec2 runner feature #206

merged 1 commit into from
Aug 7, 2024

Conversation

steven94kr
Copy link
Contributor

@steven94kr steven94kr force-pushed the OR-1789--thanos-remove-ec2-runner-feature branch from f2401af to 2af703f Compare August 7, 2024 07:10
@steven94kr steven94kr force-pushed the OR-1789--thanos-remove-ec2-runner-feature branch from 2af703f to 7ce12b3 Compare August 7, 2024 07:10
@0x6e616d
Copy link
Member

0x6e616d commented Aug 7, 2024

Hi Steven, Do you don't need arm-64 images?

@steven94kr
Copy link
Contributor Author

Hi Steven, Do you don't need arm-64 images?

Yes, I think. Do you need arm-64 images?

@0x6e616d
Copy link
Member

0x6e616d commented Aug 7, 2024

Hi Steven, Do you don't need arm-64 images?

Yes, I think. Do you need arm-64 images?

No, just a question

@steven94kr
Copy link
Contributor Author

steven94kr commented Aug 7, 2024 •
edited
Loading

Hi Steven, Do you don't need arm-64 images?

Yes, I think. Do you need arm-64 images?

No, just a question

Thank you very much.
Due to incident about AWS billing, we have to remove ARM architecture made by AWS EC2.
If you need some ARM Image, I think you can create it in local or, use the previous image in the docker hub.
I'm sorry again about the incident... 🙇🏼‍♂️

0x6e616d
0x6e616d approved these changes Aug 7, 2024
View reviewed changes
Copy link
Member

@0x6e616d 0x6e616d left a comment

Choose a reason for hiding this comment

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

+1

@steven94kr steven94kr merged commit 8cae7ee into main Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0x6e616d 0x6e616d 0x6e616d approved these changes

@theo-learner theo-learner Awaiting requested review from theo-learner

@ohbyeongmin ohbyeongmin Awaiting requested review from ohbyeongmin

@AaronLee22 AaronLee22 Awaiting requested review from AaronLee22

@xxeonge xxeonge Awaiting requested review from xxeonge

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

Successfully merging this pull request may close these issues.
2 participants
@steven94kr @0x6e616d