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

oem: port digitalocean, equinix metal (packet) and openstack #1197

Merged
merged 5 commits into from
Oct 9, 2023

Conversation

tormath1
Copy link
Contributor

@tormath1 tormath1 commented Sep 27, 2023

@pothos
Copy link
Member

pothos commented Sep 27, 2023

Note: I dropped brightbox should we keep it? I don't see any documentation regarding this.

I think it's a good idea to clean up unused/untested OEM setups, best would be in a follow-up and not linked to this PR, or?

@github-actions
Copy link

github-actions bot commented Sep 27, 2023

KEYWORDS="amd64 arm64"
IUSE=""

OEM_NAME="Packet"
Copy link
Member

Choose a reason for hiding this comment

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

We can rename this variable, it's used in the shell prompt message but otherwise I think it's not relevant

Copy link
Member

@pothos pothos left a comment

Choose a reason for hiding this comment

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

The grub cfg is gone but it has to be moved to the new place

@tormath1
Copy link
Contributor Author

Note: I dropped brightbox should we keep it? I don't see any documentation regarding this.

I think it's a good idea to clean up unused/untested OEM setups, best would be in a follow-up and not linked to this PR, or?

Ok, anyway it was making the CI failing because it was trying to build the image for this vendor too. I cherry-picked the commit and pushed to another branch to do the removal in a follow-up PR.

@tormath1 tormath1 force-pushed the tormath1/oem-openstack branch from 55821de to 88f915d Compare September 28, 2023 13:51
@tormath1 tormath1 force-pushed the tormath1/oem-openstack branch 4 times, most recently from 8754e43 to e72942e Compare October 2, 2023 10:19
@tormath1 tormath1 force-pushed the tormath1/oem-openstack branch from e72942e to 149b0a8 Compare October 2, 2023 12:42
Comment on lines 2 to 18
ConditionKernelCommandLine=|flatcar.oem.id=ec2
ConditionKernelCommandLine=|flatcar.coreos.id=ec2

ConditionKernelCommandLine=|flatcar.oem.id=azure
ConditionKernelCommandLine=|flatcar.coreos.id=azure

ConditionKernelCommandLine=|flatcar.oem.id=digitalocean
ConditionKernelCommandLine=|flatcar.coreos.id=digitalocean

ConditionKernelCommandLine=|flatcar.oem.id=gce
ConditionKernelCommandLine=|flatcar.coreos.id=gce

ConditionKernelCommandLine=|flatcar.oem.id=openstack
ConditionKernelCommandLine=|flatcar.coreos.id=openstack

ConditionKernelCommandLine=|flatcar.oem.id=packet
ConditionKernelCommandLine=|flatcar.coreos.id=packet
Copy link
Member

Choose a reason for hiding this comment

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

This should be coreos.oem.id

@tormath1 tormath1 force-pushed the tormath1/oem-openstack branch from 149b0a8 to cb25663 Compare October 2, 2023 16:13
@tormath1 tormath1 marked this pull request as ready for review October 2, 2023 16:14
@tormath1 tormath1 requested a review from a team October 2, 2023 16:16
Copy link
Member

@pothos pothos left a comment

Choose a reason for hiding this comment

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

With OEM sysext images we can also use the migration mechanism to clean up the old OEM contents. (This will also make the sysext images required on update.)

@dongsupark dongsupark added the main label Oct 9, 2023
QEMU is not supported by Afterburn, let's add an extra-check to be sure
that the service will not run on this platform.

On cloud-init provisioned system, units are wrote in '/etc' after a
certain moment, if a unit takes precedence on one in '/usr' the one in
'/usr' would have certainly already started. That's the case for
'coreos-metadata.service' (wrote in '/etc' by cloud-init but already
available in '/usr').

```
$ journalctl -u coreos-metadata.service
Sep 29 07:37:57 localhost systemd[1]: Starting coreos-metadata.service - Flatcar Metadata Agent...
Sep 29 07:37:57 localhost coreos-metadata[1266]: Error: failed to run
Sep 29 07:37:57 localhost coreos-metadata[1266]: Caused by:
Sep 29 07:37:57 localhost coreos-metadata[1266]:     0: fetching metadata from provider
Sep 29 07:37:57 localhost coreos-metadata[1266]:     1: unknown provider 'qemu'
Sep 29 07:37:57 localhost systemd[1]: coreos-metadata.service: Main process exited, code=exited, status=1/FAILURE
Sep 29 07:37:57 localhost systemd[1]: coreos-metadata.service: Failed with result 'exit-code'.
Sep 29 07:37:57 localhost systemd[1]: Failed to start coreos-metadata.service - Flatcar Metadata Agent.
Sep 29 07:37:58 core1 systemd[1]: coreos-metadata.service: Scheduled restart job, restart counter is at 1.
Sep 29 07:37:58 core1 systemd[1]: Stopped coreos-metadata.service - QEMU metadata agent.
Sep 29 07:37:58 core1 systemd[1]: Starting coreos-metadata.service - QEMU metadata agent...
Sep 29 07:37:58 core1 systemd[1]: coreos-metadata.service: Deactivated successfully.
Sep 29 07:37:58 core1 systemd[1]: Finished coreos-metadata.service - QEMU metadata agent.
```

Signed-off-by: Mathieu Tortuyaux <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants