Page MenuHomePhabricator

Implement ADR-0016 for Secondary CI in Wikibase (Use Github Actions)
Closed, ResolvedPublic8 Estimated Story Points

Description

T270158: [12h] Decide on the "secondary" CI for Wikibase code repository created an ADR, now we must implement the ADR for secondary CI in WIkibase.

Acceptance Criteria:

  • Wikibase secondary CI no longer uses travis
  • Wikibase secondary CI uses github actions
  • The same matrix of jobs is run with at least email notifications
  • No longer needed travis references are cleaned up in Wikibase.git

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
darthmon_wmde set the point value for this task to 8.
Addshore renamed this task from Implement ADR-0016 for Secondary CI in WIkibase (Use Github Actions) to Implement ADR-0016 for Secondary CI in Wikibase (Use Github Actions).Feb 23 2021, 1:48 PM

Change 666647 had a related patch set uploaded (by Noa wmde; owner: Noa wmde):
[mediawiki/extensions/Wikibase@master] Add scondary-ci job to existing github action on Wikibase

https://gerrit.wikimedia.org/r/666647

Change 667111 had a related patch set uploaded (by Noa wmde; owner: Noa wmde):
[mediawiki/extensions/Wikibase@master] Merge remote-tracking branch 'origin/master' into T273447-implement-adr16-secondary-ci-github-actions

https://gerrit.wikimedia.org/r/667111

Change 667111 abandoned by Noa wmde:
[mediawiki/extensions/Wikibase@master] Merge remote-tracking branch 'origin/master' into T273447-implement-adr16-secondary-ci-github-actions

Reason:

https://gerrit.wikimedia.org/r/667111

Change 666647 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Add scondary-ci job to existing github action on Wikibase

https://gerrit.wikimedia.org/r/666647

Change 667846 had a related patch set uploaded (by Noa wmde; owner: Noa wmde):
[mediawiki/extensions/Wikibase@master] Wikibase: remove travis CI configuration

https://gerrit.wikimedia.org/r/667846

Change 667846 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Wikibase: remove travis CI configuration

https://gerrit.wikimedia.org/r/667846

Change 668442 had a related patch set uploaded (by Noa wmde; owner: Noa wmde):
[mediawiki/extensions/Wikibase@master] Github Actions - fix DB connection issues and separate actions

https://gerrit.wikimedia.org/r/668442

Change 668442 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Github Actions - fix DB connection issues and separate actions

https://gerrit.wikimedia.org/r/668442

Change 668653 had a related patch set uploaded (by Noa wmde; owner: Noa wmde):
[mediawiki/extensions/Wikibase@master] Github Actions CI - add mysqli extension to php setup step

https://gerrit.wikimedia.org/r/668653

Change 668653 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Github Actions CI - add mysqli extension to php setup step

https://gerrit.wikimedia.org/r/668653

Change 668677 had a related patch set uploaded (by Noa wmde; owner: Noa wmde):
[mediawiki/extensions/Wikibase@master] Github Actions CI - add mysqlnd extension to php setup step

https://gerrit.wikimedia.org/r/668677

Change 668677 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Github Actions CI - add mysqlnd extension to php setup step

https://gerrit.wikimedia.org/r/668677

Change 670149 had a related patch set uploaded (by Noa wmde; owner: Noa wmde):
[mediawiki/extensions/Wikibase@master] Github Actions CI - specify username and pass in mysql command

https://gerrit.wikimedia.org/r/670149

Change 670149 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Github Actions CI - specify username and pass in mysql command

https://gerrit.wikimedia.org/r/670149

Change 670416 had a related patch set uploaded (by Noa wmde; owner: Noa wmde):
[mediawiki/extensions/Wikibase@master] Github Actions - add link to email notification

https://gerrit.wikimedia.org/r/670416

Change 670416 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Github Actions - add link to email notification

https://gerrit.wikimedia.org/r/670416

I still see a couple of references to travis in a readme and in the scripts used for CI that probably need cleaning up one way or another.

image.png (161×648 px, 21 KB)

Right now we only run github secondary CI on master, which I think is likely fine
https://github.com/wikimedia/Wikibase/blob/master/.github/workflows/secondaryCI.yml#L4
Thus one of the referenced in the scripts checking the branch could likley be removed?

Change 672778 had a related patch set uploaded (by Tonina Zhelyazkova; owner: Tonina Zhelyazkova):
[mediawiki/extensions/Wikibase@master] Replace Travis CI badge with GitHub Actions badge

https://gerrit.wikimedia.org/r/672778

Change 672780 had a related patch set uploaded (by Tonina Zhelyazkova; owner: Tonina Zhelyazkova):
[mediawiki/extensions/Wikibase@master] Replace travis_branch with git_branch in install.sh

https://gerrit.wikimedia.org/r/672780

Right now we only run github secondary CI on master, which I think is likely fine
https://github.com/wikimedia/Wikibase/blob/master/.github/workflows/secondaryCI.yml#L4
Thus one of the referenced in the scripts checking the branch could likley be removed?

I went ahead and replaced the Travis references with the branch name from the github environment variables.
LMK in the patch if it's better to remove it all together.

Change 672780 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Remove travis_branch from install.sh

https://gerrit.wikimedia.org/r/672780

Change 672778 merged by jenkins-bot:
[mediawiki/extensions/Wikibase@master] Replace Travis CI badge with GitHub Actions badge

https://gerrit.wikimedia.org/r/672778