gouthamr | i'm not sure why the skiplist isn't enforced on the ussuri job though: https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/799779 | 00:00 |
---|---|---|
tkajinam | yeah, that's still strange. I didn't hit that issue in victoria so something would be wrong in ussuri only | 00:01 |
tkajinam | maybe I found an issue | 00:02 |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Make default of NeutronDefaultAvailabilityZones empty array https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800222 | 00:03 |
opendevreview | Merged openstack/tripleo-heat-templates master: Enable healthcheck API in Octavia and Ironic https://review.opendev.org/c/openstack/tripleo-heat-templates/+/787215 | 00:03 |
opendevreview | Takashi Kajinami proposed openstack/openstack-tempest-skiplist master: Remove duplicated "releases" key https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/800573 | 00:05 |
tkajinam | ^^^ gouthamr fyi | 00:05 |
gouthamr | tkajinam: nice catch! | 00:06 |
tkajinam | I'm just guessing but that change looks promising IMO | 00:06 |
tkajinam | https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/800474 | 00:08 |
tkajinam | seems we already have a patch to fix the same | 00:08 |
* tkajinam is rebasing it | 00:08 | |
opendevreview | Takashi Kajinami proposed openstack/openstack-tempest-skiplist master: Fix list of releases for a some tests https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/800474 | 00:09 |
opendevreview | James Slagle proposed openstack/python-tripleoclient master: Set max_template_size for ephemeral Heat https://review.opendev.org/c/openstack/python-tripleoclient/+/800574 | 00:22 |
slagle | cloudnull: that might help your traceback from ephem heat ^. thing is, we've never set that before | 00:27 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 01:07 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 01:15 |
opendevreview | Ronelle Landy proposed openstack/tripleo-common master: DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 01:35 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 01:43 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Set memcached server list from memcached_node_names https://review.opendev.org/c/openstack/puppet-tripleo/+/796832 | 01:50 |
EmilienM | fultonj: it seems like https://review.opendev.org/c/openstack/tripleo-ci/+/800563 is happy | 01:52 |
EmilienM | I don't understand why it didn't work on my box yet, we'll see later this week | 01:52 |
EmilienM | fultonj: I checked and object storage tests passed fine | 02:00 |
EmilienM | something must have gone wrong in my setup | 02:01 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove leftover of hieradata cleanup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800582 | 02:09 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Deprecate useless ManilaCephFSCephFSEnableSnapshots https://review.opendev.org/c/openstack/tripleo-heat-templates/+/799099 | 02:13 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Deprecate useless ManilaCephFSCephFSEnableSnapshots https://review.opendev.org/c/openstack/tripleo-heat-templates/+/799099 | 02:21 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Update default of ManilaCephFSCephFSEnableSnapshots https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800583 | 02:21 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Set memcached server list from memcached_node_names https://review.opendev.org/c/openstack/puppet-tripleo/+/796832 | 02:31 |
opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Dynamic defaults for roles, networks, vips and bm https://review.opendev.org/c/openstack/python-tripleoclient/+/797979 | 03:15 |
*** pmannidi is now known as pmannidi|brb | 04:11 | |
*** ykarel|away is now known as ykarel | 04:27 | |
*** pmannidi|brb is now known as pmannidi | 04:54 | |
bhagyashris_ | tkajinam, gouthamr hey the issue is already reported https://bugs.launchpad.net/tripleo/+bug/1934879 i have also submitted tempest skiplist patch https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/799779 and that got merged but there is some syntax issue that's why it's not getting skip for ussuri and for that arx submitted one more patch to correct the syntax https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/8004 | 05:25 |
bhagyashris_ | 74 once that get merge hopefully the tes will skip | 05:25 |
*** bhagyashris_ is now known as bhagyashris|ruck | 05:25 | |
bhagyashris|ruck | ysandeep|away, ykarel chkumar|rover plz add this is in your review list https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/800474 | 05:26 |
tkajinam | bhagyashris|ruck, yeah we discussed that bug and I noticed that fix and rebased that patch to resolve merge conflict | 05:26 |
bhagyashris|ruck | tkajinam, thanks | 05:27 |
*** ysandeep|away is now known as ysandeep | 05:27 | |
tkajinam | so once that fix is merged I'll rerun ci to see whether it can solve the problem | 05:27 |
bhagyashris|ruck | tkajinam, sure | 05:27 |
ykarel | bhagyashris|ruck, ack | 05:29 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: vncproxy: removal of tags and steps change from post_upgrade_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797720 | 05:32 |
*** pojadhav- is now known as pojadhav | 05:50 | |
ysandeep | bhagyashris|ruck, hi o/ looking | 05:51 |
opendevreview | Gregory Thiemonge proposed openstack/tripleo-heat-templates master: Use DeploymentUser for ssh key creation in the Octavia playbook https://review.opendev.org/c/openstack/tripleo-heat-templates/+/783823 | 05:55 |
opendevreview | Gregory Thiemonge proposed openstack/tripleo-ansible master: Generate Octavia ssh key from the octavia playbook https://review.opendev.org/c/openstack/tripleo-ansible/+/783824 | 05:56 |
opendevreview | Merged openstack/openstack-tempest-skiplist master: Fix list of releases for a some tests https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/800474 | 06:01 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Utilize t-o-a role for import/introspect in train https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800592 | 06:10 |
opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: Add ports filtering support to XtremIO Cinder driver https://review.opendev.org/c/openstack/tripleo-heat-templates/+/781196 | 06:23 |
*** amoralej|off is now known as amoralej | 06:51 | |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM callback adjustment for no hosts matched https://review.opendev.org/c/openstack/validations-common/+/800169 | 07:22 |
xek_ | amoralej, ykarel, hey, I have a question: how can we get the latest eventlet into tripleo containers? | 07:25 |
xek_ | I see there is https://github.com/rdo-common/python-eventlet/branches, but the latest updates were 2 years ago for train | 07:25 |
*** xek_ is now known as xek | 07:25 | |
amoralej | xek, we only maintain in rdo-common repo forks | 07:26 |
amoralej | but not packages that we directly rebuild from fedora | 07:26 |
amoralej | the actual versions are maintained in rdoinfo repo | 07:26 |
amoralej | you chan check using rdopkg | 07:26 |
amoralej | rdopkg info python-eventlet | 07:27 |
amoralej | what version of eventlet you want? | 07:27 |
xek | amoralej, 0.31.1, released yesterday | 07:27 |
amoralej | in train? | 07:27 |
amoralej | note that we follow what is in upper-constraints.txt files by default | 07:28 |
amoralej | for train https://github.com/openstack/requirements/blob/stable/train/upper-constraints.txt#L175 | 07:28 |
amoralej | in master we have 0.30.0 and there we may update to 0.31.1 once it bumps in upper-constraints.txt | 07:29 |
xek | amoralej, we need at least one fix from this release in train... we already have it downstream, but it would be needed to run memcached tls in the gate upstream | 07:30 |
amoralej | what patch is that? | 07:30 |
amoralej | i mean, upstream one | 07:31 |
xek | amoralej, https://github.com/eventlet/eventlet/pull/705/commits/1a64cd700ec917fb408dff43d804c51061802c08 | 07:31 |
amoralej | xek you'll run memcached tls in tripleo jobs? | 07:32 |
xek | amoralej, yes in fs39 and standalone-on-multinode-ipa https://review.opendev.org/c/openstack/tripleo-ci/+/787955 | 07:33 |
amoralej | seems easy to apply... | 07:33 |
*** ysandeep is now known as ysandeep|lunch | 07:34 | |
xek | amoralej, so in case of train we could use the rdo-common/python-eventlet repo, or do we need to do it in fedora? | 07:36 |
amoralej | xek, we can do it in rdo-common | 07:37 |
amoralej | i'm doing a quick test xek | 07:37 |
amoralej | we may patch 0.25.2 and apply to all from train to victoria | 07:37 |
*** rpittau|afk is now known as rpittau | 07:39 | |
opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade master: Add log collection during update. https://review.opendev.org/c/openstack/tripleo-upgrade/+/763790 | 07:42 |
opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade stable/train: Add log collection during update. https://review.opendev.org/c/openstack/tripleo-upgrade/+/763247 | 07:44 |
opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM callback adjustment for no hosts matched https://review.opendev.org/c/openstack/validations-common/+/800169 | 07:45 |
amoralej | xek, https://github.com/rdo-common/python-eventlet/commit/058a0bc3e7f0a7dfb1f71a0b98fcb50c6d58d852 | 08:10 |
amoralej | i'm building it | 08:10 |
xek | amoralej, cool, thanks! | 08:12 |
amoralej | xek, we'll need some time to get it promoted into testing repo | 08:14 |
amoralej | xek, you'll run this only in CentOS 8, right? | 08:14 |
opendevreview | Jose Luis Franco proposed openstack/tripleo-ansible stable/train: Avoid hardcoding /home/stack in cli-undercloud-db-backup.yaml. https://review.opendev.org/c/openstack/tripleo-ansible/+/800387 | 08:16 |
xek | amoralej, I put a review up to update eventlet on master: https://review.opendev.org/c/openstack/requirements/+/800597 | 08:17 |
amoralej | ok, good | 08:18 |
opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Move undercloud-upgrade from deprecated to master check & gating https://review.opendev.org/c/openstack/tripleo-ci/+/793393 | 08:19 |
opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797865 | 08:19 |
opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797865 | 08:21 |
ykarel | xek, i see bot proposed review already contains never eventlet https://review.opendev.org/c/openstack/requirements/+/800593/1/upper-constraints.txt#161 | 08:21 |
ykarel | CI failing there though | 08:22 |
opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797865 | 08:27 |
opendevreview | Merged openstack/tripleo-ci master: Revert "standalone-upgrade ussuri/victoria nv" https://review.opendev.org/c/openstack/tripleo-ci/+/800383 | 08:41 |
*** ykarel is now known as ykarel|lunch | 08:47 | |
opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-upgrade master: Add log collection during update. https://review.opendev.org/c/openstack/tripleo-upgrade/+/763790 | 08:47 |
*** ysandeep|lunch is now known as ysandeep | 08:54 | |
opendevreview | David Vallee Delisle proposed openstack/tripleo-docs master: Adding the vDPA deployment guide https://review.opendev.org/c/openstack/tripleo-docs/+/798237 | 09:12 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Utilize t-o-a role for import/introspect in train https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800592 | 09:22 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-docs master: Adding the vDPA deployment guide https://review.opendev.org/c/openstack/tripleo-docs/+/798237 | 09:29 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-docs master: Adding the vDPA deployment guide https://review.opendev.org/c/openstack/tripleo-docs/+/798237 | 09:30 |
opendevreview | Arx Cruz proposed openstack/tripleo-quickstart-extras master: Add tasks to load tempest tests to be executed on jobs https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/789286 | 09:36 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Move undercloud-upgrade from deprecated to master check & gating https://review.opendev.org/c/openstack/tripleo-ci/+/793393 | 09:41 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797865 | 09:42 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-ansible master: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-ansible/+/800505 | 09:46 |
opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart master: Remove tripleo-operator-ansible from TQE requires https://review.opendev.org/c/openstack/tripleo-quickstart/+/790701 | 10:04 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations stable/victoria: [victoria-only] molecule-requirements constraints https://review.opendev.org/c/openstack/tripleo-validations/+/800613 | 10:04 |
*** ykarel|lunch is now known as ykarel | 10:11 | |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations stable/wallaby: molecule-requirements constraints https://review.opendev.org/c/openstack/tripleo-validations/+/800615 | 10:17 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Remove `--old-deploy-image` var for image upload https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800617 | 10:29 |
opendevreview | Ananya Banerjee proposed openstack/tripleo-ci-health-queries master: Add queries from Sova https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/798958 | 10:39 |
opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Temp mark undercloud-upgrade-wallaby non voting for swift bug https://review.opendev.org/c/openstack/tripleo-ci/+/800620 | 10:50 |
marios | fyi chkumar|rover bhagyashris|ruck ^^^ for new https://bugs.launchpad.net/tripleo/+bug/1935961 | 10:50 |
opendevreview | Merged openstack/tripleo-quickstart master: Test against the latest puppet-placement https://review.opendev.org/c/openstack/tripleo-quickstart/+/787883 | 10:51 |
opendevreview | Merged openstack/tripleo-ansible stable/ussuri: [tripleo_transfer] Add more thorough testing https://review.opendev.org/c/openstack/tripleo-ansible/+/799888 | 10:51 |
opendevreview | Merged openstack/tripleo-ci master: Add missing trigger for octavia playbook to CI https://review.opendev.org/c/openstack/tripleo-ci/+/798716 | 10:51 |
chkumar|rover | marios: ack | 10:59 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: WIP: Make tripleo-repos a collection (ansible-test) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 11:02 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: WIP: Make tripleo-repos a collection (ansible-test) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 11:05 |
opendevreview | Jiri Podivin proposed openstack/tripleo-validations stable/ussuri: [ussuri-only] molecule-requirements constraints https://review.opendev.org/c/openstack/tripleo-validations/+/799851 | 11:06 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: WIP: Make tripleo-repos a collection (ansible-test) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 11:19 |
*** dviroel|out is now known as dviroel | 11:20 | |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Utilize tripleo-operator for OvercloudImage upload https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800623 | 11:46 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: WIP: Make tripleo-repos a collection (ansible-test) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 11:48 |
opendevreview | Merged openstack/tripleo-heat-templates stable/train: Remove NovaVncProxyNetwork from ServiceNetMap https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798834 | 11:51 |
opendevreview | Merged openstack/tripleo-heat-templates master: Remove ineffective CinderPowerStoreAppliances parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800334 | 11:52 |
opendevreview | James Slagle proposed openstack/python-tripleoclient master: Always write ansible-playbook-command.sh https://review.opendev.org/c/openstack/python-tripleoclient/+/772786 | 11:52 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: WIP: Make tripleo-repos a collection (ansible-test) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 12:14 |
*** amoralej is now known as amoralej|lunch | 12:19 | |
opendevreview | Rabi Mishra proposed openstack/tripleo-quickstart-extras master: Revert "Add ansible var to enable swift on undercloud" https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800636 | 12:33 |
opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: DNM - Test Pending Ceph Pacific daemons in CI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778915 | 12:34 |
opendevreview | David Peacock proposed openstack/validations-libs master: add spinner to highlight progress https://review.opendev.org/c/openstack/validations-libs/+/795565 | 12:36 |
opendevreview | Rabi Mishra proposed openstack/tripleo-quickstart-extras master: Revert "Add ansible var to enable swift on undercloud" https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800636 | 12:37 |
opendevreview | amolkahat proposed openstack/tripleo-quickstart-extras master: Replace overcloud-delete.sh script with tripleo operator roles https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800340 | 12:38 |
opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: Make default of NeutronDefaultAvailabilityZones empty array https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800224 | 12:39 |
opendevreview | amolkahat proposed openstack/tripleo-quickstart-extras master: Replace overcloud-delete.sh script with tripleo operator roles https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800340 | 12:39 |
opendevreview | Merged openstack/python-tripleoclient master: Move FakeFlavor class to fakes.py https://review.opendev.org/c/openstack/python-tripleoclient/+/798395 | 12:39 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: WIP: Make tripleo-repos a collection (ansible-test) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 12:43 |
opendevreview | David Peacock proposed openstack/validations-libs master: add spinner to highlight progress https://review.opendev.org/c/openstack/validations-libs/+/795565 | 12:44 |
*** ricolin_ is now known as ricolin | 12:48 | |
opendevreview | Marios Andreou proposed openstack/tripleo-quickstart-extras master: Install tripleo-get-hash in tqe plays undercloud/standalone https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/793331 | 12:56 |
opendevreview | David Peacock proposed openstack/validations-libs master: improve vagrant functionality https://review.opendev.org/c/openstack/validations-libs/+/799925 | 13:02 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: add spinner to highlight progress https://review.opendev.org/c/openstack/validations-libs/+/795565 | 13:09 |
opendevreview | Marios Andreou proposed openstack/tripleo-quickstart master: Bump linters and remove dead code https://review.opendev.org/c/openstack/tripleo-quickstart/+/799475 | 13:10 |
opendevreview | Marios Andreou proposed openstack/tripleo-quickstart master: wire up tripleo_get_hash ansible module in repo-setup role https://review.opendev.org/c/openstack/tripleo-quickstart/+/791486 | 13:10 |
*** amoralej|lunch is now known as amoralej | 13:13 | |
opendevreview | David Peacock proposed openstack/validations-libs master: add spinner to highlight progress https://review.opendev.org/c/openstack/validations-libs/+/795565 | 13:17 |
rlandy | ramishra: hi ... https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/798260/2/roles/overcloud-prep-network/templates/overcloud-prep-network.sh.j2#b121 is causing an error in the internal real baremetal jobs: https://sf.hosted.upshift.rdu2.redhat.com/logs/openstack-component-tripleo/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-rhel-8-bm_envC-3ctlr_1comp-featureset001-tripleo-rhos-16.2/0da9374/logs/undercloud/home/zuul/ | 13:22 |
rlandy | overcloud_prep_network.log | 13:22 |
rlandy | worth adding a new option? | 13:23 |
cloudnull | mwhahaha so circling back from yesterday, i've gotten no combination of deploy commands to run. seems any second execution of os-net-config results in the following stack - http://paste.openstack.org/show/807428/ - first run during node provisioning seems to work perfectly, second run during deployment does not. -cc ramishra | 13:23 |
opendevreview | Merged openstack/tripleo-upgrade master: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-upgrade/+/800486 | 13:23 |
opendevreview | Merged openstack/tripleo-validations stable/train: Introducing the oslo_config_validator role https://review.opendev.org/c/openstack/tripleo-validations/+/799483 | 13:23 |
mwhahaha | cloudnull: weird ok let's troubleshoot that later this morning | 13:24 |
cloudnull | ++ | 13:24 |
opendevreview | David Peacock proposed openstack/validations-libs master: improve vagrant functionality https://review.opendev.org/c/openstack/validations-libs/+/799925 | 13:27 |
ramishra | rlandy: we can probably drop that whole thing from overcloud-prep-network.sh as queens is EOL now, or use --no-dns-nameserver with openstack subnet update, I'll propose a patch | 13:29 |
rlandy | ramishra: great - thanks | 13:29 |
ramishra | cloudnull: though this is different error you encountered, re-run of os-net-config has issues https://bugs.launchpad.net/tripleo/+bug/1934669 | 13:30 |
cloudnull | during a deployment how might i stop that rerun from happening? | 13:30 |
ramishra | it seems to work if you run before or after stack and not both | 13:31 |
cloudnull | I ran deploy with --skip-nodes-and-networks but that had no effect. | 13:31 |
ramishra | you can stop it when node provisioning as --network-config uis not the default | 13:31 |
ramishra | https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/799414 | 13:31 |
ramishra | I did that WA for fs039 in CI | 13:32 |
cloudnull | I've also tried rerunning without the network-config in the deploy command, however, it is still running os-net-config a second time | 13:32 |
cloudnull | https://github.com/cloudnull/tripleo-deployment-templates/blob/main/make-cloud.bash#L328-L344 | 13:32 |
cloudnull | thats my current deploy command | 13:33 |
cloudnull | but i've also run with the all in one as well | 13:33 |
cloudnull | same thing | 13:33 |
cloudnull | all in one being http://paste.openstack.org/show/807430/ | 13:34 |
opendevreview | David Peacock proposed openstack/validations-libs master: improve vagrant functionality https://review.opendev.org/c/openstack/validations-libs/+/799925 | 13:34 |
mwhahaha | as i mentioned yesterday i think the 2nd run is expected. the nic2 issue is not expected so we'll need to see what its' doing | 13:34 |
cloudnull | I'm re-setting things up now , i'll grab the config.yaml before and after. | 13:35 |
cloudnull | happy to troubleshoot whenever. so no rush | 13:36 |
mwhahaha | ping me after you get it provisioned and we can tmate our way to hair pulling | 13:38 |
cloudnull | ++ | 13:40 |
ramishra | cloudnull: https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/utils.py#L2895-L2897 should not be running without --network-config for overcloud deploy | 13:41 |
ramishra | unless there is a bug in overcloud deploy | 13:41 |
mwhahaha | ramishra: no he's talking about the NetworkConfig application | 13:42 |
mwhahaha | there's no way to prevent that 2nd execution on the deployment AFAIK | 13:42 |
cloudnull | ++ this is the rendered config.yaml https://gist.github.com/cloudnull/d82d8ef0a551c8eeb98be3eefcecfb45 after provisioning | 13:43 |
cloudnull | all of the configured networks are correct | 13:43 |
ramishra | AFAIK we should be running it more than once unless it's set in NetworkDeploymentActions, may be I'm missing something.. I've not seen it running twice during deployment | 13:43 |
mwhahaha | it'll run the first time because the action is create | 13:44 |
cloudnull | http://paste.openstack.org/show/807431/] | 13:44 |
ramishra | if you don't use --network-config it should not be run during provsioning | 13:44 |
cloudnull | networks from the provision | 13:44 |
* cloudnull running the deploy, will capture the results in a min | 13:44 | |
ramishra | mwhahaha: yes that's why we have not made --network-config the default | 13:44 |
mwhahaha | but that invokes something different | 13:44 |
mwhahaha | https://github.com/openstack/tripleo-heat-templates/blob/master/common/deploy-steps.j2#L606-L633 | 13:45 |
mwhahaha | that's where it's failing | 13:45 |
mwhahaha | because NetworkConfig will exists | 13:45 |
ramishra | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-overcloud-node-network-config.yaml#L32 | 13:45 |
mwhahaha | right so that's a different ansible invocation | 13:45 |
ramishra | that's the same role AFAICT | 13:46 |
mwhahaha | same role doesn't matter | 13:46 |
mwhahaha | cloudnull: wait on teh deploy command | 13:46 |
mwhahaha | cloudnull: i can haz tmate? | 13:46 |
mwhahaha | ramishra: what's happening is durring teh deploy we run tripleo_network_config (not via cli-overcloud-node-network-config) | 13:46 |
mwhahaha | we're not talking about that playbook | 13:46 |
cloudnull | its running , it takes a 5 min to fail, i can reset it all after that | 13:46 |
* cloudnull can make another session while its running if you like | 13:47 | |
ramishra | mwhahaha: you're saying during deploy steps we run it twice? | 13:47 |
mwhahaha | no | 13:47 |
ramishra | then? | 13:47 |
mwhahaha | he's applying the config up front | 13:47 |
cloudnull | ^ | 13:47 |
mwhahaha | then running the deploy and os-net-config gets run again | 13:47 |
mwhahaha | so overall it's running 2x | 13:47 |
ramishra | yes, there is no way to stop it running in deploy steps, I know that | 13:47 |
mwhahaha | right it's blowing up on that | 13:48 |
ramishra | so don't run it during provisioning is what I'm saying | 13:48 |
mwhahaha | there's likely an issue with os-net-config | 13:48 |
cloudnull | these are all the templates I'm running https://github.com/cloudnull/tripleo-deployment-templates | 13:48 |
cloudnull | i've tried to run it both inside and outside of provisioning | 13:48 |
mwhahaha | os-net-config should be a noop the 2nd time if the same config is being applied | 13:48 |
cloudnull | same result | 13:48 |
mwhahaha | there's likely a bug in os-net-config | 13:48 |
mwhahaha | or different configs are being provided to the two steps | 13:48 |
ramishra | yes, it's expected to be idempotent, but there are issues atm | 13:48 |
mwhahaha | i've never hit this because i don't use net-isolation | 13:49 |
* cloudnull is running with full net-isolation | 13:49 | |
mwhahaha | cloudnull makes terrible life choices | 13:49 |
cloudnull | and converting from a 16.x setup | 13:49 |
cloudnull | see mwhahaha comment | 13:49 |
cloudnull | :D | 13:49 |
ramishra | so I suggest you don't run it during provisoning and if you don't use --network-config with overcloud deploy, it should not run in during provisioning unless there is a bug | 13:50 |
ramishra | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-overcloud-node-network-config.yaml#L32 won't run with overcloud deploy | 13:51 |
cloudnull | I kinda want to highlight what the practical diff looks like from T going X | 13:51 |
mwhahaha | so the problem is ceph where you have to run the network config first | 13:51 |
mwhahaha | in order to get the storage network up | 13:51 |
mwhahaha | so this is something we have to fix | 13:51 |
mwhahaha | so letting the deploy works, until you need to do cephadm before th deploy (as we want people to do) | 13:51 |
ramishra | yeah I know, there is already a bug and it has to be fixed, I'm suggesting the WA for cloudnull to get going | 13:52 |
cloudnull | ramishra I can try that, I dont think i've tried that combination of things | 13:52 |
cloudnull | so provision and deployu without --network-config | 13:52 |
ramishra | cloudnull: yeah, I can look at the issue tomorrow, unless u guys fix it today | 13:52 |
cloudnull | I'll capture the config here in a min, and reset so we can tmate in a few | 13:53 |
* cloudnull needs coffee before I engage rage more. | 13:53 | |
ramishra | cloudnull: can you update the bug I linked with the details of your issues, unless u want to create a new bug | 13:53 |
cloudnull | sure | 13:53 |
opendevreview | James Slagle proposed openstack/tripleo-specs master: TripleO.Next - Container Pods Spec https://review.opendev.org/c/openstack/tripleo-specs/+/797676 | 13:54 |
mwhahaha | the proposed bug seems different but we'll poke at it today thanks for the history | 13:54 |
mwhahaha | cloudnull: why is it ens3 then ens11 | 13:58 |
cloudnull | https://gist.github.com/cloudnull/d82d8ef0a551c8eeb98be3eefcecfb45 these are the two generated config.yaml files | 13:59 |
mwhahaha | the faiure seems to be because it doesn't determine ens11 is an active nic | 13:59 |
mwhahaha | so you reference nic2 but os-net-config is not finding a 2nd nic to use | 13:59 |
cloudnull | mwhahaha the nodes have two nics | 13:59 |
mwhahaha | [2021/07/12 11:58:09 PM] [INFO] Active nics are ['ens3'] | 13:59 |
cloudnull | ens3 and ens11 | 13:59 |
mwhahaha | why is it ens11 and not ens4? | 13:59 |
* cloudnull has no idea why its named that | 13:59 | |
mwhahaha | the issue is the detection of teh 2nd nic | 14:00 |
cloudnull | the node does get rebooted after kernel args are applied | 14:00 |
cloudnull | I wonder if that is part of the cause? | 14:00 |
mwhahaha | i don't know which is why i said i wanted to troubleshoot before you ran the deploy :D | 14:00 |
cloudnull | https://github.com/cloudnull/tripleo-deployment-templates/blob/main/parameters.yaml#L35-L37 | 14:01 |
cloudnull | its all being reset now | 14:01 |
cloudnull | we can give it a bash in a min | 14:01 |
cloudnull | I dont know why the role is generating two different config.yaml files? | 14:02 |
cloudnull | 1- is from provisioning, 2- is from deploy | 14:02 |
opendevreview | David Peacock proposed openstack/validations-libs master: add spinner to highlight progress https://review.opendev.org/c/openstack/validations-libs/+/795565 | 14:03 |
mwhahaha | wonder if you have different networks in the deploy vs provisioning | 14:04 |
mwhahaha | as you're missing teh .2 and .5 netwrosk | 14:04 |
cloudnull | yeah and the dns servers | 14:05 |
cloudnull | its all the same templates? | 14:05 |
mwhahaha | dns servers likely get added from other THT stuff | 14:05 |
mwhahaha | anyway let me know when it's reset and we'll walk through what you are doing | 14:05 |
cloudnull | ssh jExFHjUCPwakCbQ949pbHpmwn@nyc1.tmate.io | 14:05 |
cloudnull | just running the inspect now | 14:06 |
cloudnull | we'll be back to broken here in a min :D | 14:06 |
opendevreview | Rabi Mishra proposed openstack/tripleo-quickstart-extras master: Don't set dns-name-servers for ctlplane subnet https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800662 | 14:07 |
ramishra | rlandy: ^^ | 14:07 |
opendevreview | Merged openstack/puppet-tripleo master: PowerStore: Remove usage of deprecated powerstore_appliances https://review.opendev.org/c/openstack/puppet-tripleo/+/800335 | 14:10 |
mwhahaha | cloudnull: https://meet.google.com/eqo-dofg-gnf | 14:11 |
rlandy | ramishra: thanks - will test with BM | 14:13 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/victoria: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-upgrade/+/800637 | 14:14 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/ussuri: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-upgrade/+/800638 | 14:15 |
*** ysandeep is now known as ysandeep|away | 14:23 | |
*** ykarel is now known as ykarel|away | 14:24 | |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: WIP: Make tripleo-repos a collection (ansible-test) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 14:35 |
opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: WIP: Make tripleo-repos a collection (ansible-test) https://review.opendev.org/c/openstack/tripleo-repos/+/800462 | 14:36 |
opendevreview | Merged openstack/tripleo-ansible master: Pass tripleo_ceph_client_cluster in ceph_client template https://review.opendev.org/c/openstack/tripleo-ansible/+/800525 | 14:41 |
opendevreview | Ananya Banerjee proposed openstack/tripleo-ci-health-queries master: Add queries from Sova https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/798958 | 14:42 |
opendevreview | David Peacock proposed openstack/validations-libs master: add spinner to highlight progress https://review.opendev.org/c/openstack/validations-libs/+/795565 | 14:43 |
opendevreview | Ananya Banerjee proposed openstack/tripleo-ci-health-queries master: Updating sova converter script https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/799514 | 14:49 |
opendevreview | David Peacock proposed openstack/validations-libs master: add spinner to highlight progress https://review.opendev.org/c/openstack/validations-libs/+/795565 | 15:14 |
cloudnull | mwhahaha it worked | 15:21 |
cloudnull | so disabling cloud-init on provisioning is the "fix" | 15:22 |
cloudnull | well cloud-init network management | 15:22 |
cloudnull | http://paste.openstack.org/show/807436/ - so i wonder if we should have our generated output just include that stanza when --network-config is used? | 15:23 |
cloudnull | -cc ramishra | 15:23 |
mwhahaha | Nice. I think we just need to disable it completely since we use os-net-config | 15:28 |
mwhahaha | Probably want to sync with Harald and Steve Baker | 15:28 |
opendevreview | Merged openstack/tripleo-quickstart master: [Master and Wallaby] Remove usage of the latest puppet-mistral/zaqar https://review.opendev.org/c/openstack/tripleo-quickstart/+/800330 | 15:29 |
ramishra | cloudnull: sounds like https://bugzilla.redhat.com/show_bug.cgi?id=1773642 which is stale | 15:31 |
cloudnull | ramishra++ | 15:31 |
mwhahaha | i don't know where the user data is put together, i think it's a metalsmith thing | 15:55 |
mwhahaha | maybe we can inject it in tripleoclient but does seems something that should be addresses in metalsmith | 15:55 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 15:58 |
opendevreview | Alex Schultz proposed openstack/puppet-tripleo master: Trim line breaks from pacemaker bundle function https://review.opendev.org/c/openstack/puppet-tripleo/+/800556 | 16:00 |
*** marios is now known as marios|out | 16:02 | |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 16:04 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 16:05 |
mwhahaha | dvd, do you know much about the virtlogd wrapper? | 16:13 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 16:13 |
mwhahaha | or mschuppert if you're around? i'm hitting errors in the virtlogd wrapper | 16:14 |
mwhahaha | 2021-07-13T16:13:05.689892031+00:00 stderr F Error: create directory `/sys/fs/cgroup/../../libpod-70fc4731fd9289440522cc561d47048a94204e8c74c9157790782ec6632f99cf.scope`: No such file or directory: OCI runtime attempted to invoke a command that was not found | 16:14 |
cloudnull | so rerunning the deployment now. with cloud-init disabled, lets see if it gets further | 16:15 |
opendevreview | Gregory Thiemonge proposed openstack/tripleo-ansible master: Generate Octavia ssh key from the octavia playbook https://review.opendev.org/c/openstack/tripleo-ansible/+/783824 | 16:18 |
opendevreview | Chris Sibbitt proposed openstack/tripleo-ansible master: Fix missing ansible_fqdn https://review.opendev.org/c/openstack/tripleo-ansible/+/800531 | 16:22 |
opendevreview | Merged openstack/tripleo-ansible stable/ussuri: tripleo-transfer: ability to customize files transfer options https://review.opendev.org/c/openstack/tripleo-ansible/+/800209 | 16:23 |
opendevreview | Merged openstack/tripleo-ansible master: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-ansible/+/800505 | 16:23 |
*** rpittau is now known as rpittau|afk | 16:23 | |
dvd | mwhahaha, I know the big picture but I didn't work on this | 16:28 |
mwhahaha | k well doesn't seem to work on centos9 for whatever reason | 16:29 |
mwhahaha | i don't know what it's supposed to do so who knows | 16:29 |
cloudnull | so here's another annoyance - http://paste.openstack.org/show/807439/ - when you rerun the deploy command, for any reason, it seems to always fail on the second execution because the "stack exists" to make it go you have delete the path ~/overcloud-deploy/rk-openstack-0 and then rerun. | 16:29 |
dvd | mwhahaha, do you have nsenter installed? | 16:29 |
*** amoralej is now known as amoralej|off | 16:30 | |
mwhahaha | i just nuked the env but shouldn't we be taking care of that via the deploy? | 16:30 |
dvd | I would be hoping so :-) | 16:30 |
mwhahaha | i'll poke at it later | 16:30 |
dvd | mwhahaha, I didn't know centos9 was released, I'll give it a try | 16:30 |
dvd | not release but I assume it's beta or alpha at this point? | 16:31 |
mwhahaha | i was able to get c9 standalone deployed with c8 containers. needed to patch puppet and puppet-tripleo and install network-scripts (manually built) | 16:31 |
mwhahaha | it's still alpha | 16:31 |
opendevreview | wes hayutin proposed openstack/tripleo-heat-templates stable/train: DNM, TEST TEST https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800685 | 16:31 |
mwhahaha | not official but we have composes to build stuff | 16:31 |
opendevreview | wes hayutin proposed openstack/tripleo-heat-templates stable/train: DNM, TEST TEST https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800685 | 16:32 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 16:37 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 16:44 |
mwhahaha | dvd yes nsenter was installed | 16:55 |
dvd | mwhahaha, ack, I'm trying to figure out how to build c9 and will try it, in the mean time I guess you can disable the wrapper if it blocks you | 16:57 |
mwhahaha | well the issue is that the virtlogd socket is missing | 16:57 |
mwhahaha | so nothing spawns | 16:57 |
mwhahaha | dvd: https://github.com/mwhahaha/cloud-images/blob/main/centos-9.stream.virt-install-cmd | 16:57 |
mwhahaha | careful tho because https://bugzilla.redhat.com/show_bug.cgi?id=1980910 | 16:58 |
dvd | nice thanks | 16:58 |
mwhahaha | dvd: http://paste.openstack.org/show/807440/ are the repos | 17:00 |
mwhahaha | and you'll need to go fetch/rebuild initscripts from an srpm from c8 | 17:01 |
mwhahaha | and install initscripts/network-scripts | 17:01 |
dvd | ack | 17:01 |
mwhahaha | and you'll need https://github.com/puppetlabs/puppet/pull/8693 and https://review.opendev.org/c/openstack/puppet-tripleo/+/800556 | 17:01 |
mwhahaha | then standalone should work (tm) | 17:01 |
dvd | hehe | 17:01 |
mwhahaha | it deploys but you can't spawn vms cause virtlogd socket is missing | 17:01 |
mwhahaha | that's actually less workarounds than we needed for c8 at least :D | 17:02 |
dvd | haha nice | 17:03 |
mwhahaha | the puppet fix was something we had to do for 8 too | 17:03 |
mwhahaha | since their providers are awful trying to future proof it led to some random failures | 17:04 |
opendevreview | Merged openstack/python-tripleoclient stable/victoria: Add --db-only option in undercloud_backup. https://review.opendev.org/c/openstack/python-tripleoclient/+/799477 | 17:18 |
opendevreview | Merged openstack/tripleo-common stable/ussuri: directly use files instread of tcib_gather_files https://review.opendev.org/c/openstack/tripleo-common/+/799204 | 17:18 |
opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Make default of NeutronDefaultAvailabilityZones empty array https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800223 | 17:18 |
opendevreview | Merged openstack/validations-common master: Callback logging unification https://review.opendev.org/c/openstack/validations-common/+/797618 | 17:18 |
opendevreview | Merged openstack/tripleo-heat-templates stable/train: Make default of NeutronDefaultAvailabilityZones empty array https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800225 | 17:30 |
-opendevstatus- NOTICE: Depends-On using https://review.opendev.org URLs are currently not working. This was due to a config change in Zuul that we are reverting and will be restarting Zuul to pick up. | 17:42 | |
dvd | mwhahaha, I'm gonna need a bit of time to reproduce this, hopefully I'll deploy tonight. I'm a wiggler so I need my hands on an issue to better understand it. | 17:47 |
mwhahaha | yea no biggie. probably something we should get a head of. i just noticed it yesterday | 17:49 |
*** zbr is now known as Guest681 | 17:50 | |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 17:50 |
opendevreview | Merged openstack/puppet-tripleo stable/victoria: Revert "Aodh deprecation" https://review.opendev.org/c/openstack/puppet-tripleo/+/799885 | 18:09 |
opendevreview | Merged openstack/puppet-tripleo stable/ussuri: Revert "Aodh deprecation" https://review.opendev.org/c/openstack/puppet-tripleo/+/799886 | 18:09 |
opendevreview | Merged openstack/puppet-tripleo stable/train: Revert "Aodh deprecation" https://review.opendev.org/c/openstack/puppet-tripleo/+/799948 | 18:09 |
opendevreview | Merged openstack/tripleo-ci master: Temp mark undercloud-upgrade-wallaby non voting for swift bug https://review.opendev.org/c/openstack/tripleo-ci/+/800620 | 18:09 |
eagles | dumb question - if you were to simply follow the basic "install the undercloud, upload the images, add nodes and introspect, etc.", do we expect a simple "openstack overcloud deploy --templates" to work? | 18:10 |
opendevreview | Merged openstack/tripleo-image-elements master: Depend on growvols element https://review.opendev.org/c/openstack/tripleo-image-elements/+/796745 | 18:10 |
opendevreview | Merged openstack/tripleo-image-elements master: Shrink whole disk image from 40G to 6G https://review.opendev.org/c/openstack/tripleo-image-elements/+/798040 | 18:10 |
opendevreview | Merged openstack/tripleo-ansible master: cephadm: add missing cluster name https://review.opendev.org/c/openstack/tripleo-ansible/+/800472 | 18:11 |
opendevreview | Merged openstack/tripleo-ansible stable/train: Avoid hardcoding /home/stack in cli-undercloud-db-backup.yaml. https://review.opendev.org/c/openstack/tripleo-ansible/+/800387 | 18:11 |
opendevreview | Merged openstack/tripleo-ansible master: Explicitly set ceph_dashboard_vip when Ceph Dashboard is enabled https://review.opendev.org/c/openstack/tripleo-ansible/+/800027 | 18:11 |
opendevreview | Merged openstack/tripleo-quickstart-extras master: Utilize t-o-a role for import/introspect in train https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/800592 | 18:11 |
eagles | afaict, it's not like we actually test that | 18:11 |
opendevreview | Merged openstack/validations-libs master: Test cross contamination prevention https://review.opendev.org/c/openstack/validations-libs/+/799754 | 18:11 |
opendevreview | Merged openstack/validations-libs master: improve vagrant functionality https://review.opendev.org/c/openstack/validations-libs/+/799925 | 18:11 |
mwhahaha | eagles: you have to provision | 18:12 |
mwhahaha | eagles: and provide the deployed yaml info bits in. but you don't need to specify much else | 18:12 |
eagles | mwhahaha: you mean like openstack overcloud node introspect --all-manageable --provide | 18:13 |
eagles | or is there another step I'm missing | 18:13 |
mwhahaha | no thats just introspection | 18:13 |
mwhahaha | you're missing a step now | 18:14 |
mwhahaha | you have to pre-provision the nodes | 18:14 |
mwhahaha | it's no longer done @ deploy time | 18:14 |
opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: vncproxy: removal of tags and steps change from post_upgrade_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797720 | 18:14 |
mwhahaha | eagles: http://paste.openstack.org/show/807444/ | 18:14 |
eagles | ah okay | 18:15 |
mwhahaha | YMMV | 18:16 |
eagles | lo; | 18:16 |
eagles | lol | 18:16 |
mwhahaha | it's missing the stackrc source | 18:16 |
mwhahaha | and the THT dir is wrong in that paste | 18:16 |
mwhahaha | but thats what i've been doing on master for the last few weeks | 18:16 |
mwhahaha | w/o issues (besides the normal ones)( | 18:16 |
eagles | mwhahaha: cool thanks... when did this change? I'm just wondering how old an environment I was working with | 18:17 |
mwhahaha | wallaby? | 18:17 |
mwhahaha | when was the last time you deployed master? | 18:17 |
eagles | it was a while ago since I started from scratch on master but I'm not sure of exactly when.. | 18:18 |
cloudnull | mwhahaha back -- so on to the next problem, when os-net-config runs the second time, reconfiguring the network with the newly generated config.yaml, it takes down the network. we have to reboot the node to get it back online. | 18:18 |
mwhahaha | also seems wrong? | 18:19 |
cloudnull | subsequent runs after reboot seem to work | 18:19 |
eagles | I remember emails about this process but missed the switchover | 18:19 |
mwhahaha | it's been a few months :D | 18:20 |
eagles | geez | 18:20 |
mwhahaha | welcome back? :D | 18:20 |
mwhahaha | eagles: btw do we still allow for the wrappers to be disabled? or should they just be always handled? | 18:20 |
mwhahaha | i was looking at the neutron bits and it seems that the wrappers can be disabled but was that just for testing? | 18:21 |
mwhahaha | https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/neutron/neutron-l3-container-puppet.yaml#L27-L47 | 18:21 |
eagles | mwhahaha: there were some that we were holding onto because the services being invoked didn't have the option to run in the foreground | 18:22 |
eagles | mwhahaha: 1s | 18:22 |
mwhahaha | the wrapper stuff is fine, i just was wondering if we ever actually disable them | 18:22 |
mwhahaha | e.g. can we just get rid of the logic and always write them out | 18:23 |
eagles | mwhahaha: we don't test disabling them so we could likely remove any that are defaulted to true. The RadvdDockerWrapper is current set to false, so I'm not sure if we've resolved the issue with radvd | 18:26 |
mwhahaha | k | 18:26 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: [TRAIN-ONLY] Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-upgrade/+/800710 | 18:33 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-ansible stable/wallaby: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-ansible/+/800711 | 18:40 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-ansible stable/victoria: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-ansible/+/800712 | 18:40 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-ansible stable/ussuri: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-ansible/+/800713 | 18:41 |
opendevreview | Sergii Golovatiuk proposed openstack/tripleo-ansible stable/train: Replace LEAPP_DEVEL_SKIP_RHSM with LEAPP_NO_RHSM https://review.opendev.org/c/openstack/tripleo-ansible/+/800714 | 18:44 |
opendevreview | Merged openstack/tripleo-heat-templates master: Introducing the ComputeVdpa role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797529 | 19:06 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 19:15 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 19:15 |
fultonj | slagle: While working on https://review.opendev.org/c/openstack/python-tripleoclient/+/777494 did you encounter pep8 R0401: Cyclic import (tripleoclient.export -> tripleoclient.utils) (cyclic-import) ? | 19:57 |
fultonj | i'm hitting it with pylint in CI https://zuul.opendev.org/t/openstack/build/4d4a5ef956f24aacacd2d59f6f8e587a | 19:57 |
fultonj | and I see tripleoclient/utils.py imports export and tripleoclient/v1/overcloud_export.py imports utils. | 19:58 |
* fultonj thinkikng of updating utils.py to "from tripleoclient import export" differently | 20:01 | |
slagle | fultonj: no. maybe something changed in the lint job to start checking for that | 20:06 |
fultonj | what's odd is that if i checkout master and "tox -e pep8" i don't see that but if I check out my patch i do, though I see no connection between the two | 20:08 |
fultonj | https://review.opendev.org/c/openstack/python-tripleoclient/+/800341 | 20:08 |
fultonj | slagle: does it look circuluar to you either way? | 20:09 |
fultonj | https://github.com/openstack/python-tripleoclient/commit/7483b62332ecf7d27a168732477f01f967bae976 | 20:09 |
fultonj | https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/export.py#L27 | 20:10 |
fultonj | https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/utils.py#L75 | 20:10 |
slagle | it does look like a cycle, but that doesn't mean it's a problem | 20:10 |
slagle | except unless the linter is configured to fail | 20:10 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 20:12 |
opendevreview | Alex Schultz proposed openstack/tripleo-operator-ansible master: Limit tripleo-repos to CentOS https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/800719 | 20:14 |
opendevreview | Gael Chamoulaud proposed openstack/validations-libs master: Remove convert_data function https://review.opendev.org/c/openstack/validations-libs/+/800457 | 20:20 |
opendevreview | Gael Chamoulaud proposed openstack/validations-libs master: Add Categories metadata key management https://review.opendev.org/c/openstack/validations-libs/+/800260 | 20:20 |
opendevreview | Gael Chamoulaud proposed openstack/validations-libs master: Add Products metadata key management https://review.opendev.org/c/openstack/validations-libs/+/800721 | 20:20 |
fultonj | i think i'll try having overcloud_deploy importing export and then moving the new export_overcloud functin out of utils and then into export | 20:24 |
opendevreview | Gael Chamoulaud proposed openstack/validations-libs master: Add Categories metadata key management https://review.opendev.org/c/openstack/validations-libs/+/800260 | 20:25 |
opendevreview | Gael Chamoulaud proposed openstack/validations-libs master: Add Products metadata key management https://review.opendev.org/c/openstack/validations-libs/+/800721 | 20:25 |
opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: DNM: Temp pre for centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 20:29 |
opendevreview | wes hayutin proposed openstack/tripleo-ci master: move scenario010 to branched jobs for voting https://review.opendev.org/c/openstack/tripleo-ci/+/800540 | 20:44 |
opendevreview | wes hayutin proposed openstack/tripleo-heat-templates stable/train: DNM, TEST TEST https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800685 | 20:47 |
opendevreview | wes hayutin proposed openstack/tripleo-heat-templates stable/train: DNM, TEST TEST https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800685 | 20:51 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: DNM: Remove cyclic import between export and utils https://review.opendev.org/c/openstack/python-tripleoclient/+/800725 | 21:09 |
dvd | mwhahaha, good news, I can reproduce the issue, good job on the doc. Bad news, I can reproduce the issue | 21:10 |
mwhahaha | :D | 21:10 |
mwhahaha | good luck | 21:11 |
dvd | haha thanks, I'll try some stuff and poke mschuppert tomorrow | 21:11 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy" https://review.opendev.org/c/openstack/python-tripleoclient/+/800341 | 21:13 |
dvd | mwhahaha, I believe it's podman or libpod incompatibility between el9 and el8 | 21:34 |
dvd | the libpod path changed between them | 21:34 |
dvd | not sure where this is coming from, if we can define this | 21:34 |
*** dviroel is now known as dviroel|out | 21:41 | |
dvd | mwhahaha, mschuppert, opened this bug for the issue and included some of my investigation. I think I have a good understanding, just not sure how to fix it yet: https://bugs.launchpad.net/tripleo/+bug/1936005 | 22:22 |
opendevreview | David Vallee Delisle proposed openstack/puppet-tripleo master: DNM [c9] nenter needs to enter the cgroup namespace https://review.opendev.org/c/openstack/puppet-tripleo/+/800731 | 22:37 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy" https://review.opendev.org/c/openstack/python-tripleoclient/+/800341 | 23:03 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: DNM: Remove cyclic import between export and utils https://review.opendev.org/c/openstack/python-tripleoclient/+/800725 | 23:45 |
opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph deploy" https://review.opendev.org/c/openstack/python-tripleoclient/+/800341 | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!