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

fix: gh built artifact names #512

Merged
merged 1 commit into from
Nov 21, 2023
Merged

fix: gh built artifact names #512

merged 1 commit into from
Nov 21, 2023

Conversation

rti
Copy link
Contributor

@rti rti commented Nov 10, 2023

With the merge of 7686b1a built artifact names changed. This PR fixes some docs and scripts that still used to old names.

should probably be backported to mw-1.39 and mw-1.40

@rti rti requested a review from a team November 10, 2023 08:05
@rti rti mentioned this pull request Nov 10, 2023
@darthmon darthmon requested a review from datn November 20, 2023 10:58
@RickiJay-WMDE
Copy link
Member

LGTM, but I'm not sure whether to approve or wait for @datn?

Copy link
Member

@datn datn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@lorenjohnson lorenjohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this naming will help conversations from getting stuck on "tarballs" so much :) Approved.

@rti rti merged commit afccbfa into main Nov 21, 2023
26 checks passed
@rti rti deleted the fix-artifacts-name branch November 21, 2023 10:07
rti added a commit that referenced this pull request Nov 21, 2023
rti added a commit that referenced this pull request Nov 21, 2023
rti added a commit that referenced this pull request Nov 27, 2023
rti added a commit that referenced this pull request Nov 27, 2023
RickiJay-WMDE pushed a commit that referenced this pull request Nov 29, 2023
rti added a commit that referenced this pull request May 16, 2024
* Set variables.env to wmde.13 values

* Test mw-* branches (#503) (#504)

* fix(github-actions): GHCR upload (#505) (#509)

* fix(github-actions): GITHUB_TOKEN access for GHCR upload

* fix(github-actions): docker image name

* chore: move if to job level

* fix: get built docker artifacts in ghcr upload job

* fix(github-actions): ghcr image name

* chore: run upload ghcr only on main and mw-?.?? branches

* fix: simplify job run check, we do not have regex here :(

* feat: upload to GHCR from prep-mw- branches as well (#514) (#516)

* fix(github-actions-workflow): ghcr upload base_ref check (#518) (#521)

* fix: gh built artifact names (#512) (#527)

* chore: update variables.env to new format

use commit hashes from WMDE.13 release CI build
https://phab.wmfusercontent.org/file/download/67e7x2izldkdqmqexnwe/PHID-FILE-atrxuifwzod3ur25xju5/BuildMetadata.zip

* fix: update test the correct versions in CI

* Release 1.39.6 (#568)

* Manually backporting formatting changes

* Revert "Releaseprep wmde.17 (#607)"

This reverts commit 090be85.

* Delete build_and_test

* Fix image urls

* Lint

* Lint

* Revert cont.

* Fix upgrade path

* Release prep

* Final release version

* Set release date

* Update example (#668)

---------

Co-authored-by: Ricki Jay <[email protected]>
Co-authored-by: Ricki <[email protected]>
Co-authored-by: adeeri <[email protected]>
Co-authored-by: adeeri <[email protected]>
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.

4 participants