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.

Sign up for GitHub

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

Jump to bottom

Introduce cloud-init test for EC2 and Azure #19816

Merged

asmorodskyi merged 1 commit into os-autoinst:master from pdostal:cloud_init_init

Aug 5, 2024

Member

pdostal commented Jul 25, 2024 •

edited

Loading

Related ticket: poo#154501, poo#163907), poo#163904), poo#163910)
Verification run:
sle-15-SP6-EC2-Updates.aarch64 publiccloud_consoletests@64bit 22 1
sle-15-SP5-Azure-Basic-Updates.aarch64 publiccloud_consoletests@64bit 23
sle-15-SP6-EC2-Updates.aarch64 publiccloud_consoletests@64bit 23
sle-15-SP6-EC2-Updates.aarch64 publiccloud_consoletests@64bit 23
sle-15-SP5-Azure-Basic-Updates.aarch64 publiccloud_consoletests@64bit 23
sle-15-SP5-Azure-Standard-Updates.x86_64 publiccloud_consoletests@64bit 23
sle-15-SP5-Azure-Standard-Updates.x86_64 publiccloud_consoletests@64bit 23
sle-15-SP2-GCE-BYOS-Updates.x86_64 publiccloud_consoletests@64bit 22 1
sle-15-SP6-Azure-BYOS-Updates.x86_64 publiccloud_img_proof@64bit 24 2

pdostal requested review from grisu48, asmorodskyi, ricardobranco777, m-dati and mpagot as code owners

July 25, 2024 10:00

ricardobranco777 reviewed

View reviewed changes

lib/publiccloud/utils.pm Outdated Show resolved Hide resolved

lib/publiccloud/instance.pm Show resolved Hide resolved

ricardobranco777 reviewed

View reviewed changes

lib/publiccloud/utils.pm Show resolved Hide resolved

mloviska reviewed

View reviewed changes

lib/publiccloud/utils.pm Show resolved Hide resolved

ricardobranco777 approved these changes

View reviewed changes

mpagot reviewed

View reviewed changes

data/publiccloud/terraform/azure.tf Outdated Show resolved Hide resolved

mpagot reviewed

View reviewed changes

data/publiccloud/cloud-init.yaml Show resolved Hide resolved

mpagot reviewed

View reviewed changes

tests/publiccloud/check_services.pm Show resolved Hide resolved

mpagot approved these changes

View reviewed changes

Contributor

mpagot left a comment

(Not related to sles4sap but) LGTM

asmorodskyi reviewed

View reviewed changes

lib/publiccloud/provider.pm Outdated Show resolved Hide resolved

asmorodskyi reviewed

View reviewed changes

lib/publiccloud/utils.pm Outdated Show resolved Hide resolved

asmorodskyi reviewed

View reviewed changes

tests/publiccloud/prepare_instance.pm Outdated Show resolved Hide resolved

asmorodskyi reviewed

View reviewed changes

variables.md Outdated Show resolved Hide resolved

asmorodskyi reviewed

View reviewed changes

tests/publiccloud/prepare_instance.pm Show resolved Hide resolved

pdostal force-pushed the cloud_init_init branch 4 times, most recently from d2a8678 to da73c8f Compare

July 29, 2024 18:49

Member Author

pdostal commented Jul 29, 2024

I renamed PUBLIC_CLOUD_CLOUD_INIT_FILE to PUBLIC_CLOUD_CLOUD_INIT and made it boolean.

pdostal force-pushed the cloud_init_init branch from da73c8f to 6cb7317 Compare

July 30, 2024 13:21


          Introduce cloud-init test for EC2 and Azure

a7968af

pdostal force-pushed the cloud_init_init branch from 6cb7317 to a7968af Compare

July 30, 2024 13:48

asmorodskyi merged commit a9f6a03 into os-autoinst:master

11 checks passed

Member Author

pdostal commented Aug 6, 2024

Check https://gitlab.suse.de/qac/qac-openqa-yaml/-/merge_requests/1777 for enablement

pdostal mentioned this pull request

Revert "Introduce cloud-init test for EC2 and Azure" #19966

Merged

pdostal added a commit to pdostal/os-autoinst-distri-opensuse that referenced this pull request


          Revert "Merge pull request os-autoinst#19816 from pdostal/cloud_init_…

e4241ff

…init"

This reverts commit a9f6a03, reversing
changes made to ae91755.

pdostal added a commit to pdostal/os-autoinst-distri-opensuse that referenced this pull request


          Reapply "Merge pull request os-autoinst#19816 from pdostal/cloud_init…

…_init"

This reverts commit e4241ff.

pdostal mentioned this pull request

Introduce cloud-init test for EC2 and Azure #19969

Merged

pdostal added a commit to pdostal/os-autoinst-distri-opensuse that referenced this pull request


          Reapply "Merge pull request os-autoinst#19816 from pdostal/cloud_init…

2168fac

…_init"

This reverts commit e4241ff.

pdostal added a commit to pdostal/os-autoinst-distri-opensuse that referenced this pull request


          Reapply "Merge pull request os-autoinst#19816 from pdostal/cloud_init…

fe862e3

…_init"

This reverts commit e4241ff.

pdostal added a commit to pdostal/os-autoinst-distri-opensuse that referenced this pull request


          Reapply "Merge pull request os-autoinst#19816 from pdostal/cloud_init…

d31812e

…_init"

This reverts commit e4241ff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

asmorodskyi asmorodskyi left review comments

mloviska mloviska left review comments

ricardobranco777 ricardobranco777 approved these changes

mpagot mpagot approved these changes

grisu48 Awaiting requested review from grisu48 grisu48 is a code owner

m-dati Awaiting requested review from m-dati m-dati is a code owner

Labels

None yet