Skip to content

remove the pattern plain: from gerrit trigger!#47

Open
khozema-nullwala wants to merge 1 commit intoAccenture:masterfrom
khozema-nullwala:master
Open

remove the pattern plain: from gerrit trigger!#47
khozema-nullwala wants to merge 1 commit intoAccenture:masterfrom
khozema-nullwala:master

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
@RobertNorthard RobertNorthard self-requested a review November 12, 2019 23:25
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