-
Notifications
You must be signed in to change notification settings - Fork 158
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
User Resource Ubuntu update #3893
Changes from 23 commits
2716749
c92494e
7aa64ba
33d14be
30ecfa2
072cfdc
19316a1
18df98c
c7ebc5b
9c2211b
d1beb03
1ea1159
271c7e0
a09af34
bc414b7
af50b39
d4a57a3
d75e442
3b7522e
3a9eecb
9fc272b
875f2f4
a3d9109
3a30d23
be7212b
8ff7f69
7db0e5f
7a43e65
ce4dc82
1049390
c08a2b7
e06dbcd
39aa284
aa4b948
9af4f05
5e2cdb5
d8fa5d4
b17bfab
85cdb98
f3b4efb
e8ba5fd
e193943
4a2036b
9a28c42
b1bb5c9
c5ce11b
86909a3
81801f8
ba9ae50
eff7050
cd1a4db
2082559
aa4713c
d5a4045
f9d9ffc
a3e5492
6980ecb
4354ca0
770e4c7
7a4d250
943e07b
760f783
c4e7094
060b6fd
f1d68da
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
--- | ||
schemaVersion: 1.0.0 | ||
name: tre-service-guacamole-linuxvm | ||
version: 0.6.9 | ||
version: 0.6.17 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am not sure if semantic versioning is used for these bundles, but it feels like this should be at least a minor version increase, if not a major. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. agreed will change then to 1.0.0 when I've finished testing |
||
description: "An Azure TRE User Resource Template for Guacamole (Linux)" | ||
dockerfile: Dockerfile.tmpl | ||
registry: azuretre | ||
|
@@ -14,29 +14,39 @@ custom: | |
"8 CPU | 32GB RAM": Standard_D8s_v5 | ||
"16 CPU | 64GB RAM": Standard_D16s_v5 | ||
image_options: | ||
"Ubuntu 18.04": | ||
"Ubuntu 22.04 LTS": | ||
source_image_reference: | ||
publisher: canonical | ||
offer: ubuntuserver | ||
sku: 18_04-lts-gen2 | ||
offer: 0001-com-ubuntu-server-jammy | ||
sku: 22_04-lts-gen2 | ||
version: latest | ||
apt-sku: 22.04 | ||
install_ui: true | ||
conda_config: false | ||
"Ubuntu 18.04 Data Science VM": | ||
"Ubuntu 20.04 LTS": | ||
source_image_reference: | ||
publisher: canonical | ||
offer: 0001-com-ubuntu-server-focal | ||
sku: 20_04-lts-gen2 | ||
version: latest | ||
apt-sku: 20.04 | ||
install_ui: true | ||
conda_config: false | ||
"Ubuntu 20.04 LTS Data Science VM": | ||
source_image_reference: | ||
publisher: microsoft-dsvm | ||
offer: ubuntu-1804 | ||
sku: 1804-gen2 | ||
offer: ubuntu-2004 | ||
sku: 2004-gen2 | ||
version: latest | ||
install_ui: false | ||
apt-sku: 20.04 | ||
install_ui: true | ||
conda_config: true | ||
# For information on using custom images, see README.me in the guacamole/user-resources folder | ||
# "Custom Image From Gallery": | ||
# source_image_name: your-image | ||
# install_ui: true | ||
# conda_config: true | ||
|
||
|
||
credentials: | ||
- name: azure_tenant_id | ||
env: ARM_TENANT_ID | ||
|
@@ -91,7 +101,7 @@ parameters: | |
default: "public" | ||
- name: os_image | ||
type: string | ||
default: "Ubuntu 18.04 Data Science VM" | ||
default: "Ubuntu 22.04 LTS Data Science VM" | ||
- name: vm_size | ||
type: string | ||
default: "2 CPU | 8GB RAM" | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In my experience, this file required additional changes for the Ubuntu 20 DSVM to work correctly. Specifically, the line to set the xcfe session config file had to be executed in the context of the user that will be logging in. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. interesting. so i wonder if running something like this from within the bootstrap will work su - username -c "command;command;command" There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Indeed, it does. This is what my line of code looks like (I am not a Linux expert, took my several tries to get it right. There might be a better way): There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also, noting that I had to do this even for the DSVM marketplace image where the UI is already installed. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @SvenAelterman could you possibly share your bootstrap script please? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, but be aware I modified it only for the DSVM Ubuntu 20 image. I did not get around to updating for plain Ubuntu 20/22. |
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.
What's the reason for not using
gen2
?