Remove hardcoded list of namespaces with iterating over all directories
in the check of valid ACLs.
Remove parameter, we don't need to pass it in.
Be more verbose and report directories scanned and number of ACL files
read.
Change-Id: I4193ac59b431d31f3ce835c93950745e5b742418
Our linters fail due to wrong sorting, fix it.
Also, fix tools/check_valid_gerrit_config.sh, openstack-dev is dead now.
(the file needs a better fix, will do a followup)
Change-Id: Iaa330a3f9507cb065ee8b3a11d91abdadb455cf2
The remaining bashate warnings was E042:
The return value of "local" is always 0; errors in subshells
used for declaration are thus hidden and will not trigger "set
-e".
Fix the few cases this triggered so that we don't get confused anymore
about the message.
Remove -v from bashate invocation, we don't need to print out all the
filenames anymore.
Change-Id: I47991a7040c8b9183bc72cce8e5d95b2cec7e6c5
For tools/check_valid_gerrit_projects.py:
* Improve one error message to mention the project as well.
For tools/check_valid_gerrit_config.sh:
* Remove output for each ACL we check, we output the entry if it's
wrong. But with 800+ ACL lines, we should not output 800+ lines of
progress and thus make it hard to find the one failing line.
Change-Id: I4434c0f0bf65a1eb69889a2362c1abae7119fdd7
Cleanup every use of `` for subshells in the nodepool and tools
directory , replacing them with $(), and finally making the scripts
consistent.
Change-Id: I2b05cd20f9c9a30ab88f8db235aa81da93b1fad3
When the ACL file normalization checker failed it only mentioned the
filename, with the actual error output from diff potentially many many
lines up in the scrollback. This change saves the diff output so it can
be reported at the end of the run where the failure happens.
Change-Id: I862f755c6ed96208c0fe8255175afb48847f8cad
Enhance Gerrit ACL check to check that the files are properly
normalized.
Co-Authored-By: Armando Migliaccio <armamig@gmail.com>
Change-Id: I9cdee60e77dab9c6943626d5fa1eda0402840277
This patch adds a check for Gerrit ACL configs. In this initial
version, the test being added checks that the config defines
at least one team that acts as a core/admin one.
The current check is intentially pretty loose to ensure some
flexibility.
Currently the check applies to stackforge and openstack-dev only.
Change-Id: I2b04f2d94130699d6fd1c16a9b00b9c81d1ba07d