search

datawire / ambassador-chart

Helm Chart for Installing Ambassador
42 stars 60 forks source link

Set additional deployment labels #114

Closed jasons42 closed 4 years ago

iNoahNothing commented 4 years ago

Could you update the Changelog and put this under the "Next Release" section?

jasons42 commented 4 years ago

Could you update the Changelog and put this under the "Next Release" section?

Updated.

jasons42 commented 4 years ago

In order to improve the CI tests, could you add some deploymentLabels in one of the ci/*-values.yaml, so the correctness of the rendered Deployment could be tested?

@inercia I added an additional values file so as not to mix the concerns of the existing tests. Please let me know if that doesn't work for you.