Skip to content

Conversation

@aweiteka
Copy link
Contributor

@aweiteka aweiteka commented Jul 6, 2017

Signed-off-by: Aaron Weitekamp aweiteka@redhat.com

@aweiteka aweiteka changed the title Improve image signing cmd docs Improve verify image signing cmd docs Jul 6, 2017
@openshift-merge-robot openshift-merge-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jul 24, 2017
@openshift-merge-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: aweiteka
We suggest the following additional approver: mfojtik

Assign the PR to them by writing /assign @mfojtik in a comment when ready.

No associated issue. Update pull-request body to add a reference to an issue, or get approval with /approve no-issue

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

Details 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

@openshift-merge-robot openshift-merge-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jul 27, 2017
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 28, 2017
@openshift-merge-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: aweiteka
We suggest the following additional approver: mfojtik

Assign the PR to them by writing /assign @mfojtik in a comment when ready.

No associated issue. Update pull-request body to add a reference to an issue, or get approval with /approve no-issue

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

Details 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

@openshift-merge-robot openshift-merge-robot removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 28, 2017
openshift-merge-robot and others added 18 commits September 7, 2017 13:16
Automatic merge from submit-queue (batch tested with PRs 14825, 15756, 16178, 16188, 16189)

dind: add support for ovn-kubernetes network plugin

@rajatchopra here's the Origin part...

Operation:
1) get a git checkout of ovn-kubernetes with my changes: ovn-kubernetes/ovn-kubernetes#141
2) rebuild your DIND images `hack/dind-cluster.sh build-images`
3) `OVN_ROOT=/path/to/ovn-kubernetes hack/dind-cluster.sh start -o`
…-handler

Automatic merge from submit-queue (batch tested with PRs 14825, 15756, 16178, 16188, 16189)

use default handler

Builds on already lgtm'd pulls.  Only the last commit is unique.

The biggest impact is to audit.

@soltysh @sttts you know want it before dcut today. :)
…clusterregistry-bz1488059

Automatic merge from submit-queue (batch tested with PRs 14825, 15756, 16178, 16188, 16189)

ClusterRegistry diagnostic: fix address mismatch

The registry for imagestreams was previously recorded as an IP, which
could change if the registry service were re-created.  Now it is a
cluster hostname, which should be unchanging even if re-created. Just
ensure it is the right hostname.

fix bug 1488059
https://bugzilla.redhat.com/show_bug.cgi?id=1488059
Automatic merge from submit-queue (batch tested with PRs 14825, 15756, 16178, 16188, 16189)

UPSTREAM: google/cadvisor: 1700: Reduce log spam when unable to get network stats

google/cadvisor#1700

xref https://bugzilla.redhat.com/show_bug.cgi?id=1328913

@derekwaynecarr
Docker changed the error for a non-existent --build-arg to a warning
and changed the text of the message slightly, breaking one of our tests.
Docker change: moby/moby@f150f42

Fixes: openshift#16197
Automatic merge from submit-queue

UPSTREAM: 48940: support fc volume attach and detach

fix https://bugzilla.redhat.com/show_bug.cgi?id=1450224
@childsb @eparis
Use the correct JSONPath syntax (this may be something to backport to
3.6.1).
Automatic merge from submit-queue

deprecate the imagestream.spec.DockerImageRepository field
…stem:masters group.

This change improves UX and allows to a member of the sudoer
group to, for example, create a privileged pod using impersonation:

$ oc create -f privileged.pod --as system:admin
Automatic merge from submit-queue

Rename pkg/deploy -> pkg/apps

@deads2k @mfojtik like promised
@tnozicka fyi
Automatic merge from submit-queue (batch tested with PRs 16202, 16198, 16212)

cleanup imagestreamtag desc
mfojtik and others added 23 commits September 18, 2017 15:52
…eamtag

Automatic merge from submit-queue (batch tested with PRs 16224, 14227)

Add `oc create imagestreamtag`
Automatic merge from submit-queue (batch tested with PRs 16224, 14227)

Node admission/authorization 

Enables the node authorizer and NodeRestriction admission plugins
This change restores the previous behavior of running
`hack/test-integration.sh <regex>`.  The timeout flag is no longer
passed to the integration test runner.  To do so correctly requires
using `go test -args` which would add further complexity to
`hack/test-go.sh`.

