Skip to content

Conversation

@pippohub
Copy link
Contributor

@pippohub pippohub commented Jul 9, 2025

I have updated the readout cfg-uri to access the config files in CONSUL

@pippohub pippohub requested a review from knopers8 July 9, 2025 13:16
readout_cfg_uri_standalone: "consul-ini://{{ consul_endpoint }}/o2/components/{{config.ResolvePath('readout/' + run_type + '/any/readout-standalone-' + it)}}"
readout_cfg_uri_stfb: "consul-ini://{{ consul_endpoint }}/o2/components/{{config.Resolve('readout', run_type, 'any', 'readout-stfb-' + it)}}"
readout_cfg_uri_standalone: "consul-ini://{{ consul_endpoint }}/o2/components/{{config.Resolve('readout', run_type, 'standalone', 'readout-standalone-' + it)}}"
readout_cfg_uri_stfb: "consul-ini://{{ consul_endpoint }}/o2/components/{{config.Resolve('readout', run_type, readout_folder, 'readout-stfb-' + it)}}"
Copy link
Collaborator

Choose a reason for hiding this comment

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

could you add some correct and backwards-compatible default for readout_folder around here? Otherwise, the template might stop working for setups where we don't use readout_folder, e.g. openstack VMs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ciao @knopers8 I did some tests and if I am not mistaken:

  • if the folders do not exists in CONSUL
  • if the variable is not set
    it goes back to any

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the path of readout if I don't set anything

consul-ini://localhost:8500/o2/components/readout/ANY/any

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the only part where it fails if I don't set anything is the standalone cfg ... as the standalone path is hardcoded.

But then I am wondering if we use it at all or if we always run with DD enabled.

If we use standalone meaning readout without DD, I can put some protection

Copy link
Collaborator

Choose a reason for hiding this comment

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

You're right, it should indeed go back to any.

It's not very important, but I would propose to anyway list it in defaults, so it is clear what are the variables that the workflow template expects.

Copy link
Collaborator

Choose a reason for hiding this comment

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

But then I am wondering if we use it at all or if we always run with DD enabled.

I do run readout alone sometimes on my VM.

@knopers8 knopers8 merged commit bf6a04d into AliceO2Group:master Jul 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants