Skip to content

[release-4.1] Bug 1720833: Gather extended network data #101

Merged
openshift-merge-robot merged 6 commits into
openshift:release-4.1from
openshift-cherrypick-robot:cherry-pick-94-to-release-4.1
Jun 25, 2019
Merged

[release-4.1] Bug 1720833: Gather extended network data #101
openshift-merge-robot merged 6 commits into
openshift:release-4.1from
openshift-cherrypick-robot:cherry-pick-94-to-release-4.1

Conversation

@openshift-cherrypick-robot
Copy link
Copy Markdown

This is an automated cherry-pick of #94

/assign sferich888

We will work in other PR/card adding relationship of objects managed
by cluster operator to RelatedObjects, just so must-gather binary
pulls the pod logs for free and there's no need to pull that in bash
script.
We leave therefore in the bash scripts the gathering of iptables,
ovs-vsctl dump and vswitchd/ovsdb-server logs (this is for better
visualization, since the OVS pods have those logs interleaved).
Also warn if no network nodes is feeded, as that will collect all
network logs on all network nodes.
@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 15, 2019
@sferich888
Copy link
Copy Markdown
Contributor

@sferich888
Copy link
Copy Markdown
Contributor

/retitle [release-4.1] Bug1720833: Gather extended network data

@openshift-ci-robot openshift-ci-robot changed the title [release-4.1] Gather extended network data [release-4.1] Bug1720833: Gather extended network data Jun 15, 2019
@sferich888
Copy link
Copy Markdown
Contributor

/bugzilla refresh

@openshift-ci-robot
Copy link
Copy Markdown

@sferich888: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

/bugzilla refresh

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.

@sferich888
Copy link
Copy Markdown
Contributor

/retitle [release-4.1] Bug 1720833: Gather extended network data
/bugzilla refresh

@openshift-ci-robot
Copy link
Copy Markdown

@sferich888: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

/retitle [release-4.1] Bug 1720833: Gather extended network data
/bugzilla refresh

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-robot openshift-ci-robot changed the title [release-4.1] Bug1720833: Gather extended network data [release-4.1] Bug 1720833: Gather extended network data Jun 15, 2019
@openshift-ci-robot
Copy link
Copy Markdown

@openshift-cherrypick-robot: This pull request references an invalid Bugzilla bug:

  • expected the bug to target the "4.1.z" release, but it targets "---" instead
Details

In response to this:

[release-4.1] Bug 1720833: Gather extended network data

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-robot openshift-ci-robot added the bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. label Jun 15, 2019
@sferich888
Copy link
Copy Markdown
Contributor

/bugzilla refresh

@openshift-ci-robot openshift-ci-robot added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. and removed bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Jun 15, 2019
@openshift-ci-robot
Copy link
Copy Markdown

@sferich888: This pull request references a valid Bugzilla bug.

Details

In response to this:

/bugzilla refresh

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.

@sferich888
Copy link
Copy Markdown
Contributor

@rcarrillocruz this should be good now!
@RickJWagner can you review

@RickJWagner
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jun 25, 2019
@knobunc
Copy link
Copy Markdown

knobunc commented Jun 25, 2019

/lgtm

@openshift-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: knobunc, openshift-cherrypick-robot, RickJWagner

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

@eparis eparis added the cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. label Jun 25, 2019
@openshift-merge-robot openshift-merge-robot merged commit 9b4d99f into openshift:release-4.1 Jun 25, 2019
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. bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants