What happened:
Multiple plugins are not exposing information about their current configuration via the PluginHelp structs as they should be.
What you expected to happen:
These plugins should expose config using the PluginHelp.Config field: https://github.com/kubernetes/test-infra/blob/fa2d9efca4e638ddc1e0499464b7ed116034806d/prow/pluginhelp/pluginhelp.go#L44-L47
lgtmupdateconfigsize (Config is included in the wrong field.)Anything else we need to know?:
This info will be used in the details sections here: https://prow.k8s.io/plugins
/area prow/plugins
/help
We don't have a way to set updateconfig configs per repo currently, do we?
We don't have a way to set updateconfig configs per repo currently, do we?
No, currently the plugin assumes that it is only enabled on a single repository.
/assign
/remove-help
These are the identified plugins:
Alright, so now I have a couple of plugins to fix regarding the validation of enabledRepos:
Thanks @fejta !
/close
@matthyx: Closing this issue.
In response to this:
Thanks @fejta !
/close
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.