Add more env vars for pingsource adapter env var preservation

[aliok]

Fixes #

Proposed Changes

• Add more env vars for pingsource adapter env var preservation
• Without this, the eventing-controller and the operator are fighting to set env var values
• Related to Env var changes of "pingsource-mt-adapter" are not respected #393
• Ideally, we should find a way to fix this problem for good! Current solution is to maintain the env vars manually.

Release Note

Add more env vars for pingsource adapter env var preservation to prevent the eventing-controller and the operator fighting to set env var values.

[knative-prow]

@aliok: 0 warnings.

In response to this:

Fixes #

Proposed Changes

• Add more env vars for pingsource adapter env var preservation
• Related to Env var changes of "pingsource-mt-adapter" are not respected #393
• Ideally, we should find a way to fix this problem for good! Current solution is to maintain the env vars manually.

Release Note

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (72e396b) 79.10% compared to head (0542aa6) 79.10%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1534   +/-   ##
=======================================
  Coverage   79.10%   79.10%           
=======================================
  Files          40       40           
  Lines        1809     1809           
=======================================
  Hits         1431     1431           
  Misses        275      275           
  Partials      103      103           

Files Changed	Coverage Δ
...knativeeventing/common/replicasenvvarstransform.go	70.58% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[matzew]

/lgtm
/approve

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aliok, matzew

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [aliok,matzew]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[matzew]

@aliok we might need backports?

[aliok]

@aliok we might need backports?

Yes, definitely. I am figuring out right now which branches to backport this.

We (RHAT) need it in 0.36.5, which is 1.9.5.

So, all the way to 1.9: 1.11, 1.10, 1.9

[aliok]

/cherrypick release-1.11

[knative-prow-robot]

@aliok: new pull request created: #1535

In response to this:

/cherrypick release-1.11

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[aliok]

/cherrypick release-1.10

[aliok]

/cherrypick release-1.9

[knative-prow-robot]

@aliok: new pull request created: #1537

In response to this:

/cherrypick release-1.10

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[knative-prow-robot]

@aliok: new pull request created: #1538

In response to this:

/cherrypick release-1.9

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.