TypeError: The 'validation_id' argument should be a List

[Rabi Mishra]

Undercloud install fails with tarceback[1]

Noticed at https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a44/800420/1/gate/tripleo-ci-centos-8-undercloud-containers/a442270/logs/undercloud/home/zuul/undercloud_install.log

Looks like https://github.com/openstack/validations-libs/commit/2f3b09d41ef8feaaf6438693551ea8aaf3a83892#diff-90b69462813c80b251c23eab92a0bbc66c389a46396df69ec705f47898eb9f9a broke it.

[1]
undercloud_admin_host or undercloud_public_host is not in the same cidr as local_ip.
Exception occured while running the command
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/tripleoclient/command.py", line 34, in run
    super(Command, self).run(parsed_args)
  File "/usr/lib/python3.6/site-packages/osc_lib/command/command.py", line 39, in run
    return super(Command, self).run(parsed_args)
  File "/usr/lib/python3.6/site-packages/cliff/command.py", line 185, in run
    return_code = self.take_action(parsed_args) or 0
  File "/usr/lib/python3.6/site-packages/tripleoclient/v1/undercloud.py", line 154, in take_action
    disable_container_prepare=parsed_args.disable_container_prepare)
  File "/usr/lib/python3.6/site-packages/tripleoclient/v1/undercloud_config.py", line 807, in prepare_undercloud_deploy
    undercloud_preflight.check(verbose_level, upgrade)
  File "/usr/lib/python3.6/site-packages/tripleoclient/v1/undercloud_preflight.py", line 501, in check
    _check_diskspace(upgrade)
  File "/usr/lib/python3.6/site-packages/tripleoclient/v1/undercloud_preflight.py", line 98, in _check_diskspace
    validation_name=playbook_args['playbook'])
  File "/usr/lib/python3.6/site-packages/validations_libs/validation_actions.py", line 348, in run_validations
    group)
  File "/usr/lib/python3.6/site-packages/validations_libs/utils.py", line 210, in get_validations_playbook
    raise TypeError("The 'validation_id' argument should be a List")
TypeError: The 'validation_id' argument should be a List
The 'validation_id' argument should be a List

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/python-tripleoclient/+/801019

[Jiri Podivin (jpodivin)]

I have submitted a fix to the python-tripleoclient. [0]

[0]https://review.opendev.org/c/openstack/python-tripleoclient/+/801019

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/validations-libs/+/801022

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/python-tripleoclient/+/801019
Committed: https://opendev.org/openstack/python-tripleoclient/commit/efdefd92de54d2dd0803f0367bf89bb577927037
Submitter: "Zuul (22348)"
Branch: master

commit efdefd92de54d2dd0803f0367bf89bb577927037
Author: Jiri Podivin <email address hidden>
Date: Fri Jul 16 08:39:22 2021 +0200

    Passes playbook to the validation action as a list

    Recent removal of the 'validations_libs.utils.convert_data'
    function, implemented by the Ief9459e914ea73fbd59a85c07a55c26078e0126a,
    resulted in a change of accepted argument types of
    the 'validations_libs.validation_actions' methods.

    This patch turns the passed argument from string into a list.

    Closes-bug: #1936465

    Signed-off-by: Jiri Podivin <email address hidden>
    Change-Id: I4e96f5a0e013de5abcde837dad3c5f9d5228dd4c

[Bhagyashri Shewale (bhagyashri-shewale)]

Hi All,

I see this issue is hitting on wallaby, victoria and ussuri as well.

[1]: https://logserver.rdoproject.org/openstack-periodic-integration-stable1/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-undercloud-containers-wallaby/7906a41/logs/undercloud/home/zuul/undercloud_install.log.txt.gz
[2]: https://logserver.rdoproject.org/openstack-periodic-integration-stable2/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-undercloud-containers-victoria/93af5fb/logs/undercloud/home/zuul/undercloud_install.log.txt.gz
[3]: https://logserver.rdoproject.org/openstack-periodic-integration-stable2/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-undercloud-containers-victoria/93af5fb/logs/undercloud/home/zuul/undercloud_install.log.txt.gz

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/python-tripleoclient/+/801232

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/python-tripleoclient/+/801234

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/python-tripleoclient/+/801258

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/python-tripleoclient/+/801232
Committed: https://opendev.org/openstack/python-tripleoclient/commit/e0f12ac4be6222fc2b1634ff167b5c83bed54815
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit e0f12ac4be6222fc2b1634ff167b5c83bed54815
Author: Jiri Podivin <email address hidden>
Date: Fri Jul 16 08:39:22 2021 +0200

    Passes playbook to the validation action as a list

    Recent removal of the 'validations_libs.utils.convert_data'
    function, implemented by the Ief9459e914ea73fbd59a85c07a55c26078e0126a,
    resulted in a change of accepted argument types of
    the 'validations_libs.validation_actions' methods.

    This patch turns the passed argument from string into a list.

    Closes-bug: #1936465

    Signed-off-by: Jiri Podivin <email address hidden>
    Change-Id: I4e96f5a0e013de5abcde837dad3c5f9d5228dd4c
    (cherry picked from commit efdefd92de54d2dd0803f0367bf89bb577927037)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/python-tripleoclient/+/801234
Committed: https://opendev.org/openstack/python-tripleoclient/commit/9b3bd5f5bec92655893a127c45083cbb618c5b66
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 9b3bd5f5bec92655893a127c45083cbb618c5b66
Author: Jiri Podivin <email address hidden>
Date: Fri Jul 16 08:39:22 2021 +0200

    Passes playbook to the validation action as a list

    Recent removal of the 'validations_libs.utils.convert_data'
    function, implemented by the Ief9459e914ea73fbd59a85c07a55c26078e0126a,
    resulted in a change of accepted argument types of
    the 'validations_libs.validation_actions' methods.

    This patch turns the passed argument from string into a list.

    Closes-bug: #1936465

    Signed-off-by: Jiri Podivin <email address hidden>
    Change-Id: I4e96f5a0e013de5abcde837dad3c5f9d5228dd4c
    (cherry picked from commit efdefd92de54d2dd0803f0367bf89bb577927037)

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/python-tripleoclient/+/801258
Committed: https://opendev.org/openstack/python-tripleoclient/commit/265c15ab2ff13f2a50f0e75773550a824365b517
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 265c15ab2ff13f2a50f0e75773550a824365b517
Author: Jiri Podivin <email address hidden>
Date: Fri Jul 16 08:39:22 2021 +0200

    Passes playbook to the validation action as a list

    Recent removal of the 'validations_libs.utils.convert_data'
    function, implemented by the Ief9459e914ea73fbd59a85c07a55c26078e0126a,
    resulted in a change of accepted argument types of
    the 'validations_libs.validation_actions' methods.

    This patch turns the passed argument from string into a list.

    Closes-bug: #1936465

    Signed-off-by: Jiri Podivin <email address hidden>
    Change-Id: I4e96f5a0e013de5abcde837dad3c5f9d5228dd4c
    (cherry picked from commit efdefd92de54d2dd0803f0367bf89bb577927037)

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Jiri Podivin <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/validations-libs/+/801022
Reason: After discussion it was decided that the revert won't be necessary, considering the merge of other fixes.

[Jiri Podivin (jpodivin)]

Considering the actions regarding the patches I think we can consider this matter to be resolved.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/python-tripleoclient 13.4.5 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/python-tripleoclient 16.4.0 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/python-tripleoclient 14.2.3 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/python-tripleoclient 17.1.0 release.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/train
Review: https://review.opendev.org/c/openstack/python-tripleoclient/+/823760

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/c/openstack/python-tripleoclient/+/823760
Committed: https://opendev.org/openstack/python-tripleoclient/commit/db6a3287f465ab90d3cc0be393f6d29d10f5fc48
Submitter: "Zuul (22348)"
Branch: stable/train

commit db6a3287f465ab90d3cc0be393f6d29d10f5fc48
Author: Jiri Podivin <email address hidden>
Date: Fri Jul 16 08:39:22 2021 +0200

    Passes playbook to the validation action as a list

    Recent removal of the 'validations_libs.utils.convert_data'
    function, implemented by the Ief9459e914ea73fbd59a85c07a55c26078e0126a,
    resulted in a change of accepted argument types of
    the 'validations_libs.validation_actions' methods.

    This patch turns the passed argument from string into a list.

    Closes-bug: #1936465

    Signed-off-by: Jiri Podivin <email address hidden>
    Change-Id: I4e96f5a0e013de5abcde837dad3c5f9d5228dd4c
    (cherry picked from commit efdefd92de54d2dd0803f0367bf89bb577927037)
    (cherry picked from commit 265c15ab2ff13f2a50f0e75773550a824365b517)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/python-tripleoclient train-eol release.