| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace hiera by lookup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837786 | 00:33 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace hiera by lookup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837786 | 00:37 |
| opendevreview | Merged openstack/tripleo-common master: build_catalog should loop deeper than one level https://review.opendev.org/c/openstack/tripleo-common/+/835084 | 00:44 |
| opendevreview | Merged openstack/tripleo-ansible master: Set the log_level opt https://review.opendev.org/c/openstack/tripleo-ansible/+/835519 | 00:45 |
| opendevreview | Merged openstack/tripleo-common master: Remove "deprecated" key in inventory https://review.opendev.org/c/openstack/tripleo-common/+/835845 | 00:45 |
| opendevreview | Merged openstack/python-tripleoclient master: Check for both config.yaml/config.json https://review.opendev.org/c/openstack/python-tripleoclient/+/837450 | 00:45 |
| *** rlandy is now known as rlandy|out | 00:52 | |
| opendevreview | Michael Johnson proposed openstack/diskimage-builder master: Mark our source trees as safe for git to use as other users https://review.opendev.org/c/openstack/diskimage-builder/+/837776 | 00:53 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: tripleo_firewall: Allow injecting frontend rules https://review.opendev.org/c/openstack/tripleo-ansible/+/831547 | 01:12 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Define frontend firewall rules separately https://review.opendev.org/c/openstack/tripleo-heat-templates/+/831549 | 01:12 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace hiera by lookup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837786 | 01:31 |
| opendevreview | Steve Baker proposed openstack/tripleo-docs master: Rework profile matching for direct provisioning https://review.opendev.org/c/openstack/tripleo-docs/+/837813 | 02:32 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: ovn: Remove usage of hiera CLI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837814 | 02:54 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: ovn: Remove usage of hiera CLI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837814 | 02:56 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: ovn: Remove usage of hiera CLI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837814 | 02:57 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: OVN: Remove usage of hiera CLI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837814 | 03:07 |
| opendevreview | Steve Baker proposed openstack/tripleo-docs master: Remove virtualbmc specific environment https://review.opendev.org/c/openstack/tripleo-docs/+/837816 | 03:12 |
| opendevreview | Steve Baker proposed openstack/tripleo-docs master: Replace openstack baremetal commands with standalone baremetal https://review.opendev.org/c/openstack/tripleo-docs/+/837817 | 03:46 |
| *** pojadhav|afk is now known as pojadhav | 03:56 | |
| opendevreview | Alan Bishop proposed openstack/tripleo-heat-templates stable/train: Fix CinderDefaultVolumeType constraints https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837818 | 03:58 |
| opendevreview | Rajesh Tailor proposed openstack/tripleo-heat-templates master: Deprecate `NovaVGPUTypesDeviceAddressesMapping parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/835440 | 04:13 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Exclude empty string from no_proxy https://review.opendev.org/c/openstack/python-tripleoclient/+/837819 | 04:52 |
| opendevreview | amolkahat proposed openstack/tripleo-ci master: Added standalone-upgrade distro jobs. https://review.opendev.org/c/openstack/tripleo-ci/+/837820 | 05:09 |
| opendevreview | Ian Wienand proposed openstack/diskimage-builder master: source-repositories : use explicit sudo/-C args when in REPO_DEST https://review.opendev.org/c/openstack/diskimage-builder/+/837824 | 06:12 |
| opendevreview | Rabi Mishra proposed openstack/python-tripleoclient master: Set deployment_status during config generation https://review.opendev.org/c/openstack/python-tripleoclient/+/837825 | 06:33 |
| *** chem is now known as Guest1901 | 06:45 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Replace hiera by lookup https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837786 | 08:08 |
| marios | o/ 11:28 < elodilles> marios: fyi, your stable/ussuri branches were deleted successfully: https://paste.opendev.org/show/bZZ3hG3jkovZpknYaFMR/ | 08:29 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: OVN: Remove usage of hiera CLI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837814 | 08:38 |
| opendevreview | Bogdan Dobrelya proposed openstack/puppet-tripleo master: Override oslo.log config for involved services https://review.opendev.org/c/openstack/puppet-tripleo/+/834846 | 08:51 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: [DNM] testing log_global_request_id https://review.opendev.org/c/openstack/tripleo-heat-templates/+/834275 | 08:53 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Manage the masquerade rules via ansible instead of puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/836872 | 09:14 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Remove reference of Mitaka from built-test-packages https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/837851 | 10:07 |
| opendevreview | Merged openstack/tripleo-ansible stable/victoria: Reference to vars_files is missing from playbook https://review.opendev.org/c/openstack/tripleo-ansible/+/836772 | 10:24 |
| *** rlandy|out is now known as rlandy | 10:31 | |
| *** dasm|off is now known as dasm | 10:46 | |
| *** pojadhav is now known as pojadhav|afk | 11:00 | |
| rlandy | dpeacock: can we merge https://review.opendev.org/c/openstack/validations-common/+/835073 | 11:02 |
| rlandy | see broken https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835072 - comment from ramishra | 11:03 |
| rlandy | or should we revet | 11:03 |
| rlandy | revert | 11:03 |
| rlandy | chkumar|ruck: ^^ | 11:03 |
| chkumar|ruck | rlandy: thanks! | 11:03 |
| dpeacock | rlandy: yes - we can - thank you for the prompt - I'll take care of it. | 11:08 |
| dpeacock | rlandy++ chkumar++ | 11:08 |
| slagle | could I get a review plz? https://review.opendev.org/c/openstack/tripleo-common/+/830970 | 11:11 |
| *** dviroel|afk is now known as dviroel | 11:24 | |
| slagle | dpeacock: hello. any thoughts what might cause this failure? https://34e9586208b3d2d1b836-40001c60cc2cc6a7bddcc7547f0d5486.ssl.cf2.rackcdn.com/837825/1/check/tripleo-ci-centos-9-standalone/5100f29/job-output.txt | 11:27 |
| slagle | looks like the error is validation run: error: argument --inventory/-i: expected one argument | 11:27 |
| slagle | i see this in a couple different standalone jobs | 11:27 |
| chkumar|ruck | rlandy: hello | 11:33 |
| chkumar|ruck | rlandy: I think validation-commons patch got +w , but it is not going to help | 11:33 |
| dpeacock | slagle: yeah we have a few inventory related patches up that should correct some such things | 11:33 |
| opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient master: Remove masquerade-networks-baremetal-puppet inclusion https://review.opendev.org/c/openstack/python-tripleoclient/+/836903 | 11:33 |
| chkumar|ruck | for that we need to follow the whole promotion chain exercise | 11:33 |
| chkumar|ruck | rlandy: https://opendev.org/openstack/tripleo-quickstart/src/branch/master/config/release/tripleo-ci/CentOS-9/master.yml#L131 | 11:34 |
| slagle | dpeacock: ok. which patch is it, is there a bug? | 11:35 |
| rlandy | chkumar|ruck: will look in a sec | 11:36 |
| chkumar|ruck | rlandy: I think we need to revert the TQE patch | 11:36 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Manage the masquerade rules via ansible instead of puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/836872 | 11:38 |
| slagle | chkumar|ruck: this one? https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/835072 | 11:38 |
| chkumar|ruck | slagle: yes | 11:39 |
| chkumar|ruck | slagle: need to chat with rlandy and then look for revert | 11:39 |
| slagle | chkumar|ruck: ok, thanks! | 11:43 |
| *** chem is now known as Guest1920 | 11:48 | |
| chkumar|ruck | dpeacock: https://review.opendev.org/c/openstack/validations-common/+/835073 onces merged, they it will go through production chain pipeline and then comes to current-tripleo upstream | 11:56 |
| chkumar|ruck | till then jobs will be breaking | 11:56 |
| dpeacock | chkumar|ruck: thank you | 11:56 |
| chkumar|ruck | best case is we can revert tqe patch | 11:56 |
| chkumar|ruck | and then you can change the depends on order revert revert | 11:56 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Revert "Set default to empty string for Validation inventory variable" https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/837801 | 11:58 |
| chkumar|ruck | slagle: ramishra rlandy ^^ please merge | 11:58 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Add new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-upgrade/+/833999 | 11:58 |
| *** pojadhav|afk is now known as pojadhav | 11:58 | |
| slagle | chkumar|ruck: done | 12:07 |
| dpeacock | chkumar|ruck: which depends do you need me to change? what to what? | 12:08 |
| chkumar|ruck | dpeacock: you can revert this patch https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/837801 and add depends-on <validations-common> patch in the revert. | 12:12 |
| dpeacock | chkumar|ruck: thanks | 12:12 |
| *** dviroel is now known as dviroel|brb | 12:19 | |
| *** dviroel|brb is now known as dviroel | 12:51 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: OVN: Remove usage of hiera CLI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837814 | 12:56 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-common master: Adding testing on build_catalog https://review.opendev.org/c/openstack/tripleo-common/+/837866 | 13:05 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: Add new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-upgrade/+/833999 | 13:09 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-common stable/wallaby: build_catalog should loop deeper than one level https://review.opendev.org/c/openstack/tripleo-common/+/837803 | 13:12 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-common stable/victoria: build_catalog should loop deeper than one level https://review.opendev.org/c/openstack/tripleo-common/+/837804 | 13:14 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-common stable/victoria: build_catalog should loop deeper than one level https://review.opendev.org/c/openstack/tripleo-common/+/837804 | 13:15 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-common stable/train: build_catalog should loop deeper than one level https://review.opendev.org/c/openstack/tripleo-common/+/837805 | 13:15 |
| *** artom_ is now known as artom | 13:29 | |
| opendevreview | Merged openstack/python-tripleoclient stable/victoria: Exposes the community validation init command https://review.opendev.org/c/openstack/python-tripleoclient/+/825466 | 13:53 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-heat-templates stable/train: [train-only] Add a new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837876 | 13:55 |
| opendevreview | Kevin Carter proposed openstack/python-tripleoclient master: Add warning for duplicate parameters https://review.opendev.org/c/openstack/python-tripleoclient/+/837877 | 13:56 |
| Tengu | cloudnull: uho, thought it wouldn't be "that" easy! -^ | 13:57 |
| cloudnull | its potentially very noisy https://paste.opendev.org/show/bXLEP866fUvE3YFv6mV8/ | 13:58 |
| cloudnull | but works | 13:58 |
| Tengu | cloudnull: maybe an option to turn that down? though... | 13:58 |
| Tengu | ok. fun. my tweak in t-h-t for the masquerade seems to work... I should be able to push a new set. | 13:59 |
| cloudnull | I'm just inspecting user provided templates, and not taking the entire set. | 13:59 |
| cloudnull | https://paste.opendev.org/show/bbHpMFadRqRO29fIBVqW/ -- this is the entire set | 13:59 |
| cloudnull | which was VERY noisy | 14:00 |
| Tengu | a bit too much | 14:00 |
| cloudnull | so I think its about as concise as we can make it | 14:00 |
| Tengu | but if it's "just" the user provided content, that should be enough | 14:00 |
| cloudnull | ++ | 14:00 |
| Tengu | and if it's written in some log, it would make it easier to debug for them - or the support. | 14:00 |
| cloudnull | support folks pinged me about it directly yesterday. its something they REALLY want. so there it is :D | 14:01 |
| Tengu | heh | 14:01 |
| Tengu | and the change isn't that big, compared to the actual benefits | 14:01 |
| cloudnull | if it makes operator/deployer lives easier ++ | 14:03 |
| Tengu | yup | 14:03 |
| Tengu | in addition, I think I have a "not too stupid" solution for the masquerade thing - ramishra had some concerns about it being exposed to all the nodes. I apparently could limit to the UC. more to come with the next patch update in t-h-t | 14:04 |
| Tengu | though it's probably not as smart as it could be. | 14:04 |
| Tengu | under testing now - UC deployed fine with the actual rules. | 14:05 |
| Tengu | let's see the OC. | 14:05 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade stable/train: [train-only] Add a new scheme of driver removal https://review.opendev.org/c/openstack/tripleo-upgrade/+/833999 | 14:12 |
| Tengu | marios: heya! care to have another look at https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/836951 + https://review.opendev.org/c/openstack/tripleo-quickstart/+/836952 ? | 14:13 |
| Tengu | marios: the depends-on tripleo-ansible has merged now. guess we can move on, unless you have any concern, of course. | 14:13 |
| Tengu | maybe getting chkumar|ruck (and rlandy ) on them might be good. | 14:14 |
| marios | Tengu: ack adding to review list | 14:18 |
| Tengu | thanks! | 14:18 |
| chkumar|ruck | Tengu: will take a look, thanks! | 14:18 |
| Tengu | chkumar|ruck: thanks :) | 14:18 |
| Tengu | that would make one step closer to the removal of puppet-firewall. | 14:19 |
| opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient master: Remove masquerade-networks-baremetal-puppet inclusion https://review.opendev.org/c/openstack/python-tripleoclient/+/836903 | 14:20 |
| Tengu | bogdando: -^^ updated the release-note here as well. | 14:20 |
| Tengu | the t-h-t one should be fine soon. | 14:21 |
| Tengu | erf.... and there's a dependency loop.... | 14:21 |
| Tengu | hmm. | 14:21 |
| Tengu | t-h-t patch depends on tripleoclient, but tripleoclient should ALSO depend on the th-t | 14:22 |
| *** pojadhav is now known as pojadhav|afk | 14:22 | |
| opendevreview | Kevin Carter proposed openstack/python-tripleoclient master: Simplify template rendering output https://review.opendev.org/c/openstack/python-tripleoclient/+/837887 | 14:24 |
| Tengu | hmmm. funky... | 14:31 |
| *** chkumar|ruck is now known as raukadah | 14:31 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Manage the masquerade rules via ansible instead of puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/836872 | 14:33 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add tripleo_keystone role https://review.opendev.org/c/openstack/tripleo-ansible/+/836641 | 14:55 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add keystone install tasks https://review.opendev.org/c/openstack/tripleo-ansible/+/836642 | 14:55 |
| *** dviroel is now known as dviroel|lunch | 15:12 | |
| jcapitao | hello guys, need some help on https://review.rdoproject.org/r/c/openstack/tripleo-ansible-distgit/+/41882 | 15:35 |
| jcapitao | we have 404 on https://tarballs.openstack.org/tripleo-ansible/tripleo-ansible-4.2.0.tar.gz | 15:35 |
| opendevreview | yatin proposed openstack/openstack-tempest-skiplist master: Revert "Skip test_update_router_admin_state tests" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/837808 | 15:45 |
| *** marios is now known as marios|out | 15:52 | |
| opendevreview | yatin proposed openstack/tripleo-heat-templates master: [DNM] Test dvr in master https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837898 | 15:54 |
| opendevreview | yatin proposed openstack/tripleo-heat-templates stable/wallaby: [DNM] check no dvr https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837900 | 15:58 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Heat: Remove non-existing puppet parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/837902 | 16:02 |
| opendevreview | Merged openstack/tripleo-ansible stable/victoria: bnr - Fix bad UEFI working https://review.opendev.org/c/openstack/tripleo-ansible/+/836598 | 16:16 |
| opendevreview | Merged openstack/validations-common master: Determine if there is any known good tripleo inventory https://review.opendev.org/c/openstack/validations-common/+/835073 | 16:16 |
| opendevreview | Merged openstack/tripleo-ci master: Remove unused scenario-standalone-options definitions https://review.opendev.org/c/openstack/tripleo-ci/+/837516 | 16:16 |
| *** dasm is now known as dasm|off | 16:20 | |
| *** dviroel|lunch is now known as dviroel | 16:20 | |
| *** rlandy is now known as rlandy|brb | 16:26 | |
| opendevreview | Kevin Carter proposed openstack/python-tripleoclient master: Add warning for duplicate parameters https://review.opendev.org/c/openstack/python-tripleoclient/+/837877 | 16:27 |
| opendevreview | Michael Johnson proposed openstack/diskimage-builder master: source-repositories : use explicit sudo/-C args when in REPO_DEST https://review.opendev.org/c/openstack/diskimage-builder/+/837824 | 16:53 |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Revert "Set default to empty string for Validation inventory variable" https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/837801 | 16:53 |
| opendevreview | Merged openstack/tripleo-puppet-elements master: Remove unused puppetlabs-vcsrepo https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/834989 | 16:53 |
| opendevreview | Merged openstack/tripleo-puppet-elements master: Remove unused puppet-keepalived https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/834988 | 16:53 |
| *** rlandy|brb is now known as rlandy | 16:53 | |
| opendevreview | Merged openstack/tripleo-puppet-elements master: Remove unused puppet-certmonger https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/834983 | 16:53 |
| opendevreview | Merged openstack/tripleo-puppet-elements master: Remove unused puppet modules https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/834990 | 16:53 |
| opendevreview | Merged openstack/tripleo-puppet-elements master: Remove modules to deploy Contrail components https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/834984 | 16:53 |
| Tengu | rlandy: fyi, I may have broken a "little" thing without any interest in the multi-node jobs - nothing important, "just" the firewall on the overcloud | 17:28 |
| Tengu | "oops".. | 17:28 |
| rlandy | Tengu: may have? | 17:40 |
| rlandy | Tengu: evidence of your breakage? | 17:41 |
| Tengu | rlandy: yeah, I'm seeing something weird with the way the iptables rules are applied on the OC, and I suspect this: https://review.opendev.org/c/openstack/tripleo-ansible/+/836871 - though... might NOT be the only cause. | 17:41 |
| rlandy | well we have not seen consistent failures there | 17:41 |
| Tengu | rlandy: nothing yet, "just" on my lab (and apparently tkajinam also hits it in his testing) - but I heard multi-node was having issues. Didn't check for now | 17:41 |
| Tengu | ok. I have a consistent failure here. | 17:42 |
| rlandy | tripleo-ansible changes are consume on merge | 17:42 |
| rlandy | Tengu: we did see this: https://bugs.launchpad.net/tripleo/+bug/1968732 | 17:43 |
| rlandy | wallaby and 17 | 17:43 |
| Tengu | trying to load.... | 17:43 |
| Tengu | hmmmmmmmmm not related I think | 17:44 |
| *** abishop_ is now known as abishop | 17:44 | |
| rlandy | https://review.rdoproject.org/zuul/build/f076d7e5da7b4419ba2ee57ad03440ab | 17:46 |
| rlandy | looks ok | 17:46 |
| Tengu | hopefully it will stay green - but frankly, I have some issues understanding why it's failing on 2 envs, and not in CI | 17:47 |
| Tengu | yeah, iptables rules seem fine there. dang. what's going on then X( | 17:48 |
| rlandy | https://zuul.opendev.org/t/openstack/builds?job_name=tripleo-ci-centos-9-containers-multinode&skip=0 | 17:57 |
| rlandy | so far - ok | 17:57 |
| Tengu | fun. | 17:58 |
| Tengu | though I have the feeling the tripleo_iptables is doing things in the wrong order... | 17:58 |
| Tengu | (i.e. inserting instead of appending, so it starts with the lock-out rule..) | 17:59 |
| Tengu | so that may be an issue anyway. | 17:59 |
| Tengu | and I don't know why I'm seeing this only now | 17:59 |
| Tengu | 'k. my solution is fine then. default the action to "append", remove a "reverse" filter, and we're far, far better. | 18:00 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Defaults tripleo_iptables to "append" instead of "insert" https://review.opendev.org/c/openstack/tripleo-ansible/+/837994 | 18:05 |
| Tengu | rlandy: -^^ that should ensure we never ever face the issue (again?) :) | 18:05 |
| rlandy | Tengu: nice fix | 18:06 |
| rlandy | I see the order issue now | 18:07 |
| Tengu | yeah...... still wondering why I'm hitting it only now | 18:07 |
| Tengu | anyway. that new order and the "append" seems more robust | 18:07 |
| Tengu | and easier to understand | 18:07 |
| Tengu | just one more test for the |combine filter - I first suspected it was some weird things with "combine 3 dicts" - apparently it wasn't, but I just want to ensure something before leaving for good. | 18:10 |
| Tengu | \o/ all is good! | 18:15 |
| Tengu | signing off. | 18:16 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Create Ceph pools/cephx keys with cli-deployed-ceph.yaml https://review.opendev.org/c/openstack/tripleo-ansible/+/837998 | 18:39 |
| opendevreview | John Fulton proposed openstack/python-tripleoclient master: Create Ceph pools/cephx keys before overcloud deploy https://review.opendev.org/c/openstack/python-tripleoclient/+/837999 | 18:40 |
| opendevreview | John Fulton proposed openstack/tripleo-docs master: Document Creating Pools/CephX keys before overcloud deployment https://review.opendev.org/c/openstack/tripleo-docs/+/838003 | 19:01 |
| opendevreview | John Fulton proposed openstack/tripleo-docs master: Document Creating Pools/CephX keys before overcloud deployment https://review.opendev.org/c/openstack/tripleo-docs/+/838003 | 19:42 |
| opendevreview | Merged openstack/diskimage-builder master: source-repositories : use explicit sudo/-C args when in REPO_DEST https://review.opendev.org/c/openstack/diskimage-builder/+/837824 | 19:45 |
| opendevreview | John Fulton proposed openstack/python-tripleoclient master: Create Ceph pools/cephx keys before overcloud deploy https://review.opendev.org/c/openstack/python-tripleoclient/+/837999 | 20:33 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add tripleo_keystone role https://review.opendev.org/c/openstack/tripleo-ansible/+/836641 | 20:34 |
| opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add keystone install tasks https://review.opendev.org/c/openstack/tripleo-ansible/+/836642 | 20:34 |
| opendevreview | Merged openstack/python-tripleoclient master: Drop StackAction related hack from tripleo deploy https://review.opendev.org/c/openstack/python-tripleoclient/+/835533 | 20:37 |
| opendevreview | Merged openstack/tripleo-ansible master: Use node UUID for bridge_mapping agent check https://review.opendev.org/c/openstack/tripleo-ansible/+/834926 | 20:38 |
| opendevreview | Merged openstack/tripleo-ci master: Setting branch overrides for VF repositories https://review.opendev.org/c/openstack/tripleo-ci/+/834631 | 20:52 |
| fultonj | popd | 21:07 |
| *** dviroel is now known as dviroel|out | 21:24 | |
| rlandy | cloudnull: Tengu: ok - so we see the firewall problem in OVB: https://logserver.rdoproject.org/cb/cbabb15dac13bde54666606ec978b75cb2e205a0/openstack-periodic-integration-main/periodic-tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001-master/dbe28f8/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz | 22:08 |
| rlandy | I'm going to log a bug for it | 22:09 |
| rlandy | since I see you -1'ed the fix | 22:09 |
| rlandy | rcastillo: ^^ fyi | 22:11 |
| rlandy | failing master OVB | 22:11 |
| rlandy | https://bugs.launchpad.net/tripleo/+bug/1969158 master OVB jobs are failing overcloud deployment - UNREACHABLE | Manage firewall rules | 22:20 |
| rlandy | rcastillo: cloudnull: Tengu: ^^ | 22:20 |
| cloudnull | rlandy is Tengu's review a fix to that issue? | 22:21 |
| cloudnull | https://review.opendev.org/c/openstack/tripleo-ansible/+/837994 | 22:21 |
| rlandy | cloudnull: I think it's supposed to be from conversation above | 22:21 |
| rlandy | <Tengu> rlandy: yeah, I'm seeing something weird with the way the iptables rules are applied on the OC, and I suspect this: https://review.opendev.org/c/openstack/tripleo-ansible/+/836871 - though... might NOT be the only cause. | 22:22 |
| rlandy | showed up in Tengu's lab | 22:22 |
| rlandy | now we see it in OVB | 22:22 |
| cloudnull | has something else recently merged that impacts the firewall? | 22:22 |
| rlandy | I can check | 22:22 |
| rlandy | https://trunk.rdoproject.org/centos9-master/report.html | 22:23 |
| cloudnull | Tengu's patch is similar to a review we had a while back that was breaking OVB, so it was abandoned | 22:23 |
| cloudnull | if that review is fixing master then I'll gladly remove -w | 22:23 |
| rlandy | cloudnull: idk if it fixes it | 22:24 |
| cloudnull | this recently merged - https://review.opendev.org/c/openstack/tripleo-ansible/+/836871 maybe related? | 22:25 |
| rlandy | https://logserver.rdoproject.org/94/837994/1/openstack-check/tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001/abe1486/ the check job on that review gets to tempest | 22:25 |
| rlandy | I can testproject his review on a periodic fs001 | 22:25 |
| cloudnull | changes the masquarade rules and adds another set of rules to the execution est | 22:26 |
| cloudnull | maybe that's not doing what we expect? | 22:26 |
| rlandy | cloudnull: yep - that was the cause tengu pointed to | 22:26 |
| rlandy | added that to the bug | 22:26 |
| rlandy | <Tengu> rlandy: yeah, I'm seeing something weird with the way the iptables rules are applied on the OC, and I suspect this: https://review.opendev.org/c/openstack/tripleo-ansible/+/836871 | 22:26 |
| cloudnull | maybe we revert that ? | 22:28 |
| cloudnull | looks like Tengu's review is failing too https://review.rdoproject.org/zuul/build/abe14866efc0432ca412478b9ccbea0b | 22:28 |
| rlandy | sure | 22:28 |
| rlandy | we can try that | 22:28 |
| cloudnull | i guess that review is failing tempest, and not deployment | 22:29 |
| cloudnull | so maybe its better? | 22:29 |
| rlandy | yes | 22:29 |
| rlandy | it got past deploy | 22:29 |
| rlandy | also that patch merged on 04/08 | 22:29 |
| rlandy | and we are only seeing the failure today | 22:29 |
| rlandy | let me testproject Tengu's change | 22:29 |
| rlandy | sec | 22:29 |
| cloudnull | ++ | 22:30 |
| rlandy | https://review.rdoproject.org/r/c/testproject/+/36255 - ok - let's see what this does | 22:32 |
| rlandy | I'll comment on the bug | 22:32 |
| cloudnull | ++ | 22:32 |
| rlandy | cloudnull: happy long weekend! | 22:32 |
| cloudnull | I'll be around a bit tomorrow and will check in on it | 22:32 |
| rlandy | sorry for the report so late | 22:32 |
| rlandy | I'm on PTO for next week | 22:32 |
| rlandy | rcastillo is rucking | 22:32 |
| cloudnull | if it passes I'll merge Tengu 's review | 22:32 |
| rcastillo | I'll keep an eye on it | 22:33 |
| cloudnull | ++ | 22:33 |
| rlandy | he can follow up | 22:33 |
| rlandy | cloudnull++ perfect - thanks | 22:33 |
| cloudnull | rcastillo rlandy have a great weekend too :D | 22:33 |
| rlandy | thank you! | 22:33 |
| rcastillo | cloudnull: you too! | 22:33 |
| *** rlandy is now known as rlandy|out | 23:48 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!