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

community-platforms: add scaleway #326

Merged
merged 1 commit into from
May 30, 2024
Merged

community-platforms: add scaleway #326

merged 1 commit into from
May 30, 2024

Conversation

tormath1
Copy link
Collaborator

@tormath1 tormath1 commented Apr 19, 2024

Scaleway documentation: from the upload to the deployment.

TODO:


⚠️ Wait for next release before merging this as we need to provide qcow2 images to be in sync with the documentation

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-326.westeurope.1.azurestaticapps.net

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-326.westeurope.1.azurestaticapps.net

@tormath1 tormath1 changed the title [wip] community-platforms: add scaleway community-platforms: add scaleway Apr 23, 2024
@tormath1 tormath1 marked this pull request as ready for review April 23, 2024 14:07
@tormath1 tormath1 requested a review from a team April 23, 2024 14:08
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-326.westeurope.1.azurestaticapps.net

Comment on lines 21 to 24
$ bzip2 --decompress ./flatcar_production_scaleway_image.img.bz2
$ mv flatcar_production_scaleway_image.{img,qcow2}
$ aws s3api create-bucket --bucket flatcar
$ aws s3 cp flatcar_production_scaleway_image.qcow2 s3://flatcar/flatcar_production_scaleway_image.qcow2
Copy link
Member

Choose a reason for hiding this comment

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

Is this renaming to qcow2 necessary? If so, maybe we should think about creating scaleway images with qcow2.bz2 extension instead of img.bz2?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, that's necessary. That's actually a good idea to rename images, I did not check if we can do this.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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


## Launch machine

Boot the machine with the CLI, referencing the snapshot ID from the import step above and your [Ignition file from Butane][butane-configs]:
Copy link
Member

Choose a reason for hiding this comment

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

Maybe mention that the command will print the ID of the instance, which will be used in follow-up commands? Or add an example output of the scw instance server create?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for the suggestion. I modified the commands to use JSON output and get the ID(snapshot, instance) in a variable. One can just copy paste the commands without editing. :)

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-326.westeurope.1.azurestaticapps.net

1 similar comment
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-326.westeurope.1.azurestaticapps.net

krnowak
krnowak previously approved these changes Apr 24, 2024
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-326.westeurope.1.azurestaticapps.net

@tormath1
Copy link
Collaborator Author

Freezing the PR until the next release to provide qcow2 images for Scaleway.

@andypost
Copy link

is this the only thing to fix to declare support?

@tormath1
Copy link
Collaborator Author

is this the only thing to fix to declare support?

@andypost you can already try Flatcar on Scaleway, the image is available on Alpha / Beta: https://beta.release.flatcar-linux.net/amd64-usr/current/flatcar_production_scaleway_image.img.bz2

This PR is just about adding some user documentation (which was one of the final steps: flatcar/Flatcar#1368)

@tormath1 tormath1 requested a review from krnowak May 30, 2024 07:20
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
@tormath1
Copy link
Collaborator Author

@krnowak nothing has changed for the review, I just rebased and asked for a new review since approval is required to merge in this repository.

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://lemon-wave-085522403-326.westeurope.1.azurestaticapps.net

@tormath1 tormath1 merged commit ee8bb82 into master May 30, 2024
2 checks passed
@tormath1 tormath1 deleted the tormath1/scaleway branch May 30, 2024 10:05
krishjainx pushed a commit to krishjainx/flatcar-website that referenced this pull request Jul 5, 2024
Our butane examples for customising locksmith contain an error:
"overwrite: true" is missing when modifying "update.conf".
This leads to ignition panicking during provisioning as "update.conf"
already exists:

ignition[1009]: CRITICAL : Ignition failed: failed to create files: failed to create files: error creating /sysroot/etc/flatcar/update.conf: error creating file "/sysroot/etc/flatcar/update.conf": A file exists there already and overwrite is false

This change updates our examples and adds "overwrite:true".

Signed-off-by: Thilo Fromm <thilofromm@microsoft.com>
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.

None yet

3 participants