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

Combines the example preseed/late_command and updates example iso url to stable 11.7 endpoint #141

Merged

Conversation

jburns24
Copy link
Contributor

This PR addressed a number of issues that I ran into when testing the provided example. The first issue is the iso url in the example is no longer valid. I updated it to 11.7 the latest minor version of bullseye that has a stable endpoint. The next issue addressed in this pr is in the supplied preseed file. I noticed in testing that the vagrant user could not do passwordless sudo. Upon further inspection I could see in /var/log/installer/syslog that only the last sed late_command was ran. I did some digging and found a number of random SO posts/comments that claimed you can only have a single preseed/late_command. I attempted to find some official Debian documentation that supported that claim and this.

This was tested on an M1 mac running VMWare Fusion 13. I verified that with this change passwordless sudo was enabled for the vagrant user and the cdrom source was commented out of the source.list file.

I had a coworker test this change on an intel Mac and they said it 'worked' though I was not there to validate what 'worked' meant to them.

Updates example iso url to stable 11.7 endpoint
@jburns24 jburns24 requested a review from a team as a code owner June 21, 2023 18:23
@hashicorp-cla
Copy link

hashicorp-cla commented Jun 21, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@nywilken nywilken left a comment

Choose a reason for hiding this comment

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

Thanks for this fix. I confirmed on ARM and Intel. 🚢

@nywilken nywilken added the docs label Jun 22, 2023
@nywilken nywilken merged commit 6e11bec into hashicorp:main Jun 22, 2023
11 checks passed
@jburns24 jburns24 deleted the bug/consolodate-preseed-late-commands branch June 22, 2023 16:57
@hashicorp hashicorp locked and limited conversation to collaborators Jun 29, 2024
@tenthirtyam tenthirtyam added documentation Improvements or additions to documentation and removed docs labels Jul 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants