Skip to content

[4.3] pkg/daemon: MCD constants fixup#1058

Merged
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
sgreene570:mcd-constants-fixup
Oct 3, 2019
Merged

[4.3] pkg/daemon: MCD constants fixup#1058
openshift-merge-robot merged 2 commits intoopenshift:masterfrom
sgreene570:mcd-constants-fixup

Conversation

@sgreene570
Copy link
Copy Markdown
Contributor

@sgreene570 sgreene570 commented Aug 13, 2019

- What I did
Add the "openshift.io/operator-managed" label from #832 to the MCD global constants file.
This way the operator managed label can be easily swapped out if the need arises (once #817 is merged the label will be used more than once, and it is an important label).

I also cleaned up some small typos in the constants file.

- How to verify it
CI

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 13, 2019
@kikisdeliveryservice
Copy link
Copy Markdown
Contributor

since it's so small i think you can squash commits 1 & 3

@kikisdeliveryservice kikisdeliveryservice changed the title pkg/daemon: MCD constants fixup [4.3] pkg/daemon: MCD constants fixup Aug 13, 2019
@kikisdeliveryservice
Copy link
Copy Markdown
Contributor

just adding a hold for now since this isn't a bugfix. thanks for this!

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 13, 2019
@sgreene570 sgreene570 force-pushed the mcd-constants-fixup branch from 4aff356 to 555110d Compare August 13, 2019 22:42
@sgreene570 sgreene570 force-pushed the mcd-constants-fixup branch from 555110d to d99a7dc Compare August 13, 2019 22:46
@runcom runcom removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 24, 2019
@runcom
Copy link
Copy Markdown
Member

runcom commented Sep 24, 2019

/approve
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 24, 2019
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: runcom, sgreene570

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 Sep 24, 2019
@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

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

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

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

/retest

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

@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

@kikisdeliveryservice
Copy link
Copy Markdown
Contributor

these retests are flooding my inbox.
until ci straightens itself out
/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 26, 2019
@kikisdeliveryservice
Copy link
Copy Markdown
Contributor

/retest

@runcom
Copy link
Copy Markdown
Member

runcom commented Oct 3, 2019

/hold cancel

@openshift-ci-robot openshift-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 3, 2019
@openshift-bot
Copy link
Copy Markdown
Contributor

/retest

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

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. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants