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

fix: improve emulator cloud provider reliability #16

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

Unix4ever
Copy link
Member

@Unix4ever Unix4ever commented Aug 9, 2024

Copy the labels from machine request to the machine request
status.

Use a single client for rtnetlink watches and management.
Retry service account creation.

@Unix4ever Unix4ever force-pushed the cloud-provider-mode-fixes branch 2 times, most recently from edecbfe to 653dae5 Compare August 16, 2024 15:26
@Unix4ever Unix4ever changed the title fix: retry Omni client creation and secrets creation fix: improve emulator cloud provider reliability Aug 16, 2024
Copy the labels from machine request to the machine request
status.

Use a single client for rtnetlink watches and management.
Retry service account creation.

Signed-off-by: Artem Chernyshev <[email protected]>
@smira
Copy link
Member

smira commented Aug 17, 2024

I don't know how hard would it be, but it might be also cool to use a single controller instance for all emulated machines if possible... but I know we want to keep separate states, to it becomes tricky.

@Unix4ever
Copy link
Member Author

Unix4ever commented Aug 19, 2024

I don't know how hard would it be, but it might be also cool to use a single controller instance for all emulated machines if possible... but I know we want to keep separate states, to it becomes tricky.

Having a single controller for all machine level logic you mean?

@Unix4ever
Copy link
Member Author

/m

@talos-bot talos-bot merged commit 749f8e4 into siderolabs:main Aug 19, 2024
13 checks passed
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.

3 participants