| openstackgerrit | Aakarsh proposed openstack/browbeat master: Adding stockpile to collect data https://review.openstack.org/604456 | 00:50 |
|---|---|---|
| *** agopi is now known as agopi|brb | 00:54 | |
| *** agopi|brb has quit IRC | 00:59 | |
| *** agopi|brb has joined #openstack-browbeat | 01:52 | |
| *** noama has joined #openstack-browbeat | 08:38 | |
| *** ssbarnea has joined #openstack-browbeat | 09:50 | |
| *** ssbarnea has quit IRC | 14:02 | |
| *** rook has quit IRC | 14:02 | |
| *** ssbarnea has joined #openstack-browbeat | 14:07 | |
| *** agopi|brb has quit IRC | 14:21 | |
| *** agopi|brb has joined #openstack-browbeat | 14:46 | |
| *** agopi|brb is now known as agopi | 14:46 | |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Adding extra-vars needed for deployment of queens/rocky https://review.openstack.org/616281 | 15:38 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Temporarily disabling BML https://review.openstack.org/616600 | 15:58 |
| *** noama has quit IRC | 16:06 | |
| *** agopi is now known as agopi|food | 16:41 | |
| *** agopi|food is now known as agopi | 17:10 | |
| agopi | ssbarnea, not sure what's wrong with https://review.openstack.org/#/c/615974/3 , I tested the package locally with twine check dist/* and even published it to https://pypi.org/project/browbeat/#history | 17:36 |
| ssbarnea | agopi: let me have a look, i was planning to check it anyway. | 17:37 |
| agopi | oh thanks ssbarnea cool :) lmk what you need. i added you to https://test.pypi.org/project/browbeat | 17:39 |
| openstackgerrit | Joe Talerico proposed openstack/browbeat master: Pbench FIO integration https://review.openstack.org/493910 | 17:39 |
| ssbarnea | agopi: i think you are referring to this error http://logs.openstack.org/74/615974/3/check/test-release-openstack/5682347/job-output.txt.gz#_2018-11-08_17_22_19_298514 right? | 17:48 |
| agopi | yes ssbarnea | 17:49 |
| ssbarnea | to me it looks like a bug unrelated to browbeat but I am planning to dig more to identify exactly what causes it | 17:49 |
| ssbarnea | as you can see the wheel is passing the test | 17:49 |
| ssbarnea | it happens only with the source dist | 17:49 |
| agopi | ack ssbarnea, because I built the tar ball locally through setup.py bdist | 17:49 |
| agopi | and then ran twine check | 17:50 |
| agopi | it ran fine | 17:50 |
| agopi | and then also i uploaded it here https://pypi.org/project/browbeat/ | 17:50 |
| agopi | thanks ssbarnea :) | 17:50 |
| ssbarnea | probably one package version is different on that job than on our local machines, it works for me too locally. | 17:50 |
| agopi | ubuntu xenial machine being used i guess maybe that's why? | 17:51 |
| openstackgerrit | Merged openstack/browbeat master: Temporarily disabling BML https://review.openstack.org/616600 | 18:00 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Setting ES requests timeout to a higher value https://review.openstack.org/611966 | 18:07 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Setting ES requests timeout to a higher value https://review.openstack.org/611966 | 18:08 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Moving browbeat-config file to /etc/browbeat https://review.openstack.org/605865 | 18:09 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: fix tox python3 overrides https://review.openstack.org/615517 | 18:17 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Add pypi publishing testing https://review.openstack.org/613582 | 18:18 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Adding stockpile to collect data https://review.openstack.org/604456 | 18:19 |
| openstackgerrit | Merged openstack/browbeat master: Added more testing to compare soft metadata in CLI https://review.openstack.org/614347 | 18:20 |
| openstackgerrit | Sorin Sbarnea proposed openstack/browbeat master: Fix package building failure due to empty description https://review.openstack.org/615974 | 18:42 |
| agopi | thanks ssbarnea | 18:43 |
| ssbarnea | lets hope it will work | 18:44 |
| ssbarnea | it did work, don't forget to merge it. | 19:09 |
| agopi | yep ssbarnea just waiting on getting more clarity of the behavior, I posted a comment. ideally we want to simulate the test-release job in which case using same env for running twine check will not work | 19:11 |
| agopi | ssbarnea, this https://review.openstack.org/#/c/616676/1 helps | 19:14 |
| agopi | i'll go ahead and merge it | 19:14 |
| ssbarnea | ... well is just avoiding the error without fixing the missing description or enabling the sdist build testing locally. | 19:16 |
| ssbarnea | shortly they can both go in | 19:16 |
| openstackgerrit | Merged openstack/browbeat master: Fix package building failure due to empty description https://review.openstack.org/615974 | 19:26 |
| agopi | merged ssbarnea ^ | 19:26 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Add numpy to extras https://review.openstack.org/616677 | 19:31 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Adding extra-vars needed for deployment of queens/rocky https://review.openstack.org/616281 | 19:59 |
| openstackgerrit | Merged openstack/browbeat master: Adding extra-vars needed for deployment of queens/rocky https://review.openstack.org/616281 | 20:23 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Add numpy to extras https://review.openstack.org/616677 | 21:00 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Add numpy to extras https://review.openstack.org/616677 | 21:00 |
| agopi | https://review.openstack.org/#/c/616677/ this good ssbarnea ? | 21:00 |
| ssbarnea | agopi: yes but you can improve it with one more line. i added comment | 21:03 |
| ssbarnea | just run pip check before and after | 21:03 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Add numpy to extras https://review.openstack.org/616677 | 21:07 |
| agopi | added thanks ssbarnea :) | 21:07 |
| openstackgerrit | Aakarsh proposed openstack/browbeat master: Add numpy to extras https://review.openstack.org/616677 | 22:02 |
| *** agopi is now known as agopi|brb | 22:04 | |
| *** agopi|brb has quit IRC | 22:09 | |
| *** agopi|brb has joined #openstack-browbeat | 23:14 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!