Skip to content

Add NicApp CRD template for NIC integration - #8

Merged
aktech merged 3 commits into
mainfrom
aktech/nic-crd-integration
Feb 5, 2026
Merged

Add NicApp CRD template for NIC integration#8
aktech merged 3 commits into
mainfrom
aktech/nic-crd-integration

Conversation

@aktech

@aktech aktech commented Feb 4, 2026

Copy link
Copy Markdown
Member

No description provided.

@aktech
aktech force-pushed the aktech/nic-crd-integration branch from edd0fe4 to 2b39f4b Compare February 4, 2026 17:15
- Add templates/nicapp.yaml for nebari-operator integration
- Add nicapp config block to values.yaml (enabled by default)
- Configures routing, TLS, and Keycloak auth via operator
- Set nicapp.enabled=false when deploying outside NIC
@aktech
aktech force-pushed the aktech/nic-crd-integration branch from 2b39f4b to a467efa Compare February 4, 2026 17:18
Comment thread values.yaml Outdated
Comment thread values.yaml Outdated
Comment thread values.yaml Outdated
Comment thread templates/nicapp.yaml Outdated
Comment thread templates/nicapp.yaml Outdated

@viniciusdc viniciusdc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looking good, some important changes required after the recent rename of the CRD

@aktech
aktech force-pushed the aktech/nic-crd-integration branch from d661668 to f3a649d Compare February 5, 2026 11:16
  - Rename templates/nicapp.yaml to nebariapp.yaml
  - Update CRD to apiVersion: apps.nebari.dev/v1, kind: NebariApp
  - Rename values nicapp -> nebariapp
  - Remove tls and gateway sections (handled by operator)
  - Update service name to proxy-public
  - Disable nebariapp in Tiltfile for local dev
@aktech
aktech force-pushed the aktech/nic-crd-integration branch from f3a649d to 472dddf Compare February 5, 2026 11:36
@aktech
aktech merged commit 9c55c91 into main Feb 5, 2026
2 checks passed
@aktech
aktech deleted the aktech/nic-crd-integration branch February 5, 2026 11:39
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.

3 participants