*** ysandeep is now known as ysandeep|afk | 00:09 | |
*** dviroel|rover is now known as dviroel|out | 00:10 | |
*** ysandeep|afk is now known as ysandeep | 01:16 | |
*** ysandeep is now known as ysandeep|afk | 03:19 | |
*** ysandeep|afk is now known as ysandeep | 04:44 | |
*** ysandeep is now known as ysandeep|afk | 06:44 | |
*** ysandeep|afk is now known as ysandeep | 07:40 | |
*** ysandeep is now known as ysandeep|lunch | 08:25 | |
*** anbanerj is now known as frenzy_friday | 09:17 | |
opendevreview | Dmitry Tantsur proposed openstack/ansible-collections-openstack master: Add a wait argument to baremetal_node https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849653 | 09:22 |
---|---|---|
opendevreview | Jakob 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/+/849657 | 10:13 |
opendevreview | Dmitry Tantsur proposed openstack/ansible-collections-openstack master: Add a wait argument to baremetal_node https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849653 | 10:25 |
*** rlandy|out is now known as rlandy | 10:26 | |
*** ysandeep|lunch is now known as ysandeep | 10:40 | |
*** rlandy is now known as rlandy|rover | 11:15 | |
dtantsur | 'Node' object has no attribute 'uuid' | 11:41 |
dtantsur | hmm, baremetal_node_info broken? | 11:41 |
dtantsur | ah, probably new openstacksdk strikes again | 11:43 |
opendevreview | Dmitry Tantsur proposed openstack/ansible-collections-openstack master: Remove deprecated stuff from baremetal_node https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849668 | 11:51 |
*** dviroel is now known as dviroel|rover | 12:12 | |
*** rlandy|rover is now known as rlandy | 12:23 | |
*** ysandeep is now known as ysandeep|afk | 12:59 | |
*** ysandeep|afk is now known as ysandeep | 13:31 | |
jm1 | rcastillo, 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 role | 14:08 |
jm1 | rcastillo, sshnaidm: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849657 | 14:14 |
rcastillo | oops, I was already changing my patch to do that :p https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844859 | 14:15 |
rcastillo | I'm fine going with yours though | 14:15 |
sshnaidm | rcastillo, jm1 we can't install collection from galaxy, we must install from opendev.org | 14:16 |
sshnaidm | just set a right branch for it | 14:16 |
jm1 | sshnaidm: why? we are already installing other collections from galaxy too | 14:17 |
sshnaidm | jm1, no, not from galaxy | 14:17 |
sshnaidm | we clone them in required projects | 14:17 |
sshnaidm | galaxy sucks and is not always accessible, it will break all tripleo jobs | 14:18 |
jm1 | sshnaidm: i see a lot of galaxy here: https://opendev.org/openstack/tripleo-quickstart/src/branch/master/ansible-collection-requirements.yml | 14:18 |
rcastillo | we install utils, posix, from galaxy | 14:18 |
sshnaidm | just add "version: stable/1.0.0" | 14:18 |
sshnaidm | no, you don't :D | 14:18 |
sshnaidm | I think this file is for manual run only, and all collection are installed from cloned | 14:19 |
sshnaidm | even if you run it, it won't install anything because it's already installed by quickstart | 14:19 |
jm1 | sshnaidm: will update my patch, gimme a sec | 14:20 |
sshnaidm | I bet we don't touch galaxy at all in CI | 14:20 |
rcastillo | why are we removing collections from required projects then? | 14:21 |
rcastillo | isn't this equivalent? | 14:21 |
rcastillo | since we'll be setting a static version anyway | 14:22 |
jm1 | rcastillo: 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#L123 | 14:28 |
jm1 | rcastillo: maybe we could change aoc in required-projects to stable instead of removing it. feel free to propose patches ^^ | 14:31 |
jm1 | rcastillo: or rather unabandon your patch from the very beginning ^^ | 14:32 |
rcastillo | right, this: https://review.opendev.org/c/openstack/tripleo-ci/+/843220 | 14:33 |
jm1 | rcastillo: 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-projects | 14:34 |
rcastillo | oh ok, that makes sense | 14:39 |
rcastillo | not sure if we need it in collection requirements.yml since quickstart.sh does install it | 14:39 |
jm1 | rcastillo: we need aoc in collection requirements if we drop it from required-projects | 14:40 |
jm1 | rcastillo: 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 possible | 14: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 recovery | 14:52 | |
*** dviroel|rover is now known as dviroel|rover|biab | 14:53 | |
jm1 | rcastillo, sshnaidm: https://review.opendev.org/c/openstack/tripleo-quickstart/+/849620 has been updated, testproject is rerunning | 15:00 |
jm1 | rcastillo, sshnaidm: added all patches to "fix" tripleo to our review list | 15:02 |
jm1 | rcastillo, 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|out | 15:04 | |
sshnaidm | jm1[m], leaving aoc with override-checkout should be fine, isn't it? | 15:12 |
*** dviroel|rover|biab is now known as dviroel|rover | 16:11 | |
*** dviroel__ is now known as dviroel|rover|biab | 16:42 | |
*** dviroel|rover|biab is now known as dviroel|rover | 17:21 | |
*** dviroel|rover is now known as dviroel|rover|Afk | 22:58 | |
*** dviroel|rover|Afk is now known as dviroel|rover | 23:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!