Hi Everyone,
Can integration folks review and merge this set of patches with priority?
https://git.opendaylight.org/gerrit/#/q/topic:revert-testplan
This reverts csit templates adding nested variables into defaults.yaml. Adding things like:
    streamtestplan: '{project}-{functionality}-{stream}.txt'
to defaults.yaml will cause every single template in releng/builder regardless of if they care about the variables {project}, {functionality}, and {stream} to now be required to set these variables.
This is not a good way to develop job templates. We need to revert the patches please!
I discovered that most of the variables could be removed from defaults.yaml without issue as projects were not overriding them yet, with the exception of {testplan}. I spent time reworking some templates in a way that will allow the projects that started overriding testplan to not break due to the revert. I believe I've caught everything and reworked the affected projects and the specific patch that handles this is:
https://git.opendaylight.org/gerrit/77712
I'd like to work with the integration team on these templates if certain variables need overriding by projects in a way that avoids using defaults.yaml for this stuff. I think we are abusing defaults.yaml more than it's original intent and I'd like to find a solution that makes things easier for everyone without forcing unneeded variables onto templates that don't require them.
Thanks,
Thanh
_______________________________________________
dev mailing list
dev@...
https://lists.opendaylight.org/mailman/listinfo/dev