Skip to content
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

[test-operator] Move away from Jobs to Pods #2663

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
17a08ad
Log Ceph health and do not fail on HEALTH_WARN
fultonj Dec 17, 2024
5731e27
[test_operator] Common default values for container images
eduolivares Dec 11, 2024
84edd17
[DOCS] Add note about Zuul job reproducer support and testing
lewisdenny Dec 12, 2024
d4072ba
Save kuttl JUnitXML result files
tosky Dec 17, 2024
18f4002
Increase the SSHd reasy timeout
pablintino Dec 18, 2024
796278d
Adoption - hook post OSP 17.1 OC ironic
hjensas Dec 13, 2024
1cfce89
Bump golang.org/x/crypto
dependabot[bot] Dec 16, 2024
fe85ae8
[test_operator][tobiko] Download default customized guest image
eduolivares Dec 18, 2024
63f8e1b
Add RHOSO kubeconfigfile to shiftstackclient pod
itzikb-redhat Dec 11, 2024
19910b3
[ci_gen_kustomize_values] Allow reject vars patches
pablintino Dec 16, 2024
a3d4e43
Consider that the zuul executor may be an IP
pablintino Dec 16, 2024
843c6f0
Add nodeset2 for scenarios using 2 nodesets
Aug 29, 2024
e53622f
[shiftstack]Enable the option to run the role isolated
rlobillo Nov 26, 2024
fc30400
Support new OpenStack initialization resource for RHOSO install
abays Nov 15, 2024
607e0e6
Add hook to enable horizon
ashu-011 Nov 21, 2024
c2e4f9f
[autohold playbook] Add missing bracket
karelyatin Dec 24, 2024
556924a
Add the hypervisors to the inventory
pablintino Dec 20, 2024
a53fd0b
Pin cifmw dependencies
pablintino Nov 27, 2024
5a88ea6
Switch antelope branch to unmaintained/2023.1
karelyatin Dec 4, 2024
88bd379
Add retry to cephadm install
jirimacku Jan 8, 2025
501c9de
Quote the custom_fields paramtert to be shell safe
jirimacku Jan 8, 2025
52af24e
KUTTL: don't bail out immediately, save the test reports
tosky Jan 7, 2025
deb0100
[Test-operator] Set resource limits
lpiwowar Dec 18, 2024
bdf4ec8
Add hypervisor param to shiftstack playbook execution
eurijon Dec 27, 2024
4afdeec
Collect image for OpenStack init controller pod separately
abays Jan 6, 2025
8ff18c6
Refines codeowners file to add ownership of template files
frenzyfriday Dec 20, 2024
a86d3fe
Adoption ironic hook - IPA var/lib/ironic/httpboot
hjensas Dec 19, 2024
0484493
Add configuration variables for control plane testing during update.
sathlan Dec 18, 2024
7a168d2
[repo_setup] Download rhos-release using kerberos
pablintino Jan 7, 2025
3cd906d
Add possibility to run HorizonTest in debug mode
ashu-011 Dec 30, 2024
16394c4
Make requests_kerberos optional if endpoint is not secured
pablintino Jan 14, 2025
61287d8
Ensure EPEL is reinstalled
pablintino Jan 13, 2025
5852999
Adds cifmw squad to codeowners and owners files
frenzyfriday Jan 14, 2025
bb6ae8e
[repo_setup] Use a role scoped copy of url_request
pablintino Jan 14, 2025
84ac038
Remove auto draft GH action
pablintino Jan 16, 2025
53adb63
[url_request] Use gssapi lib by default
pablintino Jan 16, 2025
fe0f727
[test-operator] Move away from Jobs to Pods
lpiwowar Dec 13, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 15 additions & 2 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
@@ -1,15 +1,20 @@
# Default code owners
* @pablintino @lewisdenny
* @pablintino @lewisdenny @evallesp @bshewale @frenzyfriday

# Adoption
roles/adoption_osp_deploy @openstack-k8s-operators/adoption-core-reviewers

# BGP
roles/ci_gen_kustomize_values/templates/bgp @openstack-k8s-operators/bgp
roles/ci_gen_kustomize_values/templates/bgp_dt01 @openstack-k8s-operators/bgp

# Compliance
roles/compliance @openstack-k8s-operators/security

# DCN
roles/ci_dcn_site @openstack-k8s-operators/dcn
playbooks/dcn.yml @openstack-k8s-operators/dcn
roles/ci_gen_kustomize_values/templates/dcn @openstack-k8s-operators/dcn

# Must gather
roles/os_must_gather @openstack-k8s-operators/must-gather
Expand All @@ -18,7 +23,11 @@ roles/os_must_gather @openstack-k8s-operators/must-gather
roles/idrac_configuration @openstack-k8s-operators/nfv
roles/switch_config @openstack-k8s-operators/nfv
roles/tofu @eshulman2 @openstack-k8s-operators/nfv

roles/ci_gen_kustomize_values/templates/nfv-ovs-dpdk-sriov-hci @openstack-k8s-operators/nfv
roles/ci_gen_kustomize_values/templates/ovs-dpdk @openstack-k8s-operators/nfv
roles/ci_gen_kustomize_values/templates/ovs-dpdk-sriov @openstack-k8s-operators/nfv
roles/ci_gen_kustomize_values/templates/ovs-dpdk-sriov-2nodesets @openstack-k8s-operators/nfv
roles/ci_gen_kustomize_values/templates/sriov @openstack-k8s-operators/nfv

# Polarion
roles/polarion @tosky @ciecierski @jparoly @jirimacku
Expand All @@ -28,6 +37,7 @@ roles/reportportal @jirimacku @dsariel @sdatko

# Shiftstack
roles/shiftstack @rlobillo @eurijon
roles/ci_gen_kustomize_values/templates/shiftstack @rlobillo @eurijon

# Storage
roles/cifmw_block_device @openstack-k8s-operators/storage
Expand All @@ -42,6 +52,9 @@ roles/nat64_appliance @hjensas
roles/ci_lvms_storage @fultonj @fmount @abays
roles/ci_local_storage @fultonj @fmount @abays

# Uni
roles/ci_gen_kustomize_values/templates/uni* @openstack-k8s-operators/ciops

# Update
roles/update @openstack-k8s-operators/updates

Expand Down
35 changes: 0 additions & 35 deletions .github/workflows/auto-draft.yml

This file was deleted.

6 changes: 3 additions & 3 deletions OWNERS
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
approvers:
- abays
- bshewale
- cescgina
- evallesp
- frenzyfriday
- fultonj
- lewisdenny
- pablintino
- raukadah
- rlandy
- viroel

reviewers:
- adrianfusco
Expand Down
40 changes: 40 additions & 0 deletions ci/playbooks/kuttl/run-kuttl-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,8 @@
ansible.builtin.include_role:
name: 'install_yamls_makes'
tasks_from: 'make_{{ operator }}_kuttl.yml'
apply:
ignore_errors: true # noqa: ignore-errors

- name: 'Get resource status after {{ operator }}_kuttl run'
environment:
Expand All @@ -61,3 +63,41 @@
{{ item }} > {{ cifmw_artifacts_basedir }}/logs/cmd_after_{{ operator }}_kuttl.log
loop: "{{ commands_after_kuttl_run }}"
ignore_errors: true

# Some of the xml files may not be the JUnitXML results, but getting the exact file
# name (parsing the KUTTL configuration) may be too cumbersome.
- name: Find the generated JUnitXML files
ansible.builtin.find:
paths: "{{ cifmw_installyamls_repos }}"
file_type: file
patterns: '*.xml'
register: "_cifmw_kuttl_xml_files"

- name: Copy the log results of {{ operator }}
vars:
_kuttl_test_result_dir: "{{ cifmw_artifacts_basedir }}/tests/kuttl_{{ operator }}"
block:
- name: Create the test results directory
ansible.builtin.file:
path: "{{ _kuttl_test_result_dir }}"
state: directory
mode: '0755'

- name: Copy the generated test results to the test results directory
ansible.builtin.copy:
src: "{{ item.path }}"
dest: "{{ _kuttl_test_result_dir }}"
mode: '0644'
loop: "{{ _cifmw_kuttl_xml_files.files }}"
ignore_errors: true # noqa: ignore-errors

# Note: the variable checked here is set by the specific autogenerated task
# for the autogenerated install_yamls_makes role.
- name: Fail when any tests failed
vars:
_kuttl_curr_operator_result: "{{ lookup('vars', 'make_' + operator + '_kuttl_status')|default(1) }}"
ansible.builtin.assert:
that:
- _kuttl_curr_operator_result.rc == 0
success_msg: "KUTTL tests for {{ operator }} passed"
fail_msg: "KUTTL tests for {{ operator }} failed"
10 changes: 8 additions & 2 deletions ci/playbooks/multinode-autohold.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,16 @@
block:
- name: Fetch existing autoholds from zuul
vars:
_zuul_host: >-
{{
zuul.executor.hostname
if (zuul.executor.hostname is ansible.utils.ip) else
(zuul.executor.hostname | split('.'))[1:] | join('.')
}}
_zuul_api_url: >-
{{
[
('https://'+ (zuul.executor.hostname | split('.'))[1:] | join('.')),
('https://'+ _zuul_host),
'zuul',
'api',
'tenant',
Expand All @@ -32,7 +38,7 @@
] | join('/')
}}
ansible.builtin.uri:
url: "{{ _zuul_api_url }}"
url: "{{ zuul_autohold_endpoint | default(_zuul_api_url) }}"
method: GET
headers:
Content-Type: "application/json"
Expand Down
16 changes: 7 additions & 9 deletions common-requirements.txt
Original file line number Diff line number Diff line change
@@ -1,9 +1,7 @@
ansible-core>=2.14.0
oauthlib>=3.2.0 # k8s lib that requires manual upgrade
kubernetes
kubernetes-validate
openstacksdk
jsonschema>=4.20.0

