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

New Debian container #33

Merged
merged 1 commit into from
Dec 12, 2023
Merged

Conversation

cfelipesp
Copy link
Contributor

Added debian folder to warewulf-node-images with three files in it:

  • Containerfile
  • excludes
  • container_exit.sh

It is a contribution to issue #867.

@anderbubble
Copy link
Collaborator

This is great, @cfelipesp!

Could you integrate this into https://github.com/hpcng/warewulf-node-images/blob/main/.github/workflows/container-publish.yml as well?

And can you sign-off on your commit to indicate your attestation to https://github.com/hpcng/warewulf-node-images/blob/main/DCO.md?

@anderbubble anderbubble self-requested a review November 27, 2023 18:19
@anderbubble anderbubble added the enhancement New feature or request label Nov 27, 2023
debian/Containerfile Show resolved Hide resolved
debian/Containerfile Outdated Show resolved Hide resolved
@cfelipesp
Copy link
Contributor Author

And can you sign-off on your commit to indicate your attestation to https://github.com/hpcng/warewulf-node-images/blob/main/DCO.md?

I'm sorry. I'm new to git and github. Only now I read I should have made all my commits using commit -s . Is there anything I can do now to sign-off on them? Or should I just make sure I sign-off the new commits from now on? I'm sorry...

@anderbubble
Copy link
Collaborator

@cfelipesp I apologize for my absence here.

You can fix sign past commits with git rebase --signoff. Check out git help rebase for more info, and let me know if you have further questions. Or you can tell me what name / email address to use here and I'll sign them off for you.

Meanwhile, I'll get testing this, and I look forward to getting them merged in.

@anderbubble
Copy link
Collaborator

Do be advised, I've rebased this onto main; so you'll need to pull changes back to your machine if you want to continue working from the current branch.

@anderbubble
Copy link
Collaborator

Glorious. It appears to be working.

[janderson@wwctl1 debian-node-image]$ sudo wwctl node set wwnode1 --container debian-12.0
Are you sure you want to modify 1 nodes(s): y
? Are you sure you want to modify 1 nodes(s)? [y/N] y█

[janderson@wwctl1 debian-node-image]$ sudo wwctl overlay build
Building system overlays for wwnode1: [wwinit]
WARN   : context(system) and overlays([wwinit]) specified: prioritizing context(system)
Created image for overlay wwnode1/[wwinit]: /var/lib/warewulf/provision/overlays/wwnode1/__SYSTEM__.img
Compressed image for overlay wwnode1/[wwinit]: /var/lib/warewulf/provision/overlays/wwnode1/__SYSTEM__.img.gz
Building runtime overlays for wwnode1: [generic]
WARN   : context(runtime) and overlays([generic]) specified: prioritizing context(runtime)
Created image for overlay wwnode1/[generic]: /var/lib/warewulf/provision/overlays/wwnode1/__RUNTIME__.img
Compressed image for overlay wwnode1/[generic]: /var/lib/warewulf/provision/overlays/wwnode1/__RUNTIME__.img.gz

[janderson@wwctl1 debian-node-image]$ sudo ssh root@wwnode1
Linux wwnode1 6.1.0-14-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.64-1 (2023-11-30) x86_64

The programs included with the Debian GNU/Linux system are free software;
the exact distribution terms for each program are described in the
individual files in /usr/share/doc/*/copyright.

Debian GNU/Linux comes with ABSOLUTELY NO WARRANTY, to the extent
permitted by applicable law.

root@wwnode1:~# cat /etc/os-release 
PRETTY_NAME="Debian GNU/Linux 12 (bookworm)"
NAME="Debian GNU/Linux"
VERSION_ID="12"
VERSION="12 (bookworm)"
VERSION_CODENAME=bookworm
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

@anderbubble anderbubble self-requested a review December 9, 2023 22:38
Copy link
Collaborator

@anderbubble anderbubble left a comment

Choose a reason for hiding this comment

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

@cfelipesp I squashed your commits together, and added you to the contributors file. I think this is ready to merge., except that I still need to get a signoff on it. Just let me know here for the record and I'll do it.

edit: looks like I spoke too soon. The container built for me locally, but isn't building in the GH action. I'll look into why.

@anderbubble
Copy link
Collaborator

anderbubble commented Dec 9, 2023

#11 16.05 E: Unable to locate package linux-image-amd64

Aha: I expect it's failing to build because it can't install the amd64 kernel in an aarch64 container. And well it shouldn't.

@anderbubble anderbubble force-pushed the debian-node-image branch 2 times, most recently from 358f9ee to 87242e5 Compare December 9, 2023 22:58
@anderbubble
Copy link
Collaborator

I updated the container files to install the correct kernel dependent on the intended architecture, and it's working now.

@anderbubble anderbubble changed the title Added debian folder to warewulf-node-images with Containerfile and more New Debian container Dec 10, 2023
@cfelipesp
Copy link
Contributor Author

@cfelipesp I apologize for my absence here.

You can fix sign past commits with git rebase --signoff. Check out git help rebase for more info, and let me know if you have further questions. Or you can tell me what name / email address to use here and I'll sign them off for you.

Meanwhile, I'll get testing this, and I look forward to getting them merged in.

Hi, @anderbubble. I hope this is still in time, my e-mail is [email protected] . Could you please sign them off for me, then? I'm afraid I could make a mess if I tried to do that myself. Thanks

Signed-off-by: C Felipe S Pinheiro <[email protected]>
Signed-off-by: Jonathon Anderson <[email protected]>
@anderbubble anderbubble merged commit 1e64182 into warewulf:main Dec 12, 2023
2 of 14 checks passed
@anderbubble
Copy link
Collaborator

@cfelipesp it's merged. Thanks for contributing!

@cfelipesp
Copy link
Contributor Author

@cfelipesp it's merged. Thanks for contributing!

My pleasure, @anderbubble !
Thank you for tutoring me in my first contribution to an open source project. And specially for being so patient with me. I have learned immensely.

@cfelipesp cfelipesp deleted the debian-node-image branch May 16, 2024 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants