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.

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

5º - v1.3.0 - Bugfix/cmp 738/edc postgres configuration: Resolve EDC postgres database credentials (user and password) #187

Merged
merged 10 commits into from
Nov 2, 2023
Merged

5º - v1.3.0 - Bugfix/cmp 738/edc postgres configuration: Resolve EDC postgres database credentials (user and password) #187

merged 10 commits into from
Nov 2, 2023

Conversation

saudkhan116
Copy link

@saudkhan116 saudkhan116 commented Oct 19, 2023 •
edited
Loading

Why we create this PR?

This PR contains the following changes:

  • There was an issue in the EDC postgres configuration, the database credentials was not configured correctly in helm values
  • There was a condition missing to the code build step in publish dpp backend workflow

What we want to achieve with this PR?

To fix the database configuration.

What is new?

Issues Fixed

  • Fix database credentials in the edc postgres configuration
  • Fix the condition to the publish dpp backend workflow

PR Linked to:

Tickets
cmp-738

matbmoser and others added 9 commits October 16, 2023 10:36
@matbmoser
Merge pull request #181 from catenax-ng/release/v1.2.0-digital-produc…
ab2816a 
…t-pass

Release/v1.2.0 digital product pass:update develop
feat: change the passport api to data
1934da8
@matbmoser
Merge pull request #182 from catenax-ng/release/v1.2.0-digital-produc…
4b63133 
…t-pass

feat: change the passport api to data
feat: change the passport api to data
6628b62
@matbmoser
Merge pull request #183 from catenax-ng/release/v1.2.0-digital-produc…
f096b07 
…t-pass

feat: change the passport api to data
@saudkhan116
Merge branch 'develop' of https://github.com/catenax-ng/tx-digital-pr…
4aaa153 
…oduct-pass into develop
@saudkhan116
Merge branch 'develop' of https://github.com/catenax-ng/tx-digital-pr…
1253559 
…oduct-pass into develop
@saudkhan116
Merge branch 'develop' of https://github.com/catenax-ng/tx-digital-pr…
1421bd5 
…oduct-pass into develop
@saudkhan116
fix: resolve postgres database credentials in edc configuration
256eb4f
@saudkhan116 saudkhan116 requested a review from matbmoser October 19, 2023 10:16
@matbmoser matbmoser changed the title Bugfix/cmp 738/edc postgres configuration: Resolve EDC postgres database credentials (user and password) v1.3.0 - Bugfix/cmp 738/edc postgres configuration: Resolve EDC postgres database credentials (user and password) Oct 30, 2023
@matbmoser matbmoser changed the base branch from develop_1 to develop October 31, 2023 10:51
@matbmoser matbmoser changed the title v1.3.0 - Bugfix/cmp 738/edc postgres configuration: Resolve EDC postgres database credentials (user and password) 5º - v1.3.0 - Bugfix/cmp 738/edc postgres configuration: Resolve EDC postgres database credentials (user and password) Oct 31, 2023
@saudkhan116 saudkhan116 merged commit 817a825 into develop Nov 2, 2023
@saudkhan116 saudkhan116 deleted the bugfix/cmp-738/edc-postgres-configuration branch November 2, 2023 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matbmoser matbmoser matbmoser approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@saudkhan116 @matbmoser