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.

Sign up for GitHub

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

Jump to bottom

azure: lower log priority for skipped nic update message #34730

Merged
merged 1 commit into from
Oct 14, 2016
Merged

azure: lower log priority for skipped nic update message #34730

merged 1 commit into from
Oct 14, 2016

Conversation

colemickens
Copy link
Contributor

@colemickens colemickens commented Oct 13, 2016 •
edited by jessfraz
Loading

What this PR does / why we need it: Very minor, just wanted to remove some log noise I introduced in #34526.

I chose V(3) since it aligns with the other nicupdate message printed out here, and will be hidden for the usual default of --v=2.

Release note:

azure: lower log priority for skipped nic update message

This change is Reviewable

@k8s-github-robot k8s-github-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. release-note-label-needed labels Oct 13, 2016
@colemickens
Copy link
Contributor Author

@brendandburns can you take a look? Thanks.

@brendandburns brendandburns added release-note-none Denotes a PR that doesn't merit a release note. lgtm "Looks good to me", indicates that a PR is ready to be merged. and removed release-note-label-needed labels Oct 13, 2016
@brendandburns
Copy link
Contributor

lgtm.

@k8s-github-robot
Copy link

Automatic merge from submit-queue

@k8s-github-robot k8s-github-robot merged commit 4f36c23 into kubernetes:master Oct 14, 2016
@colemickens colemickens deleted the colemickens-azure-specify-availabilityset branch October 14, 2016 18:34
@colemickens colemickens mentioned this pull request Oct 17, 2016
Merged
@jessfraz jessfraz added this to the v1.4 milestone Oct 17, 2016
@jessfraz jessfraz mentioned this pull request Oct 18, 2016
Merged
@jessfraz jessfraz added cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. release-note Denotes a PR that will be considered when it comes time to generate release notes. and removed release-note-none Denotes a PR that doesn't merit a release note. labels Oct 18, 2016
k8s-github-robot pushed a commit that referenced this pull request Oct 18, 2016
Merge pull request #35001 from jessfraz/automated-cherry-pick-of-#34526-
914ff99 
#34730-origin-release-1.4

Automatic merge from submit-queue

Automated cherry pick of #34526 #34730

Cherry pick of #34526 #34730 on release-1.4.

#34526: azure: filter load balancer backend nodes to
#34730: azure: lower log priority for skipped nic update message
@k8s-cherrypick-bot
Copy link

Commit found in the "release-1.4" branch appears to be this PR. Removing the "cherrypick-candidate" label. If this is an error find help to get your PR picked.

shyamjvs pushed a commit to shyamjvs/kubernetes that referenced this pull request Dec 1, 2016
Merge pull request kubernetes#35001 from jessfraz/automated-cherry-pi…
87fa2ae 
…ck-of-#34526-kubernetes#34730-origin-release-1.4

Automatic merge from submit-queue

Automated cherry pick of kubernetes#34526 kubernetes#34730

Cherry pick of kubernetes#34526 kubernetes#34730 on release-1.4.

kubernetes#34526: azure: filter load balancer backend nodes to
kubernetes#34730: azure: lower log priority for skipped nic update message
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@brendandburns brendandburns

Labels
cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
v1.4
Development

Successfully merging this pull request may close these issues.
7 participants
@colemickens @brendandburns @k8s-github-robot @k8s-cherrypick-bot @jessfraz @googlebot @wojtek-t