Skip to content

Comments

Add initial platform dependencies, linting, and licenses#1

Merged
emsearcy merged 15 commits intomainfrom
ems/init
Jul 9, 2025
Merged

Add initial platform dependencies, linting, and licenses#1
emsearcy merged 15 commits intomainfrom
ems/init

Conversation

@emsearcy
Copy link
Contributor

@emsearcy emsearcy commented Jul 7, 2025

Add dependencies based on the proof of concept code.

Aside from the linting and formatting files, all of the helm files were added via AI assistance, largely via the following prompt:

create a folder structure for hosting LFXv2 platform charts in
lfx-v2-helm. see "{proof-of-concept-folder}/README.md" for services.
traefik, OpenFGA, heimdall, and NATS should be included as subcharts. Do
not include any of the docker-built custom services yet. OpenFGA should
have an option to either deploy the included PostgreSQL, or provide a
value with the connection string for an external PostgreSQL database.

Add dependencies based on the proof of concept code.

Aside from the linting and formatting files, all of the `helm` files
were added via AI assistance, largely via the following prompt:

```text
create a folder structure for hosting LFXv2 platform charts in
lfx-v2-helm. see "{proof-of-concept-folder}/README.md" for services.
traefik, OpenFGA, heimdall, and NATS should be included as subcharts. Do
not include any of the docker-built custom services yet. OpenFGA should
have an option to either deploy the included PostgreSQL, or provide a
value with the connection string for an external PostgreSQL database.
```

Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
Copilot AI review requested due to automatic review settings July 7, 2025 19:50
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

7 similar comments
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@jordane jordane mentioned this pull request Jul 7, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
@emsearcy emsearcy marked this pull request as draft July 7, 2025 22:02
jordane and others added 9 commits July 7, 2025 15:16
`global` is in used by nats and using it ourselves breaks
the nats chart. Instead use `lfx` for our own specific values.

Signed-off-by: Jordan Evans <jevans@linuxfoundation.org>
the values.yaml file was missing the `config` key for nats.

Signed-off-by: Jordan Evans <jevans@linuxfoundation.org>
Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
Delete and recreate all PVS (or the entire namespace) to resolve.

Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
Signed-off-by: Jordan Evans <jevans@linuxfoundation.org>
This actually doesn't do anything, but it allows the pod to start
successfully.

Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
In the absence of subchart schema, Megalinter's helm linter doesn't
understand the nested value properties.

Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
@emsearcy emsearcy marked this pull request as ready for review July 8, 2025 21:58
@emsearcy emsearcy requested a review from jordane July 8, 2025 22:07
Signed-off-by: Jordan Evans <jevans@linuxfoundation.org>
emsearcy and others added 3 commits July 9, 2025 08:56
This was unblocked with an upstream fix and release yesterday.

Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
Signed-off-by: Eric Searcy <eric@linuxfoundation.org>
Add missing core component (OpenSearch)

Signed-off-by: Jordan Evans <jevans@linuxfoundation.org>
@emsearcy emsearcy merged commit 65a687a into main Jul 9, 2025
3 checks passed
@emsearcy emsearcy deleted the ems/init branch July 9, 2025 20:32
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.

2 participants