Skip to content

feat(cloud-native): use endpoints for managing admin-ui session#2583

Merged
moabu merged 4 commits intomainfrom
cn-aui-session
Jan 18, 2026
Merged

feat(cloud-native): use endpoints for managing admin-ui session#2583
moabu merged 4 commits intomainfrom
cn-aui-session

Conversation

@iromli
Copy link
Contributor

@iromli iromli commented Jan 15, 2026

Adjust images to support new endpoints for managing admin-ui sessions.

Closes #2582

Summary by CodeRabbit

  • Chores
    • Updated base source versions used in Docker image builds for admin-ui, all-in-one, and persistence loader components.
    • Added schema configuration template file to the admin-ui image.

✏️ Tip: You can customize this high-level summary in your review settings.

Signed-off-by: iromli <isman.firmansyah@gmail.com>
@iromli iromli self-assigned this Jan 15, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 15, 2026

📝 Walkthrough

Walkthrough

Three Dockerfiles are updated with new source version identifiers for JANS and FLEX assets, and the docker-admin-ui build now copies an additional flex_schema.json file into the container's schema directory.

Changes

Cohort / File(s) Change Summary
Version pin updates
docker-admin-ui/Dockerfile, docker-flex-all-in-one/Dockerfile, docker-persistence-loader/Dockerfile
Updated JANS_SOURCE_VERSION and FLEX_SOURCE_VERSION environment variables to new commit hashes across three Dockerfiles to fetch updated assets
Admin-UI schema addition
docker-admin-ui/Dockerfile
Added file copy operation to include flex_setup_dir/flex_schema.json into /app/schema/ directory during FLEX build stage

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • moabu
  • duttarnab

Poem

🐰 Version hashes hop along with care,
Fresh commits checked out through the air,
Schema files bundled, neat and clean,
Docker images built, lean and keen!
A rabbit's joy—these pins so bright! 🚀

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'feat(cloud-native): use endpoints for managing admin-ui session' directly aligns with the main objective: updating Docker images to support new endpoints for managing admin-ui sessions.
Linked Issues check ✅ Passed Code changes update JANS_SOURCE_VERSION and FLEX_SOURCE_VERSION across three Dockerfiles and add flex_schema.json copying, which implements the required changes to support new admin-ui session management endpoints.
Out of Scope Changes check ✅ Passed All changes are scoped to updating source versions in Dockerfiles and copying flex_schema.json, which are directly related to implementing the admin-ui session management endpoint feature.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mo-auto mo-auto added comp-docker-admin-ui Component affected by issue or PR kind-feature Issue or PR is a new feature request labels Jan 15, 2026
Signed-off-by: iromli <isman.firmansyah@gmail.com>
Signed-off-by: iromli <isman.firmansyah@gmail.com>
Signed-off-by: iromli <isman.firmansyah@gmail.com>
@iromli iromli marked this pull request as ready for review January 16, 2026 20:53
@iromli iromli requested a review from moabu as a code owner January 16, 2026 20:53
@sonarqubecloud
Copy link

@moabu moabu merged commit 5d4f4c3 into main Jan 18, 2026
9 of 13 checks passed
@moabu moabu deleted the cn-aui-session branch January 18, 2026 04:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp-docker-admin-ui Component affected by issue or PR kind-feature Issue or PR is a new feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(cloud-native): use endpoints for managing admin-ui session

3 participants