Skip to content

[Feat] Support logon session affinity #260

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

[Feat] Support logon session affinity #260

wants to merge 6 commits into from

Conversation

anirudhprasad-sap
Copy link
Contributor

@anirudhprasad-sap anirudhprasad-sap commented Sep 1, 2025 •
edited
Loading

Support previous login sessions during the upgrade.
Enabled by setting the annotation sme.sap.com/enable-session-affinity: true
Also, allow custom logout endpoint to be set via annotation sme.sap.com/logout-endpoint on CAV

@anirudhprasad-sap anirudhprasad-sap added the build-pr-images Set this label on PRs to build docker images for changes label Sep 1, 2025
@anirudhprasad-sap anirudhprasad-sap added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Sep 12, 2025
@anirudhprasad-sap anirudhprasad-sap marked this pull request as ready for review September 15, 2025 12:39
Pavan-SAP
Pavan-SAP reviewed Sep 15, 2025
View reviewed changes
@anirudhprasad-sap anirudhprasad-sap added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Sep 16, 2025
@Pavan-SAP Pavan-SAP force-pushed the logonSessionAffinity branch from cc88851 to ae39325 Compare October 14, 2025 12:22
@Pavan-SAP Pavan-SAP added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Oct 14, 2025
@anirudhprasad-sap @Pavan-SAP
[Feat] Operator: Optional Session Affinity added
da7d705 
Provide an option to enable session affinity for CAP applications
during version upgrades to ensure user sessions remain intact.
@Pavan-SAP Pavan-SAP force-pushed the logonSessionAffinity branch from ae39325 to da7d705 Compare October 14, 2025 12:29
@Pavan-SAP
Copy link
Member

I think we just need to update the new tests (introduced with this PR) to add the new subscription guid and other labels introduced recently

@Pavan-SAP Pavan-SAP added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Oct 14, 2025
@anirudhprasad-sap
Copy link
Contributor Author

I think we just need to update the new tests (introduced with this PR) to add the new subscription guid and other labels introduced recently

Yeah, I will update the tests

@anirudhprasad-sap anirudhprasad-sap added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Oct 15, 2025
@anirudhprasad-sap anirudhprasad-sap added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Oct 17, 2025
@Pavan-SAP Pavan-SAP added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Oct 17, 2025
@Pavan-SAP Pavan-SAP added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Oct 17, 2025
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
90.5% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Pavan-SAP Pavan-SAP Pavan-SAP left review comments

@skrishnan-sap skrishnan-sap Awaiting requested review from skrishnan-sap

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

build-pr-images Set this label on PRs to build docker images for changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@anirudhprasad-sap @Pavan-SAP