opendevreview | Merged openstack/tripleo-upgrade stable/train: Stop the upgrade script if some of the commands fail. https://review.opendev.org/c/openstack/tripleo-upgrade/+/802002 | 00:37 |
---|---|---|
opendevreview | Steve Baker proposed openstack/tripleo-quickstart-extras master: WIP Mount whole-disk lvm volumes in modify-image https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/801927 | 03:28 |
*** ykarel|away is now known as ykarel | 04:44 | |
opendevreview | Merged openstack/openstack-tempest-skiplist master: Skip failing scenario004-standalone-cinder-train test https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/802555 | 04:45 |
opendevreview | yatin proposed openstack/tripleo-quickstart master: [DNM] Test tripleo-ci release files in ci.centos gate job https://review.opendev.org/c/openstack/tripleo-quickstart/+/802533 | 04:55 |
*** marios is now known as marios|ruck | 05:49 | |
opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/wallaby: nova-compute: Adding failed_when to nova-compute stop task https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802466 | 06:05 |
opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/wallaby: Nova: Remove useless authtoken hieradata from computes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802467 | 06:08 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Ansible lint custom rule expansion https://review.opendev.org/c/openstack/tripleo-validations/+/798636 | 06:40 |
marios|ruck | needs reviews please https://review.opendev.org/c/openstack/tripleo-ansible/+/801615 | 06:45 |
Tengu | lemme check | 06:46 |
Tengu | marios|ruck: the depends-on is on a master patch, but iirc ooo-q is branchless? | 06:48 |
marios|ruck | Tengu: yes oooq is branchless, so we have conditionals in there that do stuff depending on which branch code is using it | 06:49 |
marios|ruck | Tengu: but what is the question | 06:49 |
Tengu | just wanted to be sure there wasn't a false value for the depends-on | 06:49 |
Tengu | patch is in gate :) | 06:49 |
marios|ruck | Tengu: no its fine, anyway merged now and also see commit message . The depends-on | 06:49 |
marios|ruck | is unrelated but needed to pass centos7 jobs in ci. | 06:50 |
marios|ruck | Tengu: thanks for checking | 06:50 |
Tengu | np | 06:52 |
*** rpittau|afk is now known as rpittau | 07:02 | |
*** amoralej|off is now known as amoralej | 07:07 | |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations stable/ussuri: DNM https://review.opendev.org/c/openstack/tripleo-validations/+/802662 | 07:25 |
ykarel | bandini, hi | 07:29 |
ykarel | bandini, can u check https://bugs.launchpad.net/tripleo/+bug/1937035 | 07:31 |
ykarel | on what causing those timeouts | 07:32 |
ykarel | in a patch it failed thrice in a row | 07:32 |
ykarel | i have a node held if u can check there | 07:32 |
opendevreview | Matthias Runge proposed openstack/tripleo-heat-templates master: Example configurable cap add https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801511 | 07:36 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations stable/train: DNM https://review.opendev.org/c/openstack/tripleo-validations/+/802666 | 07:40 |
marios|ruck | ykarel: please check https://bugs.launchpad.net/tripleo/+bug/1937035/comments/10 and file a new bug for the issue you are seeing please | 07:41 |
ykarel | marios|ruck, tempest is just fallback | 07:41 |
ykarel | issue is in pcs timeouts | 07:41 |
ykarel | so underlying issue is same, but ok to file seperate bug | 07:42 |
marios|ruck | ykarel: sure so lets track a new issue for it please so we can get a clean cix if needed | 07:42 |
marios|ruck | ykarel: thanks | 07:42 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: callback adjustment for no hosts matched https://review.opendev.org/c/openstack/validations-common/+/800169 | 07:46 |
opendevreview | Yariv proposed openstack/tripleo-upgrade master: [FFWD upgrade] Adding support of live/cold migration with external workload https://review.opendev.org/c/openstack/tripleo-upgrade/+/802524 | 07:51 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations stable/ussuri: DNM https://review.opendev.org/c/openstack/tripleo-validations/+/802662 | 07:54 |
opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Move build-test-packages as a part of undercloud setup https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/790926 | 07:59 |
opendevreview | chandan kumar proposed openstack/tripleo-heat-templates master: [DNM] tripleo-operator-ansible rpm testing https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801296 | 08:00 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: callback adjustment for no hosts matched https://review.opendev.org/c/openstack/validations-common/+/800169 | 08:33 |
*** ykarel is now known as ykarel|lunch | 08:34 | |
*** zbr is now known as Guest2544 | 09:09 | |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Ansible lint custom rule expansion https://review.opendev.org/c/openstack/tripleo-validations/+/798636 | 09:22 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible master: DNM check ansible 2.11 with tripleo-ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/802673 | 09:26 |
marios|ruck | need vote here please https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/801736 test at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802618 (scen4 tested) | 09:37 |
marios|ruck | thank you sshnaidm|afk chandankumar | 09:40 |
*** sshnaidm|afk is now known as sshnaidm | 09:45 | |
*** pojadhav is now known as pojadhav|brb | 09:53 | |
opendevreview | Marios Andreou proposed openstack/openstack-tempest-skiplist master: Revert "Skip tempest.scenario.test_object_storage_basic_ops.TestObjectStorageBasicOps" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/801584 | 09:53 |
opendevreview | Merged openstack/openstack-tempest-skiplist master: Revert "Skip manila_tempest_tests.tests.api.test_shares.SharesNFSTest test" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/801736 | 09:55 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible master: DNM check ansible 2.11 with tripleo-ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/802673 | 10:04 |
*** ykarel|lunch is now known as ykarel | 10:17 | |
opendevreview | Juan Badia Payno proposed openstack/tripleo-quickstart-extras master: WIP Create overcloud_baremetal_deploy.yaml file properly https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797452 | 10:20 |
opendevreview | Marios Andreou proposed openstack/openstack-tempest-skiplist master: Revert "Skip tempest.scenario.test_object_storage_basic_ops.TestObjectStorageBasicOps" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/801584 | 10:24 |
opendevreview | Marios Andreou proposed openstack/tripleo-heat-templates master: NO MERGE test scen4 tempest skip revert for related-bug https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802679 | 10:24 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: Removing validations-libs from test-requirements https://review.opendev.org/c/openstack/validations-common/+/802680 | 10:26 |
opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/train: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802469 | 10:34 |
opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802470 | 10:35 |
opendevreview | yatin proposed openstack/tripleo-quickstart master: Use dnf config-manager in CentOS8 releases https://review.opendev.org/c/openstack/tripleo-quickstart/+/802681 | 10:39 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Unit test refactor https://review.opendev.org/c/openstack/validations-libs/+/802682 | 10:39 |
*** jcapitao is now known as jcapitao_lunch | 10:42 | |
opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates master: Parameter to set api policy to limit live migration to role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802027 | 10:54 |
*** pojadhav|brb is now known as pojadhav | 11:03 | |
opendevreview | Lukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [FFWD][train-only] Ensure we get ovn_controller cert if needed https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798843 | 11:35 |
opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: WIP ceph systemd call managed https://review.opendev.org/c/openstack/tripleo-ansible/+/802688 | 11:44 |
opendevreview | Sorin Sbârnea proposed openstack/ansible-role-collect-logs master: Upgrade ansible-lint/molecule runs https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/792652 | 11:44 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Ansible lint custom rule expansion https://review.opendev.org/c/openstack/tripleo-validations/+/798636 | 11:47 |
*** rlandy is now known as rlandy|ruck | 11:52 | |
*** jcapitao_lunch is now known as jcapitao | 12:19 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/802693 | 12:25 |
ykarel | marios|ruck, bandini reported https://bugs.launchpad.net/tripleo/+bug/1938283 | 12:29 |
ykarel | added the cause for it | 12:30 |
ykarel | amoralej, fyi ^ | 12:31 |
*** pojadhav is now known as pojadhav|afk | 12:31 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/802694 | 12:32 |
amoralej | ykarel, so that timeout change is root cause for the issues with pacemaker we have been hitting lastly | 12:33 |
ykarel | amoralej, yes 20s is much smaller to see such timeout | 12:33 |
ykarel | in previous bug i see it failed even at 120s in wallaby, but that's rare, in master it's more frequent due to that low setting | 12:34 |
ykarel | ramishra, fyi ^ as caused with https://review.opendev.org/c/openstack/tripleo-heat-templates/+/791416 | 12:37 |
* ykarel prepares fix | 12:38 | |
marios|ruck | ykarel: thank you in call will check in bit | 12:41 |
marios|ruck | ykarel: promotion blocker? | 12:42 |
ykarel | marios|ruck, yes | 12:42 |
ykarel | promotion jobs also impacted | 12:42 |
ykarel | added logs | 12:43 |
marios|ruck | ykarel: ack i see one from the 26th at https://logserver.rdoproject.org/openstack-periodic-integration-main/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-master/3a8026d/logs/overcloud-controller-0/var/log/ the others are few days old | 12:43 |
ykarel | yes happening randomly from few months | 12:43 |
opendevreview | yatin proposed openstack/tripleo-heat-templates master: Fix condition for pacemaker resource_op_defaults https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802696 | 12:49 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: Reducing the sensitivity of the coverchange job https://review.opendev.org/c/openstack/validations-common/+/802693 | 12:53 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-validations master: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/794866 | 12:57 |
*** artom_ is now known as artom | 12:58 | |
opendevreview | David Vallee Delisle proposed openstack/tripleo-validations master: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/794866 | 13:04 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/802694 | 13:09 |
*** amoralej is now known as amoralej|lunch | 13:16 | |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible master: DNM check ansible 2.11 with tripleo-ansible https://review.opendev.org/c/openstack/tripleo-ansible/+/802673 | 13:25 |
opendevreview | Jesse Pretorius proposed openstack/tripleo-heat-templates stable/train: Remove libvirt package from overcloud nodes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800912 | 13:25 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/802694 | 13:26 |
marios|ruck | dvd: o/ hey waiting to hear back from infra but looks like your patch is stuck in the gate we may have to abandon and restore (hold for now please though) | 13:27 |
marios|ruck | dvd: if there is a better way we'll take it otherwise we will do that in a few mins ^^^ https://zuul.openstack.org/status#800848 that patch | 13:27 |
marios|ruck | https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800848/ | 13:27 |
* dvd is looking | 13:28 | |
rlandy|ruck | all: ^^ note stuck gate - may require reset | 13:28 |
dvd | marios|ruck, I see some progress in the pytest jobs | 13:29 |
marios|ruck | :( whole bunch of green runs below that will get thrown away if we abandon/restore | 13:29 |
marios|ruck | dvd: https://zuul.openstack.org/status#800848 | 13:29 |
dvd | marios|ruck, apparentlysomeone unstuck it? | 13:29 |
marios|ruck | dvd: the tripleo-ci-centos-8-scenario000-multinode-oooq-container-updates is queued | 13:29 |
marios|ruck | dvd: it is stuck | 13:29 |
dvd | oh wait | 13:29 |
dvd | you're right | 13:30 |
marios|ruck | dvd: the rest of the jobs there are good | 13:30 |
marios|ruck | dvd: but that one is just sitting there | 13:30 |
dvd | ack let me know if I need to abandon/restore | 13:30 |
marios|ruck | dvd: ack don't do anythign please | 13:30 |
marios|ruck | dvd: fungi: is checking for us now | 13:30 |
marios|ruck | thank you fungi ! | 13:30 |
fungi | yeah, i should be able to let you know momentarily | 13:31 |
marios|ruck | ack anything is worth a try but if we really have to we'll just abandon thank you fungi take your time | 13:31 |
marios|ruck | thanks rlandy|ruck++ completely missed that ;) | 13:33 |
*** amoralej|lunch is now known as amoralej | 13:38 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/802694 | 13:43 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible master: Convert all OS module imports to collections imports https://review.opendev.org/c/openstack/tripleo-ansible/+/802715 | 13:45 |
fungi | marios|ruck: dvd: it's running now | 13:58 |
fungi | rlandy|ruck: ^ | 13:58 |
dvd | fungi, nice thanks | 13:58 |
rlandy|ruck | fungi++ | 13:58 |
dvd | fungi++ | 13:58 |
rlandy|ruck | thank you!! | 13:58 |
rlandy|ruck | really didn't want to rerun the whole gate line | 13:58 |
fungi | no problem, sorry we still haven't found whatever causes nodepool launchers to sometimes squat node request locks indefinitely | 13:58 |
marios|ruck | fungi: thank you! | 13:59 |
fungi | any time | 13:59 |
marios|ruck | fungi++ all the beers :) | 13:59 |
marios|ruck | fungi: saving a bit of the planet in the process so many jobs would have had to rerun ;) | 14:00 |
marios|ruck | need another vote on this please https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/801584 (tested at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802679 scen4) cc Tengu around? :) | 14:01 |
marios|ruck | chandankumar: maybe ? ^ please thank you | 14:01 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Default value of the constructor parameter https://review.opendev.org/c/openstack/validations-libs/+/796482 | 14:02 |
marios|ruck | thanks chandankumar | 14:02 |
opendevreview | Matthias Runge proposed openstack/tripleo-heat-templates master: Example configurable cap add https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801511 | 14:03 |
opendevreview | Merged openstack/openstack-tempest-skiplist master: Revert "Skip tempest.scenario.test_object_storage_basic_ops.TestObjectStorageBasicOps" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/801584 | 14:10 |
kashyap | ykarel: Hi, I was told you'll be able to put a machine on hold to debug these failures: https://review.rdoproject.org/r/c/testproject/+/32157 | 14:24 |
ykarel | kashyap, sure adding to autohold | 14:24 |
kashyap | ykarel: Thank you. So this box a physical machine, or a VM? | 14:25 |
ykarel | kashyap, vm | 14:26 |
ykarel | kashyap, added https://softwarefactory-project.io/zuul/api/tenant/rdoproject.org/autohold, so if that job fails it will get held | 14:27 |
ykarel | your public key? | 14:27 |
kashyap | ykarel: Hmm; so a level-1 VM (which is running Compute node, and Nova instances are level-2 VMs as nested) | 14:27 |
kashyap | ykarel: Public key: https://paste.centos.org/view/6d1b3df0 | 14:28 |
kashyap | ykarel: This is also an AMD host, I'm hoping - otherwise it's not useful, afraid. | 14:29 |
ykarel | kashyap, ssh zuul@zuul@38.102.83.158 | 14:29 |
ykarel | i see svm in /proc/cpuinfo | 14:30 |
ykarel | AMD EPYC 7402 24-Core Processor | 14:30 |
kashyap | Okay; thanks. I'll see if I can get anything useful out of it | 14:31 |
opendevreview | yatin proposed openstack/tripleo-heat-templates master: Fix condition for pacemaker resource_op_defaults https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802696 | 14:31 |
kashyap | ykarel: I got permission denied when I try to SSH | 14:31 |
ykarel | kashyap, added shared keys in authorized_keys | 14:34 |
ykarel | can u try again | 14:34 |
* kashyap tries | 14:34 | |
kashyap | ykarel: Nope; still no luck | 14:35 |
kashyap | ykarel: That's the last 29 lines of SSH: https://paste.centos.org/view/62a6837a | 14:36 |
ykarel | kashyap, sorry for typo earlier, try ssh zuul@38.102.83.158 | 14:39 |
ykarel | earlier user was specified twice | 14:39 |
kashyap | Ah, right; I thought it's part of the weird host name | 14:40 |
kashyap | ykarel: Okay; I'm in | 14:41 |
*** ykarel is now known as ykarel|away | 14:45 | |
opendevreview | Ronelle Landy proposed openstack/openstack-tempest-skiplist master: Add back tripleo component tests in skip https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/802733 | 14:52 |
rlandy|ruck | marios|ruck: ^^ pls vote | 14:52 |
rlandy|ruck | should sort out the issues in ci.centos on the next promotion | 14:52 |
opendevreview | Julia Kreger proposed openstack/tripleo-common master: Add dnsmasq to ironic-pxe container https://review.opendev.org/c/openstack/tripleo-common/+/802736 | 14:54 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Redundant conditional removal https://review.opendev.org/c/openstack/validations-libs/+/802737 | 14:55 |
marios|ruck | rlandy|ruck: acv | 14:57 |
marios|ruck | rlandy|ruck: ack :_ | 14:57 |
rlandy|ruck | testprojecting those tests | 14:58 |
opendevreview | chandan kumar proposed openstack/python-tripleoclient master: [DNM] tripleo-ansible-operator changes testing https://review.opendev.org/c/openstack/python-tripleoclient/+/802739 | 14:58 |
rlandy|ruck | let's see if we can clear the tripleo component lines | 14:58 |
marios|ruck | thank you rlandy|ruck | 14:59 |
rlandy|ruck | :) | 15:03 |
ykarel|away | jpodivin, i see u have couple of bugs filed for tempest failures, i commented on all of them as those looks duplicate of https://bugs.launchpad.net/tripleo/+bug/1938283 | 15:07 |
ykarel|away | please check and if agree can close those as duplicate | 15:07 |
jpodivin | ykarel|away: I'll take a look tomorrow if that is fine with you. I'm about to leave :) | 15:08 |
ykarel|away | jpodivin, sure nothing urgent, just cleanup | 15:08 |
kashyap | ykarel|away: Hi, it doesn't contain a TripleO deployment already | 15:09 |
opendevreview | Merged openstack/openstack-tempest-skiplist master: Add back tripleo component tests in skip https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/802733 | 15:09 |
kashyap | ykarel|away: How do I get it there? (Sorry for the questionw when you're about to head out) | 15:09 |
kashyap | s/questionw/questions/ | 15:09 |
ykarel|away | kashyap, deployment is in progress https://review.rdoproject.org/zuul/stream/3e70704ca5b94f078dd201929782e5a3?logfile=console.log | 15:09 |
kashyap | ykarel|away: Oh, I see | 15:10 |
kashyap | ykarel|away: How long will it take; and how long can I have the VM for debugging? | 15:10 |
ykarel|away | kashyap, can also check logs at tail -f standalone_deploy.log, once deployment finishes tempest will run | 15:11 |
kashyap | ykarel|away: I need to head out now; will check in a bit. Thanks! | 15:12 |
ykarel|away | kashyap, will be there atleast till tomorrow same time, not sure for exact time | 15:12 |
kashyap | Okay, noted | 15:12 |
opendevreview | Alex Schultz proposed openstack/tripleo-ansible master: Action module for bulk host prep https://review.opendev.org/c/openstack/tripleo-ansible/+/799712 | 15:12 |
opendevreview | Julia Kreger proposed openstack/tripleo-heat-templates master: Swap xinetd/tftp-server for dnsmasq to support ironic-pxe operations https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802740 | 15:17 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-ansible master: Configure vhost-postcopy-support https://review.opendev.org/c/openstack/tripleo-ansible/+/802742 | 15:24 |
rlandy|ruck | mwhahaha: back with image upload on OVB ... all releases are passing now except ussuri which is passing the '--local' option and failing: https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset035-ussuri/942ccd1/logs/undercloud/home/zuul/overcloud_image_upload.log.txt.gz | 15:28 |
mwhahaha | yea i saw that | 15:28 |
rlandy|ruck | expect for fs002 https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-1ctlr_1comp-featureset002-ussuri/1fdac31/logs/undercloud/home/zuul/overcloud-image-upload.sh.txt.gz which doesn't pass --local and passes | 15:29 |
rlandy|ruck | what is the correct setting for ussuri? | 15:29 |
mwhahaha | it doesn't appear so | 15:29 |
rlandy|ruck | do we need a setting change or is there a bug? | 15:29 |
mwhahaha | so it looks like ussuri needs to not have --local even though nova is disabled | 15:29 |
rlandy|ruck | ok - if that is correct, I will fix the fs setting accordingly | 15:29 |
rlandy|ruck | thanks | 15:29 |
rlandy|ruck | marios|ruck: ^^ putting in fix for ussuri OVB | 15:30 |
marios|ruck | rlandy|ruck: thanks ping if need review | 15:31 |
*** chem is now known as Guest2598 | 15:36 | |
rlandy|ruck | sshnaidm: is it possible to set a var for an operator from a featureset? | 15:45 |
rlandy|ruck | https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/overcloud-prep-images/tasks/overcloud-prep-images.yml#L24 | 15:45 |
rlandy|ruck | ^^ want to change that value just for ussuri | 15:45 |
rlandy|ruck | ie: keep undercloud_enable_nova asits value | 15:46 |
rlandy|ruck | as its | 15:46 |
rlandy|ruck | and change tripleo_overcloud_image_upload_local only | 15:46 |
opendevreview | Merged openstack/tripleo-heat-templates master: tripleo-podman: Enable podman.socket https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800848 | 15:48 |
opendevreview | Merged openstack/puppet-tripleo stable/wallaby: Do not include client classes https://review.opendev.org/c/openstack/puppet-tripleo/+/800186 | 15:48 |
opendevreview | Merged openstack/tripleo-heat-templates master: Use yaml.safe_load to load YAML files https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801850 | 15:48 |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Remove leftover of hieradata cleanup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802205 | 15:48 |
opendevreview | Merged openstack/tripleo-heat-templates master: Enable archive task_log records while archiving the database https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801939 | 15:48 |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: libvirt: fix formatting of the log_outputs parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/792764 | 15:48 |
opendevreview | Merged openstack/tripleo-heat-templates master: Nova: Remove unused password parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/795012 | 15:48 |
opendevreview | Merged openstack/tripleo-ansible stable/train: More descriptive task names https://review.opendev.org/c/openstack/tripleo-ansible/+/801729 | 15:48 |
opendevreview | Merged openstack/tripleo-ansible stable/train: Add support for v2 registries.conf https://review.opendev.org/c/openstack/tripleo-ansible/+/801615 | 15:48 |
opendevreview | Merged openstack/tripleo-ci master: Adds wallaby upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/796771 | 15:48 |
opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Remove all docker related tasks from upgrades https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802215 | 15:48 |
opendevreview | Merged openstack/tripleo-heat-templates master: [c9] nova_libvirt with host's cgroupns https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802489 | 15:48 |
sshnaidm | rlandy|ruck, yep, I think it works | 15:49 |
rlandy|ruck | tripleo_overcloud_image_upload_local: "{% if release in ['ussuri'] %}false{% else %}{{ not undercloud_enable_nova|default(false) }}{% endif %}" | 15:49 |
rlandy|ruck | otheriwise ^^ | 15:49 |
sshnaidm | just {{ release in ['ussuri'] }} | 15:50 |
sshnaidm | or {{ release not in ['ussuri'] }} | 15:50 |
marios|ruck | rlandy|ruck: its already set from the featureset https://opendev.org/openstack/tripleo-quickstart/src/commit/c9bc2dd1df3ce7723f1907794beabada9dd04186/config/general_config/featureset039.yml#L14 | 15:50 |
marios|ruck | rlandy|ruck: can just set it there? ^ | 15:50 |
marios|ruck | rlandy|ruck: for the relevant featureset (also that should point to line 190 for undercloud_enable_nova https://opendev.org/openstack/tripleo-quickstart/src/commit/c9bc2dd1df3ce7723f1907794beabada9dd04186/config/general_config/featureset039.yml#L190 ) | 15:51 |
marios|ruck | rlandy|ruck: instead of adding convoluted special case into the operator args | 15:51 |
rlandy|ruck | I want to separate the condition and the undercloud_enable_nova setting | 15:51 |
marios|ruck | rlandy|ruck: oh i thought you wanted to make undercloud_enable_nova false generally | 15:52 |
marios|ruck | rlandy|ruck: then do it directly in vars if you dont want it to depend on undercloud_enable_nova sorry | 15:52 |
rlandy|ruck | per comment above "so it looks like ussuri needs to not have --local even though nova is disabled" | 15:52 |
rlandy|ruck | for ussuri the case is diff | 15:52 |
sshnaidm | rlandy|ruck, then your code above should work | 15:53 |
rlandy|ruck | tripleo_overcloud_image_upload_local: "{% if release in ['ussuri'] %}false{% else %}{{ not undercloud_enable_nova|default(false) }}{% endif %} | 15:53 |
rlandy|ruck | ^^ that one? | 15:53 |
marios|ruck | yeah | 15:53 |
rlandy|ruck | k | 15:53 |
rlandy|ruck | let's try it | 15:53 |
sshnaidm | yep | 15:53 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Allow customizing reload commands https://review.opendev.org/c/openstack/tripleo-ansible/+/801216 | 15:55 |
*** amoralej is now known as amoralej|off | 15:57 | |
opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add ussuri case for --local in image upload https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/802747 | 15:57 |
opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: WIP ceph systemd call managed https://review.opendev.org/c/openstack/tripleo-ansible/+/802688 | 16:04 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: Address setuptools deprecation warnings https://review.opendev.org/c/openstack/tripleo-repos/+/802750 | 16:08 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ansible master: Convert all OS module imports to collections imports https://review.opendev.org/c/openstack/tripleo-ansible/+/802715 | 16:10 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: setup.cfg: Use underscore instead of dash https://review.opendev.org/c/openstack/tripleo-repos/+/802750 | 16:12 |
opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: WIP ceph systemd call managed https://review.opendev.org/c/openstack/tripleo-ansible/+/802688 | 16:16 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: setup.cfg: Use underscore instead of dash https://review.opendev.org/c/openstack/tripleo-repos/+/802750 | 16:20 |
*** rpittau is now known as rpittau|afk | 16:21 | |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: setup.cfg: Use underscore instead of dash https://review.opendev.org/c/openstack/tripleo-repos/+/802750 | 16:21 |
opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: WIP ceph systemd call managed https://review.opendev.org/c/openstack/tripleo-ansible/+/802688 | 16:22 |
opendevreview | Marios Andreou proposed openstack/tripleo-specs master: Moving TripleO repos to independent release model https://review.opendev.org/c/openstack/tripleo-specs/+/801512 | 16:37 |
*** marios|ruck is now known as marios|out | 16:38 | |
opendevreview | Merged openstack/tripleo-validations stable/train: Corrected the task 'Get file_descriptors total_limit' to get the 'fs.file-max' value. https://review.opendev.org/c/openstack/tripleo-validations/+/802250 | 16:42 |
*** pojadhav is now known as pojadhav|out | 16:57 | |
opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: DNM Enabling vhost-postcopy-support when live_migration_permit_post_copy https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802760 | 16:58 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: Enable ansible-test sanity checks (nv) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 17:09 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/train: [train-only] Don't auto-enable post-copy in DPDK deployment https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802764 | 17:10 |
mwhahaha | rlandy|ruck: so i think our assumptions about ussuri are incorrect, something else is happening | 17:38 |
mwhahaha | https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset035-ussuri/942ccd1/logs/undercloud/home/zuul/tripleo_overcloud_image_upload.sh.txt.gz | 17:38 |
mwhahaha | https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-1ctlr_1comp-featureset002-ussuri/1fdac31/logs/undercloud/home/zuul/overcloud-image-upload.sh.txt.gz | 17:38 |
mwhahaha | they are the same | 17:38 |
mwhahaha | i think something else is happening | 17:38 |
mwhahaha | https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-1ctlr_1comp-featureset002-ussuri/1fdac31/logs/undercloud/home/zuul/tripleo_overcloud_image_upload.sh.txt.gz | 17:40 |
mwhahaha | fs002 upload was fine | 17:41 |
opendevreview | Merged openstack/tripleo-ansible stable/victoria: Fix tripleo_backup_and_restore_rear_simulate boolean check https://review.opendev.org/c/openstack/tripleo-ansible/+/802341 | 17:42 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: DNM Enabling vhost-postcopy-support when live_migration_permit_post_copy https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802760 | 17:42 |
* rlandy|ruck looks | 17:43 | |
opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: DNM Enabling vhost-postcopy-support when live_migration_permit_post_copy https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802760 | 17:43 |
mwhahaha | rlandy|ruck: i don't know where those jobs came from so i went back to look at the ipa job test and it seemed that they are different | 17:44 |
mwhahaha | https://logserver.rdoproject.org/95/24995/71/check/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-ussuri/41f09b3/logs/undercloud/home/zuul/overcloud-image-upload.sh.txt.gz | 17:44 |
mwhahaha | https://logserver.rdoproject.org/95/24995/71/check/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-ussuri/41f09b3/logs/undercloud/home/zuul/tripleo_overcloud_image_upload.sh.txt.gz | 17:44 |
mwhahaha | enable_nova is false | 17:45 |
mwhahaha | do we not set the undercloud_enable_nova to false in ussuri configs? | 17:45 |
mwhahaha | https://logserver.rdoproject.org/95/24995/71/check/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-ussuri/41f09b3/logs/undercloud/home/zuul/undercloud.conf.txt.gz | 17:45 |
rlandy|ruck | confirming line failure failure first - then the test job ... | 17:46 |
rlandy|ruck | https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset035-ussuri/942ccd1/logs/undercloud/home/zuul/overcloud-image-upload.sh.txt.gz | 17:47 |
rlandy|ruck | ^^ what I see in the last fs035 ussuri logs | 17:47 |
rlandy|ruck | and in fs002 | 17:47 |
mwhahaha | right so comparing the two files in the same job, the t-o-a version doesn't have --local https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset035-ussuri/942ccd1/logs/undercloud/home/zuul/tripleo_overcloud_image_upload.sh.txt.gz | 17:47 |
mwhahaha | which is why it fails | 17:48 |
mwhahaha | but that seems weird because it should have true if undercloud_enable_nova is set to false | 17:48 |
mwhahaha | https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset035-ussuri/942ccd1/logs/undercloud/home/zuul/undercloud.conf.txt.gz | 17:48 |
mwhahaha | enable_nova = false | 17:48 |
mwhahaha | so it feels like that's coming from something else in ussuri | 17:48 |
rlandy|ruck | fs002 does not have the local setting | 17:49 |
rlandy|ruck | so the difference I was looking at is in the featureset | 17:49 |
mwhahaha | right but in fs002, the two upload scripts are teh same | 17:50 |
rlandy|ruck | https://github.com/openstack/tripleo-quickstart/blob/master/config/general_config/featureset035.yml#L196 | 17:50 |
rlandy|ruck | which has a undercloud_enable_nova: >- setting | 17:50 |
rlandy|ruck | but fs002 does not | 17:50 |
mwhahaha | right so fs002 is using nova (the default in U) | 17:51 |
mwhahaha | and fs035 is using metalsmith | 17:51 |
rlandy|ruck | correct | 17:51 |
rlandy|ruck | so we're inconsistent | 17:51 |
mwhahaha | but for some reason the logic you have in quickstart is not being picked up correctly | 17:51 |
rlandy|ruck | (how unsual) | 17:51 |
mwhahaha | it's fine that they test different things | 17:51 |
mwhahaha | the issue seems to be why does the logic fail for fs035 | 17:51 |
mwhahaha | but i looked at the failres from yesterday for https://review.rdoproject.org/r/c/testproject/+/24995 | 17:52 |
mwhahaha | and fs001 is also inconsistent in terms of the image upload bits | 17:52 |
rlandy|ruck | https://review.rdoproject.org/zuul/stream/0acd6bc8e6554459a0719145c62dc9b1?logfile=console.log running in the testproject now reaches overcloud deploy | 17:53 |
* rlandy|ruck files bug so we can track this | 17:54 | |
rlandy|ruck | https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/802747/1/roles/overcloud-prep-images/tasks/overcloud-prep-images.yml#24 is a bad hack | 17:54 |
mwhahaha | yea that's wrong | 17:55 |
mwhahaha | do we have the ansible vars dumpped somewhere in the log output? | 17:55 |
rlandy|ruck | yeah | 17:55 |
mwhahaha | i'm looking at https://logserver.rdoproject.org/95/24995/71/check/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-ussuri/41f09b3/logs/ | 17:56 |
mwhahaha | role-vars.yaml is empty | 17:56 |
rlandy|ruck | check var to see if have something there | 17:57 |
mwhahaha | undercloud_enable_nova: 'false' | 17:57 |
mwhahaha | it's a string | 17:57 |
mwhahaha | you need to |bool|default | 17:57 |
mwhahaha | i think? | 17:57 |
rlandy|ruck | https://logserver.rdoproject.org/openstack-periodic-integration-stable3/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-1ctlr_1comp-featureset002-ussuri/7c677c9/logs/undercloud/var/log/extra/dump_variables_vars.yaml.txt.gz | 17:58 |
rlandy|ruck | general vars if that helps | 17:58 |
mwhahaha | i think the issue is we need to |bool it | 17:58 |
rlandy|ruck | can try - funny it works on other releases | 17:58 |
rlandy|ruck | trying new patch with |bool | 17:59 |
rlandy|ruck | will see if that addresses the problem | 17:59 |
rlandy|ruck | sec - just resetting 17 | 18:01 |
mwhahaha | yea sure | 18:01 |
mwhahaha | i'm testing some stuff | 18:01 |
mwhahaha | eh that should be fine | 18:01 |
mwhahaha | wtf | 18:01 |
opendevreview | Merged openstack/tripleo-ansible stable/ussuri: Fix tripleo_backup_and_restore_rear_simulate boolean check https://review.opendev.org/c/openstack/tripleo-ansible/+/802342 | 18:03 |
rlandy|ruck | k - back | 18:04 |
rlandy|ruck | so where are we are now? | 18:04 |
rlandy|ruck | |bool not needed | 18:04 |
mwhahaha | yea shouldn't be needed | 18:04 |
mwhahaha | i wonder if it's a vars precidence thing | 18:08 |
mwhahaha | because undercloud_enable_nova is defined in extras-common/defaults.yml | 18:08 |
mwhahaha | and set to true for ussuri | 18:08 |
mwhahaha | which would account for this | 18:08 |
* mwhahaha doesn't like that role | 18:09 | |
rlandy|ruck | hmmm ... defaults should have lower priority | 18:09 |
mwhahaha | should (tm) | 18:09 |
opendevreview | Alex Schultz proposed openstack/tripleo-quickstart-extras master: DNM: debug undercloud_enable_nova https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/802767 | 18:11 |
rlandy|ruck | the setting takes - just that --local fails in ussuri | 18:11 |
mwhahaha | no it shouldn't | 18:11 |
mwhahaha | it should work | 18:11 |
mwhahaha | per https://logserver.rdoproject.org/95/24995/71/check/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-ussuri/41f09b3/logs/undercloud/home/zuul/overcloud-image-upload.sh.txt.gz | 18:11 |
mwhahaha | it's what used to be there | 18:11 |
mwhahaha | it's failing because it's *not* passing --local but enable_nova is false | 18:11 |
rlandy|ruck | alright ... let's rekick the test project with https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/802767 | 18:13 |
rlandy|ruck | and see what value we are looking at | 18:14 |
mwhahaha | yea sec | 18:14 |
mwhahaha | https://review.rdoproject.org/r/c/testproject/+/34725 | 18:14 |
rlandy|ruck | ok | 18:14 |
opendevreview | Merged openstack/tripleo-ansible stable/victoria: Use serial or parallel execution depending on the kind of backup https://review.opendev.org/c/openstack/tripleo-ansible/+/802217 | 18:30 |
opendevreview | Merged openstack/tripleo-ansible stable/ussuri: Use serial or parallel execution depending on the kind of backup https://review.opendev.org/c/openstack/tripleo-ansible/+/802338 | 18:31 |
opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates master: Add a role to run independent designate bind backends https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801620 | 18:42 |
mwhahaha | rlandy|ruck: ok so i confirmed that undercloud_enable_nova is false | 19:13 |
rlandy|ruck | k - I abandoned the previous change | 19:15 |
* mwhahaha is thoroughly confused | 19:16 | |
mwhahaha | it's the setting of tripleo_overcloud_image_upload_local | 19:17 |
mwhahaha | it's because it's a string | 19:18 |
mwhahaha | https://paste.opendev.org/show/807776/ | 19:19 |
mwhahaha | errr https://paste.opendev.org/show/807777/ | 19:19 |
mwhahaha | the usage of > is dangerous | 19:20 |
mwhahaha | because it converts it to a string | 19:20 |
* mwhahaha patches | 19:21 | |
opendevreview | Alex Schultz proposed openstack/tripleo-quickstart-extras master: Add bool to image upload vars https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/802767 | 19:24 |
mwhahaha | rlandy|ruck: -^ | 19:24 |
opendevreview | Alex Schultz proposed openstack/tripleo-quickstart-extras master: Add bool to image upload vars https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/802767 | 19:25 |
rlandy|ruck | k - thanks for digging through that | 19:25 |
mwhahaha | who needs strong typing | 19:25 |
* mwhahaha flips tables | 19:25 | |
mwhahaha | THANKS YAML | 19:25 |
rlandy|ruck | watching testproject | 19:33 |
rlandy|ruck | this is fun | 19:33 |
mwhahaha | like watching paint dry | 19:33 |
rlandy|ruck | three zuuls, paint dry on multiple walls | 19:35 |
opendevreview | Juan Pablo Lopez proposed openstack/python-tripleoclient master: [WIP] Integrating restoration process to client https://review.opendev.org/c/openstack/python-tripleoclient/+/801861 | 20:21 |
opendevreview | Juan Pablo Lopez proposed openstack/tripleo-ansible master: Adding playbook to perform the restoration of a controller node using Ironic/PXE https://review.opendev.org/c/openstack/tripleo-ansible/+/799512 | 20:22 |
mwhahaha | so looks like ussuri fs001 got past the image upload | 20:36 |
mwhahaha | so promissing | 20:36 |
mwhahaha | train too | 20:36 |
mwhahaha | so i think that fixes it | 20:36 |
rlandy|ruck | nice - looking good so far | 20:41 |
opendevreview | Juan Pablo Lopez proposed openstack/tripleo-ansible master: Add backup and restore mode for ReaR configuration https://review.opendev.org/c/openstack/tripleo-ansible/+/797035 | 20:46 |
opendevreview | Steve Baker proposed openstack/tripleo-ansible stable/wallaby: Copy generated /boot/grub2/grubenv to EFI dirs https://review.opendev.org/c/openstack/tripleo-ansible/+/802804 | 20:51 |
opendevreview | Steve Baker proposed openstack/tripleo-ansible stable/victoria: Copy generated /boot/grub2/grubenv to EFI dirs https://review.opendev.org/c/openstack/tripleo-ansible/+/802805 | 20:52 |
opendevreview | Steve Baker proposed openstack/tripleo-ansible stable/ussuri: Copy generated /boot/grub2/grubenv to EFI dirs https://review.opendev.org/c/openstack/tripleo-ansible/+/802806 | 20:53 |
opendevreview | Steve Baker proposed openstack/tripleo-ansible stable/train: Copy generated /boot/grub2/grubenv to EFI dirs https://review.opendev.org/c/openstack/tripleo-ansible/+/802808 | 20:55 |
opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations master: Remove run-validations.sh script https://review.opendev.org/c/openstack/tripleo-validations/+/781761 | 20:59 |
opendevreview | Steve Baker proposed openstack/tripleo-quickstart-extras master: WIP Mount whole-disk lvm volumes in modify-image https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/801927 | 22:06 |
rlandy|ruck | stevebaker: hi - how's it going with ^^? | 22:10 |
stevebaker | rlandy|ruck: I've been trying to solve the mystery of why the lvm loopback devices don't get created when kpartx is called. Based on local centos8 testing I think it is because lvm2 isn't installed | 22:11 |
rlandy|ruck | just fyi ... https://images.rdoproject.org/centos8/master/rdo_trunk/current-tripleo/ | 22:12 |
rlandy|ruck | we've promoted master building the new image | 22:13 |
rlandy|ruck | so you can use current-tripleo (regular check) for your testing | 22:13 |
stevebaker | yeah I noticed that :D which means this should start passing when the kpartx thing is solved https://review.opendev.org/c/openstack/tripleo-quickstart/+/801409 | 22:13 |
rlandy|ruck | ok ... | 22:14 |
rlandy|ruck | feel free to leave me email if you want help from CI and we're off hours | 22:15 |
stevebaker | rlandy|ruck: much appreciated, will do | 22:15 |
stevebaker | rlandy|ruck: if installing lvm2 works we should probably get that into the image, I have no idea where that would happen. Some RDO nodepool build thing? | 22:16 |
rlandy|ruck | yes if you need lvm2 on the undercloud | 22:17 |
rlandy|ruck | RDO nodepool images are built by the rdo infra team | 22:18 |
rlandy|ruck | upstream images are much harder to change | 22:18 |
stevebaker | oh if this is an undercloud then lvm2 could just be added to python-tripleoclient | 22:18 |
rlandy|ruck | sometimes it is :) | 22:20 |
rlandy|ruck | OVB yes | 22:20 |
rlandy|ruck | real baremetal not | 22:20 |
stevebaker | hmm | 22:20 |
rlandy|ruck | can we install it with undercloud_rpm_dependencies? | 22:23 |
rlandy|ruck | per https://github.com/openstack/tripleo-quickstart/search?q=undercloud_rpm_dependencies | 22:23 |
rlandy|ruck | where we install python-tripleoclient | 22:23 |
rlandy|ruck | or adding it is fine | 22:23 |
rlandy|ruck | better than relying on the node config | 22:23 |
stevebaker | undercloud_rpm_dependencies might be best | 22:24 |
opendevreview | Julia Kreger proposed openstack/tripleo-heat-templates master: Swap xinetd/tftp-server for dnsmasq to support ironic-pxe operations https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802740 | 22:25 |
stevebaker | actually if its python-tripleoclient then end-users who customize images will have a easier time | 22:27 |
rlandy|ruck | k - either way | 22:29 |
mwhahaha | feels like it should be on tripleo-image-elements | 22:29 |
mwhahaha | cause i don't think it's specific to tripleoclient | 22:29 |
mwhahaha | and we tried to make much of the DIB bits not a hard requirement | 22:29 |
mwhahaha | https://review.rdoproject.org/r/c/openstack/tripleoclient-distgit/+/31261 | 22:30 |
mwhahaha | https://opendev.org/openstack/python-tripleoclient/src/branch/master/tripleoclient/v1/overcloud_image.py#L47-L53 | 22:32 |
mwhahaha | or there | 22:32 |
stevebaker | mwhahaha: on the other hand, downloading a overcloud-hardened-uefi-full.qcow2 and making a minor change is a different use-case than building a whole image | 22:34 |
mwhahaha | which we don't support? | 22:35 |
mwhahaha | do you mean via virt-customize? | 22:35 |
stevebaker | lol good point | 22:35 |
mwhahaha | it should be build or virt-customize or bust | 22:35 |
mwhahaha | anything beyond that means a bad time | 22:35 |
stevebaker | this particular case is running kpartx then mounting all the loopback devices https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/801927/6/roles/modify-image/tasks/manual.yml | 22:36 |
mwhahaha | pffft quickstart | 22:36 |
mwhahaha | quickstart has always done questionable things to our images | 22:36 |
mwhahaha | that stuff should likely be virt-customize instead of mounting and doing terrible things in the image | 22:37 |
stevebaker | testing virt-customize with overcloud-hardened-uefi-full.qcow2 is on my list of things to do | 22:43 |
mwhahaha | that's what we tell 3rd parties to do (not that they do, but it's what we document) | 22:44 |
stevebaker | mwhahaha: another wrinkle, infrared are aparently moving away from virt-customize to some different tool but I have again forgotton what its called. jparoly was looking at that | 22:53 |
mwhahaha | Perhaps they should use what we document and not pick random tools | 22:54 |
mwhahaha | Though to be honey we shouldn't be virt customizing anything | 22:55 |
mwhahaha | Stupid autocorrect, s/honey/honest | 22:55 |
mwhahaha | Probably should evaluate why we're doing itin the first place | 22:56 |
stevebaker | yep | 22:58 |
stevebaker | mwhahaha: the modify-image role used virt-customize until ~4 years ago, the commit message from sshnaidm suggests it was due to slowness https://opendev.org/openstack/tripleo-quickstart-extras/commit/c2dc459ac27fb9047adee7e8497fc33e288ab632 | 23:06 |
*** pmannidi|AFK is now known as pmannidi | 23:08 | |
opendevreview | Merged openstack/tripleo-heat-templates stable/train: Role parameter for tripleo_kernel_defer_reboot https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801897 | 23:38 |
opendevreview | Steve Baker proposed openstack/tripleo-quickstart-extras master: WIP Mount whole-disk lvm volumes in modify-image https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/801927 | 23:40 |
opendevreview | Merged openstack/tripleo-ansible stable/train: Use serial or parallel execution depending on the kind of backup https://review.opendev.org/c/openstack/tripleo-ansible/+/802339 | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!