| *** rlandy|ruck|biab is now known as rlandy|ruck | 00:07 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: Do not log amphora ssh keys https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801501 | 00:28 |
|---|---|---|
| opendevreview | Merged openstack/tripleo-ansible stable/train: Backup and Restore playbook is not 100% idempotent when running in snapshot mode https://review.opendev.org/c/openstack/tripleo-ansible/+/801725 | 01:02 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Enable image copy for multiple RBD Glance stores https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800960 | 01:02 |
| *** pmannidi is now known as pmannidi|brb | 01:33 | |
| *** pmannidi|brb is now known as pmannidi | 02:06 | |
| opendevreview | Merged openstack/diskimage-builder master: Update IRC networks https://review.opendev.org/c/openstack/diskimage-builder/+/801758 | 02:06 |
| *** pmannidi is now known as pmannidi|brb | 03:50 | |
| *** pmannidi|brb is now known as pmannidi | 04:38 | |
| *** bhagyashris__ is now known as bhagyashris | 04:41 | |
| 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 | 05:14 |
| *** marios is now known as marios|ruck | 05:20 | |
| *** pojadhav|afk is now known as pojadhav | 05:29 | |
| marios|ruck | ramishra_: o/ morning can you please check https://review.opendev.org/c/openstack/puppet-neutron/+/796359/2#message-1f16050d59e65982069ae6244f080b4f9e1c507b https://bugs.launchpad.net/tripleo/+bug/1937333 i posted the revert but you may have other ideas | 05:49 |
| marios|ruck | tkajinam: o/ ^^ | 05:49 |
| marios|ruck | blocks the gate | 05:50 |
| tkajinam | marios|ruck, https://review.opendev.org/c/openstack/puppet-neutron/+/801933 | 05:55 |
| tkajinam | marios|ruck, sorry that was my silly mistake. I'm confident that would fix that issue but it'd be nice if you can test that before it lands | 05:56 |
| tkajinam | I mean that fix in puppet-neutron fixes the issue | 05:56 |
| marios|ruck | thanks tkajinam looks good voted on your patch i will abandon mine once we see green ci there | 05:56 |
| tkajinam | marios|ruck, ok | 05:57 |
| marios|ruck | tkajinam: it will be tested by the ci run there already so no need for extra | 05:57 |
| marios|ruck | tkajinam: oh let me check which jobs are running there actually | 05:57 |
| marios|ruck | tkajinam: ah right puppet-neturon ok we need a test | 05:57 |
| tkajinam | marios|ruck, I think puppet-neutron doesn't trigger undercloud job | 05:57 |
| marios|ruck | tkajinam: yup sec will post | 05:58 |
| marios|ruck | tkajinam: https://bugs.launchpad.net/tripleo/+bug/1937333/comments/2 | 06:01 |
| marios|ruck | tkajinam: https://review.rdoproject.org/zuul/status#34650 | 06:01 |
| tkajinam | marios|ruck, thx | 06:06 |
| ramishra_ | marios|ruck: Hi! Looks like it's been addressed by tkajinam | 06:15 |
| marios|ruck | yeah thanks ramishra_ | 06:16 |
| ramishra_ | tkajinam: Any reason we don't have an undercloud job in puppet-tripleo? | 06:16 |
| ramishra_ | probably we should add one | 06:16 |
| tkajinam | ramishra_, https://review.opendev.org/c/openstack/puppet-neutron/+/801936 | 06:18 |
| ramishra_ | tkajinam: ack, thanks! | 06:20 |
| tkajinam | basically we have only one of standalone job or undercloud job but for puppet-neutron it tests different thing | 06:22 |
| tkajinam | ml2+ovn vs networking-baremetal | 06:23 |
| marios|ruck | tkajinam: i noticed it is running a lot of scenarios... if number of jobs is a rpoblem you can consider removing scen4 and have undercloud instead (scen1 and scen4 both test ceph) | 06:23 |
| tkajinam | marios|ruck, each scenario tests something not supported in tripleo | 06:25 |
| tkajinam | like watcher, bgpvpn, lg2w and etc | 06:25 |
| tkajinam | so we can't remove it | 06:25 |
| marios|ruck | tkajinam: ah i see sorry i confused them with the tripleo scenarios obviously these are different i thought they were based on the tripleo scen jobs | 06:25 |
| tkajinam | marios|ruck, just fyi https://github.com/openstack/puppet-openstack-integration#description | 06:27 |
| marios|ruck | tkajinam: small nit at https://review.opendev.org/c/openstack/puppet-neutron/+/801936/2#message-07f432e72c2efb25fd2bb9c263f7fa8437620d0f | 06:28 |
| marios|ruck | tkajinam: i see thank you bookmarking | 06:28 |
| tkajinam | marios|ruck, thx | 06:28 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Introducing the ComputeVdpa role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801320 | 06:33 |
| jpodivin | So I took a look at the way we have pylint setup in the python-tripleoclient. And I think that we should set the failure threshold bit bellow 100% | 06:52 |
| jpodivin | *below | 06:53 |
| opendevreview | Merged openstack/tripleo-ansible stable/train: Remove become: true from the role https://review.opendev.org/c/openstack/tripleo-ansible/+/801239 | 06:56 |
| opendevreview | Merged openstack/tripleo-ansible stable/ussuri: Switch from pacemaker standby to pacemaker stop https://review.opendev.org/c/openstack/tripleo-ansible/+/801578 | 06:56 |
| *** amoralej|off is now known as amoralej | 06:56 | |
| marios|ruck | pojadhav: no need to recheck @ https://review.opendev.org/c/openstack/tripleo-ci/+/797865 hitting https://bugs.launchpad.net/tripleo/+bug/1937333 fyi | 07:01 |
| *** marios|ruck is now known as marios | 07:01 | |
| pojadhav | marios, oops.. thanks for update !! | 07:01 |
| *** ChanServ sets mode: +o marios | 07:02 | |
| *** marios changes topic to "gate blocker - please hold recheck https://bugs.launchpad.net/tripleo/+bug/1937333 in progress" | 07:03 | |
| marios | pojadhav: np | 07:05 |
| *** marios is now known as marios|ruck | 07:05 | |
| marios|ruck | also sent there, in case anyone actually reads that ;0 http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023805.html | 07:05 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates master: Enable archive task_log records while archiving the database https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801939 | 07:09 |
| marios|ruck | tkajinam: looks good undercloud-containers completed https://review.rdoproject.org/zuul/status#34650 | 07:12 |
| tkajinam | marios|ruck, good news :-) | 07:12 |
| jpodivin | so | 07:16 |
| jpodivin | The cliff dependencies in *requirements.txt | 07:17 |
| marios|ruck | jpodivin: yeah so as we discussed i didn't realise that change was for /doc/requirements | 07:17 |
| marios|ruck | jpodivin: i see you already have it in https://opendev.org/openstack/validations-libs/src/commit/9a594f70c98e60217a1bc6d4870774b6d4a57945/requirements.txt#L10 | 07:17 |
| jpodivin | marios|ruck: yeah | 07:17 |
| marios|ruck | arxcruz: fyi https://opendev.org/openstack/validations-libs/src/commit/9a594f70c98e60217a1bc6d4870774b6d4a57945/requirements.txt#L10 | 07:17 |
| marios|ruck | arxcruz: :) | 07:17 |
| jpodivin | marios|ruck: but We should figure this out. I don't like to see things working without knowing how they work :D | 07:18 |
| marios|ruck | jpodivin: removed vote for now @ https://review.opendev.org/c/openstack/validations-libs/+/801623/4#message-f960feaf28ea9985a6fe94263ac5002c4bf83502 | 07:19 |
| jpodivin | marios|ruck: thank you. | 07:19 |
| jpodivin | marios|ruck: I think the answer might be in the timing. The VL package is being installed before tripleoclient. | 07:20 |
| jpodivin | Or, at very least, before bulk of the tripleo is in place. | 07:21 |
| jpodivin | At some point, cliff gets in, possibly pulled as a dependency of a dependency. We all know how deep those can be. | 07:21 |
| marios|ruck | jpodivin: hmm i can't see cliff in https://9a504d3846d8605cd77d-462e76888179d96b05b951c4b3bdf8c0.ssl.cf5.rackcdn.com/801623/4/check/tripleo-ci-centos-8-standalone-validation-libs/2a20e92/logs/undercloud/var/log/dnf.log | 07:21 |
| marios|ruck | jpodivin: only python3-cliff | 07:21 |
| marios|ruck | jpodivin: so where is it coming from | 07:22 |
| jpodivin | marios|ruck: python3-cliff? | 07:22 |
| opendevreview | Merged openstack/tripleo-ansible stable/train: Switch from pacemaker standby to pacemaker stop https://review.opendev.org/c/openstack/tripleo-ansible/+/801731 | 07:22 |
| opendevreview | Merged openstack/tripleo-validations master: yaml.load: Set Loader to avoid deprecation warning https://review.opendev.org/c/openstack/tripleo-validations/+/801851 | 07:22 |
| opendevreview | Merged openstack/tripleo-ci master: Make standalone-ipa jobs voting again - minus train https://review.opendev.org/c/openstack/tripleo-ci/+/762766 | 07:22 |
| jpodivin | the names rarely match | 07:22 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Support cinder image conversion on an NFS share https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801121 | 07:22 |
| jpodivin | Case and point: Validations-libs themselves. | 07:22 |
| gchamoul | jpodivin, marios|ruck, when installing tripleoclient the vf rpms will come | 07:22 |
| gchamoul | vlibs are not installed before tripleoclient | 07:24 |
| marios|ruck | jpodivin: i assumed python3-cliff isn't the actual cliff package but python library/modules for cliff ... maybe that is the actual package this what you mean ? | 07:24 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates master: Enable archive task_log records while archiving the database https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801939 | 07:24 |
| jpodivin | would be the most straightforward explanation | 07:25 |
| marios|ruck | jpodivin: yeah looks like https://koji.fedoraproject.org/koji/buildinfo?buildID=1773950 | 07:25 |
| marios|ruck | arxcruz: ^^^ | 07:25 |
| jpodivin | marios|ruck: in that case there is just one question left. Why did it cause the jobs to fail in other projects? | 07:26 |
| marios|ruck | jpodivin: i am not sure i was out those 2 days it happened | 07:27 |
| marios|ruck | arxcruz: did your patch actually break jobs or did we just revert it 'in case' ? | 07:27 |
| ykarel | marios|ruck, tripleo-repos is special case, the same will not apply to validation or other openstack projects | 07:29 |
| marios|ruck | ykarel: you mean, because it comes in before tripleoclient, that kind of thing? | 07:30 |
| ykarel | tripleo-repos is used to setup repos, and should be installable from base repos | 07:30 |
| marios|ruck | ykarel: and in fact in the job in questino here it comes from delorean repos... so this is what alex meant by 'centos repos' | 07:31 |
| ykarel | marios|ruck, alex means base centos repos | 07:31 |
| marios|ruck | ykarel: ack thanks makes sense... so there you go jpodivin ^^^ that's why | 07:31 |
| ykarel | http://mirror.centos.org/centos/8-stream/ | 07:31 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Do not log amphora ssh keys https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801500 | 07:32 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Do not log amphora ssh keys https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801499 | 07:32 |
| ykarel | the idea for tripleo-repos was it should be installable just with base repos https://trunk.rdoproject.org/centos8-master/component/tripleo/current/python3-tripleo-repos-0.1.1-0.20210714130758.06f41a6.el8.noarch.rpm | 07:33 |
| ykarel | with addition of cliff it was not possible | 07:33 |
| ykarel | thus the revert | 07:33 |
| *** rpittau|afk is now known as rpittau | 07:35 | |
| marios|ruck | jpodivin: https://review.opendev.org/c/openstack/validations-libs/+/801623/4#message-6906f83d88ab6f26d76e4202591f59ee34679ffb | 07:37 |
| marios|ruck | sorry for the noise and thanks for responding quickly jpodivin | 07:37 |
| jpodivin | ykarel, marios|ruck: np. And thanks for the explanation. | 07:38 |
| jpodivin | ykarel, marios|ruck: now we can proceed with the changes without worry. Well, more worry than usual anyway. | 07:39 |
| opendevreview | Bogdan Dobrelya proposed openstack/python-tripleoclient master: Adjust max_template_size for ephemeral heat https://review.opendev.org/c/openstack/python-tripleoclient/+/801955 | 07:40 |
| jpodivin | marios|ruck++ I see you have updated the comment on the patch with more info. Thank you. | 07:55 |
| marios|ruck | ack jpodivin | 07:56 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-common master: Re-add tripleoclient and tempest container https://review.opendev.org/c/openstack/tripleo-common/+/801987 | 08:05 |
| *** marios_ is now known as marios | 08:06 | |
| *** marios is now known as marios|ruck | 08:08 | |
| marios|ruck | ramishra_: o/ fyi test OK so abandoned the revert https://review.opendev.org/c/openstack/puppet-neutron/+/801879/1#message-53b81874f99230d60526b5b190b76b731f35d180 | 08:22 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-quickstart master: Make use of tripleo.repo.get_hash in repo-setup role https://review.opendev.org/c/openstack/tripleo-quickstart/+/801776 | 08:24 |
| *** pmannidi is now known as pmannidi|AFK | 08:30 | |
| ramishra_ | marios|ruck: ack, sounds good, hopefully that fix will land soon to clear the gate | 08:32 |
| marios|ruck | ramishra_: yeah i saw bogdan hit recheck already there are a lot of jobs there so a bit concerned about that lets see | 08:33 |
| *** chem is now known as Guest1877 | 08:38 | |
| arxcruz | marios|ruck: it does break the jobs | 08:39 |
| marios|ruck | arxcruz: ack yeah worked it out... it broke the jobs because it couldn't install tripleo-repos. in this case it was OK https://review.opendev.org/c/openstack/validations-libs/+/801623/4#message-6906f83d88ab6f26d76e4202591f59ee34679ffb | 08:40 |
| marios|ruck | arxcruz: specifically we can't depend on cliff for *tripleo-repos* | 08:41 |
| arxcruz | marios|ruck: yes, i think we should keep as it is | 08:45 |
| marios|ruck | arxcruz: thanks | 08:48 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Pylint setting adjustment https://review.opendev.org/c/openstack/python-tripleoclient/+/801991 | 08:52 |
| marios|ruck | amoralej: o/ hey can you please check https://bugs.launchpad.net/tripleo/+bug/1937035/comments/6 | 08:53 |
| bandini | marios|ruck, arxcruz: i noticed quay.io/tripleomaster/openstack-*:current-tripleo are 6 days old (and so neutron server container lacks python3-mod_wsgi). Is that expected or is there a sync hiccup? | 09:00 |
| marios|ruck | bandini: we haven't promoted master for a few days cos of blockers | 09:00 |
| bandini | ack got it | 09:00 |
| arxcruz | bandini: although we have all hashes there, the current-tripleo hash is based on promotion | 09:01 |
| marios|ruck | bandini: current-tripleo/2021-07-17 11:48 https://trunk.rdoproject.org/centos8-master/ | 09:01 |
| marios|ruck | bandini: the last blocker is getting cleared today hopefully i am going to chase the master promotion | 09:01 |
| opendevreview | Sorin Sbârnea proposed openstack/ansible-role-collect-logs master: Configure molecule to use podman https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/801993 | 09:01 |
| bandini | ack, thanks folks | 09:01 |
| arxcruz | bandini: you can check https://trunk.rdoproject.org/api-centos8-master-uc/api/promotions?promote_name=current-tripleo&limit=1 and notice that current-tripleo on quai is pointing to this hash | 09:01 |
| bandini | ack /me bookmarks | 09:02 |
| *** ykarel is now known as ykarel|lunch | 09:07 | |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Replacing MD5 use to comply with FIPS https://review.opendev.org/c/openstack/python-tripleoclient/+/801460 | 09:27 |
| amoralej | marios|ruck, i just left a comment, i hope that helps | 09:34 |
| marios|ruck | amoralej: thank you will check in sec | 09:35 |
| marios|ruck | arxcruz: do we use the same skiplist os-tempest for downstream jobs too can i add jobs into openstack-tempest-skiplist (e.g. periodic-tripleo-ci-rhel-8-scenario010-standalone-network-rhos-17 ) | 09:39 |
| arxcruz | marios|ruck: yes | 09:42 |
| *** jpodivin_ is now known as jpodivin | 09:43 | |
| arxcruz | marios|ruck: you can also add under releases the key installer: osp if you want to skip it only on osp, default is to skip in both upstream and downstream if you don't add the installer key | 09:43 |
| arxcruz | marios|ruck: https://docs.openstack.org/openstack-tempest-skiplist/latest/yaml/formatting.html#with-releases-and-installers | 09:44 |
| opendevreview | Merged openstack/tripleo-repos master: Make tripleo-repos be an ansible collection https://review.opendev.org/c/openstack/tripleo-repos/+/801621 | 09:45 |
| marios|ruck | arxcruz: thank you | 09:47 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-common master: Re-add tripleoclient and tempest container https://review.opendev.org/c/openstack/tripleo-common/+/801987 | 09:56 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-quickstart master: Removing inventory file path from tox https://review.opendev.org/c/openstack/tripleo-quickstart/+/801999 | 10:05 |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible stable/victoria: Allow for cron programming of backups https://review.opendev.org/c/openstack/tripleo-ansible/+/801733 | 10:19 |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible stable/wallaby: Remove the usage of ansible_date_time https://review.opendev.org/c/openstack/tripleo-ansible/+/801883 | 10:28 |
| opendevreview | Anand Bhat proposed openstack/tripleo-heat-templates master: Migrate from testr to stestr https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802000 | 10:31 |
| *** ykarel|lunch is now known as ykarel | 10:33 | |
| opendevreview | Francesco Pantano proposed openstack/tripleo-common stable/ussuri: Bump nautilus containers to v4.0.19 https://review.opendev.org/c/openstack/tripleo-common/+/801884 | 10:33 |
| opendevreview | Bernard Cafarelli proposed openstack/tripleo-heat-templates stable/victoria: [stable only] Fix NeutronL3AgentAvailabilityZone parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802001 | 10:35 |
| opendevreview | Jose Luis Franco proposed openstack/tripleo-upgrade stable/train: Stop the upgrade script if some of the commands fail. https://review.opendev.org/c/openstack/tripleo-upgrade/+/802002 | 10:42 |
| opendevreview | Juan Larriba proposed openstack/tripleo-ansible stable/wallaby: Remove the usage of ansible_date_time https://review.opendev.org/c/openstack/tripleo-ansible/+/801883 | 10:45 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-common stable/train: Bump nautilus containers to v4.0.19 https://review.opendev.org/c/openstack/tripleo-common/+/802003 | 10:46 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-common stable/train: Bump nautilus containers to v4.0.19 https://review.opendev.org/c/openstack/tripleo-common/+/802003 | 10:49 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798427 | 11:07 |
| marios|ruck | ramishra_: i am tempted to bring the revert back https://review.opendev.org/c/openstack/puppet-neutron/+/801933/1#message-5a0eef40cd55034a153012c35c10fbfd49ab7912 and current run failing again https://zuul.openstack.org/status#801933 | 11:18 |
| marios|ruck | ramishra_: ah damn... the revert is the same thing/repo so same problem :/ | 11:19 |
| opendevreview | Mikolaj Ciecierski proposed openstack/ansible-role-tripleo-modify-image master: Adds patch mechanism for testing changes from OpenDev https://review.opendev.org/c/openstack/ansible-role-tripleo-modify-image/+/802006 | 11:23 |
| ykarel | Tengu, hi | 11:34 |
| ykarel | are u aware about command not allowed ; TTY=unknown ; PWD=/ ; USER=heat ; COMMAND=/usr/bin/find /proc/7/fd/ -ilname socket* -printf already in healcheck? | 11:34 |
| Tengu | ykarel: errrr.... nope? though I'd say: one more reason (if needed.....) to remove those bloody scripts. | 11:35 |
| ykarel | ex: https://logserver.rdoproject.org/openstack-periodic-integration-main/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-scenario002-standalone-master/c6bcfc7/logs/undercloud/var/log/extra/journal_errors.txt.gz | 11:35 |
| Tengu | ykarel: though.... not that surprising. "heat" user can't access everything. Is it really creating an issue? | 11:35 |
| Tengu | ykarel: (and there's also barbican, nova, cinder.... well not that surprising) | 11:36 |
| ykarel | Tengu, just noticed, and many containers have this error | 11:36 |
| Tengu | yeah - as said, "expected" in the end.. still, it shouldn't fail anything at this point? | 11:37 |
| ykarel | not seen failure due to it yet | 11:38 |
| Tengu | I'd be surprised if there were any | 11:38 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-heat-templates master: Disable all container healthchecks by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/799960 | 11:38 |
| Tengu | ykarel: -^^ moved to "active" (ping mrunge - since Martin doesn't seem to be here?) | 11:39 |
| ykarel | ack | 11:39 |
| Tengu | I'd rather lock it (-w) until CloudOps has a view on it. | 11:39 |
| ykarel | sure | 11:40 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-heat-templates stable/train: [train-only] Add the ability to remove leapp actors https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801777 | 12:06 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: EndpointNotFound is now handled gracefully https://review.opendev.org/c/openstack/tripleo-validations/+/786867 | 12:08 |
| *** jcapitao is now known as jcapitao_afk | 12:14 | |
| cloudnull | mornings | 12:31 |
| mrunge | Tengu ?? | 12:38 |
| mrunge | oh, that! | 12:38 |
| Tengu | mrunge: yeah. "that". the removal of healthchecks :). | 12:39 |
| mrunge | Tengu: I remember Martin mentioned a smarter way of doing health checks | 12:39 |
| mrunge | I forgot the details here though | 12:40 |
| Tengu | mrunge: hmmm it was more about a smarter way to query healthcheck status - avoiding log parsing, and using the podman socket | 12:40 |
| mrunge | yes, I agree on that goal, no question | 12:41 |
| Tengu | but.... it doesn't solve the root of all evil: the way podman is launching healthchecks, and the issues it creates :/ | 12:41 |
| mrunge | I am not sure if I'd agree with just disabling these | 12:41 |
| mrunge | and I am quite sure operators will want health checks run like every minute (or even more often) | 12:42 |
| mrunge | replacing individual checks with a script querying all containers sequentially doesn't scale very well | 12:43 |
| *** jcapitao_afk is now known as jcapitao | 12:44 | |
| Tengu | mrunge: (I'm on a meeting - sorry for slow answers) I've pushed a spec a couple of months ago and... well, I'm pretty sure I've pinged you about it :/ | 12:44 |
| mrunge | you did not | 12:44 |
| Tengu | mrunge: https://specs.openstack.org/openstack/tripleo-specs/specs/xena/healthcheck-cleanup.html | 12:44 |
| mrunge | so, the cloudops team was ... surprised by this | 12:44 |
| mrunge | I found it myself just by coincidence | 12:45 |
| Tengu | bleh.. so my brain apparently split on that one. | 12:45 |
| mrunge | you can imagine the joy | 12:45 |
| Tengu | completly | 12:45 |
| opendevreview | Ananya proposed openstack/ansible-role-collect-logs master: Update url for sova query source https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/787502 | 12:45 |
| *** ramishra_ is now known as ramishra | 12:49 | |
| *** amoralej is now known as amoralej|lunch | 13:09 | |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible master: WIP - Managed ceph when is installed with cephadm https://review.opendev.org/c/openstack/tripleo-ansible/+/795318 | 13:13 |
| opendevreview | Ananya proposed openstack/ansible-role-collect-logs master: Update url for sova query source https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/787502 | 13:16 |
| opendevreview | Ananya proposed openstack/ansible-role-collect-logs master: Update url for sova query source https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/787502 | 13:25 |
| opendevreview | Ananya proposed openstack/ansible-role-collect-logs master: Update url for sova query source https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/787502 | 13:46 |
| opendevreview | Ananya proposed openstack/ansible-role-collect-logs master: Update url for sova query source https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/787502 | 13:47 |
| opendevreview | Ananya proposed openstack/ansible-role-collect-logs master: Update url for sova query source https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/787502 | 13:49 |
| mwhahaha | zbr: tripleo-repos still broken | 13:56 |
| mwhahaha | https://logserver.rdoproject.org/17/34617/1/check/DLRN-rpmbuild-centos8/1b5723a/job-output.txt | 13:56 |
| mwhahaha | spec needs update to handle new /usr/bin stuff | 13:57 |
| zbr | mwhahaha: yeah, you cannot imagine what kind of day i had... | 13:57 |
| mwhahaha | :D | 13:57 |
| mwhahaha | considering i started with cix pings, yea | 13:57 |
| mwhahaha | i'll update the spec later | 13:57 |
| zbr | we really need to assure rpm building happens on changes made to tripleo-repos, it counts as packaging so likely more important thant running unitests ;) | 13:58 |
| zbr | thanks. | 13:58 |
| ykarel | would need to backport to other releases in spec as tripleo-repos branchless | 13:58 |
| bogdando | https://bugs.launchpad.net/tripleo/+bug/1936138 did we break the create vs update logic, again? | 13:59 |
| bogdando | globally? | 13:59 |
| mwhahaha | probably | 13:59 |
| mwhahaha | we talked about solutions | 13:59 |
| mwhahaha | it's WIP i think | 13:59 |
| bogdando | this looks also related to idempotency of net config runs mentioned by cloudnull | 13:59 |
| mwhahaha | naw that was different | 13:59 |
| bogdando | ack | 14:00 |
| 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 | 14:01 |
| *** amoralej|lunch is now known as amoralej | 14:01 | |
| opendevreview | Ananya proposed openstack/tripleo-ci-health-queries master: Updating the queries https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/802028 | 14:05 |
| 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 | 14:06 |
| *** rpittau is now known as rpittau|afk | 14:13 | |
| marios|ruck | mwhahaha: o/ hey please check when you next have a chance replied to your comment there https://review.opendev.org/c/openstack/tripleo-specs/+/801512/2#message-2cb17dd14e53251ad0c60ecbb23f912e07e9d950 | 14:14 |
| mwhahaha | marios|ruck: it needs text as what's there isn't exactly correct | 14:15 |
| mwhahaha | will add details later | 14:15 |
| marios|ruck | mwhahaha: k i can check again. i replied to your comment though I don't think it will be *more* complex should be the same no? just skipping the in between bits. k thanks add comments on the review and i will check and update next week | 14:16 |
| mwhahaha | it's more complex | 14:16 |
| mwhahaha | because the difference is much larger | 14:16 |
| *** pojadhav- is now known as pojadhav | 14:16 | |
| mwhahaha | the drift exists between each single release today, but it's not as big as a 3 verrsion drift | 14:16 |
| mwhahaha | so it's less complex to resolve a single version backport than 3 | 14:17 |
| marios|ruck | mwhahaha: the difference between U-T is larger than W-T but going W-T in both cases is the same | 14:17 |
| mwhahaha | it's larger with more chance to screw up | 14:17 |
| mwhahaha | you get little validations each version W->V->U->T | 14:17 |
| mwhahaha | so the changes are 1) smaller and 2) validated at each phase | 14:18 |
| marios|ruck | mwhahaha: right that's what i mean about "It might make it harder to follow the context behind some of the larger diffs, without seeing the 'in between ' bits but the diff should be the same, i think?" | 14:18 |
| marios|ruck | mwhahaha: k i can add some text aroudn that | 14:18 |
| mwhahaha | not necessarily, depends on what/where things change | 14:18 |
| mwhahaha | like if something moved from tripleo-common to tripleoclient | 14:18 |
| mwhahaha | anyway i just don't want to gloss over that there is a dev impact | 14:19 |
| mwhahaha | and it's not just rainbows and unicorns :D | 14:19 |
| marios|ruck | mwhahaha: yeah i don't mean to pass that message i mean i tried to dig a bit into the 'no place to backport fixes to if we use tags' in Other End User Impact | 14:19 |
| mwhahaha | it's a dev impact not an end user impact | 14:20 |
| mwhahaha | end users won't care | 14:20 |
| marios|ruck | mwhahaha: well hey care if they want the fix but i'm equating end-user with dev and probably shouldn't ... can move that bit too | 14:20 |
| marios|ruck | ramishra: rlandy: finally in the gate https://zuul.openstack.org/status#801933 lets see | 14:25 |
| rlandy | :) | 14:27 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/wallaby: vncproxy: removal of tags and steps change from post_upgrade_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801885 | 14:30 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/victoria: vncproxy: removal of tags and steps change from post_upgrade_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801886 | 14:30 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/ussuri: vncproxy: removal of tags and steps change from post_upgrade_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801887 | 14:30 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/train: vncproxy: removal of tags and steps change from post_upgrade_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801888 | 14:30 |
| opendevreview | Anand Bhat proposed openstack/tripleo-heat-templates master: Migrate from testr to stestr https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802000 | 14:42 |
| *** jcapitao1 is now known as jcapitao | 14:43 | |
| *** ykarel is now known as ykarel|away | 14:56 | |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Enabling pylint no-else-return https://review.opendev.org/c/openstack/python-tripleoclient/+/802032 | 14:57 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/train: vncproxy: removal of tags and steps change from post_upgrade_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801888 | 14:58 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/train: vncproxy: removal of tags and steps change from post_upgrade_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/801888 | 14:59 |
| *** rlandy is now known as rlandy|ruck | 15:01 | |
| opendevreview | Kamil Sambor proposed openstack/tripleo-heat-templates master: Add param to configure neutron agent_down_time https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802035 | 15:09 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-ci master: Future proof ansible test code https://review.opendev.org/c/openstack/tripleo-ci/+/802038 | 15:13 |
| opendevreview | Jose Luis Franco proposed openstack/tripleo-heat-templates stable/train: WIP: Add upgrade_rollback stage to restore from FFU. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/800898 | 15:20 |
| *** holser is now known as holser_ | 15:24 | |
| opendevreview | Marios Andreou proposed openstack/puppet-tripleo stable/ussuri: Bump metadata 12.7.0 for ussuri tripleo repos release https://review.opendev.org/c/openstack/puppet-tripleo/+/802042 | 15:29 |
| marios|ruck | mwhahaha: please when you next have a minute thank you https://review.opendev.org/c/openstack/puppet-tripleo/+/802042 | 15:32 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-ci master: Future proof ansible test code https://review.opendev.org/c/openstack/tripleo-ci/+/802038 | 15:49 |
| *** holser_ is now known as holser | 15:51 | |
| marios|ruck | ramishra: finally https://review.opendev.org/c/openstack/puppet-neutron/+/801933 | 16:03 |
| *** marios|ruck is now known as marios | 16:03 | |
| *** ChanServ sets mode: +o marios | 16:04 | |
| *** marios changes topic to "next irc meeting 06 August https://etherpad.opendev.org/p/tripleo-meeting-items " | 16:05 | |
| marios | rlandy|ruck: fyi http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023810.html | 16:07 |
| rlandy|ruck | thanks | 16:07 |
| marios | rlandy|ruck: have a good 1 | 16:08 |
| *** marios is now known as marios|out | 16:08 | |
| rlandy|ruck | marios|out: you too | 16:08 |
| rlandy|ruck | recheck master failing jobs | 16:08 |
| simondodsley | Now that the RDO CI issue has been resolved can I get an extra +2 and a +W for https://review.opendev.org/c/openstack/tripleo-heat-templates/+/792850 | 16:09 |
| *** amoralej is now known as amoralej|off | 16:29 | |
| opendevreview | Owen McGonagle proposed openstack/tripleo-validations master: remove using containers-prepare-parameter.yaml https://review.opendev.org/c/openstack/tripleo-validations/+/796061 | 16:44 |
| rlandy|ruck | gchamoul: looking in validation-libs failure | 17:22 |
| rlandy|ruck | missing containers | 17:22 |
| opendevreview | Alan Bishop proposed openstack/tripleo-heat-templates stable/train: Support cinder image conversion on an NFS share https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802076 | 17:29 |
| opendevreview | Owen McGonagle proposed openstack/tripleo-validations master: remove using containers-prepare-parameter.yaml https://review.opendev.org/c/openstack/tripleo-validations/+/796061 | 17:39 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: Fix typo in ovn-dbs-cluster northd kolla config file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802092 | 18:06 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-docs master: Add cephadm debug mode section https://review.opendev.org/c/openstack/tripleo-docs/+/802093 | 18:11 |
| opendevreview | Juan Pablo Lopez proposed openstack/python-tripleoclient master: Adding --mode option to specify the type of ReaR configuration https://review.opendev.org/c/openstack/python-tripleoclient/+/797314 | 18:19 |
| 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 | 18:31 |
| mwhahaha | rlandy|ruck: why is that validation-libs pulling from docker.io? | 18:41 |
| mwhahaha | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_315/800260/6/gate/tripleo-ci-centos-8-standalone-validation-libs/315c45e/logs/undercloud/home/zuul/containers-prepare-parameters.yaml | 18:41 |
| rlandy|ruck | mwhahaha: yeah | 18:42 |
| rlandy|ruck | they are not using content provider | 18:42 |
| rlandy|ruck | or quay | 18:42 |
| mwhahaha | should be quay | 18:42 |
| rlandy|ruck | and docker.io didn't have containers with that hash | 18:42 |
| rlandy|ruck | quay did | 18:42 |
| mwhahaha | we've moved the defaults to quay since forever | 18:42 |
| rlandy|ruck | I know | 18:42 |
| rlandy|ruck | looking at their config | 18:43 |
| rlandy|ruck | they keep their own version of the job | 18:43 |
| mwhahaha | but doesn't that stuff come from quickstart? | 18:43 |
| gchamoul | rlandy|ruck: Hi Ronelle, you mean validations-libs patches failing in the gate? | 18:43 |
| rlandy|ruck | gchamoul: yep | 18:43 |
| rlandy|ruck | they were failing to pull containers | 18:44 |
| rlandy|ruck | namespace: docker.io/tripleomaster | 18:44 |
| rlandy|ruck | should be quay | 18:44 |
| rlandy|ruck | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_315/800260/6/gate/tripleo-ci-centos-8-standalone-validation-libs/315c45e/logs/undercloud/home/zuul/containers-prepare-parameters.yaml | 18:44 |
| rlandy|ruck | per ^^ | 18:44 |
| rlandy|ruck | https://zuul.opendev.org/t/openstack/builds?job_name=tripleo-ci-centos-8-standalone-validation-libs | 18:45 |
| rlandy|ruck | admit it was working for a while | 18:45 |
| rlandy|ruck | but we should still be using quay | 18:45 |
| mwhahaha | is it using an old tripleo-common/ | 18:46 |
| mwhahaha | 2021-07-23 18:18:00.030315 | primary | "_prepare_cmd": "openstack tripleo container image prepare default --output-env-file /home/zuul/containers-prepare-parameters.yaml " | 18:46 |
| mwhahaha | cause it should have been generated | 18:46 |
| mwhahaha | i'm very confused where that's coming from | 18:46 |
| rlandy|ruck | https://opendev.org/openstack/validations-libs/src/branch/master/.zuul.yaml#L62 | 18:47 |
| rlandy|ruck | vars: | 18:47 |
| rlandy|ruck | build_container_images: true | 18:47 |
| rlandy|ruck | should be building images anyways | 18:47 |
| rlandy|ruck | but the gate is not | 18:47 |
| rlandy|ruck | so it passes check | 18:48 |
| rlandy|ruck | and fails gate | 18:48 |
| mwhahaha | it should be in gate | 18:48 |
| mwhahaha | it uses the yaml ref | 18:48 |
| mwhahaha | https://opendev.org/openstack/validations-libs/src/branch/master/.zuul.yaml#L83 | 18:48 |
| mwhahaha | oh wait | 18:48 |
| mwhahaha | that's the wrong spot | 18:48 |
| mwhahaha | it's actually putting it in vars: vars: | 18:49 |
| mwhahaha | the anchor on L62 shold be on L63 | 18:49 |
| mwhahaha | still shouldn't use docker.io tho | 18:49 |
| rlandy|ruck | well we should fix the anchor first | 18:50 |
| mwhahaha | yes | 18:50 |
| rlandy|ruck | as the same job should run in check and gate | 18:50 |
| * rlandy|ruck puts in patch | 18:51 | |
| gchamoul | rlandy|ruck: was on it but go on ;-) | 18:51 |
| rlandy|ruck | gchamoul: well you might want to decide which way to do this | 18:52 |
| rlandy|ruck | you have the anchor on params | 18:52 |
| rlandy|ruck | and then some are repeated | 18:52 |
| gchamoul | yes I was at least fixing the anchor | 18:52 |
| rlandy|ruck | and then the reference anchor on vars | 18:52 |
| mwhahaha | probablyu should be on vas cause the deps are different | 18:52 |
| mwhahaha | unless cover is voting then that's another bug | 18:53 |
| rlandy|ruck | gchamoul: k - let me put in the vars anchor and you can edit if you want | 18:54 |
| rlandy|ruck | sec | 18:54 |
| gchamoul | ack | 18:54 |
| opendevreview | Ronelle Landy proposed openstack/validations-libs master: Build containers in gate - to match check https://review.opendev.org/c/openstack/validations-libs/+/802096 | 18:58 |
| rlandy|ruck | gchamoul: mwhahaha: ^^ pls check if you agree | 18:59 |
| gchamoul | on it | 19:00 |
| mwhahaha | wfm, don't really need to keep the other anchor but doesn't hurt to leave it | 19:00 |
| gchamoul | yes the params anchor is useless | 19:00 |
| rlandy|ruck | idk if someone else wanted a params anchor - so left it | 19:00 |
| rlandy|ruck | feel free to remove it | 19:00 |
| gchamoul | works for me | 19:00 |
| rlandy|ruck | can rebase the failing jobs on this review to get through the gate | 19:01 |
| gchamoul | doing it | 19:01 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-libs master: Add Categories metadata key management https://review.opendev.org/c/openstack/validations-libs/+/800260 | 19:02 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-libs master: Add Products metadata key management https://review.opendev.org/c/openstack/validations-libs/+/800721 | 19:02 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-libs master: Add CLI auto generated documentation using Cliff https://review.opendev.org/c/openstack/validations-libs/+/801623 | 19:02 |
| gchamoul | thx rlandy|ruck & mwhahaha! | 19:03 |
| rlandy|ruck | sure | 19:03 |
| mwhahaha | oh docker_registry_host | 19:05 |
| mwhahaha | that's still docker.io somewhere | 19:05 |
| mwhahaha | that's what's dong it | 19:05 |
| mwhahaha | tripleo-quickstart/config/release/trunk/* | 19:06 |
| opendevreview | Alex Schultz proposed openstack/tripleo-quickstart master: Switch default registry to quay.io https://review.opendev.org/c/openstack/tripleo-quickstart/+/802100 | 19:07 |
| mwhahaha | rlandy|ruck: -^ it's doing it from there | 19:08 |
| mwhahaha | there are other docker.io references but i think those are the ones controlling if you don't build/have a provider | 19:08 |
| rlandy|ruck | should parent directly off parent: tripleo-ci-centos-8-standalone | 19:09 |
| rlandy|ruck | https://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-8/master.yml | 19:09 |
| rlandy|ruck | should reference ^^ | 19:09 |
| mwhahaha | yea that's probably also a spot | 19:10 |
| mwhahaha | i can update that one too | 19:10 |
| rlandy|ruck | np - I'll fix those | 19:10 |
| * rlandy|ruck updates | 19:10 | |
| opendevreview | Alex Schultz proposed openstack/tripleo-quickstart master: Switch default registry to quay.io https://review.opendev.org/c/openstack/tripleo-quickstart/+/802100 | 19:11 |
| mwhahaha | rlandy|ruck: -^ i got those too | 19:12 |
| mwhahaha | sorry had already done it byt he time i read your message :D | 19:12 |
| rlandy|ruck | very quick :) | 19:12 |
| * rlandy|ruck barely rebasing repo | 19:12 | |
| rlandy|ruck | k - thanks | 19:12 |
| rlandy|ruck | arxcruz: fyi ^^ if you are still around | 19:13 |
| rlandy|ruck | default changed | 19:13 |
| rlandy|ruck | if no content provider in use | 19:13 |
| rlandy|ruck | and no image_build within the job | 19:13 |
| arxcruz | ack | 19:14 |
| simondodsley | Now that the RDO CI issue has been resolved can I get an extra +2 and a +W for https://review.opendev.org/c/openstack/tripleo-heat-templates/+/792850 | 19:32 |
| *** holser is now known as holser_ | 21:07 | |
| *** holser_ is now known as holser | 21:54 | |
| *** pmannidi|AFK is now known as pmannidi | 22:02 | |
| opendevreview | Merged 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 | 22:12 |
| opendevreview | Merged openstack/tripleo-common-tempest-plugin master: tripleo-common-tempest-plugin - Step 3: Remove Project Content https://review.opendev.org/c/openstack/tripleo-common-tempest-plugin/+/800155 | 22:12 |
| opendevreview | Merged openstack/tripleo-ansible stable/victoria: Explicitly set ceph_dashboard_vip when Ceph Dashboard is enabled https://review.opendev.org/c/openstack/tripleo-ansible/+/801727 | 22:12 |
| opendevreview | Merged openstack/tripleo-ansible stable/ussuri: Explicitly set ceph_dashboard_vip when Ceph Dashboard is enabled https://review.opendev.org/c/openstack/tripleo-ansible/+/801728 | 22:12 |
| opendevreview | Merged openstack/puppet-tripleo stable/ussuri: Bump metadata 12.7.0 for ussuri tripleo repos release https://review.opendev.org/c/openstack/puppet-tripleo/+/802042 | 22:12 |
| *** pmannidi is now known as pmannidi|brb | 22:19 | |
| *** pmannidi|brb is now known as pmannidi | 22:40 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!