Skip to content

cli: allow specifying custom qcnl config when installing MarbleRun#851

Merged
daniel-weisse merged 4 commits intomasterfrom
dw/dcap-conf
Sep 26, 2025
Merged

cli: allow specifying custom qcnl config when installing MarbleRun#851
daniel-weisse merged 4 commits intomasterfrom
dw/dcap-conf

Conversation

@daniel-weisse
Copy link
Copy Markdown
Member

Proposed changes

  • Add a new flag to to marblerun install: --dcap-qcnl-config-file
    • Allows specifying the path to a qcnl config, which will be added to the Helm chart instead of using the default values in --dcap-pccs-url and --dcap-secure-cert
  • The Helm chart similarly received a new settable value: dcap.qcnlConfig
    • Allows defining a qcnl config inline as a string
    • Alternatively the value may be set by reading a file using Helm's --set-file flag:
      helm install -n marblerun ./charts --set-file dcap.qcnlConfig=/path/to/sgx_qcnl.conf
      
  • Update the default qcnl generated by the CLI and the Helm chart to use the newer JSON formatted config

Signed-off-by: Daniel Weiße <dw@edgeless.systems>
@daniel-weisse daniel-weisse added the feature This change introduces new functionality label Sep 22, 2025
@netlify
Copy link
Copy Markdown

netlify bot commented Sep 22, 2025

Deploy Preview for marblerun-docs canceled.

Name Link
🔨 Latest commit d878b2c
🔍 Latest deploy log https://app.netlify.com/projects/marblerun-docs/deploys/68d645d950b1330008e12aae

USE_SECURE_CERT={{ .Values.dcap.useSecureCert }}
{
"pccs_url": "{{ .Values.dcap.pccsUrl }}",
"use_secure_cert": "{{ .Values.dcap.useSecureCert }}"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think this must be a json bool, not, e.g., "TRUE".

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed
Also added a fallback for older CLI versions that still supply this value as a string

Signed-off-by: Daniel Weiße <dw@edgeless.systems>
Signed-off-by: Daniel Weiße <dw@edgeless.systems>
Signed-off-by: Daniel Weiße <dw@edgeless.systems>
@daniel-weisse daniel-weisse merged commit 439f7f6 into master Sep 26, 2025
12 checks passed
@daniel-weisse daniel-weisse deleted the dw/dcap-conf branch September 26, 2025 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature This change introduces new functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants