OSASINFRA-3363: ShiftStack CAPO integration #7939
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
openshift-merge-robot
added
the
needs-rebase
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
Skipping CI for Draft Pull Request. |
pierreprinetti
force-pushed
the
capo-integration-la-totale
branch
from
fc61c23
to
5fe364c
Compare
openshift-merge-robot
removed
the
needs-rebase
pierreprinetti
force-pushed
the
capo-integration-la-totale
branch
3 times, most recently
from
7122edf
to
a18f2c1
Compare
pierreprinetti
force-pushed
the
capo-integration-la-totale
branch
2 times, most recently
from
a9208fe
to
b22180b
Compare
stephenfin
force-pushed
the
capo-integration-la-totale
branch
from
b22180b
to
65ef7aa
Compare
pierreprinetti
force-pushed
the
capo-integration-la-totale
branch
from
65ef7aa
to
3e7a492
Compare
|
Re-rebased on # 7824 rather than master |
pierreprinetti
force-pushed
the
capo-integration-la-totale
branch
from
3e7a492
to
8657c78
Compare
MaysaMacedo
force-pushed
the
capo-integration-la-totale
branch
from
8657c78
to
37f03f8
Compare
|
I just rebased again on top of 7824 PR with the following command: |
pierreprinetti
force-pushed
the
capo-integration-la-totale
branch
from
37f03f8
to
8657c78
Compare
|
We will have to live with that "Conflicting files" warning until @patrickdillon's PR is either rebased on or merged to master. For now, I have removed the commits that were merged to master, so that no further action should be needed when it's time to rebase onto master. |
openshift-merge-robot
added
the
needs-rebase
pierreprinetti
force-pushed
the
capo-integration-la-totale
branch
2 times, most recently
from
616dd51
to
3bb1261
Compare
openshift-merge-robot
removed
the
needs-rebase
MaysaMacedo
force-pushed
the
capo-integration-la-totale
branch
from
3bb1261
to
2b054a6
Compare
mandre
force-pushed
the
capo-integration-la-totale
branch
from
2b054a6
to
b5aaa09
Compare
|
/test e2e-openstack |
|
@mandre: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
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. |
|
/test pull-ci-openshift-installer-master-e2e-openstack-ovn |
|
@mandre: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
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. |
|
/test e2e-openstack-proxy |
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
This was generated from the CAPO repo using the same release version, like so: make release-manifests (the file then gets dumped to 'out/infrastructure-components.yaml') They can also be sourced from the following: https://storage.googleapis.com/artifacts.k8s-staging-capi-openstack.appspot.com/components/v0.7.0/infrastructure-components.yaml per the CAPO README. We probably don't want to be syncing these manually but this is how things work for every other provider currently so 🤷. Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com> Co-authored-by: Maysa Macedo <mdemaced@redhat.com> Co-authored-by: Martin André <m.andre@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com> Co-authored-by: Maysa Macedo <mdemaced@redhat.com> Co-authored-by: Martin André <m.andre@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Co-Authored-By: Maysa Macedo <mdemaced@redhat.com>
Co-Authored-By: Pierre Prinetti <pierreprinetti@redhat.com> Co-Authored-By: Martin André <m.andre@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
This is created in the postprovision step, since we'll attach it once the bootstrap machine has been created. Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
MaysaMacedo
force-pushed
the
capo-integration-la-totale
branch
from
4e758fb
to
58f46e5
Compare
openshift-merge-robot
removed
the
needs-rebase
|
/test altinfra-e2e-openstack-capi-ovn |
|
/lgtm |
|
/skip |
|
/override ci/prow/altinfra-e2e-openstack-capi-ovn |
|
@patrickdillon: Overrode contexts on behalf of patrickdillon: ci/prow/altinfra-e2e-openstack-capi-ovn 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. |
|
@pierreprinetti: The following tests failed, say
Full PR test history. Your PR dashboard. 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-bot
bot
merged commit 6e9d05c
into
openshift:master
52 of 53 checks passed
|
[ART PR BUILD NOTIFIER] This PR has been included in build ose-installer-altinfra-container-v4.16.0-202402272209.p0.g6e9d05c.assembly.stream.el8 for distgit ose-installer-altinfra. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Opening a PR so that Github keeps an history of the changes in this branch