Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

cleanup: Add ContainerFileNotices for Ubuntu & MachineConfig #101

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

ivatet-amd
Copy link
Collaborator

@ivatet-amd ivatet-amd commented Dec 8, 2023

There are two new Dockefiles: one to generate SFC MachineConfig, and another is an Onload kernel module builder for the Ubuntu kernels. This patch adds the corresponding ContainerFileNotice even though the latter image is a template.


Re on Dockerfiles:

  • I chose the specific version of Butane (v0.19.0), which must be the latest release tag.
  • Ubuntu is borderline, as we don't build these images, but added for completeness.

@ivatet-amd ivatet-amd requested a review from a team as a code owner December 8, 2023 15:27
@pcolledg-amd
Copy link
Collaborator

"Missing" in the commit message could be more specific by naming the two.

@ivatet-amd ivatet-amd force-pushed the reviews/iteterev/cleanup-container-file-notice branch from 22c034a to 671168c Compare December 12, 2023 16:31
@ivatet-amd ivatet-amd changed the title cleanup: Add missing ContainerFileNotice cleanup: Add two missing ContainerFileNotices Dec 12, 2023
@ivatet-amd ivatet-amd requested review from pcolledg-amd and a team December 12, 2023 16:33
@ivatet-amd
Copy link
Collaborator Author

@pcolledg-amd Thanks, I've expanded the commit message. 👍

@pcolledg-amd
Copy link
Collaborator

Apologies, I didn't mean literally quantity of two, but naming the two, ie. Add ContainerFileNotices for ubuntu & machineconfig

@ivatet-amd
Copy link
Collaborator Author

I've listed them in the body of the commit message, but your version works for me too.

There are two new Dockefiles: one to generate SFC MachineConfig, and
another is an Onload kernel module builder for the Ubuntu kernels. This
patch adds the corresponding ContainerFileNotice even though the latter
image is a template.
@ivatet-amd ivatet-amd force-pushed the reviews/iteterev/cleanup-container-file-notice branch from 671168c to 3cb5dd2 Compare December 12, 2023 16:38
@ivatet-amd ivatet-amd changed the title cleanup: Add two missing ContainerFileNotices cleanup: Add ContainerFileNotices for Ubuntu & MachineConfig Dec 12, 2023
@ivatet-amd ivatet-amd merged commit 609101f into master Dec 13, 2023
5 checks passed
@ivatet-amd ivatet-amd deleted the reviews/iteterev/cleanup-container-file-notice branch December 13, 2023 08:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants