e7f77f283f
libyaml now needs to Loader keyword to not throw a warning, use safe_load instead everywhere. sort_keys isn't supported for reasons surpassing understanding, so switch to ruamel.yaml for the places where we write contents back out. Our ruamel.yaml wrapper strips extra leading space, which is needed when the root object is a list. When the root object is NOT a list, it's not needed. Add a flag to the constructor to toggle the behavior. For gerrit/projects.yaml this leads to a different wrapping of description fields, which is included here. Let ruamel.yaml handle comments Since we've switched to ruamel.yaml, we don't need to strip comments. Stop stripping them. Intra-list comments aren't supported, so just remove them. Please don't ask me why we can't have a space before openstack-merlin. Change-Id: I72776774eabd4e397ee5e8bdd58b3632be8fd3a1
76 lines
2.4 KiB
Python
Executable File
76 lines
2.4 KiB
Python
Executable File
#! /usr/bin/env python
|
|
|
|
# Copyright 2015 SUSE Linux GmbH
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
|
|
import sys
|
|
import yaml
|
|
|
|
|
|
def access_gerrit_check():
|
|
"Check that all channels in gerritbot are in accessbot as well."
|
|
|
|
errors = False
|
|
|
|
access_config = yaml.safe_load(open('accessbot/channels.yaml', 'r'))
|
|
|
|
access_channels = []
|
|
for channel in access_config['channels']:
|
|
access_channels.append(channel['name'])
|
|
|
|
access_channel_set = set(access_channels)
|
|
|
|
gerrit_config = yaml.safe_load(open('gerritbot/channels.yaml'))
|
|
|
|
print("Basic check of gerritbot/channels.yaml")
|
|
REQUIRED_ENTRIES = ("branches", "events", "projects")
|
|
VALID_EVENTS = ("change-merged", "patchset-created")
|
|
for channel in gerrit_config:
|
|
for entry in REQUIRED_ENTRIES:
|
|
if entry not in gerrit_config[channel]:
|
|
print("ERROR: Required entry '%s' not specified for channel "
|
|
"'%s'" % (entry, channel))
|
|
errors = True
|
|
elif not gerrit_config[channel][entry]:
|
|
print("ERROR: Entry '%s' has no content for channel '%s'"
|
|
% (entry, channel))
|
|
errors = True
|
|
|
|
for event in gerrit_config[channel]['events']:
|
|
if event not in VALID_EVENTS:
|
|
print("ERROR: Event '%s' for channel '%s' is invalid"
|
|
% (event, channel))
|
|
errors = True
|
|
|
|
print("\nChecking that all channels in gerritbot are also in accessbot")
|
|
for channel in gerrit_config:
|
|
if channel not in access_channel_set:
|
|
print("ERROR: %s is missing from accessbot" % channel)
|
|
errors = True
|
|
|
|
return errors
|
|
|
|
|
|
def main():
|
|
errors = access_gerrit_check()
|
|
|
|
if errors:
|
|
print("Found errors in channel setup!")
|
|
else:
|
|
print("No errors found in channel setup!")
|
|
return errors
|
|
|
|
if __name__ == "__main__":
|
|
sys.exit(main())
|