Skip to content

Conversation

@bcrochet
Copy link

@bcrochet bcrochet commented Sep 1, 2020

This change is in support of bugzilla 1872080. It modifies the
configuration file for the baremetal-runtimecfg component by adjusting their
dockerfile to match the newly created "Dockerfile.openshift" file.

This change also depends on another pull request and should not be
merged until they are:
openshift/baremetal-runtimecfg#93

@openshift-bot
Copy link

Build #1

None: Max retries exceeded with url: /openshift/baremetal-runtimecfg/blob/release-4.6/Dockerfile.rhel8 (Caused by None)
Validating images/baremetal-runtimecfg.yml

@openshift-bot
Copy link

Build #2

None: Max retries exceeded with url: /openshift/baremetal-runtimecfg/blob/release-4.6/Dockerfile.openshift (Caused by None)
Validating images/baremetal-runtimecfg.yml

@bcrochet
Copy link
Author

bcrochet commented Sep 1, 2020

/retitle Bug 1872080: Update dockerfile paths for baremetal-runtimecfg configurations

@openshift-ci-robot openshift-ci-robot changed the title [baremetal-runtimecfg] Use Dockerfile.rhel8 instead of Dockerfile Bug 1872080: Update dockerfile paths for baremetal-runtimecfg configurations Sep 1, 2020
@openshift-ci-robot
Copy link

@bcrochet: This pull request references Bugzilla bug 1872080, which is invalid:

  • expected Bugzilla bug 1872080 to depend on a bug in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

Bug 1872080: Update dockerfile paths for baremetal-runtimecfg configurations

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot openshift-ci-robot added bugzilla/severity-medium Referenced Bugzilla bug's severity is medium for the branch this PR is targeting. bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Sep 1, 2020
@openshift-ci-robot
Copy link

@bcrochet: This pull request references Bugzilla bug 1872080, which is invalid:

  • expected Bugzilla bug 1872080 to depend on a bug in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

Bug 1872080: Update dockerfile paths for baremetal-runtimecfg configurations

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@yselkowitz
Copy link

/retitle Update dockerfile paths for baremetal-runtimecfg configurations

@openshift-ci-robot openshift-ci-robot changed the title Bug 1872080: Update dockerfile paths for baremetal-runtimecfg configurations Update dockerfile paths for baremetal-runtimecfg configurations Sep 1, 2020
@openshift-ci-robot openshift-ci-robot removed the bugzilla/severity-medium Referenced Bugzilla bug's severity is medium for the branch this PR is targeting. label Sep 1, 2020
@openshift-ci-robot
Copy link

@bcrochet: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

Update dockerfile paths for baremetal-runtimecfg configurations

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot openshift-ci-robot removed the bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. label Sep 1, 2020
@yselkowitz
Copy link

The new dockerfile needs to land first: openshift/baremetal-runtimecfg#93
/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 1, 2020
@bcrochet
Copy link
Author

bcrochet commented Sep 1, 2020

/retest

@yselkowitz
Copy link

/hold cancel
/lgtm
/assign @jupierce

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 1, 2020
@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 1, 2020
@crawford
Copy link

crawford commented Sep 1, 2020

/retest

@bcrochet
Copy link
Author

bcrochet commented Sep 1, 2020

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 1, 2020
@bcrochet
Copy link
Author

bcrochet commented Sep 1, 2020

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 1, 2020
@bcrochet
Copy link
Author

bcrochet commented Sep 1, 2020

/retest

This change is in support of bugzilla 1872080. It modifies the
configuration file for the baremetal-runtimecfg component by adjusting their
dockerfile to match the newly created "Dockerfile.openshift" file.

This change also depends on another pull request and should not be
merged until they are:
openshift/baremetal-runtimecfg#93
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Sep 2, 2020
@openshift-bot
Copy link

Build #3

Validating images/baremetal-runtimecfg.yml

@yselkowitz
Copy link

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 2, 2020
@jupierce jupierce merged commit c2f12af into openshift-eng:openshift-4.6-rhel-8 Sep 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants