Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SCC admission plugin: extract name to a constant #17856

Conversation

php-coder
Copy link
Contributor

I hope that it won't create unnecessary package coupling.

PTAL @pweil- @liggitt
CC @simo5

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 18, 2017
@php-coder
Copy link
Contributor Author

Test flake #17873
/test extended_conformance_gce

@php-coder php-coder changed the title PSP admission plugin: extract name to a constant SCC admission plugin: extract name to a constant Dec 19, 2017
@php-coder
Copy link
Contributor Author

@pweil- @enj PTAL

@enj
Copy link
Contributor

enj commented Jan 2, 2018

/approve

@deads2k please LGTM if you are happy with the change to the dependency graph.

@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 2, 2018
@deads2k
Copy link
Contributor

deads2k commented Jan 2, 2018

/approve

@deads2k please LGTM if you are happy with the change to the dependency graph.

We may end up taking this in the other direction (dependency-wise), but I'm ok with it for now since we already depend in that direction.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 2, 2018
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deads2k, enj, php-coder

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

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@php-coder
Copy link
Contributor Author

Test flake #17969
/test extended_conformance_gce

@php-coder
Copy link
Contributor Author

Test flake #17605
/test extended_conformance_install

@php-coder
Copy link
Contributor Author

Tests failed because of #17985
I'll re-run them later when it will be fixed.

@php-coder
Copy link
Contributor Author

/retest

@php-coder
Copy link
Contributor Author

e2e test failed because of #18000
/test end_to_end

@php-coder
Copy link
Contributor Author

Test flake #16994
/test extended_conformance_install

@php-coder
Copy link
Contributor Author

The following 3 tests failed:

  • [Feature:Builds][Conformance] build without output image building from templates should create an image from a S2i template without an output image reference defined [Suite:openshift/conformance/parallel]
  • [sig-api-machinery] Downward API should provide host IP as an env var [Conformance] [Suite:openshift/conformance/parallel] [Suite:k8s]
  • [sig-storage] Projected should be able to mount in a volume regardless of a different secret existing with same name in different namespace [Suite:openshift/conformance/parallel] [Suite:k8s]

Most likely all they were killed by timeout:

[INFO] test/extended/conformance.sh exited with code 0 after 01h 38m 58s
make: *** [test-extended] Terminated

	 -------------------------------------------------------------------
	|                                                                   |
	|  Ginkgo timed out waiting for all parallel nodes to report back!  |
	|                                                                   |
	 -------------------------------------------------------------------

/test extended_conformance_gce

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue (batch tested with PRs 17856, 16934, 17979, 17993, 18001).

@openshift-merge-robot openshift-merge-robot merged commit 58e3606 into openshift:master Jan 5, 2018
@php-coder php-coder deleted the improve_scc_admission_plugin branch January 6, 2018 00:12
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/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants