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

feat: import orabos #8

Merged
merged 112 commits into from
Nov 22, 2024
Merged

feat: import orabos #8

merged 112 commits into from
Nov 22, 2024

Conversation

toanju
Copy link
Contributor

@toanju toanju commented Sep 27, 2024

  • Import orabos features from https://github.com/cobaltcore-dev/gardenlinux to build in the gardenlinux org.
  • Adjust build workflow to only build images for openstackbaremetal-metal3-orabos, openstackbaremetal-orabos and vmware-orabos on amd64
  • Currently GardenLinux main is being used

Skipped patches:

@toanju toanju force-pushed the import-orabos branch 15 times, most recently from 0ec4306 to 60d429d Compare October 8, 2024 07:56
toanju and others added 15 commits November 13, 2024 20:31
* add submodules for checkout workflow
* fixup VERSION file
* only build orabos related targets
* link missing features
* link test related files and folders
- Include mlx5 drivers in initramdisk and load it, otherwise we have
  problems with the setup of the bond (NIC isn't known yet)

- Set the DNS on the bond, as cloud-init sets here things up statically,
  and therefore no DHCP info about the DNS
This file overrides the kublet config specifically put
there for gardener, and fails with it being there.
So we rather remove it.
Simply adding a second root.conf instructs systemd-repart to
create another root parition of equal size.
It isn't part of the gardenlinux repo (yet),
so get it in that way for now.

It's needed for the neutron OVS and the OVN driver.

Also add the loci-users for now. This probably
should be replaced with k8s means to map the users.
The gardenlinux kernel crashes currently
with the mlx5 driver.
The neutron agent will be member of the group openvswitch,
but not the openvswitch user itself.
This ensures, that they have the right permissions
The kernel can automatically detect the root based
on the GUID value. This marks the second partition as non-auto,
to disambiguate between the two partitions.
(Aside from the fact that the actual root has the label ROOT).
This is a bit of a sledge-hammer method, but for now disable
the watchdog, because it kicks in when doing a soft-reboot
or a system kexec call.
Both cut down the reboot time tremendously.
The proper way would be to find out why the watchdog is not
fed in those situations.
fwiesel and others added 11 commits November 13, 2024 20:32
This reverts commit 7e68d836a51858462ebcb22b008f5a32617f9b2a.
Netplan doesn't seem to do that anymore, and we need it
in ignition based setups anyway.
Needed for cold-migration.
The openvswitch bridge will assume the same macaddress automatically,
but if we do not set it on the bond, it will be a random one.
That's on our systems much more than the 32GiB before
and matches what we reserve in the flavours.
Use ignition package built in cobaltcore-dev/package-ignition
Switch gardenlinux to main branch
The build workflow so far used arm64 and amd64 architectures. These are
now configurable via the arch input. The default is still arm64 and
amd64. However, for the dev workflow only amd64 arch will be used.
@toanju toanju marked this pull request as ready for review November 14, 2024 13:55
@@ -0,0 +1,21 @@
#!/usr/bin/env bash
Copy link
Contributor

Choose a reason for hiding this comment

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

should use glrm (gardenlinux repomanager)

@@ -0,0 +1 @@
omit_drivers+=" iTCO_wdt "
Copy link
Contributor

Choose a reason for hiding this comment

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

what is the issue with the watchdog on your side? does it bevae wrong?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

watchdog seems to cause issues and did sometimes reset during boot

@@ -0,0 +1 @@
CMDLINE_LINUX="console=tty0 $CMDLINE_LINUX"
Copy link
Contributor

Choose a reason for hiding this comment

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

really no serial output ? no remote visibility of errors?

@@ -0,0 +1,7 @@
description: "ORA Baremetal OS"
Copy link
Contributor

Choose a reason for hiding this comment

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

this combination is difficult. the gardener feature has some bias here that could be avoided by the chost feature.
the other is the combination vhost + chost is that really intended?

@gehoern gehoern merged commit 1356aa9 into gardenlinux:main Nov 22, 2024
9 checks passed
@pnpavlov pnpavlov added this to the 2024-12 milestone Nov 22, 2024
@toanju toanju deleted the import-orabos branch January 10, 2025 08:18
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.

7 participants