Skip to content

Conversation

piersharding
Copy link
Contributor

@piersharding piersharding commented Jul 29, 2022

This PR adds some things that have helped with build test environments with less post deploy manual config:

  • add arbitrary bootstrap arguments
  • add generic post bootstrap config commands
  • parameterise distribution release (required on Jammy or future newish releases)
  • make ssh_user specification optional
  • add generic post bootstrap and osd commands

Happy to further work on this if things are missing.
Cheers,
Piers.

Signed-off-by: Piers Harding piers@ompka.net

* add arbitrary bootstrap arguments
* add generic post bootstrap config commands
* parameterise distribution release (required on Jammy)
* make ssh_user specification optional
* add generic post bootstrap and osd commands

Signed-off-by: Piers Harding <piers@ompka.net>
Copy link

@markgoddard markgoddard left a comment

Choose a reason for hiding this comment

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

Thanks Piers

* tidy up gratuitous copies in commands
* rename variable specifically for debian
* clear up documentation

Signed-off-by: Piers Harding <piers@ompka.net>
Signed-off-by: Piers Harding <piers@ompka.net>
Copy link

@markgoddard markgoddard left a comment

Choose a reason for hiding this comment

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

Almost there

Signed-off-by: Piers Harding <piers@ompka.net>
@piersharding piersharding force-pushed the bootstrap-for-single-host branch from 2a6f2aa to 2e63e31 Compare August 2, 2022 18:16
@piersharding
Copy link
Contributor Author

Hi - I resolved a merge conflict with upstream, but it would be good if you consider whether it still does what you intended - thanks.

@markgoddard
Copy link

Thanks @piersharding

@markgoddard markgoddard merged commit 2ada6f1 into stackhpc:master Aug 2, 2022
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.

2 participants