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

Firstrun 2 #3774

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Firstrun 2 #3774

wants to merge 5 commits into from

Conversation

ManoftheSea
Copy link
Contributor

@ManoftheSea ManoftheSea commented May 9, 2022

Description

Cleanup and order firstrun scripts to use systemd firstboot logic.
Cleanup image according to systemd so multiple flashes don't share machine-id or random seed
Cleanup espressobin-related logic enabling networkd and resolved services.

Replaces #3642 and #3769

Jira reference number AR-1174

How Has This Been Tested?

  • Flash the new image and see if it completes firstrun scripts and populates /etc/machine-id

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@ManoftheSea ManoftheSea added Ready to merge Reviewed, tested and ready for merge and removed Ready to merge Reviewed, tested and ready for merge labels May 9, 2022
@ManoftheSea
Copy link
Contributor Author

Still an issue with dpkg-reconfigure openssl... and the failure of the armbian-firstboot script does NOT stop the system from being considered to have booted (and writing to machine-id)

@igorpecovnik igorpecovnik added the Ready to merge Reviewed, tested and ready for merge label May 10, 2022
@igorpecovnik
Copy link
Member

It timeouts on first run and it doesn't auto login ...

@igorpecovnik igorpecovnik marked this pull request as draft May 11, 2022 17:41
@ManoftheSea ManoftheSea removed the Ready to merge Reviewed, tested and ready for merge label May 23, 2022
@igorpecovnik
Copy link
Member

Can we squeeze this in for 22.08? But has to be well tested.

@hzyitc hzyitc mentioned this pull request Aug 14, 2022
…em.service

Co-authored-by: hzyitc <h1260071766@126.com>
Co-authored-by: hzyitc <h1260071766@126.com>
hzyitc added a commit to hzyitc/armbian-onecloud that referenced this pull request Oct 17, 2022
@igorpecovnik igorpecovnik added Work in progress Unfinished / work in progress 23.05 and removed 23.05 labels May 19, 2023
@igorpecovnik igorpecovnik added Can be closed? Ping developers on stalled issue / PR and removed 23.08 Work in progress Unfinished / work in progress labels Nov 15, 2023
@github-actions github-actions bot removed the Software label Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Can be closed? Ping developers on stalled issue / PR
Development

Successfully merging this pull request may close these issues.

3 participants