openstack/metalsmith
Code Issues Proposed changes

Provide a work around to fix the CI

Browse Source 
Somehow, openstacksdk ends up logging to stdout. Strip the log message
before parsing it as JSON.

Change-Id: Id8e635de7bd8a246af7ae1750a556b752dd7dc85
This commit is contained in:
Dmitry Tantsur 2022-05-24 09:45:32 +02:00
parent a3e205fc05
commit 5816be550b
1 changed files with 18 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
playbooks/integration/exercise.yaml
View File

@ -29,8 +29,16 @@
- "{{ ssh_key_file }}" - "{{ ssh_key_file }}"
user_name: "{{ configure_instance_user | default('') }}" user_name: "{{ configure_instance_user | default('') }}"
# FIXME(dtantsur): openstacksdk issues a deprecation warning here, which
# somehow ends up in stdout, presumably because of Zuul quirks.
# Hack around it while we're solving the issue.
- name: Get instance info via CLI show - name: Get instance info via CLI show
command: metalsmith --format=json show test shell: |
errout="$(mktemp)"
if ! metalsmith --format=json show test 2> "$errout"; then
cat "$errout"
exit 1
fi
register: instance_info register: instance_info
- name: Register instance information - name: Register instance information
@ -38,8 +46,16 @@
instance: "{{ (instance_info.stdout | from_json).test }}" instance: "{{ (instance_info.stdout | from_json).test }}"
failed_when: instance.state != 'active' or instance.node.provision_state != 'active' failed_when: instance.state != 'active' or instance.node.provision_state != 'active'
# FIXME(dtantsur): openstacksdk issues a deprecation warning here, which
# somehow ends up in stdout, presumably because of Zuul quirks.
# Hack around it while we're solving the issue.
- name: Get instance info via CLI list - name: Get instance info via CLI list
command: metalsmith --format=json list shell: |
errout="$(mktemp)"
if ! metalsmith --format=json list 2> "$errout"; then
cat "$errout"
exit 1
fi
register: instance_info_via_list register: instance_info_via_list
- name: Verify that instance info via list is also correct - name: Verify that instance info via list is also correct