opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Barbican: Replace hiera by lookup https://review.opendev.org/c/openstack/puppet-tripleo/+/839222 | 00:14 |
---|---|---|
opendevreview | John Fulton proposed openstack/tripleo-docs master: Update Distributed Multibackend Storage for cephadm https://review.opendev.org/c/openstack/tripleo-docs/+/839281 | 00:16 |
*** dviroel|rover|afk is now known as dviroel|out | 00:16 | |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Use built-in function to remove undef values from array https://review.opendev.org/c/openstack/puppet-tripleo/+/839282 | 00:16 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Use stdlib function to remove undef values from array https://review.opendev.org/c/openstack/puppet-tripleo/+/839282 | 00:17 |
*** rlandy|bbl is now known as rlandy | 00:34 | |
*** rlandy is now known as rlandy|out | 00:55 | |
*** ysandeep|out is now known as ysandeep | 02:12 | |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Ssh: Replace hiera by lookup https://review.opendev.org/c/openstack/puppet-tripleo/+/839287 | 02:23 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Ceilometer: Replace hiera by lookup https://review.opendev.org/c/openstack/puppet-tripleo/+/839288 | 02:30 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: ceilometer-polling: : Require redis backend https://review.opendev.org/c/openstack/puppet-tripleo/+/839289 | 02:31 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Ceilometer: Replace hiera by lookup https://review.opendev.org/c/openstack/puppet-tripleo/+/839288 | 02:36 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove unused implementation to trigger ceilometer-upgrade https://review.opendev.org/c/openstack/puppet-tripleo/+/839290 | 02:36 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove unused implementation to trigger ceilometer-upgrade https://review.opendev.org/c/openstack/puppet-tripleo/+/839290 | 02:38 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Ceilometer: Replace hiera by lookup https://review.opendev.org/c/openstack/puppet-tripleo/+/839288 | 02:38 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Remove unused implementation to manage stunnel by systemd https://review.opendev.org/c/openstack/puppet-tripleo/+/839291 | 02:52 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove parameter to run stunnel by systemd https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839292 | 02:53 |
opendevreview | Steve Baker proposed openstack/tripleo-image-elements master: Add element reset-bls-entries https://review.opendev.org/c/openstack/tripleo-image-elements/+/837430 | 02:58 |
opendevreview | Ian Wienand proposed openstack/diskimage-builder master: Move image validation into extract-image https://review.opendev.org/c/openstack/diskimage-builder/+/839294 | 03:53 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Ensure the dependent service is started in advance https://review.opendev.org/c/openstack/tripleo-ansible/+/839295 | 04:01 |
*** ysandeep is now known as ysandeep|afk | 04:18 | |
opendevreview | Ian Wienand proposed openstack/diskimage-builder master: Move image validation into extract-image https://review.opendev.org/c/openstack/diskimage-builder/+/839294 | 05:07 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: WIP: Manage CA certificates using ansible https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839296 | 05:15 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: WIP: Manage CA certificates using ansible https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839296 | 05:18 |
*** ysandeep|afk is now known as ysandeep | 06:13 | |
*** pojadhav- is now known as pojadhav|pto | 06:20 | |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Use Validations libs Client for main validation entry point https://review.opendev.org/c/openstack/python-tripleoclient/+/823644 | 06:29 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Overriding the log path argument for tripleoclient cli https://review.opendev.org/c/openstack/python-tripleoclient/+/823645 | 06:31 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Fix the validations_log_basedir overriding https://review.opendev.org/c/openstack/python-tripleoclient/+/823646 | 06:31 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Setting default inventory path for VF interface with ooo client. https://review.opendev.org/c/openstack/python-tripleoclient/+/839228 | 06:32 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: ceilometer-polling: : Require redis backend https://review.opendev.org/c/openstack/puppet-tripleo/+/839289 | 06:44 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: ceilometer-polling: : Require redis backend https://review.opendev.org/c/openstack/puppet-tripleo/+/839289 | 06:46 |
opendevreview | Ian Wienand proposed openstack/diskimage-builder master: Move image validation into extract-image https://review.opendev.org/c/openstack/diskimage-builder/+/839294 | 06:46 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: Ensure the dependent service is started in advance https://review.opendev.org/c/openstack/tripleo-ansible/+/839295 | 06:51 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: [Train-Only] Use Validations libs Client for main validation entry point https://review.opendev.org/c/openstack/python-tripleoclient/+/823644 | 07:02 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Overriding the log path argument for tripleoclient cli https://review.opendev.org/c/openstack/python-tripleoclient/+/823645 | 07:05 |
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 | 07:05 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Fix the validations_log_basedir overriding https://review.opendev.org/c/openstack/python-tripleoclient/+/823646 | 07:05 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/train: Setting default inventory path for VF interface with ooo client. https://review.opendev.org/c/openstack/python-tripleoclient/+/839228 | 07:05 |
*** amoralej|off is now known as amoralej | 07:11 | |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-quickstart-extras master: Move the masquerade override to parameter https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839196 | 07:11 |
*** jpena|off is now known as jpena | 07:13 | |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: tls_proxy: Replace hiera by lookup https://review.opendev.org/c/openstack/puppet-tripleo/+/839303 | 07:16 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart master: [dnm] test non_root_user_setup: true https://review.opendev.org/c/openstack/tripleo-quickstart/+/839304 | 07:17 |
opendevreview | Ananya proposed openstack/tripleo-quickstart master: Changing container registry for train c8 and master c9 to quay https://review.opendev.org/c/openstack/tripleo-quickstart/+/839305 | 07:21 |
opendevreview | Ananya proposed openstack/tripleo-quickstart master: WIP : Changing container registry for train c8 and master c9 to quay https://review.opendev.org/c/openstack/tripleo-quickstart/+/839305 | 07:22 |
opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Logging: Replace hiera by lookup https://review.opendev.org/c/openstack/puppet-tripleo/+/839306 | 07:22 |
*** ysandeep is now known as ysandeep|brb | 07:25 | |
opendevreview | Ian Wienand proposed openstack/diskimage-builder master: centos: avoid head pipe failure https://review.opendev.org/c/openstack/diskimage-builder/+/839307 | 07:28 |
*** ysandeep|brb is now known as ysandeep | 07:30 | |
opendevreview | Ian Wienand proposed openstack/diskimage-builder master: Move image validation into extract-image https://review.opendev.org/c/openstack/diskimage-builder/+/839294 | 07:34 |
opendevreview | Jiri Podivin proposed openstack/validations-libs master: Validation help improvement https://review.opendev.org/c/openstack/validations-libs/+/807828 | 07:45 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Fix service dependency on libvirt daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839311 | 07:50 |
opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: Disabling container validation when running redhat-subscription https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839330 | 08:03 |
amoralej | tkajinam, i'm preparing to update puppet and facter to 7.16.0 and 4.2.8 in RDO | 08:16 |
amoralej | there were perf issues with facter 4.X in the past but i'd say they were fixed | 08:17 |
amoralej | would you review this update from puppet/tripleo side? | 08:17 |
tkajinam | amoralej, I'm not aware of any issue atm... I've submit test patches to run some jobs with 7.16.0 https://review.opendev.org/q/topic:puppet-7-16 | 08:24 |
tkajinam | amoralej, I remember there is an issue with old fater and we need to bump it to 4.x for FIPs compliance | 08:24 |
amoralej | tkajinam, yes, that's the request i got | 08:25 |
tkajinam | I'm not concerned about puppet 7.16.0 but bumping facter to 4.2.8 might be a bit tricky, because that is major bump | 08:26 |
amoralej | well, puppetlabs runs 7.x with 4.x by default | 08:26 |
tkajinam | ah, ok | 08:26 |
amoralej | https://puppet.com/docs/puppet/7/release_notes_facter.html#release_notes_facter | 08:27 |
amoralej | Released March 2022 and shipped with Puppet 7.15.0. | 08:27 |
amoralej | my concern is mostly with perf issues | 08:27 |
amoralej | but see my note in https://review.rdoproject.org/r/c/deps/facter/+/42263 | 08:27 |
tkajinam | only concern from my side is that we still use legacy facts but according to release note legacy facts are still available | 08:28 |
tkajinam | we might need to update option to generate fact cache in TripleO but we can check that once we get updated package | 08:28 |
amoralej | ah, damn | 08:28 |
tkajinam | afaik legacy facts have never been deprecated so should not be removed very soon.......... I hope | 08:31 |
amoralej | also, i guess we'll be requested to git updated facter/puppet up to wallaby | 08:32 |
tkajinam | yeah for some reason... which I hate :-) | 08:33 |
tkajinam | we are making such change too late :-) | 08:33 |
tkajinam | anyway | 08:34 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: [WIP] switch to default mol config https://review.opendev.org/c/openstack/tripleo-ansible/+/839319 | 08:35 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: [WIP] switch to default mol config https://review.opendev.org/c/openstack/tripleo-ansible/+/839319 | 08:36 |
tkajinam | amoralej, let me know once the new version is released in rdo repos. I'll check job status after that. | 08:40 |
opendevreview | Ananya proposed openstack/tripleo-ci master: Changing container registry ovb vexxhost to quay https://review.opendev.org/c/openstack/tripleo-ci/+/839323 | 09:19 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: [WIP] switch to default mol config https://review.opendev.org/c/openstack/tripleo-ansible/+/839319 | 09:20 |
opendevreview | Ananya proposed openstack/tripleo-quickstart master: WIP : Changing container registry for train c8 and master c9 to quay https://review.opendev.org/c/openstack/tripleo-quickstart/+/839305 | 09:21 |
opendevreview | Ananya proposed openstack/tripleo-quickstart master: WIP : Changing container registry for master c9 to quay https://review.opendev.org/c/openstack/tripleo-quickstart/+/839305 | 09:22 |
opendevreview | Rabi Mishra proposed openstack/tripleo-heat-templates master: Set ownership of files in external_post_deploy_tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839324 | 09:22 |
opendevreview | Rabi Mishra proposed openstack/tripleo-ansible master: Elevate privileges when deleting stack dir https://review.opendev.org/c/openstack/tripleo-ansible/+/839325 | 09:22 |
Tengu | dang... do we know why this job is failing all the time on master? https://review.rdoproject.org/zuul/builds?job_name=tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001&project=openstack/tripleo-ansible bhagyashris|ruck maybe ? | 09:36 |
Tengu | or chandankumar, marios -^^ ? | 09:36 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: collect only specific network facts https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839326 | 09:37 |
Tengu | looks like the OC nodes aren't booting | 09:38 |
Tengu | meh... | 09:38 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: collect only specific network facts https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839326 | 09:38 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: collect only specific network facts https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839326 | 09:38 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: [WIP] switch to default mol config https://review.opendev.org/c/openstack/tripleo-ansible/+/839319 | 09:40 |
chandankumar | Tengu: https://logserver.rdoproject.org/63/839163/2/openstack-check/tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001/ad39bde/logs/baremetal_2_86483_0-console.log | 09:43 |
chandankumar | [H[J[1;1Herror: ../../grub-core/disk/diskfilter.c:916:diskfilter writes are not | 09:43 |
chandankumar | supported. | 09:43 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Inventory for validations https://review.opendev.org/c/openstack/python-tripleoclient/+/839200 | 09:46 |
opendevreview | Merged openstack/diskimage-builder master: Add a job to test building jammy https://review.opendev.org/c/openstack/diskimage-builder/+/836228 | 09:47 |
opendevreview | Juan Larriba proposed openstack/tripleo-ansible stable/wallaby: Mysql has been replaced by mariadb https://review.opendev.org/c/openstack/tripleo-ansible/+/839332 | 09:47 |
marios | Tengu: o/ ruck|rover may know i have not been following that (was also out last couple days) | 09:51 |
opendevreview | Fernando Díaz Bravo proposed openstack/python-tripleoclient master: bnr - overcloud restore - fix error in the help description for the stack parameter https://review.opendev.org/c/openstack/python-tripleoclient/+/838652 | 09:52 |
opendevreview | Yamato Tanaka proposed openstack/puppet-tripleo stable/victoria: HAProxy: enable forwardfor for all http endpoints https://review.opendev.org/c/openstack/puppet-tripleo/+/838800 | 09:52 |
marios | Tengu: looks unstable beyond tripleo-ansible too | 09:52 |
marios | Tengu: https://review.rdoproject.org/zuul/builds?job_name=tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001 | 09:53 |
marios | bhagyashris|ruck: o/ do we have a bug on that? ^ | 09:53 |
opendevreview | Merged openstack/diskimage-builder master: yum-minimal: clean up release package installs https://review.opendev.org/c/openstack/diskimage-builder/+/837248 | 09:54 |
bhagyashris|ruck | marios, hey at my end i am facing issue to open the link | 09:55 |
bhagyashris|ruck | let me check | 09:56 |
marios | bhagyashris|ruck: try without the tripleo-ansible i.e. the one i pointed at should load faster | 09:56 |
bhagyashris|ruck | ack | 09:56 |
marios | bhagyashris|ruck: 3rd party fs1 very red | 09:56 |
marios | bhagyashris|ruck: might be different things but if you see 2/3 with same it is worth filing it | 09:56 |
marios | Tengu: fyi ^^ | 09:57 |
bhagyashris|ruck | marios, yup | 09:57 |
marios | thank you bhagyashris|ruck | 09:59 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible master: [WIP] switch to default mol config https://review.opendev.org/c/openstack/tripleo-ansible/+/839319 | 10:02 |
chandankumar | bogdando: hello | 10:02 |
bogdando | hi | 10:02 |
opendevreview | Merged openstack/diskimage-builder master: Move reset-bls-entries to post-install https://review.opendev.org/c/openstack/diskimage-builder/+/838792 | 10:02 |
chandankumar | bogdando: regarding this change https://review.opendev.org/c/openstack/tripleo-ansible/+/839319 Do we want to rename all the cs8 jobs to centos-stream without version | 10:02 |
chandankumar | ? | 10:02 |
bogdando | likely? | 10:03 |
chandankumar | tripleo-ansible-centos-8-role-addition - tripleo-ansible-centos-stream-role-addition | 10:03 |
chandankumar | then we donot need to update the version each time a new centos stream version comes up | 10:04 |
chandankumar | *job version | 10:04 |
bogdando | Tengu: WDYT? ^^ | 10:04 |
bogdando | makes sense | 10:04 |
opendevreview | Yamato Tanaka proposed openstack/puppet-tripleo stable/train: HAProxy: enable forwardfor for all http endpoints https://review.opendev.org/c/openstack/puppet-tripleo/+/839350 | 10:05 |
bogdando | chandankumar: do you think we should paramterize it, or add another config for driver podman? | 10:05 |
bogdando | we'll need that for standalone ansible roles work | 10:05 |
bogdando | https://review.opendev.org/c/openstack/tripleo-ansible/+/839319/5/.config/molecule/config.yml is for delegated driver | 10:05 |
opendevreview | Ananya proposed openstack/tripleo-ci master: Changing container registry ovb vexxhost to quay https://review.opendev.org/c/openstack/tripleo-ci/+/839323 | 10:06 |
chandankumar | bogdando: So basically we need to test standalone role on both c8 and c9 and similar goes for downstream | 10:07 |
chandankumar | let me think | 10:07 |
bogdando | I linked your work to slagle WIP patches... | 10:07 |
bogdando | I feel like all that can be put together nicely | 10:08 |
bogdando | just unfortunately I'm not a molecule expert at all | 10:08 |
bogdando | chandankumar: as standalone role, I mean this (not the job type) https://review.opendev.org/q/topic:standalone-roles | 10:09 |
bogdando | so there will be tens of roles, and getting rid of repeatable molecule bits for each and every would be a win | 10:09 |
bogdando | especially once we start conferting all puppet-openstack into roles :) | 10:10 |
bogdando | converting* | 10:10 |
bogdando | marios: JFYI | 10:11 |
bogdando | ofc we could offload that dedup to individual teams of contributors who will be working on those new roles, like dvd did with the tripleo kernel role | 10:15 |
bogdando | but I think having a common for all roles templating/defaulting engine like that one would benefit all | 10:16 |
bogdando | not only devs but CI infra team | 10:17 |
bhagyashris|ruck | marios, Tengu, fyi https://bugs.launchpad.net/tripleo/+bug/1970400 | 10:17 |
opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: collect only specific network facts https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839326 | 10:17 |
bogdando | chandankumar: I hope that all makes sense | 10:17 |
chandankumar | bogdando: yes yes a lot | 10:17 |
marios | thanks bogdando | 10:18 |
marios | sorry bogdando was meant for bhagyashris|ruck o/ ^ | 10:18 |
bogdando | np | 10:18 |
bogdando | you can thank me also :) | 10:19 |
marios | :) | 10:19 |
*** rlandy|out is now known as rlandy | 10:24 | |
*** ysandeep is now known as ysandeep|coffee | 10:37 | |
slagle | tkajinam: is this one good? https://review.opendev.org/c/openstack/tripleo-ansible/+/831547 | 11:03 |
slagle | the tht patch passed ci | 11:03 |
Tengu | chandankumar, bogdando makes sense yep | 11:07 |
Tengu | bhagyashris|ruck: thanks! (sorry, was afk for lunch) | 11:07 |
bhagyashris|ruck | np | 11:07 |
opendevreview | Manoj Katari proposed openstack/tripleo-heat-templates master: Remove deprecated template and parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839358 | 11:08 |
Tengu | bhagyashris|ruck: while at it - is this one also known? https://zuul.opendev.org/t/openstack/build/196fedf7c8144a3f8f41b9320aa1253b iirc I hit it already yesterday :/ | 11:09 |
*** dviroel|out is now known as dviroel|rover | 11:09 | |
bhagyashris|ruck | Tengu, hey if you see the recent results it's passing https://zuul.opendev.org/t/openstack/builds?job_name=tripleo-ansible-centos-8-molecule-tripleo_firewall&skip=0 | 11:10 |
Tengu | hmmmm | 11:11 |
Tengu | doesn't really make any sense.... | 11:11 |
* Tengu digging | 11:12 | |
opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Revert "Install jss-5.0.3 in freeipa installation" https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839333 | 11:17 |
chandankumar | dviroel|rover: bhagyashris|ruck ^^ | 11:18 |
*** ysandeep|coffee is now known as ysandeep | 11:28 | |
opendevreview | mbu proposed openstack/python-tripleoclient master: Use constant constants.ANSIBLE_INVENTORY for setting inventory default path https://review.opendev.org/c/openstack/python-tripleoclient/+/839363 | 11:28 |
opendevreview | Cedric Jeanneret proposed openstack/ansible-role-collect-logs master: Get IPTables stats in network info https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/839364 | 11:36 |
Tengu | bhagyashris|ruck, dviroel|rover, rlandy, chandankumar, marios -^^ should help debugging weird things in the network | 11:37 |
Tengu | is that ansible-role-collect-logs a trusted one, or may I depends-on it in a patch? | 11:37 |
chandankumar | Tengu: nope | 11:39 |
chandankumar | Tengu: depends on works | 11:39 |
Tengu | chandankumar: great, thanks! | 11:39 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Get all of the INPUT rules in a dedicated chain https://review.opendev.org/c/openstack/tripleo-ansible/+/839163 | 11:40 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Inventory for validations https://review.opendev.org/c/openstack/python-tripleoclient/+/839200 | 11:47 |
opendevreview | Cedric Jeanneret proposed openstack/ansible-role-collect-logs master: Get IPTables stats in network info https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/839364 | 11:51 |
chandankumar | Tengu: collect-logs is not used in molecule jobs | 11:54 |
Tengu | chandankumar: arggg | 11:55 |
Tengu | chandankumar: what's used then? | 11:55 |
Tengu | (still, that patch would be nice, and would help me debugging the other failures) | 11:55 |
chandankumar | Tengu: may be adding something in these scenario | 11:57 |
chandankumar | might give you the results in jobout | 11:57 |
chandankumar | https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_firewall/molecule | 11:57 |
*** rlandy is now known as rlandy|mtg | 11:58 | |
chandankumar | in converage.yml file | 11:58 |
Tengu | hmmmm ok | 12:00 |
Tengu | lemme chec | 12:00 |
Tengu | k | 12:00 |
chandankumar | https://opendev.org/opendev/base-jobs/src/branch/master/playbooks/base/post.yaml | 12:01 |
chandankumar | something is failing at fetch output centos-8-stream | ok: 2 changed: 1 unreachable: 0 failed: 1 skipped: 1 rescued: 0 ignored: 0 | 12:01 |
Tengu | yep | 12:01 |
Tengu | but I don't know what. | 12:02 |
Tengu | so it MAY be due to the new way of managing the rules, but that would be specific to molecule imho | 12:02 |
Tengu | though we can see other jobs failing - I didn't get any issue locally. | 12:02 |
Tengu | so the collect-log patch will already help a bit for the other jobs. | 12:02 |
Tengu | adding anything to the molecule thing will end in https://1e1b40a5c8ddcf8cc214-d94618d55c79fbc152473d20a9c08710.ssl.cf5.rackcdn.com/839163/2/check/tripleo-ansible-centos-8-molecule-tripleo_firewall/196fedf/job-output.txt afaik | 12:03 |
*** dasm|off is now known as dasm | 12:04 | |
*** amoralej is now known as amoralej|lunch | 12:05 | |
chandankumar | Tengu: we sent a testproject in rdo side https://review.rdoproject.org/r/c/testproject/+/42218 | 12:05 |
chandankumar | you can hold the node and debug | 12:05 |
Tengu | ah, cool, thanks! for that, I have to check with fungi I think? | 12:06 |
chandankumar | Tengu: for rdo one, you can ping dpawlik, he can hold the node for you | 12:07 |
Tengu | 'k. how do I find the node ID? in the console? | 12:07 |
chandankumar | Tengu: https://review.rdoproject.org/r/c/testproject/+/42218 review link is enough and job name | 12:07 |
Tengu | dpawlik: heya! would it be possible to hold the node running https://review.rdoproject.org/r/c/testproject/+/42218 and add my ssh key on it? | 12:07 |
chandankumar | the job is still queued https://review.rdoproject.org/zuul/status#42218 | 12:08 |
dpawlik | Tengu: sure, what is your ssh key? | 12:08 |
Tengu | 2s | 12:08 |
Tengu | dpawlik: ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIHUnwjB20UKmsSed9X73eGNV5AOEFccQ3NYrRW776pEk cjeanner@tengu.lain.internux.ch | 12:09 |
fungi | yeah, if you need a held node in opendev i can help, but i don't manage the rdo gerrit/zuul | 12:09 |
Tengu | fungi: no problem :) | 12:10 |
Tengu | dpawlik: thanks :) | 12:10 |
dpawlik | Tengu: ping me when the CI job will start | 12:12 |
Tengu | dpawlik: ok - trying to keep an eye on it while doing some other tasks. can't it be automated? :) | 12:14 |
Tengu | dpawlik: apparently starting now. | 12:15 |
Tengu | no more "queued" | 12:15 |
fultonj | marios: would you please comment? https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352/44/roles/standalone/tasks/main.yml#6 | 12:16 |
fultonj | I agree about using standalone_ceph. For now I added a scenario check only because I want to start with 001. However, I'll remove the scenario check as per the TODO when 004 and 010 are ready. | 12:18 |
dpawlik | Tengu: k, waiting for some more info | 12:18 |
fultonj | Should I be limiting the branches to be tested in another way? | 12:18 |
Tengu | dpawlik: apparently running for good, the console is running :) | 12:19 |
Tengu | dpawlik: guess it's node-0002464003 ? | 12:19 |
dpawlik | Tengu: try zuul@38.102.83.65 | 12:22 |
Tengu | dpawlik: I'm in! | 12:22 |
marios | fultonj: o/ ack https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352/44#message-2836bf24943dbda2da1731343d5775f385ee765b | 12:23 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Inventory for validations https://review.opendev.org/c/openstack/python-tripleoclient/+/839200 | 12:23 |
Tengu | dpawlik: I'll let you know as soon as you can unhold that node, thanks! | 12:24 |
dpawlik | Tengu: np | 12:28 |
opendevreview | John Fulton proposed openstack/tripleo-quickstart-extras master: Deploy Ceph before the Overcloud for CI scen001 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352 | 12:29 |
fultonj | marios: https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/834352/45..46/roles/standalone/tasks/main.yml | 12:30 |
Tengu | hmmm. funky. | 12:31 |
Tengu | I can ping the host. I can access it with SSH at the beginning, but suddenly not anymore. | 12:31 |
Tengu | guess there's something in iptables. | 12:31 |
fultonj | marios: thank you. so assuming this is green do you think this patch is nearly ready? | 12:32 |
marios | fultonj: i mean if it works then yeah we probably want to sanity check scen1/4 (are they both running ther ei think only 1? or may be mistaken) | 12:33 |
marios | fultonj: i'll add to my review list and once we see it is green can add to the ci team reviews list | 12:33 |
fultonj | 001 is green so i expect it to be again. 004 is excluded for now and i'll add 004 and 010 in follow up patchs. | 12:34 |
marios | fultonj: ack sounds good | 12:35 |
fultonj | then lines 7-10 will be removed | 12:35 |
marios | fultonj: the scen12 thing merged i think already | 12:35 |
fultonj | https://review.opendev.org/c/openstack/tripleo-ci/+/838557 | 12:35 |
fultonj | yes it is | 12:35 |
marios | ack | 12:35 |
fultonj | thank you marios | 12:36 |
Tengu | dpawlik: o_O I lost the access to the node..... argh! | 12:37 |
Tengu | "connection refused" - while I changed the INPUT policy to ACCEPT o_O | 12:37 |
marios | welcome fultonj | 12:39 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Get all of the INPUT rules in a dedicated chain https://review.opendev.org/c/openstack/tripleo-ansible/+/839163 | 12:41 |
dpawlik | Tengu: the job passed, right? | 12:44 |
Tengu | dpawlik: yeah, I tweaked iptables too soon apparentl. | 12:44 |
dpawlik | Tengu: so it will not hold the node when job is OK | 12:44 |
Tengu | dpawlik: you can unhold it (if it was holded), I think I may have an idea. | 12:44 |
dpawlik | Tengu: set some task with fail and hold will work | 12:44 |
Tengu | dpawlik: so if I re-check it will run on the same node? | 12:45 |
dpawlik | Tengu: is possible, but... probably not | 12:45 |
Tengu | dpawlik: ok. well, unhold it - as said, I may have an idea as to why it failed. | 12:45 |
Tengu | it's weird though. but... | 12:45 |
dpawlik | ack, unhold the node... | 12:45 |
Tengu | I guess I'll have to play a bit more with some of the injected rules during the molecule testing. | 12:46 |
Tengu | and maybe rely more on the implicit things within iptable.s | 12:46 |
dpawlik | Tengu: it is not a problem to hold it. Just let me know or nhicher, or tristanC | 12:46 |
Tengu | dpawlik: thanks! I'll ping any of you back if needed then :) | 12:47 |
opendevreview | Roger Heslop proposed openstack/tripleo-docs master: Removed `--standalone` option from deploy command of standalone OpenStack. This option is deprecated as of Victoria. See https://bugs.launchpad.net/tripleo/+bug/1969644 https://review.opendev.org/c/openstack/tripleo-docs/+/839372 | 12:53 |
opendevreview | Roger Heslop proposed openstack/tripleo-docs stable/wallaby: Removed `--standalone` option from deploy command of standalone OpenStack in stable/Wallaby. This option is deprecated as of Victoria. See https://bugs.launchpad.net/tripleo/+bug/1969644 https://review.opendev.org/c/openstack/tripleo-docs/+/839375 | 13:07 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Revert "Setting default inventory path for VF interface with ooo client." https://review.opendev.org/c/openstack/python-tripleoclient/+/839376 | 13:10 |
*** amoralej|lunch is now known as amoralej | 13:15 | |
opendevreview | chandan kumar proposed openstack/tripleo-ci master: [DNM] multi parenting exploration https://review.opendev.org/c/openstack/tripleo-ci/+/839379 | 13:29 |
opendevreview | Jiri Podivin proposed openstack/python-tripleoclient stable/wallaby: Revert "Setting default inventory path for VF interface with ooo client." https://review.opendev.org/c/openstack/python-tripleoclient/+/839383 | 13:41 |
opendevreview | Merged openstack/python-tripleoclient master: Fix node delete for unprovision confirmation https://review.opendev.org/c/openstack/python-tripleoclient/+/838980 | 13:45 |
*** pdeore_ is now known as pdeore | 14:25 | |
*** ysandeep is now known as ysandeep|afk | 14:36 | |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Get all of the INPUT rules in a dedicated chain https://review.opendev.org/c/openstack/tripleo-ansible/+/839163 | 14:57 |
*** ysandeep|afk is now known as ysandeep | 14:59 | |
Tengu | bogdando: -^^ I learned the difference between goto and jump in iptables the hard way. | 15:00 |
Tengu | pffff. | 15:00 |
Tengu | nasty one. | 15:00 |
bogdando | :) | 15:00 |
bogdando | another turing complete language discouraging goto? | 15:01 |
Tengu | goto is basically a dead-end | 15:01 |
Tengu | if nothing matches in the chain called by goto, you're stuck. | 15:01 |
Tengu | with jump, packet will go back to the source chain, and continue navigating through the rules. | 15:01 |
*** pojadhav|pto is now known as pojadhav | 15:03 | |
*** lbragstad8 is now known as lbragstad | 15:05 | |
bogdando | bu stuck you mean the rules processing stops there? | 15:08 |
Tengu | bogdando: yep. | 15:15 |
Tengu | confirmation: molecule is now green. | 15:15 |
bogdando | cool | 15:15 |
bogdando | Tengu: just a tiiiny question about loggied packets :) | 15:21 |
bogdando | logged | 15:22 |
*** xek_ is now known as xek | 15:23 | |
Tengu | bogdando: good question, pushed the answer :) | 15:25 |
Tengu | tl;dr: nothing has changed at that point | 15:25 |
Tengu | if you want me to push that answer in the commit message, or as a release note, lemme know | 15:26 |
bogdando | no, that works | 15:27 |
Tengu | :) | 15:27 |
rcastillo | 712492 | 15:35 |
rcastillo | disregard :) | 15:35 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: wip: testing potential fix for missing tripleo_firewall_frontend_enabled https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839408 | 16:10 |
*** dviroel|rover is now known as dviroel|rover|lunch | 16:11 | |
*** marios is now known as marios|out | 16:12 | |
*** rlandy|mtg is now known as rlandy | 16:14 | |
*** ysandeep is now known as ysandeep|out | 16:17 | |
*** jpena is now known as jpena|off | 16:17 | |
*** blarnath is now known as d34dh0r53 | 16:23 | |
*** dviroel|rover|lunch is now known as dviroel|rover | 17:02 | |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Disabling container validation when running redhat-subscription https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839330 | 18:52 |
slagle | cloudnull: could you review this one and the depends-on? https://review.opendev.org/c/openstack/python-tripleoclient/+/838910 | 19:21 |
rlandy | cloudnull: slagle: can we get another vote/w+ on https://review.opendev.org/c/openstack/tripleo-ansible/+/839187 | 19:25 |
rlandy | thank you | 19:25 |
rlandy | fultonj: ^6 fyi | 19:25 |
cloudnull | slagle done | 19:26 |
cloudnull | rlandy done | 19:26 |
rlandy | cloudnull: thank you | 19:27 |
*** dkehn_ is now known as dkehn | 19:35 | |
slagle | cloudnull: thanks! | 19:38 |
opendevreview | Merged openstack/tripleo-common stable/wallaby: Remove tftp healthcheck from ironic-pxe image https://review.opendev.org/c/openstack/tripleo-common/+/832847 | 20:38 |
opendevreview | Clark Boylan proposed openstack/diskimage-builder master: Fix dhcp-all-interfaces on debuntu systems https://review.opendev.org/c/openstack/diskimage-builder/+/839080 | 20:45 |
opendevreview | Clark Boylan proposed openstack/diskimage-builder master: Revert "Fallback to persistent netifs names with systemd" https://review.opendev.org/c/openstack/diskimage-builder/+/838863 | 20:45 |
opendevreview | Merged openstack/tripleo-docs master: Add warning to ceph-ansible integration docuemtnation https://review.opendev.org/c/openstack/tripleo-docs/+/839096 | 21:09 |
*** dasm is now known as dasm|off | 21:40 | |
opendevreview | Merged openstack/tripleo-quickstart-extras master: Change DNS test to use dig instead of ping https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/839094 | 22:31 |
*** rlandy is now known as rlandy|out | 22:53 | |
*** dviroel|rover is now known as dviroel|rover|out | 22:56 | |
opendevreview | Merged openstack/tripleo-common stable/wallaby: Add TypeError exception when parsing addresses https://review.opendev.org/c/openstack/tripleo-common/+/839050 | 23:17 |
opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Update overcloud credentials to use ephemeral Heat https://review.opendev.org/c/openstack/python-tripleoclient/+/838910 | 23:22 |
opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Restoration of the 'export_passwords' exclusion https://review.opendev.org/c/openstack/python-tripleoclient/+/838911 | 23:22 |
opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: Remove upgrade converge https://review.opendev.org/c/openstack/python-tripleoclient/+/817127 | 23:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!