Skip to content
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.

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

Initial SSA structure for new deployments #2577

Merged
merged 1 commit into from
Oct 10, 2022
Merged

Initial SSA structure for new deployments #2577

merged 1 commit into from
Oct 10, 2022

Conversation

Milton-Ch
Copy link
Contributor

Was able to get SSA tests working doing manual execution of "DeployServer" job first and then build using LDAP.

I'm reviewing embedded tests failures

Prepare


Description

Target issue

#2576

Implementation Details

Added required config to get SSA structure working in new deployments.


Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • [] Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

@mo-auto mo-auto added comp-jans-auth-server Component affected by issue or PR comp-jans-linux-setup Component affected by issue or PR labels Oct 10, 2022
@sonarcloud
Copy link

sonarcloud bot commented Oct 10, 2022

[jans-linux-setup] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sonarcloud
Copy link

sonarcloud bot commented Oct 10, 2022

[Jans authentication server parent] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@yuriyz
Copy link
Contributor

yuriyz commented Oct 10, 2022

Lets merge it. I think it's right move.

@yuriyz yuriyz marked this pull request as ready for review October 10, 2022 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-auth-server Component affected by issue or PR comp-jans-linux-setup Component affected by issue or PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants