Skip to content

(Hopefully) fix Gitlab timeout errors#2050

Open
StephaneTrebel wants to merge 8 commits intomainfrom
moar-gitlab-logs
Open

(Hopefully) fix Gitlab timeout errors#2050
StephaneTrebel wants to merge 8 commits intomainfrom
moar-gitlab-logs

Conversation

@StephaneTrebel
Copy link
Copy Markdown
Collaborator

🤞

@StephaneTrebel StephaneTrebel self-assigned this Apr 2, 2026
@StephaneTrebel StephaneTrebel added this to the MCO milestone Apr 2, 2026
@github-actions github-actions bot added the built label Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

🤖 Hey !

The security scan report for the current pull request is available here.

@StephaneTrebel StephaneTrebel added the preview Deploy preview app with Argo-cd label Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

🤖 Hey !

A preview of the application is available at : https://console-pr-2050.dso.cpin-hp.numerique-interieur.fr

Please be patient, deployment may take a few minutes.

@StephaneTrebel StephaneTrebel force-pushed the moar-gitlab-logs branch 3 times, most recently from c1a9548 to 420ffb3 Compare April 3, 2026 12:16
@StephaneTrebel
Copy link
Copy Markdown
Collaborator Author

@shikanime On ne va ptet pas fusionner ça comme ça, mais je veux bien ton avis

console.log(`[GITLAB] addGroupMember ${userId} ${accessLevel}`)
const group = await this.getOrCreateProjectGroup()
return this.api.GroupMembers.add(group.id, userId, accessLevel)
return this.api.GroupMembers.add(group.id, userId, accessLevel as any)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I witness, je reconnais bien l'erreur, c'est un soucis avec db:generate ?

Copy link
Copy Markdown
Member

@shikanime shikanime left a comment

Choose a reason for hiding this comment

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

En theorie je vois pas ce que ca changerais. Apres, Promise.all a certaine regles de scheduling qui fait que ce n'est pas forcement un "all" en meme temps mais est fait en batch.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

C'est vrai que tres moche les () ?? (), p'tet faire des p'tit variable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

built preview Deploy preview app with Argo-cd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants