Skip to content

Follow redirects when truncating/deleting - #15722

Merged
ktf merged 1 commit into
AliceO2Group:devfrom
mrtineide:dev
Aug 26, 2026
Merged

Follow redirects when truncating/deleting#15722
ktf merged 1 commit into
AliceO2Group:devfrom
mrtineide:dev

Conversation

@mrtineide

Copy link
Copy Markdown
Contributor

The config on the ccdb-test.cern.ch changed.
Now the truncate endpoint and the DELETE is HTTPS. Trying with HTTP will now redirect to HTTPS with the Location header, which needs to be followed.

My local CCDB ctest is no longer failing.

The config on the ccdb-test.cern.ch changed.
Now the truncate endpoint and the DELETE is HTTPS.
Trying with HTTP will now redirect to HTTPS with the
Location header, which needs to be followed.
@mrtineide
mrtineide requested review from a team, Barthelemy, costing and sawenzel as code owners August 25, 2026 21:19
@sawenzel

Copy link
Copy Markdown
Collaborator

The unit test CCDB/test/testCcdbApi.cxx is failing on AARCH64. Is it related?

@mrtineide

Copy link
Copy Markdown
Contributor Author

The unit test CCDB/test/testCcdbApi.cxx is failing on AARCH64. Is it related?

Yes it is related, that is the test that was failing before on my machine. The changes should have fixed it.
I run the unit test locally again and it is passing on my machine. Without my changes, when I run the CCDB unit test locally (again just now) they fail the same way as AARCH64.
So I am no sure why AARCH64 would fail.

@ktf

ktf commented Aug 26, 2026

Copy link
Copy Markdown
Member

I suspect the issue is some expired / missing certificate on the arm builder. Before the fix of yesterday on the CCDB server side, the tests were nevertheless passing when unauthenticated. Now they do not anymore. Merging this and investigating the remaining issue separately.

@ktf
ktf merged commit 7fb2640 into AliceO2Group:dev Aug 26, 2026
10 of 11 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.

4 participants