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

[EDU-474] Studios doc audit #407

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

Conversation

justinegeffen
Copy link
Contributor

@justinegeffen justinegeffen commented Jan 29, 2025

Fixes https://seqera.atlassian.net/browse/EDU-474 and https://seqera.atlassian.net/browse/EDU-491

Rolled into this:

  • Renaming of Data Studios to Studios

@jason-seqera
Copy link
Contributor

I've merged #404

If you rebase this or merge in master (update branch 🙃 ), the Vale warnings for Xpra ect. will cease.

@seqeralabs seqeralabs deleted a comment from netlify bot Feb 17, 2025
@justinegeffen justinegeffen marked this pull request as ready for review February 17, 2025 21:55
@justinegeffen justinegeffen added 1. Editor review Needs a language review 1. Dev/PM/SME Needs a review by a Dev/PM/SME 2. UX review Needs a review by the UX team labels Feb 19, 2025
Copy link
Contributor Author

@justinegeffen justinegeffen left a 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


```
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.
Copy link
Contributor Author

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?

Copy link
Member

@robnewman robnewman left a 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!

justinegeffen and others added 4 commits February 25, 2025 21:24
Co-authored-by: Chris Hakkaart <[email protected]>
Co-authored-by: Rob Newman <[email protected]>
Signed-off-by: Justine Geffen <[email protected]>
…studios_troubleshooting.mdx

Signed-off-by: Justine Geffen <[email protected]>
Merge branch 'master' into justinegeffen-edu-474
@justinegeffen justinegeffen changed the title [EDU-474] Data Studios doc audit [EDU-474] Studios doc audit Feb 25, 2025
Copy link
Member

@robnewman robnewman left a 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.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
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.
Copy link
Member

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.
Copy link
Member

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.
Copy link
Member

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.

Reference: https://seqera.slack.com/archives/C040MFU06AE/p1740407438484439?thread_ts=1739987472.109469&cid=C040MFU06AE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. Dev/PM/SME Needs a review by a Dev/PM/SME 1. Editor review Needs a language review 2. UX review Needs a review by the UX team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants