| *** openstack has joined #openstack-sahara | 01:19 | |
| *** ChanServ sets mode: +o openstack | 01:19 | |
| openstackgerrit | Merged openstack/puppet-sahara master: add release note link in readme https://review.openstack.org/577201 | 02:22 |
|---|---|---|
| openstackgerrit | Vu Cong Tuan proposed openstack/sahara-dashboard master: Add py36 testenv https://review.openstack.org/577687 | 02:26 |
| *** Bhujay has joined #openstack-sahara | 05:37 | |
| *** Bhujay has quit IRC | 05:44 | |
| *** Bhujay has joined #openstack-sahara | 05:44 | |
| *** Bhujay has quit IRC | 05:53 | |
| *** hoonetorg has quit IRC | 06:20 | |
| *** hoonetorg has joined #openstack-sahara | 06:32 | |
| *** Bhujay has joined #openstack-sahara | 06:41 | |
| *** pcaruana has joined #openstack-sahara | 07:07 | |
| *** tesseract has joined #openstack-sahara | 07:20 | |
| *** rogeryu_ has joined #openstack-sahara | 07:23 | |
| rogeryu_ | https://review.openstack.org/#/c/577610/ | 07:24 |
| rogeryu_ | Has anyone helped me to review this commit? thank you very much! :) | 07:24 |
| ltosky[m] | rogeryu_: it will be probably a -2 | 07:37 |
| ltosky[m] | Check https://docs.openstack.org/sahara/latest/contributor/development-guidelines.html#modification-of-upstream-files | 07:41 |
| ltosky[m] | and the comments to similar commits abandoned, like https://review.openstack.org/#/c/576325/ | 07:41 |
| *** pgadiya has joined #openstack-sahara | 07:59 | |
| *** pgadiya has quit IRC | 07:59 | |
| rogeryu_ | @ltosky Thanks for your reply, the demo clearly explains my doubts : ) | 08:01 |
| *** brad[] has joined #openstack-sahara | 09:15 | |
| *** pgadiya has joined #openstack-sahara | 11:14 | |
| *** pgadiya has quit IRC | 11:14 | |
| *** tosky has joined #openstack-sahara | 11:28 | |
| *** rogeryu_ has quit IRC | 11:41 | |
| Bhujay | tosky, if ranger , oozie or hive is included in cluster template , it fails during mysql-server installation first and then for root access next , here is the log http://paste.openstack.org/show/724198/ | 12:26 |
| Bhujay | http://paste.openstack.org/show/724217/ | 12:26 |
| tosky | which plugin? | 12:26 |
| Bhujay | ambari2.4 | 12:26 |
| tosky | created with sahara-image-pack on centos? | 12:27 |
| Bhujay | yes | 12:28 |
| tosky | uhm, I'm totally sure that I tested it | 12:28 |
| tosky | is it still the case of no internet access? | 12:28 |
| tosky | that's the case that I did not test | 12:28 |
| Bhujay | here is few more details , i am back in few minutes ...http://paste.openstack.org/show/724223/ | 12:30 |
| tosky | no need to have an additional repository; mysql-server should expand to mariadb-server | 12:31 |
| tosky | check if that's installed | 12:31 |
| tosky | zigo: hi! Small reminder about that sahara patch (wsgi), if you can apply that change... | 12:36 |
| tosky | last week the gates were blocked, but now we can merge things again | 12:36 |
| zigo | tosky: Didn't I do it already? | 12:36 |
| zigo | tosky: https://review.openstack.org/#/c/573179/ | 12:37 |
| zigo | Is it what you were talking about? | 12:37 |
| tosky | zigo: we rechecked and python has short circuit evaluation, so the two conditions can be put together | 12:38 |
| zigo | tosky: Oh, thanks for the reminder, doing it. | 12:38 |
| tosky | perfect :) | 12:39 |
| openstackgerrit | Thomas Goirand proposed openstack/sahara master: Fix flask.request.content_length is None https://review.openstack.org/573179 | 12:44 |
| tellesnobrega | zigo, it will fail pep8 | 12:53 |
| tellesnobrega | you need to break the line in 2 | 12:53 |
| tellesnobrega | tosky, also, I think I've hit a new issue with python 3 | 12:54 |
| openstackgerrit | Thomas Goirand proposed openstack/sahara master: Fix flask.request.content_length is None https://review.openstack.org/573179 | 12:54 |
| tellesnobrega | thanks zigo | 12:54 |
| zigo | :P | 12:57 |
| tosky | tellesnobrega: I tried to start the scenario tests but they fails in the tooz/memcached codepath | 12:58 |
| *** prasen has joined #openstack-sahara | 12:58 | |
| tosky | run them with python3 | 12:58 |
| tellesnobrega | hmm | 12:59 |
| tellesnobrega | I was trying to start a storm cluster with new image | 12:59 |
| tellesnobrega | it is failing with a list has no .lower() function | 13:00 |
| tellesnobrega | python 2 works | 13:00 |
| *** prasen has quit IRC | 13:08 | |
| Bhujay | tosky , u r right , it was the absense of mariadb creatin the problem .... it is quiet wired java-1.8 , mariadb etc are getting uninstalled after the os hardening ... i will check and report back | 13:09 |
| tosky | oh | 13:10 |
| tosky | not something I would expect O.o | 13:10 |
| openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack/sahara master: Adding Storm 1.2.0 and 1.2.1 https://review.openstack.org/577815 | 13:25 |
| tosky | tellesnobrega: I'm testing the extjs fix; so far I can run the images that were failing (sie), I'm going to test with pack images, both generated with the current code and without | 13:27 |
| tellesnobrega | awesome | 13:27 |
| tellesnobrega | thanks | 13:27 |
| tosky | I think that, while we don't need a release note for the fix as the issue didn't go in a released version (ok, m2, but that's not even a beta), | 13:28 |
| tosky | the backport to queens should have a release notes, because the buggy commit ended up in a point release | 13:28 |
| tosky | but let's see after I finish the testing | 13:28 |
| openstackgerrit | Merged openstack/sahara-dashboard master: Add py36 testenv https://review.openstack.org/577687 | 13:29 |
| tellesnobrega | tosky, sure, once we merge this, you can write the backport (so I don't merge my own) and release note makes sense | 13:30 |
| openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack/sahara-image-elements master: Updating Storm version list https://review.openstack.org/577826 | 13:59 |
| *** Bhujay has quit IRC | 15:12 | |
| openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack/sahara master: Adding Storm 1.2.0 and 1.2.1 https://review.openstack.org/577815 | 15:12 |
| *** pcaruana has quit IRC | 15:13 | |
| *** tesseract has quit IRC | 16:18 | |
| *** Bhujay has joined #openstack-sahara | 16:21 | |
| tosky | and a small reminder about the two "migrate to stestr" reviews (sahara and sahara-tests) | 16:40 |
| *** jeremyfreudberg has joined #openstack-sahara | 17:32 | |
| openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack/sahara master: Adding Storm 1.2.0 and 1.2.1 https://review.openstack.org/577815 | 17:32 |
| jeremyfreudberg | tosky, do you expect me to merge this: https://review.openstack.org/#/c/567430/ ? (uwsgi in gate thing) | 17:33 |
| tosky | jeremyfreudberg: if you don't see any relevant issue, sure :) | 17:58 |
| jeremyfreudberg | hmm, maybe sahara-image-pack should default to debug as true... i've been sitting here 20 minutes with no output | 18:22 |
| jeremyfreudberg | kind of off-putting as a first time user of that | 18:23 |
| tosky | uhm | 18:23 |
| jeremyfreudberg | hmm, or maybe nothing is actually happening | 18:25 |
| jeremyfreudberg | looking at the gate INFO still tends to be somewhat useful | 18:25 |
| jeremyfreudberg | so i wonder what i'm stuck on, then | 18:25 |
| jeremyfreudberg | yeah, i was stuck on https://github.com/openstack/sahara/blob/master/sahara/cli/image_pack/api.py#L71 the whole time | 18:27 |
| tosky | oh, initial start of libguestfs I guess | 18:27 |
| jeremyfreudberg | precisely | 18:27 |
| openstackgerrit | Merged openstack/sahara master: Updating plugins status for Rocky https://review.openstack.org/566130 | 18:45 |
| *** jeremyfreudberg has quit IRC | 18:47 | |
| openstackgerrit | Merged openstack/sahara master: Deploy using wsgi by default https://review.openstack.org/567430 | 19:08 |
| *** jeremyfreudberg has joined #openstack-sahara | 19:09 | |
| *** pcaruana has joined #openstack-sahara | 19:27 | |
| jeremyfreudberg | tellesnobrega, do you think it is premature to deprecate vanilla 2.7.1? | 19:34 |
| openstackgerrit | Jeremy Freudberg proposed openstack/sahara-extra master: Host some patched jars in common artifacts https://review.openstack.org/577449 | 19:36 |
| tellesnobrega | jeremyfreudberg, hmm, how long have we had 2.8.2 on? | 20:22 |
| tellesnobrega | it was queens | 20:23 |
| tellesnobrega | we could keep 2.7.1 now, and if 3 comes up we can deprecate it, if not we wait another cycle | 20:23 |
| *** whooligan_md has quit IRC | 20:24 | |
| *** whooligan_md has joined #openstack-sahara | 20:24 | |
| jeremyfreudberg | tellesnobrega: makes perfect sense | 20:34 |
| jeremyfreudberg | thanks! | 20:34 |
| tellesnobrega | yeah | 20:35 |
| tellesnobrega | :) | 20:35 |
| tellesnobrega | I'm testing spark 2.3 now | 20:35 |
| *** pcaruana has quit IRC | 20:43 | |
| openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack/sahara master: Fix flask.request.content_length is None https://review.openstack.org/573179 | 20:45 |
| openstackgerrit | Jeremy Freudberg proposed openstack/sahara stable/queens: Better default value for domain in swift config https://review.openstack.org/577927 | 20:47 |
| * tosky whispers "stestr" | 20:50 | |
| jeremyfreudberg | tosky, just these two? https://review.openstack.org/#/c/571468/ https://review.openstack.org/#/c/575489/ ? | 20:51 |
| tellesnobrega | approved the sahara one | 20:52 |
| tosky | yes, those two are the only ones right now; we are almost done | 20:53 |
| jeremyfreudberg | tellesnobrega, you'll have to approve sahara-tests patch too, as i've already +2ed | 20:53 |
| tellesnobrega | yes | 20:53 |
| tellesnobrega | I'm on it | 20:53 |
| tosky | I have only a pending not-yet-submitted for sahara which migrates cover.sh in sahara.git, and then we are testrepository-free | 20:53 |
| tellesnobrega | I have 2 comments on https://review.openstack.org/#/c/575489/3 | 20:57 |
| tellesnobrega | can you clarify those so I can +A? | 20:57 |
| jeremyfreudberg | tellesnobrega: the regex is fine (it could alternatively be no caret and no trailing period) | 20:59 |
| jeremyfreudberg | select/run should be one or the other, but not both :p | 20:59 |
| openstackgerrit | Merged openstack/sahara master: Remove any reference to pre-built images https://review.openstack.org/575466 | 21:02 |
| tellesnobrega | tosky, can you fix the doc so we merge it now? | 21:03 |
| tosky | right | 21:03 |
| jeremyfreudberg | till tomorrow, guys | 21:06 |
| *** jeremyfreudberg has quit IRC | 21:06 | |
| openstackgerrit | Luigi Toscano proposed openstack/sahara-tests master: Port away from ostestr/testr and use stestr https://review.openstack.org/575489 | 21:07 |
| tosky | argh, a new minimum requirement | 21:10 |
| tosky | umpf | 21:10 |
| tosky | Flask removed its upper constraint | 21:11 |
| tosky | I suspect that, upon approving the flask patch, it will fail and we will need to amend it to change the minimum requirement | 21:14 |
| tosky | uh, not that patch; we already merged another flask patch | 21:16 |
| openstackgerrit | Luigi Toscano proposed openstack/sahara master: Bump Flask version according requirements https://review.openstack.org/577932 | 21:23 |
| tosky | thanks to the Ubuntu maintainers we should be compatible with that version | 21:23 |
| *** rcernin has joined #openstack-sahara | 21:45 | |
| openstackgerrit | Luigi Toscano proposed openstack/sahara master: Bump Flask version according requirements https://review.openstack.org/577932 | 21:59 |
| tosky | and I thought I understood the new way of handling requirements... | 22:03 |
| openstackgerrit | Luigi Toscano proposed openstack/sahara master: Switch ostestr to stestr https://review.openstack.org/571468 | 22:27 |
| tosky | (rebased on top of the flask bump) | 22:28 |
| *** tosky has quit IRC | 23:34 | |
| openstackgerrit | Merged openstack/sahara-tests master: Port away from ostestr/testr and use stestr https://review.openstack.org/575489 | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!