-
Notifications
You must be signed in to change notification settings - Fork 4
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
[EDU-474] Studios doc audit #407
base: master
Are you sure you want to change the base?
Conversation
Fixes https://seqera.atlassian.net/browse/EDU-474 Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
I've merged #404 If you rebase this or merge in master (update branch 🙃 ), the Vale warnings for Xpra ect. will cease. |
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
…/docs into justinegeffen-edu-474
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.
Some updates and discussion points.
cc @robnewman, @deepcshah
platform_versioned_docs/version-24.3/troubleshooting_and_faqs/studios_troubleshooting.mdx
Outdated
Show resolved
Hide resolved
platform_versioned_docs/version-24.3/troubleshooting_and_faqs/studios_troubleshooting.mdx
Outdated
Show resolved
Hide resolved
platform_versioned_docs/version-24.3/troubleshooting_and_faqs/studios_troubleshooting.mdx
Show resolved
Hide resolved
platform_versioned_docs/version-24.3/troubleshooting_and_faqs/studios_troubleshooting.mdx
Outdated
Show resolved
Hide resolved
platform_versioned_docs/version-24.3/troubleshooting_and_faqs/studios_troubleshooting.mdx
Outdated
Show resolved
Hide resolved
|
||
``` | ||
kubectl apply -f secret.yml | ||
``` | ||
|
||
1. Edit the `tower-svc.yml` file and uncomment the `volumes.cert-volume`, `volumeMounts.cert-volume`, and `env.TOWER_OIDC_PEM_PATH` fields so that the public/private key pair is available to Platform. | ||
1. Edit the `tower-src.yml` file and uncomment the `volumes.cert-volume`, `volumeMounts.cert-volume`, and `env.TOWER_OIDC_PEM_PATH` fields so that the public/private key pair is available to Platform. |
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.
@endre-seqera, this relates to https://seqera.atlassian.net/browse/EDU-491. Please could you review and confirm it's the correct fix?
platform_versioned_docs/version-24.3/getting-started/proteinfold.mdx
Outdated
Show resolved
Hide resolved
platform_versioned_docs/version-24.3/troubleshooting_and_faqs/studios_troubleshooting.mdx
Outdated
Show resolved
Hide resolved
platform_versioned_docs/version-24.3/troubleshooting_and_faqs/studios_troubleshooting.mdx
Outdated
Show resolved
Hide resolved
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.
Couple of points, but almost there!
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Co-authored-by: Rob Newman <61608+robnewman@users.noreply.github.com> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
…studios_troubleshooting.mdx Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Merge branch 'master' into justinegeffen-edu-474
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.
Super close - some very minor changes needed.
@@ -5,7 +5,7 @@ date: "12 Jun 2024" | |||
tags: [docker, compose, kubernetes, data, studios, deployment] | |||
--- | |||
|
|||
Enable Studios as part of your Seqera Platform instance. You must have Data Explorer enabled to use Data Studios. Only the Amazon AWS public cloud is currently supported. | |||
Enable Studios as part of your Seqera Platform instance. You must have Data Explorer enabled to use Studios. Only the Amazon AWS public cloud is currently supported. | |||
|
|||
:::caution | |||
You must complete the upgrade of your Platform Enterprise installation to version 24.1.4 before enabling and configuring Data Studios. |
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 must complete the upgrade of your Platform Enterprise installation to version 24.1.4 before enabling and configuring Data Studios. | |
You must complete the upgrade of your Platform Enterprise installation to version 24.2 before enabling and configuring Studios. |
Let's start requiring to upgrade to v24.2 so we start collecting telemetry data.
- **stopped**: When a session is stopped, the associated compute resources are deallocated. You can start or delete the data studio when it's in this state. | ||
- **errored**: This state most often indicates that there has been an error starting the data studio session but it is in a **stopped** state. There might be errors reported by the session itself but these will be overwritten with a **running** status if the data studio session is still running. | ||
- **stopped**: When a session is stopped, the associated compute resources are deallocated. You can start or delete the Studio when it's in this state. | ||
- **errored**: This state most often indicates that there has been an error starting the Studio session but it is in a **stopped** state. There might be errors reported by the session itself but these will be overwritten with a **running** status if the session is still running. |
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.
@robnewman, this doesn't seem correct.
If there's an error that results in a stopped state, what relevance does the second sentence have? Should it be a separate point?
Yes - perhaps move to an admonishment.
|
||
:::note | ||
By default, data studio sessions only have read permissions to mounted data paths. Write permissions can be added for specific cloud storage buckets during the compute environment configuration by defining additional **Allowed S3 Buckets**. This means that data can be written from the session back to the cloud storage path(s) mounted. To stop potential data loss, only one session per workspace can mount a unique data path. When adding a new session, data paths already mounted to other running sessions are unavailable. If a new file is uploaded to the cloud storage bucket path while a session is running, the file may not be available to the session immediately. | ||
By default, sessions only have read permissions to mounted data paths. Write permissions can be added for specific cloud storage buckets during the compute environment configuration by defining additional **Allowed S3 Buckets**. This means that data can be written from the session back to the cloud storage path(s) mounted. To stop potential data loss, only one session per workspace can mount a unique data path. When adding a new session, data paths already mounted to other running sessions are unavailable. If a new file is uploaded to the cloud storage bucket path while a session is running, the file may not be available to the session immediately. |
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.
Ah yes. This is updated. Change to:
By default, sessions only have read permissions to mounted data paths. Write permissions can be added
for specific cloud storage buckets during the compute environment configuration by defining additional
**Allowed S3 Buckets**. This means that data can be written from the session back to the cloud storage
path(s) mounted. If a new file is uploaded to the cloud storage bucket path while a session is running, the
file may not be available to the session immediately.
|
||
:::warning | ||
An active connection to a session will not prevent administrative actions that might disrupt that connection. For example, a session can be stopped by another workspace user while you are active in the session, the underlying credentials can be changed, or the compute environment can be deleted. These are independent actions and the user in the session won't be alerted to any changes - the only alert will be a server connection error in the active session browser tab. | ||
::: | ||
|
||
Once connected, the data studio session will display the status of **running** in the list, and any connected user's avatar will be displayed under the status in both the list of data studios and in each data studio session detail page. | ||
Once connected, the session will display the status of **running** in the list, and any connected user's avatar will be displayed under the status in both the list of Studios and in each Studio's detail page. |
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.
Based on the most recent discussion, it's a list of Studios
and it's each Studio's detail page
.
Fixes https://seqera.atlassian.net/browse/EDU-474 and https://seqera.atlassian.net/browse/EDU-491
Rolled into this: