eventing icon indicating copy to clipboard operation
eventing copied to clipboard

[QE] Passing a custom config template for upgrade tests is limiting (and cumbersome)

Open cardil opened this issue 3 years ago • 16 comments

Describe the bug The PR #4615 have introduced a way to pass a custom config template. But, passing it is limited to a relative path of default config.toml file.

This is cumbersome, as seen in: https://github.com/google/knative-gcp/blob/930cae634aa9fdb1ee5064091680668e3711f236/test/upgrade/continual.go#L45-L46 (cc @zhongduo), and can be used only with vendored sources.

Expected behavior It should be easy to pass a custom config path, even outside of vendor path.

Knative release version v0.20.0 +

cardil avatar Mar 29 '21 19:03 cardil

/area test-and-release /kind cleanup

cardil avatar Mar 29 '21 19:03 cardil

/assign

cardil avatar Mar 29 '21 19:03 cardil

@cardil are you working on this now? Do you expect to get it in by next release (0.23) or after that?

grantr avatar Apr 05 '21 16:04 grantr

Not so actively. I think I'll propose some solution, but i think after 0.23.

cardil avatar Apr 05 '21 17:04 cardil

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

github-actions[bot] avatar Jul 08 '21 01:07 github-actions[bot]

/remove-lifecycle stale

cardil avatar Jul 21 '21 18:07 cardil

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

github-actions[bot] avatar Oct 20 '21 01:10 github-actions[bot]

/remove-lifecycle stale

cardil avatar Oct 20 '21 13:10 cardil

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

github-actions[bot] avatar Jan 19 '22 01:01 github-actions[bot]

/remove-lifecycle stale

cardil avatar Jan 20 '22 17:01 cardil

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

github-actions[bot] avatar Apr 21 '22 01:04 github-actions[bot]

/remove-lifecycle stale /triage accepted

pierDipi avatar May 23 '22 06:05 pierDipi

@cardil is this issue still relevant?

pierDipi avatar Mar 27 '24 18:03 pierDipi

@cardil are you still interesting in this issue?

pierDipi avatar Mar 27 '24 18:03 pierDipi

Yup

cardil avatar Mar 28 '24 15:03 cardil