Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

fix(installer): MONGODB_HOST should hardcode mongo #9656

Merged
merged 2 commits into from May 2, 2023
Merged

Conversation

RealAnna
Copy link
Contributor

@RealAnna RealAnna commented May 2, 2023

This PR

as mentioned in #9527 we removed the name-override from MongoDB so this should be hardcoded in the MONGODB_HOST env var. This var will not cause any problem for external DBs since those are connected via external_connection_string

Related Issues

Fixes #9526 (comment)

How to test

https://github.com/keptn/keptn/actions/runs/4858605537

Signed-off-by: realanna <anna.reale@dynatrace.com>
@RealAnna RealAnna requested a review from a team as a code owner May 2, 2023 06:49
@RealAnna RealAnna added the CI:trigger-build-everything Trigger CI Build: Set BUILD_EVERYTHING=TRUE label May 2, 2023
Signed-off-by: realanna <anna.reale@dynatrace.com>
@sonarcloud
Copy link

sonarcloud bot commented May 2, 2023

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

@codecov
Copy link

codecov bot commented May 2, 2023

Codecov Report

Merging #9656 (60b49d0) into master (6827bdb) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #9656   +/-   ##
=======================================
  Coverage   70.90%   70.91%           
=======================================
  Files         618      618           
  Lines       32735    32735           
  Branches     1645     1645           
=======================================
+ Hits        23211    23214    +3     
+ Misses       8278     8276    -2     
+ Partials     1246     1245    -1     

see 1 file with indirect coverage changes

Flag Coverage Δ
api 75.56% <ø> (ø)
approval-service 64.34% <ø> (ø)
bridge-server 74.33% <ø> (ø)
bridge2 87.28% <ø> (ø)
cli 44.65% <ø> (ø)
distributor 70.17% <ø> (+0.29%) ⬆️
lighthouse-service 71.89% <ø> (ø)
mongodb-datastore 58.25% <ø> (ø)
remediation-service 100.00% <ø> (ø)
resource-service 77.42% <ø> (ø)
secret-service 81.35% <ø> (ø)
shipyard-controller 68.07% <ø> (ø)
statistics-service 58.41% <ø> (ø)
webhook-service 84.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@RealAnna RealAnna merged commit fecc867 into master May 2, 2023
44 checks passed
@RealAnna RealAnna deleted the fix/bridge_mongo branch May 2, 2023 11:10
@github-actions github-actions bot mentioned this pull request May 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI:trigger-build-everything Trigger CI Build: Set BUILD_EVERYTHING=TRUE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bridge can't access mongodb when oauth is enabled
2 participants