Skip to content

Azure MAG fullyprivate#29480

Merged
openshift-merge-robot merged 1 commit into
openshift:masterfrom
Amoghrd:azure-mag-fullypriv
Aug 23, 2022
Merged

Azure MAG fullyprivate#29480
openshift-merge-robot merged 1 commit into
openshift:masterfrom
Amoghrd:azure-mag-fullypriv

Conversation

@Amoghrd
Copy link
Copy Markdown
Contributor

@Amoghrd Amoghrd commented Jun 14, 2022

No description provided.

@openshift-ci openshift-ci Bot requested review from MayXuQQ and mgahagan73 June 14, 2022 20:38
@openshift-bot
Copy link
Copy Markdown
Contributor

Issues in openshift/release go stale after 30d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 15d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci openshift-ci Bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 1, 2022
@MayXuQQ
Copy link
Copy Markdown
Contributor

MayXuQQ commented Aug 1, 2022

/lifecycle frozen

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 1, 2022

@MayXuQQ: The lifecycle/frozen label cannot be applied to Pull Requests.

Details

In response to this:

/lifecycle frozen

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.

@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 1, 2022

/remove-lifecycle stale

@openshift-ci openshift-ci Bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 1, 2022
@Amoghrd Amoghrd force-pushed the azure-mag-fullypriv branch 3 times, most recently from bb1abdd to 2bcc78c Compare August 18, 2022 18:06
@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 18, 2022

/retest

@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 18, 2022

@jianlinliu PTAL

@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 18, 2022

/retest

Comment thread ci-operator/step-registry/ipi/conf/azuremag/ipi-conf-azuremag-commands.sh Outdated
@Amoghrd Amoghrd force-pushed the azure-mag-fullypriv branch from 2bcc78c to 4e673a2 Compare August 19, 2022 13:48
@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 19, 2022

/retest

1 similar comment
@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 19, 2022

/retest

@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 19, 2022

Install is very flaky for some reason

@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 19, 2022

/retest

1 similar comment
@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 22, 2022

/retest

@Amoghrd
Copy link
Copy Markdown
Contributor Author

Amoghrd commented Aug 22, 2022

@jianlinliu 4.12 rehearse seems to be passing but 4.10 is failing, is it because of the change to ipi-conf-azuremag using 4.12 image?

@jianlinliu
Copy link
Copy Markdown
Contributor

@jianlinliu 4.12 rehearse seems to be passing but 4.10 is failing, is it because of the change to ipi-conf-azuremag using 4.12 image?

I do not think so, the job failed on installer-rehearse-azure-mag-ipi-install-install, while the change to ipi-conf-azuremag is only used to creating install-config, not related to ipi-install-install. Sounds like a flake.

@jianlinliu
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 23, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 23, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Amoghrd, jianlinliu

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 23, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 23, 2022

@Amoghrd: The following tests 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/release-controller-config cf7800e049ad612496aab873316a7086594c1f7a link true /test release-controller-config
ci/rehearse/periodic-ci-openshift-verification-tests-master-installer-rehearse-4.10-installer-rehearse-azure-mag 4e673a2 link unknown /test pj-rehearse
ci/prow/pj-rehearse 4e673a2 link false /test pj-rehearse

Full PR test history. Your PR dashboard.

Details

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. I understand the commands that are listed here.

@openshift-merge-robot openshift-merge-robot merged commit 0529d10 into openshift:master Aug 23, 2022
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 23, 2022

@Amoghrd: Updated the following 2 configmaps:

  • ci-operator-master-configs configmap in namespace ci at cluster app.ci using the following files:
    • key openshift-verification-tests-master__installer-rehearse-4.12.yaml using file ci-operator/config/openshift/verification-tests/openshift-verification-tests-master__installer-rehearse-4.12.yaml
  • job-config-master-periodics configmap in namespace ci at cluster app.ci using the following files:
    • key openshift-verification-tests-master-periodics.yaml using file ci-operator/jobs/openshift/verification-tests/openshift-verification-tests-master-periodics.yaml
Details

In response to this:

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants