Skip to content

Bug 2034537: OWNERS: Update ShiftStack members#2885

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
shiftstack:owners
Dec 21, 2021
Merged

Bug 2034537: OWNERS: Update ShiftStack members#2885
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
shiftstack:owners

Conversation

@pierreprinetti
Copy link
Copy Markdown
Contributor

@pierreprinetti pierreprinetti commented Dec 21, 2021

No description provided.

@openshift-ci openshift-ci Bot added bugzilla/severity-medium Referenced Bugzilla bug's severity is medium for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. labels Dec 21, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 21, 2021

@pierreprinetti: This pull request references Bugzilla bug 2034537, which is valid. The bug has been updated to refer to the pull request using the external bug tracker.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target release (4.10.0) matches configured target release for branch (4.10.0)
  • bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)

No GitHub users were found matching the public email listed for the QA contact in Bugzilla (pprinett@redhat.com), skipping review request.

Details

In response to this:

Bug 2034537: OWNERS: Update ShiftStack members

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
Copy link
Copy Markdown
Contributor Author

pierreprinetti commented Dec 21, 2021

/skip

mandre
mandre reviewed Dec 21, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@mandre mandre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Dec 21, 2021
@sinnykumari
Copy link
Copy Markdown
Contributor

sinnykumari commented Dec 21, 2021

Out of curiosity, is Shiftstack and OpenStack refers to same team in OpenShift?

@pierreprinetti
Copy link
Copy Markdown
Contributor Author

pierreprinetti commented Dec 21, 2021

Shiftstack and OpenStack refers to same team in OpenShift

Within the context of OpenShift, we are the OpenStack team indeed. However since we officially are in the OpenStack organisation, it's weird to refer to the team as "openstack" in that context... hence the juggling 😅

@mandre
Copy link
Copy Markdown
Member

mandre commented Dec 21, 2021

Out of curiosity, is Shiftstack and OpenStack refers to same team in OpenShift?

Yes, depending on who you're talking to they may refer to our team under different names which can be very confusing. Team members usually refer to the team as "ShiftStack" that highlights we're doing "OpenShift on OpenStack", OpenShift org sees us as the "OpenStack" team, and OpenStack org sees us as the "OpenStack as Infra" team.

@pierreprinetti
Copy link
Copy Markdown
Contributor Author

pierreprinetti commented Dec 21, 2021

/skip

@sinnykumari
Copy link
Copy Markdown
Contributor

sinnykumari commented Dec 21, 2021

Interesting information, thanks for sharing :)

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 21, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mandre, pierreprinetti, sinnykumari

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 Dec 21, 2021
@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

8 similar comments
@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@sinnykumari
Copy link
Copy Markdown
Contributor

sinnykumari commented Dec 21, 2021

/skip

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Dec 21, 2021

/retest-required

Please review the full test history for this PR and help us cut down flakes.

@kikisdeliveryservice
Copy link
Copy Markdown
Contributor

kikisdeliveryservice commented Dec 21, 2021

Since this is just an OWNER's file update no need to retest

/override e2e-aws
/override e2e-gcp-op

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 21, 2021

@kikisdeliveryservice: /override requires a failed status context or a job name to operate on.
The following unknown contexts were given:

  • e2e-aws
  • e2e-gcp-op

