-
Notifications
You must be signed in to change notification settings - Fork 138
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
Switch to new machine type in semaphore CI #3462
Conversation
.semaphore/push_images.yml
Outdated
@@ -2,7 +2,7 @@ version: v1.0 | |||
name: Operator CD | |||
agent: | |||
machine: | |||
type: e1-standard-4 | |||
type: f1-standard-4 |
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.
I think this can actually be f1-standard-2, I think these machines are bigger. The 4 is apparently much larger, might want to check with @sujeet-kr though.
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.
Yes e1-standard-4
's can be replaced with f1-standard-2
s, if the reason for using e1-standard-4
was memory and storage.
type: f1-standard-4 | |
type: f1-standard-2 |
.semaphore/semaphore.yml
Outdated
@@ -137,7 +137,7 @@ blocks: | |||
task: | |||
agent: | |||
machine: | |||
type: e1-standard-4 | |||
type: f1-standard-4 |
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.
type: f1-standard-4 | |
type: f1-standard-2 |
9fa47b5
to
f1c89ca
Compare
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.
LGTM
Description
For PR author
make gen-files
make gen-versions
For PR reviewers
A note for code reviewers - all pull requests must have the following:
kind/bug
if this is a bugfix.kind/enhancement
if this is a a new feature.enterprise
if this PR applies to Calico Enterprise only.