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

Remove elemental-cli from kairos #1422

Closed
wants to merge 20 commits into from
Closed

Conversation

Itxaka
Copy link
Member

@Itxaka Itxaka commented May 17, 2023

Signed-off-by: Itxaka [email protected]

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

@Itxaka Itxaka requested a review from a team May 17, 2023 09:05
@Itxaka Itxaka linked an issue May 17, 2023 that may be closed by this pull request
@Itxaka
Copy link
Member Author

Itxaka commented May 17, 2023

Needs a repository bump to get the newer packages which include kairos-agent run stage and the new services using kairos-agent instead of elemental

1 similar comment
@Itxaka
Copy link
Member Author

Itxaka commented May 17, 2023

Needs a repository bump to get the newer packages which include kairos-agent run stage and the new services using kairos-agent instead of elemental

@Itxaka
Copy link
Member Author

Itxaka commented May 17, 2023

this one: #1423

@@ -10,7 +10,7 @@ SSH_USER="${SSH_USER:-kairos}"
SSHPASS="${SSH_PASS:-kairos}"
export SSHPASS
HD_SIZE="${HD_SIZE:-50000}"
INSTALL_COMMAND="${INSTALL_COMMAND:-sudo /bin/sh -c 'elemental install /dev/sda && sync'}"
INSTALL_COMMAND="${INSTALL_COMMAND:-sudo /bin/sh -c 'kairos-agent install /dev/sda && sync'}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't it true that kairos-agent install and kairos-agent manual-install can be both used to achieve the same things (are interchangeable) ? If yes, maybe we should get rid of one of the 2.

Copy link
Contributor

Choose a reason for hiding this comment

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

Nothing to do with this PR though, I just had that thought.

Copy link
Member Author

Choose a reason for hiding this comment

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

Basically yes, but manual install accepts a config file while install does not and expects a config to be ready in the live CD or config dirs.

We should merge both and just add a flag to override the config file.

jimmykarily
jimmykarily previously approved these changes May 18, 2023
Itxaka added 4 commits May 18, 2023 16:05
We no longer need to link stuff as kairos-agent will try to find both
names for the binaty

Signed-off-by: Itxaka <[email protected]>
Signed-off-by: Itxaka <[email protected]>
Itxaka added 13 commits May 19, 2023 19:21
Signed-off-by: Itxaka <[email protected]>
They store the files in different places

Signed-off-by: Itxaka <[email protected]>
 - machine-id should be removed to not break non systemd systems.
 - systemd will create it on first boot, then our yip files take care of
   storing it safely
 - /var/lib/dbus needs to be a PERSISTENT path for alpine systems using
   dbus, which store the machine-id in there

Signed-off-by: Itxaka <[email protected]>
Signed-off-by: Itxaka <[email protected]>
@Itxaka Itxaka linked an issue May 22, 2023 that may be closed by this pull request
mauromorales
mauromorales previously approved these changes May 22, 2023
@Itxaka
Copy link
Member Author

Itxaka commented May 23, 2023

moved to #1441

@Itxaka Itxaka closed this May 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants