Skip to content

MFT: Update the QCDB url from test to production.#796

Merged
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
tomas-herman:mft-qcdb-url-change
Aug 11, 2021
Merged

MFT: Update the QCDB url from test to production.#796
Barthelemy merged 1 commit into
AliceO2Group:masterfrom
tomas-herman:mft-qcdb-url-change

Conversation

@tomas-herman
Copy link
Copy Markdown
Collaborator

No description provided.

@tomas-herman tomas-herman changed the title MFT - Update the QCDB url from test to production. MFT: Update the QCDB url from test to production. Aug 5, 2021
@Barthelemy Barthelemy self-requested a review August 10, 2021 08:55
@Barthelemy
Copy link
Copy Markdown
Collaborator

are you sure that you want to point to production database from a config file that will most likely be used outside P2 ? I am not sure we want to have all kind of test objects in the QCDB.

@tomas-herman
Copy link
Copy Markdown
Collaborator Author

Hi Barth, we have started to test the MFT cluster QC task on EPN and from our understanding the json file used for this should be the one present in O2 and therefore it must contain the path to the production database. And because we are not allowed to make changes to existing builds of O2 on the EPN we are using a local copy of the json file located on the EPN at the moment.

I completely agree that for tests outside P2 we don't want to fill the production QCDB and the plan for that is that we would by hand change the URL in our local builds were we are allowed to make changes.

Please correct me if what I said was wrong or if there is a better solution to our situations.

@Barthelemy
Copy link
Copy Markdown
Collaborator

it all makes sense, thank you. Merging now.

@Barthelemy Barthelemy merged commit 5d08c93 into AliceO2Group:master Aug 11, 2021
@tomas-herman tomas-herman deleted the mft-qcdb-url-change branch August 17, 2021 12:50
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.

2 participants