Remove duplicate key

[rsitrifork]

Remove duplicate key to fix #58591

What

The ENV entry for AB_JWT_SIGNATURE_SECRET was duplicated in the deployment, preventing Helm upgrades.

How

Remove one of the two duplicate entries

Recommended reading order

1. deployment.yaml

Can this PR be safely reverted and rolled back?

• YES 💚
• NO ❌

[CLAassistant]

All committers have signed the CLA.

[dfeddad]

I have been waiting for this to be merged for sometimes #402

[abedzantout]

Hi all — just checking in on this. It’s been open for a few weeks, and it’s currently a key blocker for deploying the latest version of Airbyte. Happy to contribute if there's any blockers preventing this from being merged

[fabio-wmp]

I have the same problem, does uninstall delete the volumes ?

[TheNumberOne]

@abuchanan-airbyte Would it be possible to get someone on the airbyte team to review and merge this

(I'm pinging you because you have the most commits in the last month)

[perangel]

Hey, thanks for bringing this to our attention. We'll process this PR internally and get this merged shortly

[perangel]

/create-oss-pr

[mnacharov]

@pmossman @perangel @abuchanan-airbyte can you please add this patch into the next chart release ? this bug is here for quite a long time ~ 2 months already (see #402)

[perangel]

These changes have now been merged. A patch release of the chart is forthcoming

[chyzouakia]

These changes have now been merged. A patch release of the chart is forthcoming

Hello @perangel ! Hope you are doing fine 🙂
Do you know when this patch will be released ? It's blocking the upgrade on our side 😬
Thank you ! 🙏

[TheNumberOne]

@chyzouakia 1.7.1 was released with this change a few days ago ^-^

[chyzouakia]

@chyzouakia 1.7.1 was released with this change a few days ago ^-^

@TheNumberOne Hello ! No if you check the helm-charts repo and get the latest airbyte-1.7.1 tgz you'll see that the duplication is still there in charts/workload-api-server/templates/deployment.yaml. I still get the error when I try to upgrade. I upgraded manually by downloading the charts and fixing this issue. I hope it will be fixed in the next release so I don't need to have the helm charts locally 😄

[eugene-kuchansky-proxet]

@perangel @mnacharov it's "Closed with unmerged commits"
could you please fix?

[TheNumberOne]

@chyzouakia Yeah, you're right. I don't know why I thought it had the fix :/