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

Ship systemd config file for sshpassword support #42

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ydirson
Copy link
Contributor

@ydirson ydirson commented Mar 24, 2023

The sshpassword commandline parameter is documented in doc/ but
is implemented in this file which was not shipped as part of the
installer itself.

Note this PR builds on #41 to deploy this file automatically in the RPM

Reasons to have this here include:
* don't require changes to srpm when the set of installed files change
* make installer development faster by deploying directly to a filesystem

This installation process still depends by default on the sm package
being installed, but through SM_ROOT variable it can now be run on a
dev machine where this is not the case, it just needs an unpacked
version of the package.

Signed-off-by: Yann Dirson <[email protected]>
The sshpassword commandline parameter is documented in doc/ but
is implemented in this file which was not shipped as part of the
installer itself.

Signed-off-by: Yann Dirson <[email protected]>
ydirson added a commit to xcp-ng/xcp that referenced this pull request Apr 4, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.
ydirson added a commit to xcp-ng/xcp that referenced this pull request Apr 6, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.
ydirson added a commit to xcp-ng/xcp that referenced this pull request Apr 17, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.
ydirson added a commit to xcp-ng/xcp that referenced this pull request Apr 17, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.
ydirson added a commit to xcp-ng/xcp that referenced this pull request Apr 18, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.
ydirson added a commit to xcp-ng/xcp that referenced this pull request Apr 18, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.
ydirson added a commit to xcp-ng/xcp that referenced this pull request Apr 18, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.
ydirson added a commit to xcp-ng/xcp that referenced this pull request Apr 19, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.
ydirson added a commit to xcp-ng/xcp that referenced this pull request Jun 15, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.

Signed-off-by: Yann Dirson <[email protected]>
ydirson added a commit to xcp-ng/create-install-image that referenced this pull request Jun 15, 2023
This should indeed be part of host-installer-startup, see
xenserver/host-installer#42

Will need to be reverted when that PR is merged.

Signed-off-by: Yann Dirson <[email protected]>
@freddy77
Copy link
Collaborator

@ydirson I think you should add some reviewers to this PR

@stormi
Copy link
Contributor

stormi commented Apr 30, 2024

@freddy77 I don't think we have enough credentials on this repo for this.

@freddy77
Copy link
Collaborator

freddy77 commented May 2, 2024

@freddy77 I don't think we have enough credentials on this repo for this.

Added some reviewer.

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.

3 participants