-
Notifications
You must be signed in to change notification settings - Fork 113
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
Changes to use Ironic Stein #79
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but please remove mdns-related options from ironic.conf. You'll also have to update dev-scripts to stop mentioning ironic-exporter first.
891e064
to
885e7ba
Compare
8251fb5
to
c300f05
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/metal3-io/ironic-image/blob/master/runironic.sh#L29 - remove the call for the exporter here and also the runexporterapp file https://github.com/metal3-io/ironic-image/blob/master/runexporterapp.sh
ironic.conf
Outdated
@@ -27,7 +27,6 @@ deploy_logs_local_path = /shared/log/ironic/deploy | |||
|
|||
[conductor] | |||
automated_clean = true | |||
enable_mdns = True | |||
send_sensor_data = true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
send_sensor_data = true
won't be necessary since we won't have the exporter
c300f05
to
bdb7a89
Compare
@dtantsur @iurygregory thanks and done all! |
bdb7a89
to
d3f935c
Compare
Looks good to me. @iurygregory @dtantsur If one of you can double check, I'll go ahead and push the merge button. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't merge, lgtm
This cannot merge before openshift-metal3/dev-scripts#680 |
This patch contains changes needed to be able to use Ironic Stein. It has dependencies from other patches in other related repos. Full list of patches provided in PR.
d3f935c
to
59da58f
Compare
Build SUCCESS, see build http://10.8.144.11:8080/job/dev-tools/920/ |
This patch contains changes needed to be able to use Ironic Stein.
It has dependencies from other patches in other related repos
and they should all be merged at the same time, following the order below,
to guarantee a correct behavior.
Full patch set:
openshift-metal3/dev-scripts#680
#79
metal3-io/ironic-inspector-image#30
metal3-io/ironic-ipa-downloader#3