Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

OpenID configuration parameters added #13

Merged
merged 15 commits into from
Jul 22, 2024
Merged

OpenID configuration parameters added #13

merged 15 commits into from
Jul 22, 2024

Conversation

alexey-bogdanov
Copy link
Contributor

@alexey-bogdanov alexey-bogdanov commented Jul 16, 2024 •
edited by cheshi-mantu
Loading

closes #12

@alexey-bogdanov alexey-bogdanov self-assigned this Jul 16, 2024
@alexey-bogdanov alexey-bogdanov mentioned this pull request Jul 16, 2024
Closed
@cheshi-mantu
adds comments for email
000343d 
adds clomments for CryptoPass

Signed-off-by: Egor I <[email protected]>
@cheshi-mantu
datasource comments
9618f2d 
Signed-off-by: Egor I <[email protected]>
@cheshi-mantu
adds strict SMTP enabled
80fba30 
Signed-off-by: Egor I <[email protected]>
@cheshi-mantu
adds hikari logging comment
c2f79e9 
Signed-off-by: Egor I <[email protected]>
cheshi-mantu
cheshi-mantu suggested changes Jul 18, 2024
View reviewed changes
Copy link
Member

@cheshi-mantu cheshi-mantu left a comment

Choose a reason for hiding this comment

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

Linter fails in the workflow and when trying to dry run the created config I'm also getting the error.
see attached
error-dry-run.txt

@cheshi-mantu
Copy link
Member

cheshi-mantu commented Jul 18, 2024 •
edited
Loading

there is --set=image.imageName=allure-testops in the linter exec start, that needs to be updated to --set=image.imageName=testops

cheshi-mantu
cheshi-mantu approved these changes Jul 22, 2024
View reviewed changes
Copy link
Member

@cheshi-mantu cheshi-mantu left a comment

Choose a reason for hiding this comment

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

tested
merging

@cheshi-mantu cheshi-mantu merged commit 2abd4bd into master Jul 22, 2024
1 check passed
@cheshi-mantu cheshi-mantu deleted the openid branch July 22, 2024 07:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cheshi-mantu cheshi-mantu cheshi-mantu approved these changes

Assignees

@alexey-bogdanov alexey-bogdanov

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

add openid parameters for TestOps 5.6.0 to testops-deploy-helm-chart
2 participants
@alexey-bogdanov @cheshi-mantu