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

Show what project is triggered not just branch #69

Conversation

ca-stefan-cordes
Copy link
Contributor

On Info Level the log lines on a multibranch project that are produced are

Dez 08, 2019 1:54:00 PM **INFO** com.cloudbees.jenkins.plugins.BitbucketJobProbe triggerMatchingJobs
Triggering BitBucket job develop

which is not really helpful as "job.getName()" is just the branch name.
Like on the fine level log statement above the "job.getFullDisplayName()" should be printed:

Dez 08, 2019 1:54:00 PM **FINE** com.cloudbees.jenkins.plugins.BitbucketJobProbe
Considering to poke test-multibranch » develop

which is more helpful.

@tzachs
Copy link
Collaborator

tzachs commented Aug 3, 2020

@ca-stefan-cordes since the plugin Jenkins base version was updated to 2.204.1, the code was changed.
Can you please re-fork this one and make the changes you need?
This PR should be closed unless you are really against it.

@tzachs
Copy link
Collaborator

tzachs commented Aug 11, 2020

@ca-stefan-cordes ?

@ca-stefan-cordes
Copy link
Contributor Author

Hi @tzachs,
I am still interested in having a useful log (getFullDisplayName instead of getName),
update will follow...

ca-stefan-cordes added a commit to c-a-services/bitbucket-plugin that referenced this pull request Aug 11, 2020
@ca-stefan-cordes ca-stefan-cordes force-pushed the show-what-project-is-triggered-not-just-branch branch from 00c946a to b653831 Compare August 11, 2020 21:06
ca-stefan-cordes added a commit to c-a-services/bitbucket-plugin that referenced this pull request Aug 11, 2020
@ca-stefan-cordes
Copy link
Contributor Author

Hi @tzachs,
I tried to re-use this pull-request via force pushing the re-setted brach to my repo, but github closed this PR.
So I created a new one against current master: #78

@tzachs
Copy link
Collaborator

tzachs commented Aug 12, 2020

Hi @ca-stefan-cordes ,

Thanks for opening a new PR. sorry for the hassle

tzachs added a commit that referenced this pull request Aug 12, 2020
…ed-not-just-branch

Show what project is triggered, not just the branch name. #69
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants