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 packages and update docs #324

Merged
merged 7 commits into from
Jun 29, 2021
Merged

Add packages and update docs #324

merged 7 commits into from
Jun 29, 2021

Conversation

mudler
Copy link
Contributor

@mudler mudler commented Jun 25, 2021

This PR is multi-fold:

PR is draft as I need to test the examples provided first

@mudler mudler force-pushed the fixup_packages_add branch 2 times, most recently from 31e8d7c to 032bbbb Compare June 25, 2021 08:53
@mudler mudler self-assigned this Jun 25, 2021
@mudler mudler changed the title Add packages required from CI, update docs Add packages and update docs Jun 25, 2021
@mudler mudler force-pushed the fixup_packages_add branch 7 times, most recently from 31141fe to 32d4c84 Compare June 28, 2021 12:09
@mudler
Copy link
Contributor Author

mudler commented Jun 28, 2021

I can't test the "from scratch" example above without a published change of cOS that is in this PR, I will amend in case it's not working in a follow-up. I need another round to consume gomplate in any case.

@mudler mudler marked this pull request as ready for review June 28, 2021 12:10
ENV LUET_NOLOCK=true
SHELL ["/usr/bin/luet", "install", "-y", "-d"]

RUN system/cos
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the one I couldn't test till the end. It fails during install because /etc/hostname is already populated by docker

@@ -108,3 +108,9 @@ excludes:
- ^/usr/local/share
- ^/usr/local/src
- ^/usr/local/games

- ^/etc/hostname
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've added those to excludes. Hostname is generated by docker, so not really needed, similarly for the rest.

@mudler mudler force-pushed the fixup_packages_add branch 2 times, most recently from 9615bf9 to 90a2e4f Compare June 28, 2021 14:39
@mudler mudler removed their assignment Jun 28, 2021
@mudler mudler enabled auto-merge (squash) June 28, 2021 16:51
We are using it for generating CI pipelines, handy to have directly in
our repository

Signed-off-by: Ettore Di Giacinto <[email protected]>
This will allow us to free any dependency on the luet-lab/luet-repo
repository

Signed-off-by: Ettore Di Giacinto <[email protected]>
Fixes #274
Fixes #275

Signed-off-by: Ettore Di Giacinto <[email protected]>
Signed-off-by: Ettore Di Giacinto <[email protected]>
Signed-off-by: Ettore Di Giacinto <[email protected]>

4. Followed the procedure described in [AWS docs](https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/creating-an-ami-ebs.html#creating-launching-ami-from-snapshot) to register an AMI from snapshot. Used all default settings unless for the firmware, were I forced it to UEFI boot.

5. Launch instance with this simple userdata:
Copy link
Contributor

Choose a reason for hiding this comment

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

is this really needed? I booted the instance with a different user data, not creating users and root/cos was available.

Also, what is really needed if you want to launch the raw image is to provide a layout file (like in the example user-data) so its possible to use the new cos-deploy to install the system.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good point! I'll update the docs 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done!

Signed-off-by: Ettore Di Giacinto <[email protected]>
@mudler mudler merged commit 39a1419 into master Jun 29, 2021
@mudler mudler deleted the fixup_packages_add branch June 29, 2021 09:24
frelon pushed a commit to frelon/elemental-toolkit that referenced this pull request May 12, 2023
* Keep working dir after a chroot call

Signed-off-by: David Cassany <[email protected]>
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.

2 participants