Signed-off-by: Monis Khan <mkhan@redhat.com>
Automatic merge from submit-queue (batch tested with PRs 15834, 16321, 16353, 15298, 15433)

Drop pkg/util/ipcmd, port to vishvananda/netlink (mostly)

This drops our wrapper around /sbin/ip and ports the remaining code to use the netlink library instead.

Except for the one place we were using ipcmd in pod_linux because netlink doesn't yet have a `LinkSetTxQLen`. I've submitted a PR for that (vishvananda/netlink#257) but until then, I just changed the code to call `exec` directly.
…-PR383-bugzilla1490768-support-HTTP-300-response-in-pagination

Automatic merge from submit-queue (batch tested with PRs 15834, 16321, 16353, 15298, 15433)

bump(github.com/gophercloud/gophercloud): ed590d9afe113c6107cd60717b196155e6579e78

Automatic detection of Cinder API version doesn't work.
The problem is described in Bugzilla #14907688 (https://bugzilla.redhat.com/show_bug.cgi?id=1490768).

The problem is in the gophercloud library that doesn't support HTTP code 300 in pagination.
This problem is fixed in PR: gophercloud/gophercloud#383

That's why the gophercloud library version is bumped up to ed590d9afe113c6107cd60717b196155e6579e78

fix https://bugzilla.redhat.com/show_bug.cgi?id=1490768

@rootfs PTAL
Automatic merge from submit-queue (batch tested with PRs 15834, 16321, 16353, 15298, 15433)

use privileged  client in registry instead of user client when getting or creating signatures

@csrwng 

Fixes: openshift#16349
Automatic merge from submit-queue (batch tested with PRs 15834, 16321, 16353, 15298, 15433)

allow sys_chroot cap on SCCs

Removes the requirement to drop chroot. 

@smarterclayton
Automatic merge from submit-queue (batch tested with PRs 15834, 16321, 16353, 15298, 15433)

UPSTREAM: 49420: Fix c-m crash while verifying attached volumes

Fixes  https://bugzilla.redhat.com/show_bug.cgi?id=1474441
Automatic merge from submit-queue

Promote image trigger annotation to GA and correct syntax

Use the correct JSONPath syntax (this may be something to backport to
3.6.1).

@bparees @mfojtik @soltysh
Automatic merge from submit-queue (batch tested with PRs 16226, 16377, 15766, 16299, 16153)

remove install dependency from images

Use a specific, private scheme to handle the conversion of internal image apis stuff.

@bparees @soltysh found it!
Automatic merge from submit-queue (batch tested with PRs 16226, 16377, 15766, 16299, 16153)

Fix NodePort test

Pulls in the k8s test utils fix from kubernetes/kubernetes#49025 to make the NodePort test work on non-cloud installs again.

Fixes openshift#15253
Automatic merge from submit-queue (batch tested with PRs 16226, 16377, 15766, 16299, 16153)

UPSTREAM: 50334: Support iscsi volume attach and detach

Fixes: kubernetes/kubernetes#50334

/cc @rootfs @humblec @childsb @eparis
Automatic merge from submit-queue (batch tested with PRs 16226, 16377, 15766, 16299, 16153)

Allow running a subset of the integration tests

This change restores the previous behavior of running `hack/test-integration.sh <regex>`.  The timeout flag is no longer passed to the integration test runner.  To do so correctly requires using `go test -args` which would add further complexity to `hack/test-go.sh`.

Signed-off-by: Monis Khan <mkhan@redhat.com>

/assign @smarterclayton
Automatic merge from submit-queue

provide a specific order for template parameters in the TSB

fixes openshift#16383
Automatic merge from submit-queue

Switch pkg/image to use clientsets

@mfojtik @deads2k I hope this handles all of them, including the SAR bits we've talked about yesterday.
@openshift-merge-robot openshift-merge-robot added needs-api-review and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Sep 19, 2017
@openshift-ci-robot openshift-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 19, 2017
@aweiteka
Copy link
Contributor Author

Proposed changes were picked up in another commit. Closing.

@aweiteka aweiteka closed this Sep 19, 2017
@openshift-ci-robot
Copy link

openshift-ci-robot commented Sep 19, 2017

@aweiteka: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
ci/openshift-jenkins/cmd 0dc49b1 link /test cmd

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-api-review size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.