Skip to content

Conversation

@mpuccio
Copy link
Contributor

@mpuccio mpuccio commented Dec 9, 2024

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 9, 2024

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1

@mpuccio mpuccio force-pushed the mp-branch-1 branch 2 times, most recently from 2711a53 to cea36b9 Compare December 9, 2024 09:04
@mconcas
Copy link
Collaborator

mconcas commented Dec 9, 2024

@singiamtel Hi! I see that the build/O2/fullCI_slc9 — Please resolve merge conflicts requires some conflicts management. I wonder if this is related to this PR or some inconsistent state of this PR.
I cannot acces the logs, so I cannot check.
I'd like to force merge this as the failing tests are not listed as required, but I want to be sure I don't mess up anything.

Could you please have a look to that? Thanks in advance.

@singiamtel
Copy link
Collaborator

@singiamtel Hi! I see that the build/O2/fullCI_slc9 — Please resolve merge conflicts requires some conflicts management. I wonder if this is related to this PR or some inconsistent state of this PR. I cannot acces the logs, so I cannot check. I'd like to force merge this as the failing tests are not listed as required, but I want to be sure I don't mess up anything.

Could you please have a look to that? Thanks in advance.

Ciao Matteo,

The fullCI_slc9 tester is currently not working properly, feel free to bypass it when merging

Cheers

@mpuccio mpuccio marked this pull request as draft December 10, 2024 08:58
@mpuccio mpuccio marked this pull request as ready for review December 11, 2024 13:24
@mpuccio
Copy link
Contributor Author

mpuccio commented Dec 11, 2024

@mconcas it does not destroy anything, it can go

@mconcas mconcas merged commit 1432c4b into AliceO2Group:dev Dec 11, 2024
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants