Skip to content

Added charts for radar-appserver-jersey#7051

Open
this-Aditya wants to merge 6 commits intomainfrom
feature/add-appserver-jersey
Open

Added charts for radar-appserver-jersey#7051
this-Aditya wants to merge 6 commits intomainfrom
feature/add-appserver-jersey

Conversation

@this-Aditya
Copy link
Copy Markdown
Member

Summary

This PR adds new charts for the Radar Appserver Jersey. This is similar to the Radar Appserver, but it uses the Jersey framework instead of Spring Boot.

@this-Aditya
Copy link
Copy Markdown
Member Author

This PR keeps the file upload configurations the same as they were in the legacy appserver, since it has not yet been finalized whether to use the gateway file upload endpoint or to recreate it for the appserver-jersey for now.

@RADAR-base RADAR-base deleted a comment from github-actions bot Jan 8, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 8, 2026

Great PR! Please pay attention to the following items before merging:

Files matching charts/*/values.yaml:

  • Is the PR adding a new container? Please reviewer, add it to the models (internal process)
  • Is the PR adding a new parameter? Please, ensure it’s documented in the README.md

This is an automatically generated QA checklist based on modified files.

Copy link
Copy Markdown
Member

@Hsankesara Hsankesara left a comment

Choose a reason for hiding this comment

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

Looks good to me. Some minor changes. Also, the pre-commit checks are failing, I would recommend running pre-commit locally and updating commit messages.

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