Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Print all Publish Containers in crab status command #5274

Open
novicecpp opened this issue Dec 14, 2023 · 3 comments
Open

Print all Publish Containers in crab status command #5274

novicecpp opened this issue Dec 14, 2023 · 3 comments
Assignees

Comments

@novicecpp
Copy link
Contributor

crab status command in latest version only print Publish Container that have the same name as tm_output_dataset.

Screenshot from 2023-12-14 17-26-33

Now that we have multiple Publish Containers, we need to display all of them.
The info to display is already there in taskdb tm_multipub_rule.

@novicecpp novicecpp self-assigned this Dec 14, 2023
@novicecpp
Copy link
Contributor Author

novicecpp commented Dec 14, 2023

My proposal is print them when supply the --long arg.

The new version of crab status without --long (notice the Publish container's rule is gone):
Screenshot from 2023-12-14 17-24-08

with --long, it will print Publish Container's name and its rule after Publication section:
Screenshot from 2023-12-14 17-23-47

@belforte What do you think?

@belforte
Copy link
Member

I'd rather add a new option than expect/ask users to use --long with task of >100 jobs.
But.. are we talking about one line for each output file ? I.e. mostly one at at most a handful ? Why not adding them all of the times ?
I agree that the crab status output is getting long. But suggest to keep looking for ideas.
Maybe add an option to list rules ? In the end rules on Publish container are not very useful, esp. if they have no expiration date. And one can simply to rucio rule-info <container>. Maybe one line about this ?

Anyhow, I would really really stay out of status --long

@novicecpp
Copy link
Contributor Author

Maybe add an option to list rules ? In the end rules on Publish container are not very useful, esp. if they have no expiration date. And one can simply to rucio rule-info . Maybe one line about this ?

I don't think so.
I always use the rule ID as the starting point when interact with the Rucio system, mainly because it has GUI that easy to navigate around. If it not enough, then I will go to the CLI like list-file-replicas or list-account-usage command.
It also tell the users which rule is created and managed by CRAB.

.

About the --long I kinda agree with you. I personally do not mind having the longer output, and as the reason above, I still want to show both the container name and rule ID.

Maybe a new arg like --show-containers is the best option where we can keep status output short, trade-off with a little extra effort required from users to provide additional arg. We can also add more information to the output like rule expiration time or lock status summary.

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

No branches or pull requests

2 participants