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

remove the pattern plain: from gerrit trigger! #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

khozema-nullwala
Copy link

Hi,
In the DevOps Academy session while running through Module - 02 lab activity the build job does not gets trigger.

Removing the plain: from the gerrit trigger pattern of Reference Application Build job then after saving changes to this job it gets executed on code push done to master branch.

@RobertNorthard
Copy link
Contributor

RobertNorthard commented Nov 12, 2019

Thanks @khozema-nullwala and @randyspiess for your contribution.

However, our CI / CD tests (running in Travis) are failing. Would you be able to correct this too? Looks like openjdk7 and 8 are no longer available in Travis.

Once Travis is passing, I'm happy to merge these changes as I can see in the Pluggable SCM provider code base here where the gerrit provider trigger methods lives it also appends "plain:" which would result in plain:plain:master.

@RobertNorthard RobertNorthard self-assigned this Nov 12, 2019
@RobertNorthard RobertNorthard removed their assignment Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants