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

Introduce tags for Inria GitLab CI. #3181

Merged
merged 1 commit into from
Oct 16, 2024
Merged

Introduce tags for Inria GitLab CI. #3181

merged 1 commit into from
Oct 16, 2024

Conversation

Zimmi48
Copy link
Member

@Zimmi48 Zimmi48 commented Oct 16, 2024

Following an issue with having hit our quotas on GitLab.com, and by request of @mattam82 in https://coq.zulipchat.com/#narrow/stream/237656-Coq-devs-.26-plugin-devs/topic/GitLab.2Ecom.20subscription.20.2F.20compute.20minutes.

This PR introduces the tags that are needed to run jobs on Inria GitLab CI (using the shared runners there).

This PR should be merged just after coq/bot#324 is merged and deployed.

@Zimmi48
Copy link
Member Author

Zimmi48 commented Oct 16, 2024

There's a pipeline for this branch at https://gitlab.inria.fr/coq/opam/-/pipelines/1049952.

@Zimmi48
Copy link
Member Author

Zimmi48 commented Oct 16, 2024

Actually, testing on a branch is not a working solution. I should test-deploy the bot PR so that this is tested in normal PR conditions.

@Zimmi48
Copy link
Member Author

Zimmi48 commented Oct 16, 2024

Actually, testing on a branch is not a working solution. I should test-deploy the bot PR so that this is tested in normal PR conditions.

I thought this was the problem, but I misinterpreted the error in the log. I got the same error when testing this in "normal PR conditions":

$ scripts/opam-coq-list-pr-files | xargs scripts/opam-coq-lint
PR head is: 3b80d02d8a40a562f3553c447b53347462c0d220
fatal: Not a valid object name origin/master

See https://gitlab.inria.fr/coq/opam/-/jobs/4842732

@Zimmi48
Copy link
Member Author

Zimmi48 commented Oct 16, 2024

@gares I'm afraid I'll need help to debug this part.

@Zimmi48
Copy link
Member Author

Zimmi48 commented Oct 16, 2024

Going to merge as suggested by @palmskog on Zulip.

@Zimmi48 Zimmi48 merged commit c472754 into master Oct 16, 2024
4 of 9 checks passed
@Zimmi48 Zimmi48 deleted the test-tags branch October 16, 2024 14:03
Zimmi48 added a commit to coq/bot that referenced this pull request Oct 16, 2024
Following an issue with having hit our quotas on GitLab.com, and by
request of @mattam82 in
https://coq.zulipchat.com/#narrow/stream/237656-Coq-devs-.26-plugin-devs/topic/GitLab.2Ecom.20subscription.20.2F.20compute.20minutes.

This PR does the following things:

- [x] Set up mirroring.
- [x] Modify configuration for synchronizing PRs.

We should also:

- [x] Create the Inria GitLab repository.
- [x] Adjust the GitLab CI configuration to use the shared runners
available there. (Done in coq/opam#3181.)
- [x] Add a webhook on the Inria GitLab repository.
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.

1 participant