Skip to content

Add gburges to cluster-wide.yaml - #2610

Merged
openshift-merge-robot merged 2 commits into
openshift:masterfrom
gburges:master
Jan 16, 2019
Merged

Add gburges to cluster-wide.yaml#2610
openshift-merge-robot merged 2 commits into
openshift:masterfrom
gburges:master

Conversation

@gburges

@gburges gburges commented Jan 16, 2019

Copy link
Copy Markdown
Contributor

No description provided.

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 16, 2019
@openshift-ci-robot openshift-ci-robot added the sig/azure Categorizes item related to Azure jobs label Jan 16, 2019
@gburges

gburges commented Jan 16, 2019

Copy link
Copy Markdown
Contributor Author

@Kargakis pls review

0xmichalis
0xmichalis reviewed Jan 16, 2019
View reviewed changes
Comment thread projects/azure/cluster-wide.yaml Outdated
0xmichalis
0xmichalis approved these changes Jan 16, 2019
View reviewed changes

@0xmichalis 0xmichalis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 16, 2019
@openshift-ci-robot

openshift-ci-robot commented Jan 16, 2019

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gburges, kargakis

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-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 16, 2019
@openshift-merge-robot
openshift-merge-robot merged commit 60185f2 into openshift:master Jan 16, 2019
@0xmichalis

0xmichalis commented Jan 16, 2019 •
edited
Loading

Copy link
Copy Markdown
Contributor

Updated manually in the CI cluster, you should have access in the azure namespace now

[kargakis@dhcp129-200 release]$ oc apply --as=system:admin -f projects/azure/cluster-wide.yaml 
group.user.openshift.io/azure-team configured
Error from server (AlreadyExists): project.project.openshift.io "azure" already exists
[kargakis@dhcp129-200 release]$ oc get group azure-team
NAME         USERS
azure-team   asalkeld, charlesakalugwu, gburges, jim-minter, kargakis, kwoodson, Makdaam, mjudeikis, pweil-, thekad, y-cote

@birsanv birsanv mentioned this pull request Jul 2, 2026
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@asalkeld asalkeld Awaiting requested review from asalkeld
@jim-minter jim-minter Awaiting requested review from jim-minter
1 more reviewer
@0xmichalis 0xmichalis 0xmichalis approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@0xmichalis 0xmichalis

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. sig/azure Categorizes item related to Azure jobs size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gburges @openshift-ci-robot @0xmichalis @openshift-merge-robot