*** Sravan has quit IRC | 00:07 | |
*** Sravan has joined #openstack-kolla | 00:10 | |
*** Sravan has quit IRC | 00:15 | |
*** gfidente|afk has quit IRC | 00:22 | |
*** dave-mccowan has joined #openstack-kolla | 00:54 | |
*** wolverineav has quit IRC | 01:18 | |
*** wolverineav has joined #openstack-kolla | 01:18 | |
*** wolverineav has quit IRC | 01:23 | |
*** itlinux has joined #openstack-kolla | 01:24 | |
*** wolverineav has joined #openstack-kolla | 01:33 | |
*** Sravan has joined #openstack-kolla | 01:33 | |
*** wolverineav has quit IRC | 01:33 | |
*** Sravan has quit IRC | 01:38 | |
openstackgerrit | Taeha Kim proposed openstack/kolla master: Add RHEL subscription registration https://review.openstack.org/495148 | 01:46 |
---|---|---|
*** Sravan has joined #openstack-kolla | 01:47 | |
*** itlinux has quit IRC | 01:51 | |
*** hongbin has joined #openstack-kolla | 01:53 | |
*** Sravan has quit IRC | 02:19 | |
*** Sravan has joined #openstack-kolla | 02:19 | |
*** openstackstatus has quit IRC | 02:22 | |
*** openstack has joined #openstack-kolla | 02:25 | |
*** ChanServ sets mode: +o openstack | 02:25 | |
*** k_mouza has joined #openstack-kolla | 02:50 | |
*** k_mouza has quit IRC | 02:54 | |
*** Sravan has quit IRC | 02:57 | |
*** wolverineav has joined #openstack-kolla | 02:59 | |
*** Sravan has joined #openstack-kolla | 03:32 | |
*** hongbin has quit IRC | 03:33 | |
*** dave-mccowan has quit IRC | 03:40 | |
*** skramaja has joined #openstack-kolla | 03:52 | |
*** skramaja has quit IRC | 03:56 | |
*** skramaja has joined #openstack-kolla | 03:57 | |
*** mchlumsky has quit IRC | 03:58 | |
*** janki has joined #openstack-kolla | 04:38 | |
*** Sravan has quit IRC | 04:50 | |
*** Sravan has joined #openstack-kolla | 04:54 | |
spsurya | egonzalez: mgoddard hi... | 04:56 |
spsurya | you might have seen this http://lists.openstack.org/pipermail/openstack-discuss/2018-December/000837.html | 04:56 |
spsurya | can please review https://review.openstack.org/#/c/620567/ and let me know if it is required or not | 04:57 |
spsurya | thanks | 04:57 |
openstackgerrit | Dmitry Teselkin proposed openstack/kolla master: Fix image squashing https://review.openstack.org/640099 | 04:59 |
*** wolverineav has quit IRC | 05:00 | |
*** wolverineav has joined #openstack-kolla | 05:01 | |
*** Sravan has quit IRC | 05:07 | |
*** Sravan has joined #openstack-kolla | 05:10 | |
*** Sravan has joined #openstack-kolla | 05:11 | |
*** Sravan has quit IRC | 05:16 | |
*** wolverineav has quit IRC | 05:17 | |
*** wolverineav has joined #openstack-kolla | 05:18 | |
*** wolverineav has quit IRC | 05:22 | |
*** wolverineav has joined #openstack-kolla | 05:25 | |
*** wolverineav has quit IRC | 05:30 | |
*** wolverineav has joined #openstack-kolla | 05:32 | |
*** wolverineav has quit IRC | 05:40 | |
*** wolverineav has joined #openstack-kolla | 06:04 | |
*** wolverineav has quit IRC | 06:04 | |
*** jbadiapa has quit IRC | 06:05 | |
*** yankcrime has quit IRC | 06:07 | |
*** mgoddard has quit IRC | 06:07 | |
*** pomac has quit IRC | 06:07 | |
*** gary_perkins has quit IRC | 06:08 | |
*** gary_perkins has joined #openstack-kolla | 06:10 | |
*** mgoddard has joined #openstack-kolla | 06:10 | |
*** wolverineav has joined #openstack-kolla | 06:14 | |
*** wolverineav has quit IRC | 06:32 | |
*** k_mouza has joined #openstack-kolla | 06:50 | |
*** Luzi has joined #openstack-kolla | 06:53 | |
*** k_mouza has quit IRC | 06:55 | |
*** rgogunskiy has joined #openstack-kolla | 07:09 | |
mnasiadka | spsurya: I think on queens we were still on xenial, question if this will not break the gates - and we would need separate nodesets for xenial and bionic | 07:21 |
*** ivve has joined #openstack-kolla | 07:23 | |
mnasiadka | spsurya: sorry, waking up - we can leave queens on xenial - I need to remember to start replying only after first coffee ;) | 07:25 |
openstackgerrit | Michal Nasiadka proposed openstack/kolla-ansible master: [WIP] Enable tempest test in gate https://review.openstack.org/402122 | 07:37 |
*** hamzaachi has joined #openstack-kolla | 07:38 | |
*** pcaruana has joined #openstack-kolla | 07:39 | |
*** owalsh has quit IRC | 07:39 | |
*** pcaruana has quit IRC | 07:43 | |
*** pcaruana has joined #openstack-kolla | 07:43 | |
hrw | mnasiadka: can you +w https://review.openstack.org/#/c/641983/ one? | 07:50 |
mnasiadka | hrw: nope, this is kolla - I have rights only in k-a | 07:51 |
hrw | ok | 07:51 |
*** rgogunskiy_ has joined #openstack-kolla | 07:51 | |
hrw | morning | 07:53 |
*** rgogunskiy has quit IRC | 07:54 | |
*** jbadiapa has joined #openstack-kolla | 07:54 | |
*** wolverineav has joined #openstack-kolla | 07:55 | |
*** wolverineav has quit IRC | 07:59 | |
spsurya | mnasiadka: ok, but let's see what other think of | 08:02 |
hrw | spsurya: hi, can you +w https://review.openstack.org/#/c/641983/ one? | 08:03 |
spsurya | hrw: i am checking the same | 08:03 |
hrw | 3> | 08:03 |
hrw | https://review.openstack.org/#/q/owner:marcin.juszkiewicz%2540linaro.org+status:open+openstack/kolla for brave cores ;D | 08:12 |
hrw | I am now waiting for get_pip to pass zuul and then will push set with fixed interdeps | 08:14 |
openstackgerrit | Michal Nasiadka proposed openstack/kolla-ansible master: [WIP] Enable tempest test in gate https://review.openstack.org/402122 | 08:15 |
*** zhubingbing has joined #openstack-kolla | 08:18 | |
*** luksky has joined #openstack-kolla | 08:20 | |
*** dougsz has joined #openstack-kolla | 08:31 | |
*** jschlueter has quit IRC | 08:39 | |
*** jschlueter has joined #openstack-kolla | 08:40 | |
openstackgerrit | Merged openstack/kolla master: Introduce 'get_pip' macro https://review.openstack.org/641983 | 08:46 |
hrw | yay! | 08:47 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: base: drop mangling of Python directories on Ubuntu https://review.openstack.org/642398 | 08:47 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: replace 'distro_python3' variable with 'distro_python_version' https://review.openstack.org/642014 | 08:47 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: Debian/Ubuntu: move to Python 3 for binary/source images https://review.openstack.org/642375 | 08:47 |
mgoddard | Morning kolla | 08:51 |
*** luksky has quit IRC | 08:52 | |
hrw | mgoddard: ehlo | 08:54 |
mgoddard | spsurya: just looking at your bionic change. Seems like a good idea. Do you know what the problem with queens & zuul config is? | 08:54 |
hrw | I will wait for zuul to go though those patches or maybe give a break today from py3. | 08:55 |
hrw | few more patch reading and I will be afraid to open a fridge | 08:55 |
*** yankcrime has joined #openstack-kolla | 08:55 | |
*** luksky has joined #openstack-kolla | 08:57 | |
hrw | spsurya: https://review.openstack.org/#/c/642460/ https://review.openstack.org/#/c/642398/ https://review.openstack.org/#/c/612681/ are independent from any other. can you take a look? | 08:58 |
spsurya | mgoddard: no clue yet for problem with queens and zuul config | 08:58 |
mgoddard | spsurya: I think the problem is that zuul config can look at all branches. Maybe we need to give the nodeset a different name to avoid a collision | 08:59 |
mgoddard | spsurya: e.g. kolla-ubuntu-bionic | 08:59 |
mgoddard | spsurya: could always check in the zuul channel | 09:00 |
spsurya | mgoddard: so in that case we have to keep both xenial and bionic i guess | 09:00 |
spsurya | as for queens xenial in use | 09:01 |
mgoddard | spsurya: I think the definition in the old branches will be enough to keep those branches working | 09:01 |
spsurya | ok lemme test with new changes | 09:02 |
*** priteau has joined #openstack-kolla | 09:02 | |
spsurya | hrw: sure i will check | 09:06 |
hrw | spsurya: thanks | 09:06 |
dteselkin | mgoddard: hi, could you pls take a look at https://review.openstack.org/#/c/640099/ | 09:07 |
mgoddard | dteselkin: will do | 09:08 |
dteselkin | thanks! | 09:08 |
*** hamzaachi has quit IRC | 09:09 | |
hrw | dteselkin: how squashing works at all? sorry but never looked at it | 09:10 |
hrw | when I build base then it has (for example) 14 sublayers used during build. is squash at the end to make base one single layer? and similar for rest of images? | 09:11 |
hrw | dteselkin: so each image is 100% separate from others | 09:11 |
*** k_mouza has joined #openstack-kolla | 09:12 | |
dteselkin | hrw: Yes, docker build will add a tag to the image, then docker-squash called. It will compact several layers (starting from previous tagged layer to the latest one) in one layer and create a new image | 09:12 |
dteselkin | Old layers then removed, so this should be done before building child images | 09:13 |
hrw | dteselkin: so we get base:tag as layer, then openstack-base:tag will be base:tag + layer, nova-base will base:tag + openstack-base:tag layer + nova-base:tag layer etc? | 09:14 |
hrw | instead of 14 layers of base:tag + nth layers of openstack-base + yth layers of nova-base | 09:15 |
dteselkin | Yes | 09:15 |
*** k_mouza_ has joined #openstack-kolla | 09:15 | |
*** k_mouza_ has quit IRC | 09:15 | |
*** k_mouza has quit IRC | 09:15 | |
hrw | ok. let me do a buitl | 09:16 |
dteselkin | http://paste.openstack.org/show/747592/ | 09:16 |
*** k_mouza has joined #openstack-kolla | 09:16 | |
mgoddard | hrw: this is a change from the old behaviour, which squashed all layers | 09:16 |
dteselkin | The only issue with squashing is that images will be rebuilt every time from the beginning - all layers are removed so docker can't use cache | 09:17 |
mgoddard | my main concern is whether there is still a use case for squashing all layers, for those doing CI/CD | 09:17 |
hrw | (venv3) 09:17 (1s) hrw@debian:kolla$ kolla-build-images.sh debian source nova-compute squash | 09:17 |
hrw | dteselkin: I do build of all images at one run | 09:17 |
dteselkin | Well, I can add an option to select squashing depth | 09:17 |
hrw | ERROR:kolla.common.utils:"docker-squash" command is not found. try to install it by "pip install docker-squash" | 09:18 |
hrw | Traceback (most recent call last): | 09:18 |
hrw | dteselkin: please catch that exception | 09:19 |
dteselkin | you need 'pip install docker-squash' :) | 09:19 |
dteselkin | For some reason this component isn't in requirements.txt | 09:19 |
dteselkin | I don't know why | 09:19 |
hrw | dteselkin: I know that I need. kolla said it to me | 09:19 |
*** k_mouza has quit IRC | 09:19 | |
hrw | dteselkin: and then CRASHED with traceback | 09:19 |
hrw | dteselkin: send patch to add it to requirements? | 09:20 |
dteselkin | Ok, so the question is - should I add this to requirements.txt or just catch that exception ? | 09:20 |
hrw | dteselkin: both | 09:21 |
dteselkin | Ok | 09:21 |
dteselkin | Separate CR ? | 09:21 |
hrw | dteselkin: catch exception in squash patch (or seprate), requiremtsn in separate | 09:21 |
dteselkin | OK | 09:21 |
hrw | INFO:kolla.common.utils.base:Successfully tagged squash/debian-source-base:8.0.0 | 09:21 |
hrw | ERROR:kolla.common.utils.base:Unknown error when building | 09:21 |
dteselkin | s/OK/Ok/ | 09:21 |
hrw | Traceback (most recent call last): | 09:21 |
hrw | File "/home/hrw/devel/kolla/kolla/kolla/image/build.py", line 598, in builder | 09:21 |
hrw | self.squash() | 09:21 |
hrw | File "/home/hrw/devel/kolla/kolla/kolla/image/build.py", line 620, in squash | 09:22 |
hrw | squash_from = tagged[1] | 09:22 |
hrw | TypeError: 'filter' object is not subscriptable | 09:22 |
dteselkin | Hm, that's smth new | 09:22 |
*** k_mouza has joined #openstack-kolla | 09:22 | |
dteselkin | Ok, will add list(filter(...)) | 09:23 |
openstackgerrit | Surya Prakash (spsurya) proposed openstack/kolla master: DNM: Test jobs running on bionic instead of xenial https://review.openstack.org/620567 | 09:23 |
hrw | dteselkin: you use py2 or py3 to run kolla? | 09:23 |
dteselkin | py2 | 09:23 |
hrw | dteselkin: then remember to check under py3 too | 09:23 |
dteselkin | Ok | 09:23 |
hrw | dteselkin: I more or less forgot that there is python < 3.5 | 09:23 |
mgoddard | dteselkin: this suggests we need a unit test to catch this case | 09:27 |
openstackgerrit | Surya Prakash (spsurya) proposed openstack/kolla master: DNM: Test jobs running on bionic instead of xenial https://review.openstack.org/620567 | 09:33 |
mgoddard | hrw: wdyt about my comments on the distro_python_version patch about keeping debian/ubuntu on py2 until the following patch? | 09:35 |
hrw | mgoddard: makes sense | 09:36 |
hrw | mgoddard: let zuul first go through whole series and then will look at changing | 09:36 |
mgoddard | hrw: sure | 09:36 |
openstackgerrit | Bernard Cafarelli proposed openstack/kolla master: Drop tcp_wrappers package requirement https://review.openstack.org/642706 | 09:43 |
*** gfidente has joined #openstack-kolla | 09:45 | |
openstackgerrit | Dmitry Teselkin proposed openstack/kolla master: Fix image squashing https://review.openstack.org/640099 | 09:46 |
*** andrein has joined #openstack-kolla | 09:47 | |
hrw | dteselkin: let's retry | 09:47 |
*** hamzaachi has joined #openstack-kolla | 09:47 | |
hrw | INFO:kolla.common.utils:Image squash is enabled and "docker-squash" version is b'1.0.7' | 09:47 |
hrw | dteselkin: b'? | 09:47 |
hrw | INFO:kolla.common.utils.base:Successfully tagged squash2/debian-source-base:8.0.0 | 09:48 |
hrw | INFO:kolla.common.utils.base:Squashing from sha256:70bdb2644a97de24c26117a341657e92906c154c425036c28e57c4515beb35b2 (tagged as: debian:stretch-backports) | 09:48 |
hrw | INFO:kolla.common.utils.base:Image squashed successfully | 09:48 |
hrw | progress ;d | 09:48 |
dteselkin | :) | 09:48 |
hrw | dteselkin: get rid of b' and I may +2 | 09:49 |
dteselkin | Doing :) | 09:50 |
hrw | ah. and exception on missing docker-squash | 09:50 |
dteselkin | There is no exception, I'll disable squashing if command not found | 09:50 |
dteselkin | There is a check for docker-squash version already | 09:50 |
hrw | dteselkin: w8 w8... | 09:50 |
dteselkin | kolla/common/utils.py | 09:51 |
hrw | dteselkin: usecase: user John has working Kolla instalation. He did 'git up' to update to latest Kolla source and decided to squash images (as he saw in git log your change). John runs 'kolla --squash' and BAM! traceback right into face! | 09:52 |
hrw | or simple 'sorry, you lack docker-squash. install this way: pip....' and exit. | 09:52 |
dteselkin | No, now there won't be traceback - kolla-build checks version of docker-squash, I've updated it to disable squashing if it's not installeed | 09:53 |
*** hamzaachi has quit IRC | 09:53 | |
hrw | dteselkin: writing 'sorry, no docker-squash, will follow with no squashing' == pissed off John who just wasted 3h of build time as he did not noticed first line | 09:53 |
dteselkin | kolla/image/build.py L1316 | 09:53 |
*** hamzaachi has joined #openstack-kolla | 09:54 | |
hrw | dteselkin: when user gives --squash and docker-squash is missing kolla (imho) should exit with info | 09:55 |
dteselkin | Ah, ok then | 09:55 |
hrw | dteselkin: when you run kolla on machine without running docker it says "sorry, no docker daemon" and exits instead of trying to do anything (not that it can anything) | 09:55 |
dteselkin | Sounds reasonable ;) | 09:56 |
hrw | dteselkin: also it is handy to do 'git up' before 'git review' (where git up == git pull --rebase --autostash) so you do not get suprise from zuul on final merge | 09:57 |
dteselkin | ERROR:kolla.common.utils:"docker-squash" command is not found. try to install it by "pip install docker-squash" | 09:59 |
dteselkin | WARNING:kolla.common.utils:Unable to detect version of "docker-squash", is it installed? | 09:59 |
dteselkin | Then stop. Correct ? | 09:59 |
dteselkin | Second message looks redundant :) | 10:00 |
hrw | drop warning one | 10:00 |
hrw | INFO:kolla.common.utils.openstack-base:Successfully tagged squash2/debian-source-openstack-base:8.0.0 | 10:04 |
hrw | INFO:kolla.common.utils.openstack-base:Squashing from sha256:b2aec2baa38ffd75b2e9866863222636df293f7352cbc05fd8ad04a2237f5249 (tagged as: squash2/debian-source-base:8.0.0) | 10:04 |
hrw | nice, nice | 10:04 |
hrw | two more images and will do image inspect | 10:05 |
openstackgerrit | Dmitry Teselkin proposed openstack/kolla master: Fix image squashing https://review.openstack.org/640099 | 10:10 |
mgoddard | spsurya: could you reapply your +2 to https://review.openstack.org/#/c/639949? | 10:11 |
spsurya | mgoddard: yes it was applied on PS1 somehow | 10:13 |
spsurya | though done | 10:13 |
hrw | mgoddard: +2+w | 10:13 |
mgoddard | thanks both | 10:13 |
openstackgerrit | Dmitry Teselkin proposed openstack/kolla master: Fix image squashing https://review.openstack.org/640099 | 10:13 |
*** owalsh has joined #openstack-kolla | 10:14 | |
hrw | dteselkin: nova-compute from 42 layers down to 6. | 10:15 |
*** andrein has quit IRC | 10:15 | |
*** andrein has joined #openstack-kolla | 10:16 | |
hrw | 2 from debian:stretch-backports + debian-source-base + openstack-base + nova-base + nova-compute. nice work | 10:16 |
dteselkin | thanks ) debian seems to use less layers than ubuntu, which uses 4 | 10:17 |
hrw | ERROR:kolla.common.utils:"docker-squash" command is not found. try to install it by "pip install docker-squash" | 10:17 |
hrw | (venv3) 10:17 (2s) hrw@debian:kolla$ | 10:17 |
hrw | nice! | 10:17 |
hrw | +2 from me | 10:18 |
dteselkin | Thanks:) | 10:19 |
openstackgerrit | Mark Goddard proposed openstack/kolla master: DNM: Use Jinja variables for version-dependent packages https://review.openstack.org/642713 | 10:21 |
openstackgerrit | Pedro Alvarez Piedehierro proposed openstack/kolla master: Install fluentd gem on debian/ubuntu for non x86_64 https://review.openstack.org/638692 | 10:24 |
*** skramaja has quit IRC | 10:25 | |
mgoddard | hrw: bit of a PoC for setting Jinja variables in an include file: horizon_pyver | 10:26 |
mgoddard | hrw: oops, https://review.openstack.org/642713 | 10:26 |
mgoddard | hrw: could make the RHEL8 patch a bit cleaner, might help some of your work too? | 10:26 |
hrw | mgoddard: feel free to edit/rebase any of my patch. | 10:27 |
hrw | mgoddard: now and forever | 10:27 |
mgoddard | hrw: are you looking at them today? | 10:28 |
hrw | mgoddard: want to make a break for few hours at least | 10:28 |
mgoddard | hrw: I don't really want to take over, but tempted to try a couple things | 10:28 |
hrw | sure | 10:29 |
hrw | mgoddard: 642713 is imho wrong move. complicates stuff | 10:29 |
mgoddard | hrw: it makes the rhel8 patches much simpler | 10:30 |
hrw | mgoddard: nope. does not handle python-ldappool <> python3-ldappool etc | 10:30 |
mgoddard | hrw: I'd probably only use it for common packages - python, mod_wsgi | 10:30 |
hrw | mgoddard: there was one similar already | 10:30 |
mgoddard | hrw: the problem with the previous one is that it was implicit. This is explicit | 10:31 |
hrw | mgoddard: https://review.openstack.org/#/c/636457/ one iirc | 10:31 |
mgoddard | hrw: yeah I saw it. The 'Diskimage-builder model' | 10:31 |
hrw | mgoddard: please, do not remind me DIB | 10:32 |
hrw | mgoddard: https://marcin.juszkiewicz.com.pl/2018/01/12/yadibp/ | 10:32 |
mgoddard | hrw: it does solve python-ldappool / python3-ldappool - create a python_ldappool_rpm variable | 10:32 |
hrw | mgoddard: personally I prefer how we have it now - pile of if/elif/else | 10:33 |
hrw | when we start split then next step will be unifing centos/debu package lists | 10:33 |
mgoddard | hrw: I agree that would be the wrong approach. In this case we normally have 1-1 mapping though | 10:35 |
hrw | mgoddard: nearly. | 10:35 |
mgoddard | hrw: I don't feel that strongly about it, hopefully it won't be around forever | 10:35 |
hrw | mgoddard: there are some rhel7<>8 differences here and there | 10:35 |
hrw | mgoddard: sysvinit-utils for py2 but not py3 etc | 10:35 |
hrw | I hope that RDO will abandon py2 in/after train so we just drop py2 stuff | 10:36 |
mgoddard | hrw: more generally, I think adding something like variables.j2 would be useful to avoid duplication | 10:37 |
hrw | mgoddard: ok | 10:37 |
mgoddard | e.g. from distro_python_interpreter we could infer distro_python3, distro_python_interpreter, distro_python_site_packages, etc. | 10:38 |
hrw | mgoddard: /usr/libexec/platform-python - which python version it is? | 10:50 |
mgoddard | hrw: not present on my laptop | 10:51 |
hrw | mgoddard: present in rhel8 base | 10:51 |
hrw | ;D | 10:52 |
mgoddard | also on centos | 10:52 |
hrw | anyway if user want to have python in system python3.6 will be installed | 10:53 |
openstackgerrit | Martin Mágr proposed openstack/kolla master: Allow mitral-executor to perform undercloud backups https://review.openstack.org/640008 | 11:00 |
openstackgerrit | Merged openstack/kolla master: Move project templates from ubuntu.yaml to project.yaml https://review.openstack.org/639949 | 11:05 |
*** zhubingbing has quit IRC | 11:09 | |
openstackgerrit | Mark Goddard proposed openstack/kolla master: replace 'distro_python3' variable with 'distro_python_version' https://review.openstack.org/642014 | 11:22 |
*** dave-mccowan has joined #openstack-kolla | 11:23 | |
*** owalsh has quit IRC | 11:50 | |
*** owalsh has joined #openstack-kolla | 11:52 | |
*** owalsh has quit IRC | 11:56 | |
*** owalsh has joined #openstack-kolla | 11:57 | |
*** owalsh has quit IRC | 12:06 | |
*** owalsh has joined #openstack-kolla | 12:07 | |
*** owalsh has quit IRC | 12:18 | |
spsurya | mgoddard: egonzalez should this go to master now https://review.openstack.org/#/c/620567/ ? | 12:19 |
*** luksky has quit IRC | 12:19 | |
*** k_mouza_ has joined #openstack-kolla | 12:20 | |
*** owalsh has joined #openstack-kolla | 12:21 | |
*** k_mouza_ has quit IRC | 12:21 | |
*** k_mouza_ has joined #openstack-kolla | 12:22 | |
*** k_mouza has quit IRC | 12:22 | |
mgoddard | spsurya: looks good to me | 12:34 |
mgoddard | spsurya: can you remove the DNM from the commit message? | 12:35 |
mgoddard | spsurya: also update the commit message | 12:35 |
spsurya | mgoddard: sure | 12:35 |
*** mjturek has joined #openstack-kolla | 12:40 | |
hrw | spsurya: better is to W-1 so when you decide to merge you do not have to edit patch just mark as ready for review | 12:40 |
spsurya | hrw: yeah, but initial aim of this patch was to test only, please have a look in PS ;) | 12:42 |
hrw | sure | 12:42 |
hrw | spsurya: I was surprised to see that we have xenial somewhere still | 12:43 |
openstackgerrit | Surya Prakash (spsurya) proposed openstack/kolla master: Run jobs on bionic instead of xenial https://review.openstack.org/620567 | 12:44 |
spsurya | hrw: ok | 12:45 |
*** mjturek has quit IRC | 12:45 | |
*** janki has quit IRC | 12:59 | |
*** janki has joined #openstack-kolla | 12:59 | |
*** owalsh has quit IRC | 13:01 | |
*** owalsh has joined #openstack-kolla | 13:02 | |
*** FlorianFa has quit IRC | 13:03 | |
*** luksky has joined #openstack-kolla | 13:08 | |
*** Sravan has joined #openstack-kolla | 13:09 | |
*** owalsh has quit IRC | 13:10 | |
*** gfidente has quit IRC | 13:13 | |
*** yan0s has joined #openstack-kolla | 13:13 | |
*** Sravan has quit IRC | 13:14 | |
*** owalsh has joined #openstack-kolla | 13:15 | |
*** owalsh has quit IRC | 13:19 | |
*** owalsh has joined #openstack-kolla | 13:20 | |
*** dave-mccowan has quit IRC | 13:22 | |
*** irclogbot_1 has quit IRC | 13:26 | |
*** irclogbot_1 has joined #openstack-kolla | 13:29 | |
openstackgerrit | Erol Guzoglu proposed openstack/kolla-ansible master: Add Prometheus to Grafana https://review.openstack.org/642749 | 13:38 |
*** owalsh has quit IRC | 13:50 | |
*** lennyb has quit IRC | 13:51 | |
*** jbadiapa has quit IRC | 13:55 | |
*** owalsh has joined #openstack-kolla | 13:56 | |
*** openstack has joined #openstack-kolla | 15:40 | |
*** ChanServ sets mode: +o openstack | 15:40 | |
hrw | bash: python: command not found | 15:40 |
hrw | now 1+h to check 'debian/ubuntu source' builds | 15:41 |
*** jbadiapa has quit IRC | 15:41 | |
*** jchhatbar has quit IRC | 15:47 | |
*** hamzaachi_ has joined #openstack-kolla | 15:47 | |
*** hamzaachi has quit IRC | 15:50 | |
*** andrein has quit IRC | 16:05 | |
*** ivve has quit IRC | 16:07 | |
mgoddard | hrw: all binary images built except helm-repository: http://logs.openstack.org/75/642375/4/check/kolla-build-ubuntu-binary/54c386d/logs/build/000_FAILED_helm-repository.txt.gz | 16:24 |
mgoddard | couldn't get that one to build, didn't investigate much | 16:24 |
mgoddard | 6 failures for source images | 16:24 |
*** ktibi has quit IRC | 16:26 | |
*** hamzaachi_ has quit IRC | 16:34 | |
*** hamzaachi_ has joined #openstack-kolla | 16:34 | |
hrw | ok | 16:36 |
*** ktibi has joined #openstack-kolla | 16:36 | |
hrw | mgoddard: should be python not python3 with pip there | 16:38 |
mgoddard | hrw: ? | 16:38 |
hrw | mgoddard: or rather image use python3 not python | 16:38 |
mgoddard | hrw: you mean that's the check for python2 vs 3 | 16:39 |
mgoddard | ? | 16:39 |
hrw | mgoddard: image installs python. get-pip wants python3 | 16:39 |
hrw | so py3 setup lacks distutils etc - looks like very minimal install | 16:40 |
mgoddard | do we need to explcitly install distutils? | 16:40 |
hrw | no | 16:41 |
hrw | mgoddard: switch python to python3 and then check? | 16:41 |
*** yan0s has quit IRC | 16:41 | |
openstackgerrit | Alex Schultz proposed openstack/kolla master: Add python3 packages for RHEL systems https://review.openstack.org/632156 | 16:41 |
openstackgerrit | Alex Schultz proposed openstack/kolla master: Add distro_package_manager configuration https://review.openstack.org/639219 | 16:44 |
*** Sravan has joined #openstack-kolla | 16:44 | |
*** Sravan has quit IRC | 16:45 | |
*** Sravan has joined #openstack-kolla | 16:45 | |
*** wolverineav has joined #openstack-kolla | 16:45 | |
*** wolverineav has quit IRC | 16:45 | |
*** wolverineav has joined #openstack-kolla | 16:45 | |
*** Sravan has quit IRC | 16:46 | |
*** Sravan has joined #openstack-kolla | 16:46 | |
openstackgerrit | Alex Schultz proposed openstack/kolla master: WIP: Fedora support for python3 package testing https://review.openstack.org/624838 | 16:47 |
*** wolverineav has quit IRC | 16:47 | |
*** wolverineav has joined #openstack-kolla | 16:48 | |
openstackgerrit | Alex Schultz proposed openstack/kolla master: Add distro_package_manager configuration https://review.openstack.org/639219 | 16:49 |
openstackgerrit | Alex Schultz proposed openstack/kolla master: WIP: Fedora support for python3 package testing https://review.openstack.org/624838 | 16:49 |
*** Sravan has quit IRC | 16:49 | |
mwhahaha | hrw, mgoddard -^ i rebased my rhel python3 stuff on the distro_python_version | 16:49 |
hrw | mwhahaha: a moment after you got +w+2+2 for existing version ;D | 16:50 |
mgoddard | mwhahaha: great, thanks. | 16:50 |
* mwhahaha flips tables | 16:50 | |
mwhahaha | :D | 16:50 |
mgoddard | someone had to do it | 16:50 |
mwhahaha | well lets try and get it in tomorrow then | 16:50 |
mwhahaha | i'll make sure ci is green | 16:50 |
hrw | mwhahaha: I always check status before pushing new ver :D | 16:50 |
hrw | ok. looks like my patch to openstack-base works | 16:51 |
mwhahaha | meh this was going to cause a whole bunch of changes anyway | 16:51 |
mwhahaha | better to get it now | 16:51 |
hrw | sure | 16:51 |
*** Sravan has joined #openstack-kolla | 16:51 | |
hrw | now I have to recreate 'buster-py3-clean' branch with recent gerrits, apply my openstack-base change on top, do test build and send for review | 16:51 |
hrw | good to see that python=python3=3.7.2+ in all images (where python exists at all) | 16:52 |
hrw | bpc/debian-source-kube-controller-manager has 2.7 | 16:52 |
hrw | and have to dig out monasca fix as well | 16:53 |
*** Sravan has quit IRC | 16:54 | |
mgoddard | anyone around to review https://review.openstack.org/#/c/642014? | 16:54 |
* mwhahaha pokes mandre | 16:55 | |
mandre | mgoddard, mwhahaha: will review right away | 16:56 |
mgoddard | thanks mandre! | 16:56 |
*** hamzaachi__ has joined #openstack-kolla | 16:57 | |
*** hamzaachi_ has quit IRC | 16:58 | |
openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: monasca, tempest: install librdkafka-dev to get it built for non-x86/source https://review.openstack.org/642826 | 16:58 |
hrw | mgoddard: 'anyone around to review https://review.openstack.org/#/c/642014 one?' gives clickable working links ;D | 16:59 |
*** hamzaachi_ has joined #openstack-kolla | 16:59 | |
mgoddard | hrw: works for me :) | 17:00 |
hrw | mgoddard: | 17:00 |
hrw | mgoddard: '?' gets added here as part of link | 17:00 |
hrw | mgoddard: == 404 on gerrit | 17:01 |
mgoddard | hrw: I see | 17:01 |
*** hamzaachi__ has quit IRC | 17:02 | |
hrw | ok. kube-controler and kubetoolbox will have py2 due to ceph | 17:03 |
hrw | ubuntu builds fail... FWEGF@I#%@#%!@(%#!@#%@!#($! | 17:04 |
hrw | ok. now 7 patches to replace. | 17:07 |
*** priteau has quit IRC | 17:08 | |
openstackgerrit | Merged openstack/kolla-ansible master: Fix failure to mount cephfs https://review.openstack.org/642535 | 17:11 |
mgoddard | hrw: what does kafka have to do with tempest? | 17:11 |
hrw | mgoddard: fails same way as monasca | 17:11 |
mgoddard | hrw: :/ | 17:11 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: replace 'distro_python3' variable with 'distro_python_version' https://review.openstack.org/642014 | 17:18 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: Debian/Ubuntu: move to Python 3 for binary/source images https://review.openstack.org/642375 | 17:18 |
hrw | mgoddard: 642014 should be same, 642375 got openstack-base | 17:18 |
hrw | s.should.is | 17:19 |
mgoddard | hrw: would have been better to avoid a rebase - now we have to rebase the rhel8 patch again | 17:19 |
hrw | mgoddard: https://review.openstack.org/642014 is not changed. | 17:20 |
hrw | and rhel8 depend only on it | 17:20 |
mgoddard | hrw: it was rebased | 17:20 |
mgoddard | hrw: new commit ID, so dependent patches also need to rebase | 17:20 |
hrw | ah. sorry. right | 17:20 |
hrw | time for Depends-On: field. it depends on ChangeId not commitId | 17:21 |
hrw | from what I know | 17:21 |
mgoddard | hrw: it might work in this case, I don't think there was a conflict between them | 17:22 |
hrw | that was the only change from me today. have to rest a bit | 17:24 |
*** ktibi has quit IRC | 17:25 | |
mgoddard | hrw: should all/most python3 packages in debian/ubuntu have a python3 prefix? | 17:25 |
hrw | mgoddard: should | 17:25 |
hrw | mgoddard: but then we also instal packages which do not have python in their names... | 17:26 |
*** Sravan has joined #openstack-kolla | 17:26 | |
hrw | ceph-common fetches python 2.7 for us | 17:26 |
hrw | and ceph is maintained by one person so it will stay that way probably for a while | 17:27 |
mgoddard | hrw: just tried using --template-only and grep python-, get quite a few hits | 17:27 |
hrw | mgoddard: good idea ;D | 17:28 |
mgoddard | hrw: would be good if we could automate checking the python version in the gate | 17:29 |
mgoddard | maybe something like goss with dgoss - https://github.com/aelsabbahy/goss | https://github.com/aelsabbahy/goss/tree/master/extras/dgoss | 17:31 |
mgoddard | could validate lots of stuff with that | 17:31 |
*** gfidente is now known as gfidente|afk | 17:31 | |
hrw | 140GB of docker images... | 17:35 |
hrw | let add another set ;D | 17:35 |
hrw | (venv3) 17:37 (79s) hrw@debian:kolla$ kolla-build-images.sh 'debian ubuntu' 'binary source' '' bpc1808 | 17:37 |
hrw | kolla-build-images.sh is from my https://github.com/hrw/docker-utils repo | 17:38 |
*** k_mouza has joined #openstack-kolla | 17:41 | |
hrw | have a nice day/evening. | 17:42 |
* hrw off | 17:42 | |
mgoddard | night hrw | 17:42 |
*** k_mouza_ has quit IRC | 17:44 | |
*** k_mouza has quit IRC | 17:45 | |
openstackgerrit | Mark Goddard proposed openstack/kolla master: Remove distro_python3 release note https://review.openstack.org/642838 | 17:47 |
openstackgerrit | Mark Goddard proposed openstack/kolla master: Add python3 packages for RHEL systems https://review.openstack.org/632156 | 17:48 |
*** Sravan has quit IRC | 17:48 | |
*** Sravan has joined #openstack-kolla | 17:49 | |
*** luksky has quit IRC | 17:50 | |
*** Sravan has quit IRC | 17:50 | |
*** Sravan has joined #openstack-kolla | 17:51 | |
*** ivve has joined #openstack-kolla | 17:52 | |
*** hamzaachi_ has quit IRC | 17:55 | |
*** Sravan has quit IRC | 17:56 | |
*** henriqueof has joined #openstack-kolla | 17:57 | |
*** Sravan has joined #openstack-kolla | 17:57 | |
*** jbadiapa has joined #openstack-kolla | 17:58 | |
*** Sravan has quit IRC | 18:01 | |
*** Sravan has joined #openstack-kolla | 18:03 | |
*** goldyfruit has joined #openstack-kolla | 18:04 | |
*** goldyfruit has quit IRC | 18:07 | |
*** goldyfruit has joined #openstack-kolla | 18:07 | |
*** shardy has quit IRC | 18:09 | |
*** Sravan has quit IRC | 18:09 | |
*** faridda has joined #openstack-kolla | 18:10 | |
*** sshnaidm is now known as sshnaidm|afk | 18:11 | |
*** Sravan has joined #openstack-kolla | 18:12 | |
*** Sravan has quit IRC | 18:23 | |
openstackgerrit | Alex Schultz proposed openstack/kolla master: Add distro_package_manager configuration https://review.openstack.org/639219 | 18:24 |
openstackgerrit | Alex Schultz proposed openstack/kolla master: WIP: Fedora support for python3 package testing https://review.openstack.org/624838 | 18:24 |
*** gmann is now known as gmann_afk | 18:26 | |
*** pcaruana has quit IRC | 18:39 | |
*** dougsz has quit IRC | 18:41 | |
*** hamzaachi_ has joined #openstack-kolla | 19:03 | |
*** wolverineav has quit IRC | 19:09 | |
*** Sravan has joined #openstack-kolla | 19:10 | |
*** wolverineav has joined #openstack-kolla | 19:29 | |
*** wolverineav has quit IRC | 19:34 | |
*** luksky has joined #openstack-kolla | 19:48 | |
*** Sravan has quit IRC | 19:50 | |
*** andrein has joined #openstack-kolla | 19:51 | |
openstackgerrit | Alex Schultz proposed openstack/kolla master: Use virtual package for ansible https://review.openstack.org/642874 | 19:53 |
*** wolverineav has joined #openstack-kolla | 20:02 | |
mloza | Hello, I have glance-image-import.conf in custom config folder of glance but it doesn't copy it to the node's folder after doing reconfigure | 20:03 |
mloza | I'm using stable/rock branch | 20:03 |
*** wolverineav has quit IRC | 20:06 | |
*** wolverineav has joined #openstack-kolla | 20:06 | |
*** gfidente|afk has quit IRC | 20:09 | |
openstackgerrit | Merged openstack/kolla master: replace 'distro_python3' variable with 'distro_python_version' https://review.openstack.org/642014 | 20:12 |
*** Sravan has joined #openstack-kolla | 20:20 | |
*** Sravan has quit IRC | 20:24 | |
*** gmann_afk is now known as gmann | 20:54 | |
*** hamzaachi_ has quit IRC | 20:56 | |
*** mjturek has quit IRC | 20:57 | |
hrw | mgoddard: found some places where py2 was used. build test in progress | 21:16 |
hrw | http://paste.debian.net/1072903/ | 21:17 |
*** wolverineav has quit IRC | 21:20 | |
*** mloza has quit IRC | 21:26 | |
*** wolverineav has joined #openstack-kolla | 21:26 | |
*** wolverineav has quit IRC | 21:31 | |
openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: Debian/Ubuntu: move to Python 3 for binary/source images https://review.openstack.org/642375 | 21:39 |
openstackgerrit | Marcin Juszkiewicz proposed openstack/kolla master: Another set of Debian/Ubuntu py3 fixes https://review.openstack.org/642898 | 21:39 |
hrw | 642375 no changes, 642898 convert all leftovers to py3 | 21:40 |
*** wolverineav has joined #openstack-kolla | 21:45 | |
*** mloza has joined #openstack-kolla | 21:51 | |
*** Sravan has joined #openstack-kolla | 21:59 | |
openstackgerrit | Scott Solkhon proposed openstack/kolla-ansible master: Add mising handlers for external Ceph. https://review.openstack.org/642297 | 22:08 |
*** faridda has quit IRC | 22:26 | |
*** ivve has quit IRC | 22:44 | |
*** Sravan has quit IRC | 22:55 | |
*** Sravan has joined #openstack-kolla | 22:56 | |
openstackgerrit | Merged openstack/kolla master: Add python3 packages for RHEL systems https://review.openstack.org/632156 | 23:03 |
openstackgerrit | Merged openstack/kolla master: Add distro_package_manager configuration https://review.openstack.org/639219 | 23:04 |
openstackgerrit | Scott Solkhon proposed openstack/kolla-ansible master: Add mising handlers for external Ceph. https://review.openstack.org/642297 | 23:09 |
*** goldyfruit has quit IRC | 23:21 | |
*** goldyfruit has joined #openstack-kolla | 23:36 | |
*** itlinux has joined #openstack-kolla | 23:37 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!