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

Add flavor for CAPI. #203

Merged
merged 9 commits into from
Dec 12, 2024
Merged

Add flavor for CAPI. #203

merged 9 commits into from
Dec 12, 2024

Conversation

Gerrit91
Copy link
Contributor

Description

This is for our work on cluster-api-provider-metal-stack.

We need one more machine there to serve as a worker node and also we need to increase disk size and CPU cores in order to accommodate the Kubernetes components and pass the kubeadm pre-flight checks.

@Honigeintopf
Copy link

Is this working already?

@Gerrit91
Copy link
Contributor Author

I had all machines in Waiting state, but did not yet try to allocate all of them. But probably it works, yes.

@Honigeintopf
Copy link

Third Machine is stuck in PXE booting somehow. You only run make, with env set to capms right?

@Honigeintopf
Copy link

Ethernet8 not up.

root@leaf01:~# show interface status
  Interface        Lanes       Speed    MTU    FEC         Alias    Vlan    Oper    Admin    Type    Asym PFC
-----------  -----------  ----------  -----  -----  ------------  ------  ------  -------  ------  ----------
  Ethernet0  25,26,27,28  4294967.3G   9000    N/A  fortyGigE0/0   trunk      up       up     N/A         N/A
  Ethernet4  29,30,31,32  4294967.3G   9000    N/A  fortyGigE0/4   trunk      up       up     N/A         N/A
Ethernet120  33,34,35,36  4294967.3G   9216    N/A  fortyGigE0/8  routed      up       up     N/A         N/A

@Gerrit91
Copy link
Contributor Author

You must first build the SONiC image and use this when starting up the containerlab in order to include the config_db.json into the switch.

@Gerrit91 Gerrit91 marked this pull request as ready for review December 12, 2024 11:04
@Gerrit91 Gerrit91 requested a review from a team as a code owner December 12, 2024 11:04
Makefile Outdated
Comment on lines 188 to 189
password-machine01:
@$(MAKE) --no-print-directory _password MACHINE_UUID=e0ab02d2-27cd-5a5e-8efc-080ba80cf258
@$(MAKE) --no-print-directory _free-machine MACHINE_NAME=machine01 MACHINE_UUID=00000000-0000-0000-0000-000000000001
Copy link
Contributor

Choose a reason for hiding this comment

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

Why _password instead of _free-machine?

Makefile Show resolved Hide resolved
@Gerrit91 Gerrit91 requested a review from vknabel December 12, 2024 15:20
Copy link
Contributor

@vknabel vknabel left a comment

Choose a reason for hiding this comment

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

image

@Gerrit91 Gerrit91 merged commit f53e2ea into master Dec 12, 2024
2 checks passed
@Gerrit91 Gerrit91 deleted the capms-flavor branch December 12, 2024 15:22
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.

4 participants