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 workspaces UI to templates #41

Open
wants to merge 25 commits into
base: develop
Choose a base branch
from

Conversation

Aopanis
Copy link
Collaborator

@Aopanis Aopanis commented Aug 19, 2024

Description

Add new workspaces ui to azure deployment templates. Additionally reworked the unit files to use - prefix instead of || true pattern (- prefix is the unit file way of specifying acceptable errors), and reformatted the cloud-init section. Rebased on Jack's changes and changed the version to specific versions.

Testing

Testing was done by deploying all the templates on azure.

all-in-one-without-lb

Screenshot 2024-08-16 at 12 11 53 PM

all-in-one-nlb

Screenshot 2024-08-16 at 12 08 28 PM

ai-unlimited-without-lb

Screenshot 2024-08-16 at 12 03 43 PM

ai-unlimited-nlb

Screenshot 2024-08-16 at 12 02 18 PM

Copy link

@johandry johandry left a comment

Choose a reason for hiding this comment

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

The variable AiUnlimitedHttpPort should be renamed to AiUnlimitedAuthPort also the description to port to access the AI Unlimited authentication service. This port is not anymore for the HTTP or UI, it is for the Authentication Service

@johandry
Copy link

johandry commented Aug 23, 2024

If you have or need a health check for the UI, you can use "http://UI URL/healtcheck"

There is a typo, that I will change in a PR this week, should be /healthcheck 😄

cloud-work535 and others added 5 commits August 26, 2024 15:52
deleted alb templates from aws deployments
* Added public-key deployment script, added private keys to keyvault, removed public ssh firewall and output strings, hardcoded/updated version strings, and added secrets write permission to resource declaration.

* Missed hardcoded version for jupyter.

---------

Co-authored-by: Anton Panis <anton.panis@teradata.com>
cloud-work535 and others added 2 commits October 18, 2024 15:57
added permissions to support static ip
…changed the environment file location to reflect this. Additionally updated UI version to v0.0.5. (#48)

Co-authored-by: Anton Panis <anton.panis@teradata.com>
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.

7 participants