Only the following contexts were expected:

  • ci/prow/bootstrap-unit
  • ci/prow/e2e-agnostic-upgrade
  • ci/prow/e2e-aws
  • ci/prow/e2e-aws-disruptive
  • ci/prow/e2e-aws-serial
  • ci/prow/e2e-aws-single-node
  • ci/prow/e2e-aws-upgrade-single-node
  • ci/prow/e2e-aws-workers-rhel7
  • ci/prow/e2e-aws-workers-rhel8
  • ci/prow/e2e-gcp-op
  • ci/prow/e2e-gcp-op-single-node
  • ci/prow/e2e-metal-ipi
  • ci/prow/e2e-ovn-step-registry
  • ci/prow/e2e-vsphere-upgrade
  • ci/prow/images
  • ci/prow/okd-e2e-aws
  • ci/prow/okd-images
  • ci/prow/unit
  • ci/prow/verify
  • pull-ci-openshift-machine-config-operator-fcos-e2e-aws
  • pull-ci-openshift-machine-config-operator-fcos-images
  • pull-ci-openshift-machine-config-operator-fcos-unit
  • pull-ci-openshift-machine-config-operator-fcos-verify
  • pull-ci-openshift-machine-config-operator-master-bootstrap-unit
  • pull-ci-openshift-machine-config-operator-master-e2e-agnostic-upgrade
  • pull-ci-openshift-machine-config-operator-master-e2e-aws-disruptive
  • pull-ci-openshift-machine-config-operator-master-e2e-aws-serial
  • pull-ci-openshift-machine-config-operator-master-e2e-aws-single-node
  • pull-ci-openshift-machine-config-operator-master-e2e-aws-upgrade-single-node
  • pull-ci-openshift-machine-config-operator-master-e2e-aws-workers-rhel7
  • pull-ci-openshift-machine-config-operator-master-e2e-aws-workers-rhel8
  • pull-ci-openshift-machine-config-operator-master-e2e-gcp-op
  • pull-ci-openshift-machine-config-operator-master-e2e-gcp-op-single-node
  • pull-ci-openshift-machine-config-operator-master-e2e-metal-ipi
  • pull-ci-openshift-machine-config-operator-master-e2e-ovn-step-registry
  • pull-ci-openshift-machine-config-operator-master-e2e-vsphere-upgrade
  • pull-ci-openshift-machine-config-operator-master-okd-e2e-aws
  • pull-ci-openshift-machine-config-operator-master-okd-images
  • tide
Details

In response to this:

Since this is just an OWNER's file update no need to retest

/override e2e-aws
/override e2e-gcp-op

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.

@kikisdeliveryservice
Copy link
Copy Markdown
Contributor

kikisdeliveryservice commented Dec 21, 2021

/override ci/prow/e2e-aws
/override ci/prow/e2e-gcp-op

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 21, 2021

@kikisdeliveryservice: Overrode contexts on behalf of kikisdeliveryservice: ci/prow/e2e-aws, ci/prow/e2e-gcp-op

Details

In response to this:

/override ci/prow/e2e-aws
/override ci/prow/e2e-gcp-op

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-merge-robot openshift-merge-robot merged commit e70e7b4 into openshift:master Dec 21, 2021
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 21, 2021

@pierreprinetti: Some pull requests linked via external trackers have merged:

The following pull requests linked via external trackers have not merged:

These pull request must merge or be unlinked from the Bugzilla bug in order for it to move to the next state. Once unlinked, request a bug refresh with /bugzilla refresh.

Bugzilla bug 2034537 has not been moved to the MODIFIED state.

Details

In response to this:

Bug 2034537: OWNERS: Update ShiftStack members

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
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Dec 21, 2021

@pierreprinetti: 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/e2e-aws-upgrade-single-node 433f6d2 link false /test e2e-aws-upgrade-single-node
ci/prow/e2e-aws-workers-rhel8 433f6d2 link false /test e2e-aws-workers-rhel8
ci/prow/e2e-aws-serial 433f6d2 link false /test e2e-aws-serial
ci/prow/e2e-aws-workers-rhel7 433f6d2 link false /test e2e-aws-workers-rhel7
ci/prow/e2e-aws-disruptive 433f6d2 link false /test e2e-aws-disruptive
ci/prow/e2e-vsphere-upgrade 433f6d2 link false /test e2e-vsphere-upgrade

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.

@pierreprinetti pierreprinetti deleted the owners branch December 21, 2021 21:49
@openshift-ci openshift-ci Bot mentioned this pull request Jan 6, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sinnykumari sinnykumari sinnykumari approved these changes

@cheesesashimi cheesesashimi Awaiting requested review from cheesesashimi

@kikisdeliveryservice kikisdeliveryservice Awaiting requested review from kikisdeliveryservice

+1 more reviewer

@mandre mandre mandre left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@mandre mandre

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. bugzilla/severity-medium Referenced Bugzilla bug's severity is medium for the branch this PR is targeting. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@pierreprinetti @sinnykumari @mandre @openshift-bot @kikisdeliveryservice @openshift-merge-robot