Wednesday, 2022-07-13

*** ysandeep is now known as ysandeep|afk00:09
*** dviroel|rover is now known as dviroel|out00:10
*** ysandeep|afk is now known as ysandeep01:16
*** ysandeep is now known as ysandeep|afk03:19
*** ysandeep|afk is now known as ysandeep04:44
*** ysandeep is now known as ysandeep|afk06:44
*** ysandeep|afk is now known as ysandeep07:40
*** ysandeep is now known as ysandeep|lunch08:25
*** anbanerj is now known as frenzy_friday09:17
opendevreviewDmitry Tantsur proposed openstack/ansible-collections-openstack master: Add a wait argument to baremetal_node  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84965309:22
opendevreviewJakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Build and install latest RPM for openstacksdk in TripleO jobs  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84965710:13
opendevreviewDmitry Tantsur proposed openstack/ansible-collections-openstack master: Add a wait argument to baremetal_node  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84965310:25
*** rlandy|out is now known as rlandy10:26
*** ysandeep|lunch is now known as ysandeep10:40
*** rlandy is now known as rlandy|rover11:15
dtantsur'Node' object has no attribute 'uuid'11:41
dtantsurhmm, baremetal_node_info broken?11:41
dtantsurah, probably new openstacksdk strikes again11:43
opendevreviewDmitry Tantsur proposed openstack/ansible-collections-openstack master: Remove deprecated stuff from baremetal_node  https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84966811:51
*** dviroel is now known as dviroel|rover12:12
*** rlandy|rover is now known as rlandy12:23
*** ysandeep is now known as ysandeep|afk12:59
*** ysandeep|afk is now known as ysandeep13:31
jm1rcastillo, sshnaidm: wip patch on how we get latest rpm of openstacksdk in our tripleo jobs in aoc. it depends on some changes to tripleo, e.g. on rafael's patch to the build-test-packages role14:08
jm1rcastillo, sshnaidm: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84965714:14
rcastillooops, I was already changing my patch to do that :p https://review.opendev.org/c/openstack/ansible-collections-openstack/+/84485914:15
rcastilloI'm fine going with yours though14:15
sshnaidmrcastillo, jm1 we can't install collection from galaxy, we must install from opendev.org14:16
sshnaidmjust set a right branch for it14:16
jm1sshnaidm: why? we are already installing other collections from galaxy too14:17
sshnaidmjm1, no, not from galaxy14:17
sshnaidmwe clone them in required projects14:17
sshnaidmgalaxy sucks and is not always accessible, it will break all tripleo jobs14:18
jm1sshnaidm: i see a lot of galaxy here: https://opendev.org/openstack/tripleo-quickstart/src/branch/master/ansible-collection-requirements.yml14:18
rcastillowe install utils, posix, from galaxy14:18
sshnaidmjust add "version: stable/1.0.0"14:18
sshnaidmno, you don't :D14:18
sshnaidmI think this file is for manual run only, and all collection are installed from cloned14:19
sshnaidmeven if you run it, it won't install anything because it's already installed by quickstart14:19
jm1sshnaidm: will update my patch, gimme a sec14:20
sshnaidmI bet we don't touch galaxy at all in CI14:20
rcastillowhy are we removing collections from required projects then?14:21
rcastilloisn't this equivalent?14:21
rcastillosince we'll be setting a static version anyway14:22
jm1rcastillo: aoc in required projects is master branch. quickstart will force install master branch of aoc https://opendev.org/openstack/tripleo-quickstart/src/commit/b48d869e14de40444d69a107a0b718b5f721e912/quickstart.sh#L12314:28
jm1rcastillo: maybe we could change aoc in required-projects to stable instead of removing it. feel free to propose patches ^^14:31
jm1rcastillo: or rather unabandon your patch from the very beginning ^^14:32
rcastilloright, this: https://review.opendev.org/c/openstack/tripleo-ci/+/84322014:33
jm1rcastillo: required-projects also affects job variant collection, so using override-checkout might cause issues for us. if we can, i would prefer getting rid of aoc in required-projects14:34
rcastillooh ok, that makes sense14:39
rcastillonot sure if we need it in collection requirements.yml since quickstart.sh does install it14:39
jm1rcastillo: we need aoc in collection requirements if we drop it from required-projects14:40
jm1rcastillo: imho your patch https://review.opendev.org/c/openstack/tripleo-ci/+/843220 is more clean than my patch which just drops aoc from required-projects. but this zuuls job variant mechanism caused me so many headaches in the past that i really try to avoid override-checkouts etc. whenever possible14:42
-opendevstatus- NOTICE: Due to an incident in our hosting provider, the tarballs.opendev.org site (and possibly other sites served from static.opendev.org) is offline while we attempt recovery14:52
*** dviroel|rover is now known as dviroel|rover|biab14:53
jm1rcastillo, sshnaidm: https://review.opendev.org/c/openstack/tripleo-quickstart/+/849620 has been updated, testproject is rerunning15:00
jm1rcastillo, sshnaidm: added all patches to "fix" tripleo to our review list15:02
jm1rcastillo, sshnaidm: will have a break now and let memtest verify my new ram. might take a while before i am back ^^15:02
*** ysandeep is now known as ysandeep|out15:04
sshnaidmjm1[m], leaving aoc with override-checkout should be fine, isn't it?15:12
*** dviroel|rover|biab is now known as dviroel|rover16:11
*** dviroel__ is now known as dviroel|rover|biab16:42
*** dviroel|rover|biab is now known as dviroel|rover17:21
*** dviroel|rover is now known as dviroel|rover|Afk22:58
*** dviroel|rover|Afk is now known as dviroel|rover23:57

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!