Skip to content

feat: update tmforum-api in dev environment#1515

Draft
versatildefuy wants to merge 1 commit intoDOME-Marketplace:mainfrom
versatildefuy:feature/upgrade-tmforum-in-dev
Draft

feat: update tmforum-api in dev environment#1515
versatildefuy wants to merge 1 commit intoDOME-Marketplace:mainfrom
versatildefuy:feature/upgrade-tmforum-in-dev

Conversation

@versatildefuy
Copy link
Copy Markdown

Update tmforum-api in dev environment to version 1.3.6.

Adds similar sbx similar pods and tls secret generation.

Also changes bae configuration pointing to new tmforum pods uris.

# -- should tm-forum-api be enabled
enabled: true

defaultConfig:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why are you adding things here, that are already in the access-node? With your PR https://github.com/DOME-Marketplace/access-node/pull/81/files this version is already set and the additional env vars are also already in place. Duplicating that will bring inconsistencies in the future. Beside that, why dont you update the access-node chart in this PR?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Hello @wistefan, this PR is a draft until we can update the access-node version. This PR was created before the team asked us to upgrade access-node chart version. Once DOME-Marketplace/access-node#81 is closed I will update the content of this PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants