Skip to content

Conversation

@jgrosseo
Copy link
Collaborator

No description provided.

@jgrosseo jgrosseo requested a review from a team as a code owner January 12, 2021 14:45
@jgrosseo jgrosseo requested a review from ktf January 12, 2021 14:46
@jgrosseo
Copy link
Collaborator Author

@ktf Here the first part (if the metric would be saved)

if (alienFile) {
/// FIXME: get the JAlien stats
monitoringInfo += ",";
monitoringInfo += alienFile->GetSE();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if the problem with the metrics is not that GetSE() returns something which the monitoring backend does not understand as a properly formatted string? I will check.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At least the line which appears with LOGP looks proper: E.g. "alien:///alice/data/2015/LHC15o/000244918/pass5_lowIR/PWGZZ/Run3_Conversion/96_20201013-1346_child_1/0028/AO2D.root:/,631838549,ALICE::CERN::EOS"

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you don't mind, merge this, we can parse the log output if we don't get the logging up

@ktf ktf merged commit b56fc6b into AliceO2Group:dev Jan 12, 2021
@jgrosseo jgrosseo deleted the alien_se branch January 28, 2021 12:08
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