[WIP] CCDB api import from QC (O2-347)#1321
Conversation
b81b9df to
9041309
Compare
|
This is a PR to move the CCDB api with curl that I wrote for the QC to O2. This is provided as is. I made it compliant with Coding Standard and provided a set of functional tests that rely on ccdb-test.cern.ch:8080. |
| * @param startValidityTimestamp Start of validity. If omitted, current timestamp is used. | ||
| * @param endValidityTimestamp End of validity. If omitted, current timestamp + 1 year is used. | ||
| */ | ||
| void store(TObject* rootObject, std::string path, std::map<std::string, std::string> metadata, |
There was a problem hiding this comment.
I suggest to return the newly created object ID / path from the store() method
There was a problem hiding this comment.
I agree on the idea but fail to implement it. I tried to use
std::string requestOutput;
curl_easy_setopt(curl, CURLOPT_WRITEFUNCTION, CurlWrite_CallbackFunc_StdString);
curl_easy_setopt(curl, CURLOPT_WRITEDATA, &requestOutput);
It works in the method list() but here it does not. Probably because of the multipart or because of the redirection.
I propose to keep it as a future improvement if you don't mind.
|
@dberzano : Seems like the macos CI is down. |
|
Yes, I know, it's restarting; an update was installed by the machine owner and the node was rebooted. |
|
@dberzano When I click to see the details of the macos failure, I get a 404. |
|
I have to work on the macos issue, not sure what is going on. |
Co-authored-by: Alessandro Balbino <alessandro.balbino@cern.ch>
No description provided.