8000 create worktree before adding packages by gaelgatelement · Pull Request #336 · helm/chart-releaser · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

create worktree before adding packages #336

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

create worktree before adding packages #336

wants to merge 2 commits into from

Conversation

gaelgatelement
Copy link
@gaelgatelement gaelgatelement commented Sep 19, 2023 •
edited
Loading

Current behaviour is causing a race condition which prevent it from pushing multiple 8000 artifacts :
https://github.com/vector-im/ess-starter-edition-core/actions/runs/6237382710/job/16930869688#step:8:94

@gaelgatelement gaelgatelement force-pushed the gaelg/fix-detached-heads branch from 98575dc to c612f1e Compare September 19, 2023 15:41
@gaelgatelement gaelgatelement changed the title create worktree before adding packages wip: create worktree before adding packages Sep 19, 2023
@gaelgatelement gaelgatelement force-pushed the gaelg/fix-detached-heads branch from 460dcb1 to dc494fd Compare September 19, 2023 16:07
@gaelgatelement gaelgatelement changed the title wip: create worktree before adding packages create worktree before adding packages Sep 19, 2023
@gaelgatelement
Copy link
Author
gaelgatelement commented Sep 19, 2023 •
edited
Loading

Tested locally on a test repository and it worked fine :

Préparation de l'arbre de travail (HEAD détachée f3c57cf)
HEAD est maintenant à f3c57cf init gh-pages
[HEAD détachée 7d28e9f] Publishing chart package for element-operator-2.11.10
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 element-operator-2.11.10.tgz
[HEAD détachée e6eba6a] Publishing chart package for element-updater-2.11.10
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 element-updater-2.11.10.tgz
Pushing to branch "gh-pages"
Énumération des objets: 7, fait.
Décompte des objets: 100% (7/7), fait.
Compression par delta en utilisant jusqu'à 16 fils d'exécution
Compression des objets: 100% (6/6), fait.
Écriture des objets: 100% (6/6), 31.58 Kio | 7.89 Mio/s, fait.
Total 6 (delta 1), réutilisés 0 (delta 0), réutilisés du pack 0
remote: Resolving deltas: 100% (1/1), done.
To <test remote>.git
   f3c57cf..e6eba6a  HEAD -> gh-pages

It should now be ready for review.

@gaelgatelement gaelgatelement mentioned this pull request Sep 20, 2023
Merged
@cpanato cpanato closed this Jun 2, 2025
@cpanato cpanato reopened this Jun 2, 2025
@cpanato
Copy link
Member
cpanato commented Jun 2, 2025

@gaelgatelement need a rebase

@gaelgatelement
create worktree before adding packages
3b8ed36 
Signed-off-by: Gaël Goinvic <gaelg@element.io>
@gaelgatelement
push after commiting all artifacts
2ed78ee 
Signed-off-by: Gaël Goinvic <gaelg@element.io>
@gaelgatelement gaelgatelement force-pushed the gaelg/fix-detached-heads branch from dc494fd to 2ed78ee Compare June 3, 2025 08:53
@pull-request-size pull-request-size bot added size/XS and removed size/S labels Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gaelgatelement @cpanato
0