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

Downgrading a message from INFO to FINE #2510

Merged
merged 1 commit into from Aug 20, 2016

Conversation

5 participants
@jglick
Member

jglick commented Aug 16, 2016

Not really sure what this message is about, but it is printed pretty frequently when using Pipeline, and does not seem to correspond to any obvious error condition, so I would like it to shut up.

@reviewbybees

@abayer

This comment has been minimized.

Show comment
Hide comment
@abayer

abayer Aug 16, 2016

Member

🐝

Member

abayer commented Aug 16, 2016

🐝

@oleg-nenashev

This comment has been minimized.

Show comment
Hide comment
@oleg-nenashev
Member

oleg-nenashev commented Aug 16, 2016

🐝

@oleg-nenashev

This comment has been minimized.

Show comment
Hide comment
@oleg-nenashev

oleg-nenashev Aug 16, 2016

Member

Needs a JIRA issue if you want to get it backported to .3

Member

oleg-nenashev commented Aug 16, 2016

Needs a JIRA issue if you want to get it backported to .3

@reviewbybees

This comment has been minimized.

Show comment
Hide comment
@reviewbybees

reviewbybees Aug 16, 2016

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

reviewbybees commented Aug 16, 2016

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@recena

This comment has been minimized.

Show comment
Hide comment
@recena

recena Aug 16, 2016

Contributor

🐝 and 👍

Contributor

recena commented Aug 16, 2016

🐝 and 👍

@jglick

This comment has been minimized.

Show comment
Hide comment
@jglick

jglick Aug 16, 2016

Member

I do not think it needs a backport.

@stephenc any insight into this code BTW?

Member

jglick commented Aug 16, 2016

I do not think it needs a backport.

@stephenc any insight into this code BTW?

@jglick

This comment has been minimized.

Show comment
Hide comment
@jglick

jglick Aug 17, 2016

Member

Test failure looks unrelated.

Member

jglick commented Aug 17, 2016

Test failure looks unrelated.

@oleg-nenashev oleg-nenashev reopened this Aug 19, 2016

@oleg-nenashev oleg-nenashev merged commit 30ede81 into jenkinsci:master Aug 20, 2016

1 check passed

Jenkins This pull request looks good
Details

@jglick jglick deleted the jglick:Queue.maintain-will-you-please-be-quiet-please branch Oct 6, 2016

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