Drop remnants of install_type
Change-Id: I110c642da925a5f6b3bd071e7844c20fda673c66
This commit is contained in:
parent
10ca82c243
commit
18567711a4
@ -11,9 +11,6 @@ kolla_ansible_setup_any_errors_fatal: true
|
|||||||
|
|
||||||
kolla_base_distro: "{{ base_distro }}"
|
kolla_base_distro: "{{ base_distro }}"
|
||||||
# Zed dropped install_type so we have it only on upgrades
|
# Zed dropped install_type so we have it only on upgrades
|
||||||
{% if is_upgrade %}
|
|
||||||
kolla_install_type: "{{ install_type }}"
|
|
||||||
{% endif %}
|
|
||||||
network_interface: "{{ api_interface_name }}"
|
network_interface: "{{ api_interface_name }}"
|
||||||
network_address_family: "{{ address_family }}"
|
network_address_family: "{{ address_family }}"
|
||||||
docker_restart_policy: "no"
|
docker_restart_policy: "no"
|
||||||
|
@ -197,9 +197,7 @@ FAKE_DATA = {
|
|||||||
|
|
||||||
'container_inspect': {
|
'container_inspect': {
|
||||||
'Config': {
|
'Config': {
|
||||||
'Env': ['KOLLA_BASE_DISTRO=ubuntu',
|
'Env': ['KOLLA_BASE_DISTRO=ubuntu'],
|
||||||
'KOLLA_INSTALL_TYPE=source',
|
|
||||||
'KOLLA_INSTALL_METATYPE=rdo'],
|
|
||||||
'Hostname': 'node2',
|
'Hostname': 'node2',
|
||||||
'Volumes': {'/var/lib/kolla/config_files/': {}}},
|
'Volumes': {'/var/lib/kolla/config_files/': {}}},
|
||||||
'Mounts': {},
|
'Mounts': {},
|
||||||
@ -543,9 +541,7 @@ class TestContainer(base.BaseTestCase):
|
|||||||
)
|
)
|
||||||
|
|
||||||
def test_get_container_env(self):
|
def test_get_container_env(self):
|
||||||
fake_env = dict(KOLLA_BASE_DISTRO='ubuntu',
|
fake_env = dict(KOLLA_BASE_DISTRO='ubuntu')
|
||||||
KOLLA_INSTALL_TYPE='source',
|
|
||||||
KOLLA_INSTALL_METATYPE='rdo')
|
|
||||||
self.dw = get_DockerWorker({'name': 'my_container',
|
self.dw = get_DockerWorker({'name': 'my_container',
|
||||||
'action': 'get_container_env'})
|
'action': 'get_container_env'})
|
||||||
self.dw.dc.containers.return_value = self.fake_data['containers']
|
self.dw.dc.containers.return_value = self.fake_data['containers']
|
||||||
@ -1211,26 +1207,22 @@ class TestAttrComp(base.BaseTestCase):
|
|||||||
def test_compare_environment_neg(self):
|
def test_compare_environment_neg(self):
|
||||||
container_info = {'Config': dict(
|
container_info = {'Config': dict(
|
||||||
Env=['KOLLA_CONFIG_STRATEGY=COPY_ALWAYS',
|
Env=['KOLLA_CONFIG_STRATEGY=COPY_ALWAYS',
|
||||||
'KOLLA_BASE_DISTRO=ubuntu',
|
'KOLLA_BASE_DISTRO=ubuntu']
|
||||||
'KOLLA_INSTALL_TYPE=source']
|
|
||||||
)}
|
)}
|
||||||
self.dw = get_DockerWorker({
|
self.dw = get_DockerWorker({
|
||||||
'environment': dict(KOLLA_CONFIG_STRATEGY='COPY_ALWAYS',
|
'environment': dict(KOLLA_CONFIG_STRATEGY='COPY_ALWAYS',
|
||||||
KOLLA_BASE_DISTRO='ubuntu',
|
KOLLA_BASE_DISTRO='ubuntu')})
|
||||||
KOLLA_INSTALL_TYPE='source')})
|
|
||||||
|
|
||||||
self.assertFalse(self.dw.compare_environment(container_info))
|
self.assertFalse(self.dw.compare_environment(container_info))
|
||||||
|
|
||||||
def test_compare_environment_pos(self):
|
def test_compare_environment_pos(self):
|
||||||
container_info = {'Config': dict(
|
container_info = {'Config': dict(
|
||||||
Env=['KOLLA_CONFIG_STRATEGY=COPY_ALWAYS',
|
Env=['KOLLA_CONFIG_STRATEGY=COPY_ALWAYS',
|
||||||
'KOLLA_BASE_DISTRO=ubuntu',
|
'KOLLA_BASE_DISTRO=ubuntu']
|
||||||
'KOLLA_INSTALL_TYPE=source']
|
|
||||||
)}
|
)}
|
||||||
self.dw = get_DockerWorker({
|
self.dw = get_DockerWorker({
|
||||||
'environment': dict(KOLLA_CONFIG_STRATEGY='COPY_ALWAYS',
|
'environment': dict(KOLLA_CONFIG_STRATEGY='COPY_ALWAYS',
|
||||||
KOLLA_BASE_DISTRO='centos',
|
KOLLA_BASE_DISTRO='centos')})
|
||||||
KOLLA_INSTALL_TYPE='source')})
|
|
||||||
|
|
||||||
self.assertTrue(self.dw.compare_environment(container_info))
|
self.assertTrue(self.dw.compare_environment(container_info))
|
||||||
|
|
||||||
|
@ -118,7 +118,6 @@
|
|||||||
timeout: 9000
|
timeout: 9000
|
||||||
vars:
|
vars:
|
||||||
base_distro: debian
|
base_distro: debian
|
||||||
install_type: source
|
|
||||||
is_upgrade: yes
|
is_upgrade: yes
|
||||||
tls_enabled: true
|
tls_enabled: true
|
||||||
|
|
||||||
@ -134,7 +133,6 @@
|
|||||||
timeout: 9000
|
timeout: 9000
|
||||||
vars:
|
vars:
|
||||||
base_distro: ubuntu
|
base_distro: ubuntu
|
||||||
install_type: binary
|
|
||||||
is_upgrade: yes
|
is_upgrade: yes
|
||||||
tls_enabled: true
|
tls_enabled: true
|
||||||
|
|
||||||
@ -145,7 +143,6 @@
|
|||||||
timeout: 9000
|
timeout: 9000
|
||||||
vars:
|
vars:
|
||||||
base_distro: ubuntu
|
base_distro: ubuntu
|
||||||
install_type: source
|
|
||||||
is_upgrade: yes
|
is_upgrade: yes
|
||||||
tls_enabled: true
|
tls_enabled: true
|
||||||
|
|
||||||
@ -156,7 +153,6 @@
|
|||||||
timeout: 9000
|
timeout: 9000
|
||||||
vars:
|
vars:
|
||||||
base_distro: ubuntu
|
base_distro: ubuntu
|
||||||
install_type: source
|
|
||||||
is_upgrade: yes
|
is_upgrade: yes
|
||||||
|
|
||||||
- job:
|
- job:
|
||||||
@ -166,7 +162,6 @@
|
|||||||
timeout: 9000
|
timeout: 9000
|
||||||
vars:
|
vars:
|
||||||
base_distro: ubuntu
|
base_distro: ubuntu
|
||||||
install_type: source
|
|
||||||
is_upgrade: yes
|
is_upgrade: yes
|
||||||
|
|
||||||
- job:
|
- job:
|
||||||
@ -231,7 +226,6 @@
|
|||||||
timeout: 10800
|
timeout: 10800
|
||||||
vars:
|
vars:
|
||||||
base_distro: ubuntu
|
base_distro: ubuntu
|
||||||
install_type: source
|
|
||||||
is_upgrade: yes
|
is_upgrade: yes
|
||||||
|
|
||||||
- job:
|
- job:
|
||||||
@ -352,7 +346,6 @@
|
|||||||
nodeset: kolla-ansible-focal
|
nodeset: kolla-ansible-focal
|
||||||
vars:
|
vars:
|
||||||
base_distro: ubuntu
|
base_distro: ubuntu
|
||||||
install_type: source
|
|
||||||
is_upgrade: yes
|
is_upgrade: yes
|
||||||
|
|
||||||
- job:
|
- job:
|
||||||
|
Loading…
Reference in New Issue
Block a user