# Allows to unpin cryptography
pyOpenSSL>=22.1.0
ansible-core==2.15.13
oauthlib==3.2.2
kubernetes==31.0.0
kubernetes-validate==1.31.0
openstacksdk==4.1.0
jsonschema==4.23.0
pyOpenSSL==24.2.1
12 changes: 12 additions & 0 deletions docs/dictionary/en-custom.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
aaabbcc
abcdefghij
addr
afuscoar
alertmanager
ansible
ansibleee
Expand Down Expand Up @@ -105,6 +106,8 @@ ctl
ctlplane
ctrl
ctx
cve
customizations
dashboard
dataplane
dataplanedeployments
Expand Down Expand Up @@ -164,6 +167,7 @@ fbqufbqkfbzxrja
fci
fedoraproject
fil
filesystem
fips
firewalld
flbxutz
Expand All @@ -173,6 +177,7 @@ freefonts
frmo
fsid
fultonj
fusco
fwcybtb
gapped
genericcloud
Expand Down Expand Up @@ -295,13 +300,15 @@ manpage
mawxlihjizaogicbjyxbzig
mawxlihjizcbwb
maxdepth
mcs
mellanox
metallb
metalsmith
mgmt
mins
minsizegigabytes
mlnx
mls
modprobe
mountpoints
mtcylje
Expand Down Expand Up @@ -463,7 +470,11 @@ scansettingbinding
scap
scp
sdk
selevel
selinux
serole
setype
seuser
sha
shiftstack
shiftstackclient
Expand All @@ -472,6 +483,7 @@ sizepercent
skbg
skiplist
specificities
spnego
spxzvbhvtzxmsihbyb
src
sshkey
Expand Down
2 changes: 2 additions & 0 deletions docs/source/baremetal/01_baremetal_hosts_data.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,8 @@ cifmw_baremetal_hosts:
nmstate:
# interfaces: # Sample nmstate state snippet
# - name: enp6s0f0.161
# Optional: (string) app label
label: openstack

compute-1:
# Another BM host
Expand Down
4 changes: 4 additions & 0 deletions docs/source/reproducers/03-zuul.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
# Zuul job

~~~{warning}
This feature isn't tested nor used by the CI Framework team. This feature will be supported and rewritten in our new version of the tool.
~~~

~~~{tip}
It is strongly advised to run this reproducer against a dedicated hypervisor
with enough resources. The current configuration will require close to 40G
Expand Down
28 changes: 14 additions & 14 deletions galaxy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,20 +71,20 @@ tags:
# L(specifiers,https://python-semanticversion.readthedocs.io/en/latest/#requirement-specification). Multiple version
# range specifiers can be set and are separated by ','
dependencies:
'git+https://github.com/containers/ansible-podman-collections': 'main'
'git+https://github.com/ansible-collections/community.general': 'main'
'git+https://github.com/ansible-collections/ansible.posix': 'main'
'git+https://github.com/ansible-collections/ansible.utils': 'main'
'git+https://github.com/ansible-collections/community.libvirt': 'main'
'git+https://github.com/ansible-collections/community.crypto': 'main'
'git+https://github.com/ansible-collections/kubernetes.core': 'main'
'git+https://github.com/ansible-collections/ansible.netcommon': 'main'
'git+https://github.com/openstack/ansible-config_template': 'master'
'git+https://github.com/ansible-collections/junipernetworks.junos': 'main'
'git+https://github.com/ansible-collections/cisco.ios': 'main'
'git+https://github.com/ansible-collections/mellanox.onyx': 'master'
'git+https://github.com/openshift/community.okd': 'main'
'git+https://github.com/ovirt/ovirt-ansible-collection': 'master'
'git+https://github.com/containers/ansible-podman-collections': '1.16.2'
'git+https://github.com/ansible-collections/community.general': '10.0.1'
'git+https://github.com/ansible-collections/ansible.posix': '1.6.2'
'git+https://github.com/ansible-collections/ansible.utils': 'v5.1.2'
'git+https://github.com/ansible-collections/community.libvirt': '1.3.0'
'git+https://github.com/ansible-collections/community.crypto': '2.22.3'
'git+https://github.com/ansible-collections/kubernetes.core': '5.0.0'
'git+https://github.com/ansible-collections/ansible.netcommon': 'v7.1.0'
'git+https://github.com/openstack/ansible-config_template': '2.1.1'
'git+https://github.com/ansible-collections/junipernetworks.junos': 'v9.1.0'
'git+https://github.com/ansible-collections/cisco.ios': 'v9.0.3'
'git+https://github.com/ansible-collections/mellanox.onyx': 'master' # Unmaintained, no tags
'git+https://github.com/openshift/community.okd': '4.0.0'
'git+https://github.com/ovirt/ovirt-ansible-collection': '3.2.0-1'

# The URL of the originating SCM repository
repository: https://github.com/openstack-k8s-operators/ci-framework
Expand Down
Loading
Loading