Skip to content

Sync zed into 2023.1 #909

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 29 commits into from
Feb 5, 2024
Merged

Sync zed into 2023.1 #909

merged 29 commits into from
Feb 5, 2024

Conversation

jovial
Copy link
Contributor

@jovial jovial commented Feb 2, 2024

No description provided.

markgoddard and others added 17 commits January 17, 2024 11:28
Adds two upgrade test jobs: Ubuntu Jammy OVS and Rocky 9 OVN.

These jobs reuse the existing aio reusable workflow, which has a new
'upgrade' input parameter. When testing an upgrade, the previous release
is first deployed, then the aio is upgraded to the current release
before running RefStack tests.
This will make it easier to automate various tasks.
We no longer require it since Ironic dropped the iscsi deploy driver.
Use the new kolla-images.py script to automatically define tag variables
in globals.yml based on the tags that are defined in kolla_image_tags.
The previous tag was not in Ark, but this one is present.
The failing case was the haproxy image used by the neutron_tls_proxy
container, if a tag is defined for the 'neutron' prefix but not
'haproxy'.
zed: automate Kolla Ansible tag variable definitions
@jovial jovial requested a review from a team as a code owner February 2, 2024 14:44
```
TASK [stackhpc.pulp.pulp_repository : include_tasks] ************************************************************************************************************************
Friday 02 February 2024  18:54:16 +0100 (0:00:00.189)       0:00:00.189 *******
fatal: [localhost]: FAILED! =>
  msg: |-
    The conditional check 'pulp_repository_container_repos | length > 0' failed. The error was: error while evaluating conditional (pulp_repository_container_repos | length
> 0): {{ stackhpc_pulp_repository_container_repos }}: {{ (stackhpc_pulp_repository_container_repos_kolla +
        stackhpc_pulp_repository_container_repos_ceph +
        stackhpc_pulp_repository_container_repos_hashicorp) | selectattr('required') }}: {%- set repos = [] -%} {%- for image in stackhpc_pulp_images_kolla_filtered -%} {%-
if image not in stackhpc_kolla_unbuildable_images[kolla_base_distro]-%} {%- set image_repo = kolla_docker_namespace ~ "/" ~ image -%} {%- set repo = {"name": image_repo} -%}
 {%- set _ = repos.append(stackhpc_pulp_repository_container_repos_kolla_common | combine(repo)) -%} {%- endif -%} {%- endfor -%} {{ repos }}: 'dict object' has no attribute
 'rocky'

    The error appears to be in '/home/stack/kayobe-env-zed/src/kayobe-config/etc/kayobe/ansible/collections/ansible_collections/stackhpc/pulp/roles/pulp_repository/tasks/mai
n.yml': line 2, column 3, but may
    be elsewhere in the file depending on the exact syntax problem.

    The offending line appears to be:

    ---
    - include_tasks: container.yml
      ^ here
```
@jovial jovial force-pushed the sync/zed-into-2023.1 branch from c9aa5de to e7a23b7 Compare February 2, 2024 18:57
jovial and others added 6 commits February 2, 2024 23:02
```
TASK [Query images and tags] ***************************************************
Friday 02 February 2024  20:39:16 +0000 (0:00:01.453)       0:00:01.529 *******
fatal: [localhost]: FAILED! => changed=false
  cmd:
  - /stack/kayobe-automation-env/src/kayobe-config/etc/kayobe/../../tools/kolla-images.py
  - list-tags
  delta: '0:00:00.324461'
  end: '2024-02-02 20:39:17.748038'
  msg: non-zero return code
  rc: 1
  start: '2024-02-02 20:39:17.423577'
  stderr: |-
    Traceback (most recent call last):
      File "/stack/kayobe-automation-env/src/kayobe-config/etc/kayobe/../../tools/kolla-images.py", line 372, in <module>
        main()
      File "/stack/kayobe-automation-env/src/kayobe-config/etc/kayobe/../../tools/kolla-images.py", line 364, in main
        list_tags(base_distros, kolla_image_tags)
      File "/stack/kayobe-automation-env/src/kayobe-config/etc/kayobe/../../tools/kolla-images.py", line 328, in list_tags
        image_tags = get_tags(base_distros, kolla_image_tags)
      File "/stack/kayobe-automation-env/src/kayobe-config/etc/kayobe/../../tools/kolla-images.py", line 199, in get_tags
        tag = get_tag(kolla_image_tags, base_distro, container)
      File "/stack/kayobe-automation-env/src/kayobe-config/etc/kayobe/../../tools/kolla-images.py", line 187, in get_tag
        return get_parent_tag(kolla_image_tags, base_distro, container)
      File "/stack/kayobe-automation-env/src/kayobe-config/etc/kayobe/../../tools/kolla-images.py", line 178, in get_parent_tag
        return kolla_image_tags[parent_tag_name][base_distro]
    KeyError: 'rocky-9'
  stderr_lines: <omitted>
  stdout: ''
  stdout_lines: <omitted>
```
Copy link
Contributor

@markgoddard markgoddard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but no CI to test :(

@jovial
Copy link
Contributor Author

jovial commented Feb 5, 2024

LGTM, but no CI to test :(

LGTM, but no CI to test :(

Thanks, for what it is worth all the non-upgrade jobs were passing: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/7762584458 in the run prior to 7b5f902. I'll wait a little bit to see if SMS comes back online before merging.

@markgoddard
Copy link
Contributor

LGTM, but no CI to test :(

LGTM, but no CI to test :(

Thanks, for what it is worth all the non-upgrade jobs were passing: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/7762584458 in the run prior to 7b5f902. I'll wait a little bit to see if SMS comes back online before merging.

I wouldn't hold your breath...

If the non-upgrade jobs passed let's merge.

@markgoddard markgoddard merged commit 7d00fbc into stackhpc/2023.1 Feb 5, 2024
@markgoddard markgoddard deleted the sync/zed-into-2023.1 branch February 5, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants