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

update tool-use-by-group query #153

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

cat-bro
Copy link
Contributor

@cat-bro cat-bro commented Apr 4, 2024

This fails with a syntax error

cat@galaxy:~$ gxadmin query tool-use-by-group 2022-02-01 --group=QUT_users
ERROR:  invalid input syntax for type timestamp: "-01"
LINE 12: date_trunc('month', job.create_time) = '-01'

This looks like a typo with year_month vs years_month

@cat-bro cat-bro marked this pull request as draft April 4, 2024 11:18
@cat-bro
Copy link
Contributor Author

cat-bro commented Apr 4, 2024

making this a draft because I'm not sure if it uses the --group argument in the query

Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

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

yes you're correct arg_group is not present.

Based on the help text group looks like it should be mandatory as well, since it's not included in the output.

I can't make the suggestion but if you could add a

AND galaxy_group.name = '$arg_group'

that should help

parts/22-query.sh Show resolved Hide resolved
parts/22-query.sh Outdated Show resolved Hide resolved
cat-bro and others added 4 commits April 5, 2024 12:24
Co-authored-by: Helena <hexylena@galaxians.org>
Co-authored-by: Helena <hexylena@galaxians.org>
@cat-bro cat-bro marked this pull request as ready for review April 5, 2024 03:13
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