Closed yuqi-zhang closed 2 months ago
@yuqi-zhang: This pull request references Jira Issue OCPBUGS-35300, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
/jira refresh
@yuqi-zhang: This pull request references Jira Issue OCPBUGS-35300, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @sergiordlr
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sinnykumari, yuqi-zhang
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/retest-required
Remaining retests: 0 against base HEAD 0278c483093b751902be798d9c9fc9de8d976850 and 2 for PR HEAD a5dcc6fe225562079f1b3ab6667d3d2efc6c8436 in total
@yuqi-zhang: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/e2e-gcp-op-techpreview | a5dcc6fe225562079f1b3ab6667d3d2efc6c8436 | link | false | /test e2e-gcp-op-techpreview |
Full PR test history. Your PR dashboard.
@yuqi-zhang: Jira Issue OCPBUGS-35300: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-35300 has been moved to the MODIFIED state.
/cherry-pick release-4.16
@yuqi-zhang: new pull request created: #4406
[ART PR BUILD NOTIFIER]
This PR has been included in build ose-machine-config-operator-container-v4.17.0-202406121741.p0.gfc878b8.assembly.stream.el9 for distgit ose-machine-config-operator. All builds following this will include this PR.
In 4.14 we slightly tweaked the services such that mcd-pull no longer depends on network-online.target. This broke some ARO disconnected environments that needed dnsmasq to be ready before the MCD-pull runs on firstboot.
Add in a new Azure template for MCD-pull with an additional After=dnsmasq.service so the strict ordering of:
is adhered to.
- What I did
- How to verify it
- Description for the changelog