diff options
| author | Pierre Riteau <pierre@stackhpc.com> | 2021-10-08 22:37:56 +0200 |
|---|---|---|
| committer | Radosław Piliszek <radoslaw.piliszek@gmail.com> | 2021-10-09 16:04:20 +0000 |
| commit | 556e8f9eeef71d240bef961cee0e3e5d357e3711 (patch) | |
| tree | 2eb1fdeea3bb930349c3ce5b32a89c29d8a00ddd | |
| parent | 8cdf952c6be7e15c1e6a7589371bc827386d4cd2 (diff) | |
| download | kolla-556e8f9eeef71d240bef961cee0e3e5d357e3711.tar.gz kolla-556e8f9eeef71d240bef961cee0e3e5d357e3711.zip | |
Fix GitHub tarball URL for monasca-thresh
And the options docs, they were misleading.
Change-Id: Ie388938fc207c982c5650221baee354a5bb9be30
Co-Authored-By: Radosław Piliszek <radoslaw.piliszek@gmail.com>
| -rwxr-xr-x | kolla/common/config.py | 15 |
1 files changed, 11 insertions, 4 deletions
diff --git a/kolla/common/config.py b/kolla/common/config.py index a6c090f06..f2cc4cc2a 100755 --- a/kolla/common/config.py +++ b/kolla/common/config.py @@ -232,9 +232,14 @@ _CLI_OPTS = [ ' of disk IO. "docker-squash" tool is required, install' ' it by "pip install docker-squash"')), cfg.StrOpt('openstack-release', default=OPENSTACK_RELEASE, - help='OpenStack release for building kolla-toolbox'), - cfg.StrOpt('openstack-branch', default='master', - help='Branch for source images'), + help='OpenStack release for building kolla source images and ' + 'kolla-toolbox image'), + cfg.StrOpt('openstack-branch', + help='Branch for source images (internal; with a dash; ' + 'please set openstack-release instead)'), + cfg.StrOpt('openstack-branch-slashed', + help='Branch for source images (internal; with a slash; ' + 'please set openstack-release instead)'), cfg.BoolOpt('docker-healthchecks', default=True, help='Add Kolla docker healthcheck scripts in the image'), cfg.BoolOpt('quiet', short='q', default=False, @@ -498,7 +503,7 @@ SOURCES = { 'monasca-thresh': { 'type': 'url', 'location': ('https://github.com/openstack/monasca-thresh/archive/' - 'master.tar.gz')}, + '${openstack_branch_slashed}.tar.gz')}, 'monasca-thresh-additions-monasca-common': { 'type': 'url', 'location': ('$tarballs_base/openstack/monasca-common/' @@ -1018,7 +1023,9 @@ def parse(conf, args, usage=None, prog=None, conf.set_default('base_tag', DEFAULT_BASE_TAGS[conf.base]['tag']) prefix = '' if conf.openstack_release == 'master' else 'stable-' openstack_branch = '{}{}'.format(prefix, conf.openstack_release) + openstack_branch_slashed = openstack_branch.replace('-', '/') conf.set_default('openstack_branch', openstack_branch) + conf.set_default('openstack_branch_slashed', openstack_branch_slashed) if not conf.base_image: conf.base_image = DEFAULT_BASE_TAGS[conf.base]['name'] |