restructure baremetal configuration - backport 4.13 #9282
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is backport of #9274
Move BM configuraiton from
AUTH"[baremetal"]
andAUTH["ipmi"]
toENV_DATA["baremetal"]
andENV_DATA["baremetal"]["servers"]
respectively.This is just cosmetic change, but I think it will make the code and config little bit more readable and clean. (Right now for example the
AUTH["ipmi"]
section contains also information about the network configuration of the server (IP, MAC,...) and disks, so I think it make more sense to name the section differently and also have all the configuration related to the particular baremetal environment in one section (ENV_DATA["baremetal"]
).The configuration now looks like this:
This PR has to be merged together with respective MR in ocs4-jenkins repo.