-
Notifications
You must be signed in to change notification settings - Fork 923
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 section on high availability setups to Galaxy Interactive Tools training #5179
base: main
Are you sure you want to change the base?
Add section on high availability setups to Galaxy Interactive Tools training #5179
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall, some small changes needed and then happy to merge this! Thank you for documenting it!
> > <code-in-title>SQL</code-in-title> | ||
> > ```sql | ||
> > ALTER DATABASE gxitproxy OWNER TO galaxy; | ||
> > ``` | ||
> > {: data-cmd="true"} | ||
> {: .code-in} | ||
> | ||
> > <code-out-title>SQL</code-out-title> | ||
> > ``` | ||
> > ALTER DATABASE | ||
> > ``` | ||
> {: .code-out} | ||
> | ||
> 4. Sign out of the `postgres` database using `exit`. Then connect to the `gxitproxy` database as `galaxy`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you could replace steps 1+2+3 with the single CLI command: sudo -u postgres createdb -O galaxy gxitproxy
, which would avoid students potentially missing the "logout and log in again" steps.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, that makes the hands-on significantly shorter too.
Addressed in 72e272f.
…6 of HA setup section hands-on of the Galaxy Interactive Tools training Co-authored-by: Helena <[email protected]>
…p section of the Galaxy Interactive Tools training
@hexylena Upstream changes have been merged, the training can be merged now. |
- natefoo | ||
- slugger70 | ||
- hexylena | ||
- abretaud | ||
- kysrpex |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- natefoo | |
- slugger70 | |
- hexylena | |
- abretaud | |
- kysrpex | |
authorship: | |
- natefoo | |
- slugger70 | |
- hexylena | |
- abretaud | |
- kysrpex |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Committed fa2d197. Does the key contributors
work exactly the same as contributions
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks @kysrpex.
I'm just not sure if we should merge it now or if we need to wait for a proper Galaxy release.
Sorry for the previous (deleted) comment, my mind was somewhere else 😅 That's ok, after all if we merge now, it would be misleading, as it would not work |
Add an optional section on configuring Galaxy Interactive Tools in high availability Galaxy setups using PostgreSQL. Consider merging #5178 together with this one.
Related reads, issues and PRs (for context):
Please keep this PR as a draft until the PRs mentioned above are merged AND a new version of Galaxy is released (the latest is 24.1 at the moment of writing).