-
Notifications
You must be signed in to change notification settings - Fork 118
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
Sync up #204
Sync up #204
Conversation
We need to increase the size of the esp image for downstream compatibility.
Sensor data collection was intended for an earlier use of the ironic-image and resulting ironic container image in order to supply data to prometheus about hardware that has not yet been deployed upon. Since the deicsion was made not to leverage the data, it seems pointless to collect sensor data. As such, disable sensor data collection.
An improvement for testing and customization, moving all the files we need to install in the main image to a file and having dnf read its content. This allows exotic compositions of sources, like urls and local dirs, without having to touch the Dockerfile.
This will allow to pass a custom list of packages to install during build time with very little effort.
/lgtm |
@@ -58,7 +58,7 @@ enable_ssl_api = true | |||
automated_clean = {{ env.IRONIC_AUTOMATED_CLEAN }} | |||
# NOTE(dtantsur): keep aligned with [pxe]boot_retry_timeout below. | |||
deploy_callback_timeout = 4800 | |||
send_sensor_data = true | |||
send_sensor_data = false |
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 don't think we should change this.
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.
shouldn't change it at all, or upstream?
httpd-tools | ||
ipmitool | ||
iproute | ||
iscsi-initiator-utils |
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 wonder why we have this, we don't use the iscsi deploy..
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.
iirc we did at one stage (at the very start), its probably historical
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: derekhiggins, dtantsur, elfosardo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test-integration |
Bug 1986238: Supermicro X12 fails to provision using Redfish BM HW Provisioning
Syncing up patches from the downstream (openshift) image to here.