Skip to content
This repository has been archived by the owner on Mar 18, 2024. It is now read-only.

blueprint-reference: add Ignition config section #96

Merged
merged 1 commit into from
May 11, 2023

Conversation

7flying
Copy link
Member

@7flying 7flying commented Nov 29, 2022

Add info about the new ignition customization.

Please wait until osbuild/osbuild-composer#3161 gets merged to merge this (if approved oc).

@7flying 7flying force-pushed the ignition-blueprint branch 2 times, most recently from 605d566 to 4fa45c5 Compare December 2, 2022 15:02
@7flying
Copy link
Member Author

7flying commented Dec 19, 2022

Looking for a review on this, the PR that introduced the changes described here has been merged.

@7flying 7flying force-pushed the ignition-blueprint branch from 4fa45c5 to 2d70660 Compare January 12, 2023 08:46
@7flying
Copy link
Member Author

7flying commented Jan 12, 2023

Updated after some updates made on osbuild/osbuild-composer#3130

@7flying 7flying force-pushed the ignition-blueprint branch 3 times, most recently from ed4542b to 1ae3a2a Compare February 9, 2023 12:13
@7flying
Copy link
Member Author

7flying commented Feb 9, 2023

@runcom @elianerpereira changes made, let me know if it makes sense

runcom
runcom previously approved these changes Feb 9, 2023
@7flying 7flying force-pushed the ignition-blueprint branch from 1ae3a2a to 54497a7 Compare February 9, 2023 16:28
@7flying
Copy link
Member Author

7flying commented Feb 9, 2023

Updated after @miabbott comments.

@7flying
Copy link
Member Author

7flying commented Mar 8, 2023

rebased on main, can I bother you @thozza for a review? This PR has approved but then forgotten and not merged 😢

Copy link
Member

@thozza thozza 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 the PR. I added a few comments. In addition to that, please do not break lines in the same paragraph and let the MD interpreter which generates the Guides Web page deal with that. IOW, a paragraph should be a single long line.

@7flying 7flying force-pushed the ignition-blueprint branch from 196c108 to 555e342 Compare March 9, 2023 11:02
@7flying
Copy link
Member Author

7flying commented Mar 9, 2023

Replaced ISO with image in a couple of places, added full base64 working example, changed paragraphs to be single-line paragraphs, expanded the r4e acronym

@7flying 7flying requested a review from thozza March 9, 2023 11:07
Copy link
Member

@thozza thozza left a comment

Choose a reason for hiding this comment

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

Sorry, but I missed one thing in my previous review. Otherwise this looks OK from my PoV

@7flying 7flying force-pushed the ignition-blueprint branch from 555e342 to 128508c Compare March 9, 2023 12:01
thozza
thozza previously approved these changes Mar 9, 2023
Copy link
Member

@thozza thozza left a comment

Choose a reason for hiding this comment

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

👍

@7flying 7flying force-pushed the ignition-blueprint branch from fae84bc to ba8481d Compare May 8, 2023 09:13
@supakeen supakeen merged commit 4ce28ae into osbuild:main May 11, 2023
@7flying 7flying deleted the ignition-blueprint branch May 11, 2023 07:15
@7flying
Copy link
Member Author

7flying commented May 11, 2023

@supakeen thank you!!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants