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

Add mutate queries to fix missing usernames #145

Merged
merged 4 commits into from
Jan 22, 2024

Conversation

mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Jan 20, 2024

Either by setting the username to the lowercase first portion of the email address (which excludes any conflicts), or by setting the username to a uuid (you can use this for confictls).

Either by setting the username to the lowercase first portion of the
email address (which excludes any conflicts), or by setting the username
to a uuid (you can use this for confictls).
@mvdbeek
Copy link
Member Author

mvdbeek commented Jan 20, 2024

I've tested the queries individually, but the release script fails with

bash: -c: line 0: syntax error near unexpected token `('
bash: -c: line 0: `./.tmpgxadmin cluster slurm-stats() { ## --help'
config
bash: -c: line 0: syntax error near unexpected token `('
bash: -c: line 0: `./.tmpgxadmin config dump() { ## --help'
bash: -c: line 0: syntax error near unexpected token `('
bash: -c: line 0: `./.tmpgxadmin config validate() { ## --help'

so haven't pursued this any further.

@mvdbeek mvdbeek requested review from natefoo and hexylena January 20, 2024 09:47
@hexylena
Copy link
Member

Thanks @mvdbeek I'll look into this.

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, looks OK.

@hexylena
Copy link
Member

A new release doesn't need to be issued to get a new version, a github pre-release revision will be released automatically once this is merged that can be installed easily (also via ansible)

@hexylena hexylena merged commit 98c86ed into galaxyproject:main Jan 22, 2024
1 check passed
@hexylena
Copy link
Member

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