airbyte-platform icon indicating copy to clipboard operation
airbyte-platform copied to clipboard

Fix/update api port

Open Santhin opened this issue 2 years ago • 4 comments
trafficstars

What

It repairs airbyte-api-server port and adds missing ingress

How

Describe the solution

Can this PR be safely reverted / rolled back?

If you know that your PR is backwards-compatible and can be simply reverted or rolled back, check the YES box.

Otherwise if your PR has a breaking change, like a database migration for example, check the NO box.

If unsure, leave it blank.

  • [ ] YES 💚
  • [X] NO ❌

🚨 User Impact 🚨

Are there any breaking changes? What is the end result perceived by the user? If yes, please merge this PR with the 🚨🚨 emoji so changelog authors can further highlight this if needed. No

Santhin avatar Aug 17 '23 15:08 Santhin

Without this fix, helm deployments can't use terraform module :(

nurikk-sa avatar Aug 29 '23 13:08 nurikk-sa

Actually, you can but you need to create additional ingres by your hand, for PoC port forward can works as well

Santhin avatar Aug 29 '23 15:08 Santhin

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Piotr Sierkin seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar May 07 '24 21:05 CLAassistant

Your branch is not currently up-to-date with main. Please update your branch before attempting to snapshot your PR.

github-actions[bot] avatar May 07 '24 21:05 github-actions[bot]