Thursday, 2015-06-25

*** salv-orlando has joined #openstack-oslo00:05
*** david-ly_ has joined #openstack-oslo00:06
*** david-lyle has quit IRC00:08
*** salv-orlando has quit IRC00:12
*** dims_ has joined #openstack-oslo00:14
openstackgerritJoshua Harlow proposed openstack/futurist: Show a prettier callback name  https://review.openstack.org/19536200:14
*** dims has quit IRC00:16
*** salv-orlando has joined #openstack-oslo00:21
*** salv-orlando has quit IRC00:27
openstackgerritJoshua Harlow proposed openstack/futurist: Show a prettier callback name  https://review.openstack.org/19536200:27
*** rushiagr_away is now known as rushiagr00:32
openstackgerritJoshua Harlow proposed openstack/futurist: Show a prettier callback name  https://review.openstack.org/19536200:34
harlowjadims_ oops, sorry, didn't see +2, +1, not changing again i promise :-P00:36
harlowjawhoopsies, lol00:36
harlowjathose debug statements might be to much for people (but they match similar ones in oslo.service)00:38
harlowja*to much log... blah blah00:39
harlowjahttp://paste.openstack.org/show/320583/ (example if people want to use it)00:40
harlowjashould make output like in http://paste.openstack.org/show/320584/00:41
*** boris-42 has quit IRC00:42
dims_harlowja: ack00:42
harlowja:)00:44
*** mriedem has quit IRC00:45
*** rushiagr is now known as rushiagr_away00:49
openstackgerritRoxana Gherle proposed openstack/oslo.config: Add enforce_type option when setting an override  https://review.openstack.org/19535500:54
*** tobe has joined #openstack-oslo00:56
tchaypoharlowja: and now that I’m back at my desk00:58
tchaypoyou say ‘pkg_resources'00:58
*** sigmavirus24 is now known as sigmavirus24_awa00:58
harlowjadamn that was the longest trip to the airport ever :)00:58
harlowjaha00:58
tchaypodo you mean the copy in setuptools, or the copy vendored into pip? it’s likely they’re at different versions with different understandings of the world00:58
harlowjai think in setuptools from what i can see00:58
tchaypohence nakato’s patch to update the version in pip to match setuptools00:59
harlowjapossibly, ya00:59
harlowjahttp://paste.openstack.org/show/320604/ (from http://logs.openstack.org/82/193982/5/check/gate-futurist-requirements/ee09ff4/console.html )00:59
harlowjabut i think lifeless  may be ontop of this fix?00:59
harlowja* https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/project-requirements-change.py (this file)01:00
*** amotoki has joined #openstack-oslo01:01
NakatoYea, pip 7.1.0 hasn't been tagged yet.  But dstufft pulled in the updated pkg_resources from setuptools 18.0.01:01
harlowjak01:01
Nakatoharlowja: Is pastie 320604 from Py32+?01:03
harlowjalooks like 2.701:03
NakatoOh derp01:03
harlowja:)01:04
*** Rockyg has joined #openstack-oslo01:05
harlowjaeezhova if u get around https://review.openstack.org/#/c/195369/ (pretty sure thats borking cinder at the current time)01:05
harlowja*since they just switched to oslo_service it seems01:05
*** amotoki has quit IRC01:05
harlowjahttps://review.openstack.org/#/c/195358/ triggered that01:06
harlowja*somehow*01:06
Nakato320604 looks like something else though01:10
harlowjapossibly, not sure01:12
harlowjahttps://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/project-requirements-change.py#L20 does import the system pkg_resources, which is probaby out of date right?01:12
harlowjaand/or is broken when trying to parse those lines01:13
openstackgerritMerged openstack/futurist: Allow providing customized scheduling strategies  https://review.openstack.org/19116301:14
*** sputnik13 has quit IRC01:24
*** sdake has joined #openstack-oslo01:28
*** abitha has quit IRC01:29
*** sdake_ has joined #openstack-oslo01:29
*** sdake has quit IRC01:33
*** david-ly_ is now known as david-lyle01:36
dims_harlowja: https://review.openstack.org/#/c/191998/ needs a rebase01:38
harlowjayuppers01:38
harlowjaontop of that01:38
*** dims_ has quit IRC01:45
*** dims has joined #openstack-oslo01:50
openstackgerritJoshua Harlow proposed openstack/futurist: Allow adding periodic callables at runtime  https://review.openstack.org/19199801:53
harlowjaok dokie, bb01:53
*** dontalton has quit IRC01:56
*** dims has quit IRC02:04
*** dims has joined #openstack-oslo02:10
*** sdake has joined #openstack-oslo02:13
*** sdake_ has quit IRC02:17
*** Rockyg has quit IRC02:22
*** yamahata has quit IRC02:28
*** salv-orl_ has joined #openstack-oslo02:29
*** salv-orl_ has quit IRC02:34
*** manishg has quit IRC02:54
*** dims has quit IRC02:58
mtaninoHi, can I get a review for the bugfix to stable/kilo branch? https://review.openstack.org/#/c/194357/03:00
*** harlowja_at_home has joined #openstack-oslo03:01
*** amotoki has joined #openstack-oslo03:02
*** stevemar has joined #openstack-oslo03:03
*** amotoki has quit IRC03:06
openstackgerritMerged openstack/futurist: Show a prettier callback name  https://review.openstack.org/19536203:09
*** stevemar has quit IRC03:15
lifelessharlowja_at_home: so the infra script has to be rewritten03:20
harlowja_at_homelifeless,  ya, that was my guess03:20
lifelessharlowja_at_home: I sent mail about this last night03:20
lifelessNakato: tchaypo: ^03:20
harlowja_at_homeright, that converge stuff email03:21
*** tobe has quit IRC03:26
*** mtanino has quit IRC03:31
harlowja_at_homelifeless,  do u want to rewrite my anvil ones also, lol03:32
harlowja_at_homeor someone (not it!)03:33
harlowja_at_homelol03:33
lifelessharlowja_at_home: let me think about that.03:33
lifelessno.03:33
harlowja_at_home:)03:33
harlowja_at_homeu didn't think for very long03:33
harlowja_at_homeu might want to think longer03:33
harlowja_at_home*let it sit* and such03:33
*** tobe has joined #openstack-oslo03:39
*** rjaiswal has quit IRC03:44
lifelessharlowja_at_home: no03:46
harlowja_at_homelol03:46
openstackgerritJoshua Harlow proposed openstack/taskflow: Expose strategies so doc generation can easily pick them up  https://review.openstack.org/19540103:49
openstackgerritJoshua Harlow proposed openstack/taskflow: Expose strategies so doc generation can easily pick them up  https://review.openstack.org/19540103:52
*** arnaud____ has joined #openstack-oslo03:57
*** abitha has joined #openstack-oslo04:03
*** amotoki has joined #openstack-oslo04:10
*** vilobhmm has joined #openstack-oslo04:14
*** achanda has joined #openstack-oslo04:23
openstackgerritJoshua Harlow proposed openstack/taskflow: Remove usage of weakref in action_engine scheduler  https://review.openstack.org/19540804:24
*** stevemar2 has joined #openstack-oslo04:37
*** salv-orlando has joined #openstack-oslo04:41
*** salv-orlando has quit IRC04:47
*** rpodolyaka1 has joined #openstack-oslo04:59
*** arnaud____ has quit IRC05:03
*** achanda has quit IRC05:06
*** vilobhmm has quit IRC05:26
tchaypolifeless: sorry, what exactly needs to be rewritten?05:28
openstackgerritMerged openstack/taskflow: Handy access to INFO level  https://review.openstack.org/19113205:29
openstackgerritMerged openstack/taskflow: Use sphinx deprecated docstring markup  https://review.openstack.org/19046705:29
*** arnaud____ has joined #openstack-oslo05:36
*** nkrinner has joined #openstack-oslo05:39
openstackgerritMerged openstack/taskflow: Executors come in via options config, not keyword arguments  https://review.openstack.org/18215905:40
openstackgerritMerged openstack/taskflow: A few jobboard documentation tweaks  https://review.openstack.org/19048005:40
*** rpodolyaka1 has quit IRC05:42
*** browne has quit IRC06:01
*** tobe has quit IRC06:14
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Imported Translations from Transifex  https://review.openstack.org/19450706:19
lifelesstchaypo: see my mail to -dev yesterday night06:24
tchaypoI’ve read it twice06:24
tchaypowait06:25
tchaypoI was reading [openstack-dev] [all][requirements] Whats with the "python_version=='2.7'" stuff thats showing up from today?06:25
tchaypobut I just found the other thread06:25
tchaypooh. that’s much clearer06:27
*** abitha has quit IRC06:30
*** tobe has joined #openstack-oslo06:33
*** katyafervent is now known as katyafervent_awa06:37
*** mkoderer_cloud has joined #openstack-oslo06:39
*** jaosorior has joined #openstack-oslo06:41
lifelesstchaypo: :)06:41
*** katyafervent_awa is now known as katyafervent06:44
*** achanda has joined #openstack-oslo06:44
*** rpodolyaka1 has joined #openstack-oslo06:46
*** exploreshaifali has joined #openstack-oslo06:53
*** salv-orlando has joined #openstack-oslo06:54
*** salv-orlando has quit IRC06:56
*** salv-orlando has joined #openstack-oslo06:56
*** arnaud____ has quit IRC06:57
*** achanda has quit IRC06:58
*** ihrachyshka has joined #openstack-oslo07:00
*** achanda has joined #openstack-oslo07:08
*** stevemar2 has quit IRC07:13
*** viktors|afk is now known as viktors07:15
viktorsmorning, oslo!07:15
*** shardy has joined #openstack-oslo07:20
*** cdent has joined #openstack-oslo07:29
*** tobe has quit IRC07:48
*** rpodolyaka1 has quit IRC07:53
*** pblaho has joined #openstack-oslo08:00
*** ihrachyshka_ has joined #openstack-oslo08:02
*** tobe has joined #openstack-oslo08:04
*** ihrachyshka has quit IRC08:04
*** ndipanov has joined #openstack-oslo08:06
*** harlowja_still_a has joined #openstack-oslo08:10
*** harlowja_at_home has quit IRC08:14
*** exploreshaifali has quit IRC08:16
*** ozamiatin has joined #openstack-oslo08:16
*** rpodolyaka1 has joined #openstack-oslo08:24
*** e0ne has joined #openstack-oslo08:32
*** ihrachyshka_ has quit IRC08:33
*** yassine_ has joined #openstack-oslo08:36
*** e0ne has quit IRC08:37
*** haypo has joined #openstack-oslo08:38
haypohum, i'm working on a python 3 patch for nova, and a test fails with08:41
hayposqlalchemy.orm.exc.DetachedInstanceError: Parent instance <InstanceExtra at 0x7fe32da80828> is not bound to a Session; deferred load operation of attribute 'pci_requests' cannot proceed08:41
haypothe same test pass on python 2.708:41
*** katyafervent is now known as katyafervent_awa08:47
rpodolyaka1that's weird - it must have nothing to do with python version08:50
rpodolyaka1I'd suggest to take a look at the test and the corresponding DB API method and understand why pci_requests value is not requested in advance, if it's known to be used later when session is not available08:51
*** salv-orl_ has joined #openstack-oslo08:53
hayporpodolyaka1: hum. the issue is in oslo_db.sqlalchemry.ModelBase.__contains__(). this method calls hasattr(). the ORM raises a sqlalchemy.orm.exc.DetachedInstanceError exception08:54
haypoon Python 2, hasattr() catchs *any* exception... on Python 3, hasattr() only catchs AttributeError (which makes more sense)08:54
haypo>'pci_requests' not in extra< returns False on Python 2, but it raises DetachedInstanceError on Python 308:55
haypotwo options: (1) fix oslo.db to replace hasattr() with "try: getattr() except Exception: pass" to mimick Python 2 behaviour08:57
*** salv-orlando has quit IRC08:57
haypo(2) fix nova unit test08:57
hayporpodolyaka1: for me, it looks like a real bug in oslo.db, even on Python 2. DetachedInstanceError should not be hidden. what do you think?08:57
*** salv-orl_ has quit IRC08:58
haypo"it looks like a real bug in oslo.db, even on Python 2" hum, i have to rephrase: it looks like a bug. python 3 behaviour looks better to me, python 2 "hides" the error. what do you think?08:58
haypoviktors: ^^ hey. i also would like your opinion on this oslo.db / sqlalchemy / nova / python 3 issue08:59
rpodolyaka1yeah, we don't want errors to pass silently here09:00
rpodolyaka1if it's everything except AttributeError it must raised further09:00
hayporpodolyaka1: the unit test fails at https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_db_api.py#L278809:00
viktorshaypo: looking09:01
rpodolyaka1that unit tests looks weird, like someone write it that way just to make it pass :P09:02
rpodolyaka1pci_request is a valid attribute of extra from what I see09:02
rpodolyaka1it's just that we hide this error in __contains__() right now...09:02
* rpodolyaka1 never liked the idea of treating instances like dicts..09:03
hayporpodolyaka1: the traceback, http://paste.openstack.org/show/320825/09:03
rpodolyaka1I suggest we fix __contains__() and only catch AttributeError09:04
hayporpodolyaka1: i'm working on such fix09:04
hayporpodolyaka1: but it means that the nova test must also be fixed09:04
rpodolyaka1haypo: ok. thank you!09:04
rpodolyaka1haypo: right09:04
hayporpodolyaka1: sorry, i don't know oslo.db nor SQLAlchemy! can you suggest me how to fix this specific unit test?09:05
hayporpodolyaka1: FYI i'm porting the nova sqlalchemy code to Python 3, and it's the remaining test which fails on Python 309:05
rpodolyaka1\o/09:05
rpodolyaka1haypo: so we need to figure out why the test explicitly asserts on 'not in'09:05
viktorshaypo: the proposal to "fix __contains__() and only catch AttributeError" looks good to me as well09:06
rpodolyaka1and if it should be 'in' instead, then we need to take a look at db.instance_extra_get_by_instance_uuid()09:06
hayporpodolyaka1: i understand that the code fetchs only a few columns, pci_requests is declared using "deferred"09:06
rpodolyaka1and make it load pci_requests attribute on demand09:06
hayporpodolyaka1: the unit test ensures that the pci_requests content is not fetched09:07
rpodolyaka1aha, so it does not work that way09:07
rpodolyaka1'deferred' just means - 'load the attribute on first access'09:07
haypothe column: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/models.py#L360  "pci_requests = orm.deferred(Column(Text))"09:07
rpodolyaka1yeah, so the point is - the attribute is always there, 'not in' makes no sense to me. hasattr() has hidden the real error from us and made the code behave like the author expected09:09
*** e0ne has joined #openstack-oslo09:10
hayporpodolyaka1, viktors : i hope that not "too much" code relies on the current behaviour of __contains__()...09:10
rpodolyaka1if we really want to check pci_request hasn't been loaded - we can assertRaises() on sqlalchemy.orm.exc.DetachedInstanceError09:10
rpodolyaka1haypo: indeed :(09:10
haypook, at least "tox -e py27" pass on oslo.db with my change :-D09:10
rpodolyaka1:)09:13
*** shardy has quit IRC09:14
*** shardy has joined #openstack-oslo09:15
*** e0ne is now known as e0ne_09:16
openstackgerritVictor Stinner proposed openstack/oslo.db: Fix sqlalchemy.ModelBase.__contains__() behaviour  https://review.openstack.org/19547209:20
hayporpodolyaka1, viktors ^^ here you have. does gerrit runs integration tests on oslo.db, testing nova at least?09:21
*** e0ne_ has quit IRC09:22
viktorshaypo: tempest only, afaik09:22
haypoviktors: ah cool, tempest has a wide code coverage09:22
viktorshaypo: can you also add some unittests to patch? :)09:23
haypois it possible to at least test manually nova tests on a specific oslo.db change?09:23
haypoviktors: sorry, i don't know what tests are!09:23
viktors>> nova tests on a specific oslo.db change - you can install custom oslo.db into your .tox.py34/ env09:25
haypoviktors: yeah, i will do that to run unit tests. but i don't have the right setup to run functional tests.09:26
haypoi was asking if i can do it in gerrit, maybe using a test patch on nova using depends-on?09:26
viktorshaypo: yes, depends-on tag should help09:27
haypook09:27
rpodolyaka1afaik, depends-on is just a helper for Zuul to understand in which order to put patches in the gate testing09:28
rpodolyaka1i.e. it does not affect check queue09:28
rpodolyaka1so you will probably need to test this locally09:29
rpodolyaka1by pip installing  oslo.db commit into py27 of nova09:29
rpodolyaka1and/or update your devstack and run tempest09:29
rpodolyaka1as gate jobs works with releases from pypi, not specific commits of oslo.* libs09:30
haypo"gate jobs works with releases from pypi, not specific commits of oslo.* libs" ah yes09:32
*** e0ne has joined #openstack-oslo09:33
haypoit was discussed at the summit at vancouver. i'm quite sure that dhellmann has a tool to do that09:33
haypotest an oslo library on nova before creating a release09:33
haypoit was discussed to automate this using gerrit, but i don't remember the conclusion of the discussion :-D09:34
rpodolyaka1yeah, viktors can point you to the script, but I haven't heard of this being available via our usual gerrit workflow09:36
hayporpodolyaka1: do you mean that the tests must be run locally?09:37
haypofor example, in a devstack?09:37
openstackgerritVictor Stinner proposed openstack/oslo.db: Fix sqlalchemy.ModelBase.__contains__() behaviour  https://review.openstack.org/19547209:37
haypo"viktors: sorry, i don't know what tests are!" sorry for the bad joke. here is an updated patch with an unit test ^^09:38
*** katyafervent_awa is now known as katyafervent09:38
hayporpodolyaka1: to come back to nova, i propose to skip the test with a comment: http://paste.openstack.org/show/320858/09:43
viktorshaypo: np, sorry for my english :)09:43
hayporpodolyaka1: i will rework the patch when oslo.db will be released with my __contains__() fix09:43
viktorshaypo: as for patch - do we really need line 66 - "    return hasattr(self, key)" ?09:44
haypoviktors: oh. i forgot to remove it09:44
openstackgerritVictor Stinner proposed openstack/oslo.db: Fix sqlalchemy.ModelBase.__contains__() behaviour  https://review.openstack.org/19547209:45
*** alexpilotti has joined #openstack-oslo09:46
*** salv-orlando has joined #openstack-oslo09:51
lifelessharlowja_still_a: https://review.openstack.org/195490 <- may be useful to you09:55
lifelessrpodolyaka1: Depends-On affects checks as well as gates09:55
lifelessthere's glue in devstack to use the zuul ref09:56
*** salv-orl_ has joined #openstack-oslo09:56
lifelessI don't know if its enabled in cross-project checks09:56
haypolifeless: hi. i got this error when running "tox -e py27" in nova, are you aware of this subunit-trace issue? http://paste.openstack.org/show/320860/09:56
lifelesshaypo: first I've heard of it09:57
haypoby the way, what is this os-testr project? why do we need yet another wrapper on top of other wrappers? :-p (why not just running unittests ? :-p)09:57
lifelessits a shim to consolidate redundant glue across openstack09:57
lifelessit can also do openstack specific UI09:57
haypolifeless: why does openstack require special code to run tests? why not enhance testr directly?09:58
lifelesshaypo: we should enhance testr of course09:58
*** salv-orlando has quit IRC09:58
lifelessanyhow, 10pm, pumpkin time for me09:59
*** salv-orl_ has quit IRC09:59
*** salv-orlando has joined #openstack-oslo09:59
haypolifeless: ah, bauzas on #openstack-nova found https://github.com/openstack/os-testr/commit/de604832c3ece2aaeb56cc7a427dcb1dd833c0e110:00
rpodolyaka1lifeless: cool, didn't know that!10:00
*** rpodolyaka1 is now known as podolyaka_hungry10:01
*** salv-orl_ has joined #openstack-oslo10:01
haypodef _get_test_list(regex, env=None):10:02
haypo    proc = subprocess.Popen(['testr', 'list-tests', regex], env=env,10:02
haypo                            stdout=subprocess.PIPE)10:02
openstackgerritCedric Brandily proposed openstack/oslo.db: Remove implicit RequestContext decoration  https://review.openstack.org/19549410:02
haypohum, i'm not sure that it's the most efficient way to get the list of tests... why not using the testr python module here?10:03
*** salv-orlando has quit IRC10:03
haypolifeless: oh. i also wrote something similar, https://github.com/openstack/os-testr/blob/master/os_testr/os_testr.py#L13910:03
haypoi mean a function to run testr to get a nice output, and exclude tests10:04
haypo(my code was very different)10:04
openstackgerritCedric Brandily proposed openstack/oslo.db: Remove implicit RequestContext decoration  https://review.openstack.org/19549410:04
*** Kennan2 has joined #openstack-oslo10:14
*** Kennan has quit IRC10:14
*** dims has joined #openstack-oslo10:14
*** Kennan2 is now known as Kennan10:17
*** husanu3 has joined #openstack-oslo10:19
*** exploreshaifali has joined #openstack-oslo10:20
*** husanu3 has quit IRC10:25
*** husanu1 has joined #openstack-oslo10:26
*** ihrachyshka has joined #openstack-oslo10:27
*** husanu1 has quit IRC10:28
*** husanu4 has joined #openstack-oslo10:30
*** husanu4 has quit IRC10:31
*** e0ne is now known as e0ne_10:31
*** e0ne_ has quit IRC10:36
*** sdake has quit IRC10:50
openstackgerritDavanum Srinivas (dims) proposed openstack/futurist: Updated from global requirements  https://review.openstack.org/19398210:53
*** e0ne has joined #openstack-oslo10:55
*** rpodolyaka1 has joined #openstack-oslo10:56
*** rpodolyaka1 is now known as rpodolyaka_10:56
*** dims has quit IRC11:02
*** dims has joined #openstack-oslo11:05
*** ozamiatin has quit IRC11:07
*** dims_ has joined #openstack-oslo11:10
*** dims has quit IRC11:11
*** tobe has quit IRC11:11
*** salv-orl_ has quit IRC11:14
*** e0ne is now known as e0ne_11:15
*** dims_ has quit IRC11:16
*** david-lyle has quit IRC11:17
*** dims has joined #openstack-oslo11:21
*** david-lyle has joined #openstack-oslo11:21
*** achanda has quit IRC11:25
*** e0ne_ has quit IRC11:26
*** rpodolyaka_ has quit IRC11:28
*** e0ne has joined #openstack-oslo11:28
*** rpodolyaka1 has joined #openstack-oslo11:32
*** prad has joined #openstack-oslo11:32
*** EmilienM|off is now known as EmilienM11:38
openstackgerritMerged openstack/futurist: Allow adding periodic callables at runtime  https://review.openstack.org/19199811:39
openstackgerritLi Yingjun proposed openstack/oslo.config: Fixes example for cfg  https://review.openstack.org/19553611:44
*** salv-orlando has joined #openstack-oslo11:49
*** amotoki has quit IRC11:57
flaper87dims: hey, just a heads up. The fix is on its way. There was a wrong build pushed to f21 repos.12:05
flaper87We'll still need the proposed patch but we also need the fixed build which has been approved already12:05
flaper87mirrors are getting it12:05
dimsflaper87: glad to hear12:05
*** bknudson has joined #openstack-oslo12:19
*** stpierre has joined #openstack-oslo12:20
flaper87dims: and just to be clear, since I know there are confusions. The amqp1 driver is just the victim here12:21
flaper87the failure has nothing to do with the driver/proton whatsoever12:21
dimsflaper87: ack12:22
*** dims has quit IRC12:31
*** dims has joined #openstack-oslo12:31
*** rpodolyaka1 has quit IRC12:32
*** exploreshaifali has quit IRC12:32
*** rpodolyaka1 has joined #openstack-oslo12:33
*** david-ly_ has joined #openstack-oslo12:35
*** ozamiatin has joined #openstack-oslo12:38
*** david-lyle has quit IRC12:39
*** kgiusti has joined #openstack-oslo12:46
*** husanu4 has joined #openstack-oslo12:50
*** husanu4 has quit IRC12:52
*** sdake has joined #openstack-oslo12:56
*** husanu5 has joined #openstack-oslo12:56
*** husanu5 has quit IRC13:00
*** e0ne is now known as e0ne_13:04
*** e0ne_ is now known as e0ne13:04
dimsrpodolyaka: viktors: seen this yet? https://bugs.launchpad.net/oslo.db/+bug/146870713:06
openstackLaunchpad bug 1468707 in oslo.db "Remove implicit RequestContext monkey patch" [Undecided,New]13:06
*** manishg has joined #openstack-oslo13:07
*** rpodolyaka1 has quit IRC13:09
*** manishg has quit IRC13:09
*** jecarey has joined #openstack-oslo13:10
*** husanux0 has joined #openstack-oslo13:13
*** husanux0 has quit IRC13:13
*** husanux0 has joined #openstack-oslo13:15
*** shardy has quit IRC13:17
*** husanux0 has quit IRC13:18
*** husanux2 has joined #openstack-oslo13:20
*** e0ne is now known as e0ne_13:20
*** husanux2 has quit IRC13:21
*** exploreshaifali has joined #openstack-oslo13:21
*** e0ne_ is now known as e0ne13:22
*** shardy has joined #openstack-oslo13:22
*** ajo is now known as ajo|afk13:25
*** husanux3 has joined #openstack-oslo13:25
*** salv-orl_ has joined #openstack-oslo13:27
*** husanux3 has quit IRC13:29
*** salv-orlando has quit IRC13:31
openstackgerritMerged openstack/oslo.rootwrap: daemon: avoid raising UnboundLocalError to callers  https://review.openstack.org/19189513:31
*** mriedem has joined #openstack-oslo13:31
*** nkrinner has quit IRC13:32
*** rpodolyaka1 has joined #openstack-oslo13:34
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.db: Allow projects that use test_models_sync to filter some changes  https://review.openstack.org/19557813:34
flaper87mirrors seem to have gotten the package, the recheck will hopefully pass now13:37
openstackgerritMerged openstack/oslo.service: Enforce H301 check  https://review.openstack.org/19507613:38
openstackgerritMerged openstack/oslo.service: Enforce H405 check  https://review.openstack.org/19507713:38
*** zzzeek has joined #openstack-oslo13:39
*** sputnik13 has joined #openstack-oslo13:42
*** shardy has quit IRC13:42
*** shardy has joined #openstack-oslo13:44
*** crc32 has joined #openstack-oslo13:44
*** sputnik13 has quit IRC13:47
*** sputnik13 has joined #openstack-oslo13:48
*** sputnik13 has quit IRC13:48
*** sputnik13 has joined #openstack-oslo13:49
*** exploreshaifali has quit IRC13:58
*** manishg has joined #openstack-oslo14:03
*** stevemar has joined #openstack-oslo14:06
*** e0ne is now known as e0ne_14:13
*** e0ne_ has quit IRC14:23
*** shardy has quit IRC14:29
*** yamahata has joined #openstack-oslo14:30
*** shardy has joined #openstack-oslo14:30
*** vilobhmm has joined #openstack-oslo14:33
*** e0ne has joined #openstack-oslo14:36
*** rpodolyaka1 has quit IRC14:37
*** rpodolyaka1 has joined #openstack-oslo14:39
*** rpodolyaka1 has quit IRC14:40
*** stevemar has quit IRC14:55
*** rpodolyaka1 has joined #openstack-oslo14:57
*** browne has joined #openstack-oslo14:59
*** ozamiatin has quit IRC15:00
*** david-ly_ is now known as david-lyle15:01
*** mtanino has joined #openstack-oslo15:05
*** shardy has quit IRC15:08
*** shardy has joined #openstack-oslo15:08
openstackgerritMerged openstack/oslo.db: Remove implicit RequestContext decoration  https://review.openstack.org/19549415:09
*** shardy has quit IRC15:13
*** shardy has joined #openstack-oslo15:13
*** manishg has quit IRC15:14
*** mtanino_ has joined #openstack-oslo15:15
openstackgerritMerged openstack/futurist: Updated from global requirements  https://review.openstack.org/19398215:15
*** mtanino has quit IRC15:16
*** kzaitsev_mb has joined #openstack-oslo15:23
kzaitsev_mbfolks, a quick question — is there an oslo lib, that would allow me to create a rather complex cli UI (the one that asks for some values interactively, for example)15:24
*** mtanino has joined #openstack-oslo15:28
*** manishg has joined #openstack-oslo15:28
*** mtanino_ has quit IRC15:31
*** e0ne is now known as e0ne_15:35
*** e0ne_ is now known as e0ne15:37
*** mtanino has quit IRC15:40
*** mkoderer_cloud has quit IRC15:44
dhellmannkzaitsev_mb: the cliff library is the closes thing we have, and it's managed by the sdk team now. It doesn't have any built-in support for prompting for missing values though. What sort of prompting do you need?15:46
*** sdake_ has joined #openstack-oslo15:50
openstackgerritSean Dague proposed openstack-dev/pbr: WIP: first sketch of wsgi_scripts support  https://review.openstack.org/19529215:50
*** vilobhmm has quit IRC15:51
*** sdake has quit IRC15:54
openstackgerritSean Dague proposed openstack-dev/pbr: WIP: first sketch of wsgi_scripts support  https://review.openstack.org/19529215:55
*** e0ne is now known as e0ne_15:56
*** arnaud____ has joined #openstack-oslo15:57
mtreinishdims, dhellmann: am I pulling in a bad version of oslo.db here?: http://logs.openstack.org/06/186506/4/check/gate-subunit2sql-python27/a46f6d4/console.html#_2015-06-25_15_21_25_56716:00
mtreinishI feel like I'm missing something because subunit2sql doesn't use oslo_context16:00
mtreinishoh, nm I found it https://bugs.launchpad.net/oslo.db/+bug/146870716:03
openstackLaunchpad bug 1468707 in oslo.db "Remove implicit RequestContext monkey patch" [Undecided,New]16:03
*** harlowja_still_a has quit IRC16:06
*** e0ne_ has quit IRC16:06
dhellmanndims, rpodolyaka, viktors: should we get an oslo.db release out with that dependency on oslo.context fixed? https://review.openstack.org/#/c/195656/16:11
*** e0ne has joined #openstack-oslo16:11
dhellmannmtreinish: yeah, I think we spotted that and had a work-around for one other project earlier this week, but didn't see it in subunit2sql16:11
mtreinishyeah I just caught it myself, I was rechecking away because there is a unit test race on postgres16:13
mtreinishafter the 3rd or 4th time I figured I should look :)16:13
dhellmannmtreinish: it looks like at this point the fix to add oslo.context is on top of another patch to remove the oslo namespace, so we wouldn't want to release as-is. We could revert the namespace deletion, release a fix, and re-approve the namespace deletion to get around that16:13
rpodolyaka1dhellmann: you beat me to it16:14
rpodolyaka1that's an interesting case16:14
rpodolyaka1can we have like stable branches on demand in gerrit?16:14
rpodolyaka1so that we don't have to dance with reverts16:14
mtreinishdhellmann: ok, sure. In the meantime I pushed up a subunit2sql patch to set  oslo.db!=1.12.0 in requirements16:14
dhellmannrpodolyaka1: well, we could, but the names get weird. Let me think about that.16:14
mtreinishthe benefit of not being g-r tracked16:15
dhellmannmtreinish: yeah, that may be faster for you16:15
*** rpodolyaka2 has joined #openstack-oslo16:18
*** subscope has quit IRC16:20
openstackgerritDoug Hellmann proposed openstack/stevedore: Document the signature for check_func  https://review.openstack.org/19566716:24
*** sdake_ is now known as sdake16:25
dhellmannrpodolyaka1, mtreinish, dims : it looks like we could use feature/1.12 as a branch name to do a small backport, and I'm going to look into what it would take to allow us to use bug/ instead of feature/16:29
*** rpodolyaka1 has quit IRC16:29
*** achanda has joined #openstack-oslo16:31
mtreinishdhellmann: ok, cool16:32
*** cdent has quit IRC16:32
sputnik13dhellmann: found the root cause of our issue, the one you helped with yesterday16:32
sputnik13one of the modules we're testing (cmd.manage) is adding a required option with no default, and if that test is being included in the same test list as another test that tries to parse config options, the test that runs after it sees that required option as missing and barfs on the ground16:34
*** jaosorior has quit IRC16:35
sputnik13and using a fixture doesn't help here because fixture doesn't try to mock out the CONF object (which I was mistakenly thinking it did)16:36
sputnik13even though the config fixture provides a register_option and register_cli_option to wrap the CONF object's version and clean up the options upon teardown of the config fixture, we can't pass the config fixture to the function (cmd.manage.main) that does the registration...16:38
sputnik13so I'm wondering whether the "right thing" is to make the fixture provide all of the same interfaces as the CONF object so that it can then guarantee cleanup of everything when the fixture is torn down16:38
sputnik13and we can pass the fixture to anything that needs to use CONF16:39
*** yamahata has quit IRC16:40
*** ndipanov has quit IRC16:42
*** arnaud____ has quit IRC16:44
*** shardy has quit IRC16:45
*** shardy has joined #openstack-oslo16:45
*** manishg has quit IRC16:45
*** ashleighfarnham has joined #openstack-oslo16:45
*** manishg has joined #openstack-oslo16:46
*** yamahata has joined #openstack-oslo17:07
*** achanda has quit IRC17:08
*** e0ne has quit IRC17:08
openstackgerritKevin Benton proposed openstack/oslo.db: Allow additional exceptions in wrap_db_retry  https://review.openstack.org/19451117:17
*** sdake_ has joined #openstack-oslo17:20
*** yamahata has quit IRC17:20
*** yamahata has joined #openstack-oslo17:20
*** sdake has quit IRC17:24
*** ozamiatin has joined #openstack-oslo17:24
*** achanda has joined #openstack-oslo17:25
*** shardy has quit IRC17:25
*** shardy has joined #openstack-oslo17:26
harlowjadims do u want to cut a futurist release?17:27
*** ozamiatin has quit IRC17:28
*** ihrachyshka has quit IRC17:29
*** sdake_ is now known as sdake17:29
*** dikonoor has joined #openstack-oslo17:30
*** rjaiswal has joined #openstack-oslo17:31
dikonoordims:Hi ..I have a query on GMR..I am trying to use nova GMR and the report always gets generated in /var/log/messages..and as per what I understand the report always goes to where stderr is configured to.17:31
dikonoordims:I wonder if there's a way to confiure in the respective conf file something like a section (like the [oslo_policy] section and the policy_file key for instance) where we can say this is where I want my nova GMR output to go to17:33
*** ihrachyshka has joined #openstack-oslo17:34
*** ihrachyshka has quit IRC17:41
*** yassine_ has quit IRC17:45
dikonoordims: Figured out..GMR apis have that option but it's not implemented by nova17:45
openstackgerritJoshua Harlow proposed openstack/oslo.service: Track created thread and disallow more than one start being active  https://review.openstack.org/19535817:53
*** boris-42 has joined #openstack-oslo17:54
*** rpodolyaka1 has joined #openstack-oslo17:54
rpodolyaka1dhellmann: cool, that'd great!17:55
*** geguileo has joined #openstack-oslo17:59
geguileoCan someone help me with a question regarding oslo versioned objects? Specifically on DateTimeField18:02
*** husanu4 has joined #openstack-oslo18:02
*** dontalton has joined #openstack-oslo18:02
*** dikonoor has quit IRC18:03
lifelesshaypo: yeah18:13
lifelesshaypo: so I'm sure os-testr can be better18:14
lifelesshaypo: like, using the testr API18:14
*** husanu4 has quit IRC18:14
*** rpodolyaka1 has quit IRC18:15
*** pblaho has quit IRC18:21
*** rpodolyaka1 has joined #openstack-oslo18:23
dimsharlowja: in zuul as we speak18:24
harlowjadims sweethotness18:24
* dims just got back from scooping out ice cream for 2 hours for school kids18:25
* harlowja didn't get no icecream18:26
harlowja:(18:26
harlowjai just graduated 6th grade18:26
harlowjalearned me some abcs18:26
dims:)18:26
dimsharlowja: come to nova mid-cycle and i'll get you what you need :)18:27
harlowjawhere's that at, can't remember anymore, lol18:27
dimsRochester, MN18:27
harlowjai think my memory is going, lol18:27
harlowjaah18:27
dimsdhellmann: ack on the bug/ branch18:27
*** arnaud____ has joined #openstack-oslo18:33
mtreinishlifeless, haypo: yes os-testr can be much better. It's kinda of a hack right now18:35
mtreinishmostly using subprocess18:35
mtreinishI wrote it quickly to replace all the pretty_tox.sh scripts which were doing the same basic thing and were spreading like the plague18:36
mtreinishthe idea was there would be a single place that everyone would use for that kinda thing18:36
*** arnaud____ has quit IRC18:37
*** timsim has joined #openstack-oslo18:38
mtreinishinstead of using divergently implemented, misleadingly named, small bash scripts that were copy and pasted between repos18:38
*** sputnik13 has quit IRC18:40
*** dontalton has quit IRC18:44
*** sputnik13 has joined #openstack-oslo18:46
*** pblaho has joined #openstack-oslo18:47
*** ericksonsantos has joined #openstack-oslo18:56
*** ericksonsantos has quit IRC19:03
openstackgerritKevin Benton proposed openstack/oslo.db: Allow additional exceptions in wrap_db_retry  https://review.openstack.org/19451119:05
kevinbentonzzzeek: ping19:06
kevinbentondims: how often does olso do releases?19:07
*** achanda has quit IRC19:11
dimskevinbenton: every week19:14
kevinbentondims: ok. so if that DB patch merges by the end of this week, when would it be available to neutron? end of next week?19:14
kevinbentondims: trying to determine if it's worth merging a workaround in neutron19:14
dimskevinbenton: first release needs to get out, then g-r needs to be updated for the min oslo.db version, then neutron can use it19:15
kevinbentondims: how long does the second step take? does it usually involve weeks of bikeshedding and tears?19:16
dimswe should be able to swing this one pretty quickly19:16
dimswe can make a good case i think19:17
kevinbentondims: ack19:17
*** rpodolyaka1 has quit IRC19:20
harlowjasileht dims https://review.openstack.org/#/c/195711/19:27
*** kzaitsev_mb has quit IRC19:28
*** jamespage_ has joined #openstack-oslo19:31
openstackgerritJoshua Harlow proposed openstack/futurist: Ensure universal wheels are built  https://review.openstack.org/19571419:31
harlowjaalthough i'm unsure if ^ should make a 0.1.1 come out?19:31
*** jamespage_ has quit IRC19:32
*** rushiagr_away is now known as rushiagr19:32
* harlowja noticed that when looking at @ https://pypi.python.org/pypi/futurist 19:32
harlowjaand its 'py version' there19:32
*** arnaud____ has joined #openstack-oslo19:33
dimsharlowja: 0.1.0 is still usable right?19:34
harlowjaya19:34
* harlowja just has to check if installing on py3 will complain19:34
harlowjawill check19:34
harlowjadims think we do need one :-/19:36
harlowjanot installable on py3 for pip19:36
harlowjahttp://paste.openstack.org/show/321145/19:36
harlowjasoooo, ya :-/ agile releases man!19:37
harlowjalol19:37
harlowja:)19:37
dimsharlowja: ack. doing something right now. will get it out later today19:37
harlowjak19:37
harlowjathx19:37
*** rpodolyaka1 has joined #openstack-oslo19:38
*** arnaud____ has quit IRC19:38
*** rpodolyaka1 has quit IRC19:41
*** ajo|afk is now known as ajo19:41
*** shardy has quit IRC19:42
*** shardy has joined #openstack-oslo19:43
*** ozamiatin has joined #openstack-oslo19:44
zzzeekkevinbenton: hi19:48
*** ozamiatin has quit IRC20:01
*** mtanino has joined #openstack-oslo20:01
*** jecarey has quit IRC20:09
lifelessharlowja: no sdist published ?20:10
harlowjalifeless ya, thats also weird20:10
lifelessdims: I can cut a release for futurist if harlowja needs one20:10
harlowjanot in desperation, just would be nice :)20:10
* harlowja wonders why a sdist wouldn't be created (since afaik the infra scripts upload that stuff when tagged)20:11
harlowjahave a universal wheel would be nice, and then the why no sdist question20:12
harlowja*having20:14
*** kgiusti has quit IRC20:14
*** ViswaV has joined #openstack-oslo20:19
ViswaVHi ...question about oslo_messaging.... What is the proper way to delete/cleanup queues from RabbitMQ  after a compute node that was exposing RPC endpoint via oslo_messaging (backed by RabbitMQ) is terminated from service ?20:21
ViswaVThere is a separate orchestration service in our case that brings up these compute nodes and manages their lifecycle... like Trove would manage their guest agents... When these nodes come up they initialize and expose RPC endpoint. But they could either be gracefully decommissioned or they can go bad and disappear in which case their resources (like the queues in rabbitmq used by the  RPC mechanism) have to be cleaned up by the central20:22
ViswaV orchestrator.20:22
ViswaVI don't see any API available in oslo.messaging to refer to the underlying amqp queues by name, and clean them up.20:23
ViswaVThanks in advance if anyone can please point me in the right direction20:24
*** sdake_ has joined #openstack-oslo20:24
*** sdake has quit IRC20:28
*** sdake_ is now known as sdake20:29
*** kzaitsev_mb has joined #openstack-oslo20:34
*** arnaud____ has joined #openstack-oslo20:34
*** jaypipes has quit IRC20:39
*** arnaud____ has quit IRC20:39
*** ozamiatin has joined #openstack-oslo20:40
*** ozamiatin has quit IRC20:47
* sputnik13 pokes lifeless 21:04
*** prad has quit IRC21:05
sputnik13"somebody" said that you might know why pip 7.0.3 is complaining about this line in requirements.txt21:05
sputnik13python-ldap>=2.4;python_version=='2.7'21:05
*** e0ne has joined #openstack-oslo21:05
sputnik13lifeless any ideas about ^?21:05
harlowjalol21:06
* harlowja wasn't me21:06
harlowjatchaypo might be able to help also (if either is around)21:07
tchaypoMorning21:08
tchaypoWhat's the complaint?21:08
harlowjasputnik13 ^21:08
tchaypoI think you need pip 7.1 for it to understand the end marker21:09
lifelessno21:09
tchaypoBut even then I'm not sure if it will be understood without a space either before or after the ;21:09
lifelessstale pbr version21:09
tchaypoBut also I just woke up21:09
lifelesspip isn't complaining about the line21:09
lifelesssetuptools is21:09
lifelesssputnik13: http://lists.openstack.org/pipermail/openstack-dev/2015-June/067823.html21:10
openstackgerritMerged openstack/automaton: Add base exception class for this library  https://review.openstack.org/19479321:10
tchaypoThis is why I was asking for the actual error message :)21:10
sputnik13doh21:10
sputnik13lifeless thanks21:10
lifelesstchaypo: pip 6 should be fine for simple == markers21:11
tchaypoOh yeah, 7.1 just adds <= and >=21:11
sputnik13ah what the heck21:12
sputnik13I have pbr 1.2.0 but when I stack for some reason pbr is being downgraded to 0.11.021:13
lifelesssputnik13: http://lists.openstack.org/pipermail/openstack-dev/2015-June/068031.html21:13
sputnik13lifeless: ack, yeah this is happening on keystonemiddleware install21:15
sputnik13thanks21:15
lifelessI'm happy to refer you to my emails anytime :)21:16
lifeless[seriously, I'm just glad I'm out in front on this]21:16
harlowja:)21:17
sputnik13hey, I'm glad to use your emails :)21:17
*** sreshetnyak has quit IRC21:28
*** sreshetnyak has joined #openstack-oslo21:28
*** mtanino has quit IRC21:38
*** hogepodge has quit IRC21:38
*** e0ne is now known as e0ne_21:39
*** shardy has quit IRC21:40
*** e0ne_ is now known as e0ne21:41
*** stpierre has quit IRC21:44
*** mriedem has quit IRC21:44
openstackgerritMerged openstack/automaton: Add history.rst that uses generated 'ChangeLog' file  https://review.openstack.org/19485521:47
openstackgerritMerged openstack/automaton: Remove 3.3 classifier  https://review.openstack.org/19486421:47
dimsflaper87: i see bunch of failures - building proton - http://logs.openstack.org/07/194507/3/check/gate-oslo.messaging-pep8/aa202ca/console.html21:48
dimsflaper87: /tmp/pip-build-ZSveCI/python-qpid-proton/build/bundled/qpid-proton/proton-c/src/platform.c:89:2: error: #error "Don't know how to generate uuid strings on this platform"21:48
*** rushiagr is now known as rushiagr_away22:03
*** Rockyg has joined #openstack-oslo22:04
*** zzzeek has quit IRC22:07
harlowjahmmm, ya i see that elsewhere to dims flaper8722:17
harlowja* http://logs.openstack.org/11/195711/1/check/check-requirements-integration-dsvm/672f081/console.html22:17
harlowjahttp://logs.openstack.org/11/195711/1/check/check-requirements-integration-dsvm/672f081/console.html#_2015-06-25_21_18_49_58022:17
harlowjano uuid strings, hmmm, questionable error, ha22:17
harlowjaunless thats a commodore 6422:17
harlowjathen maybe, lol22:17
* harlowja didn't know infra ran using commodore64 vms22:18
*** arnaud____ has joined #openstack-oslo22:23
*** e0ne has quit IRC22:26
*** arnaud____ has quit IRC22:28
*** dims has quit IRC22:30
*** dontalton has joined #openstack-oslo22:32
*** dims_ has joined #openstack-oslo22:38
*** manishg has quit IRC22:42
*** manishg has joined #openstack-oslo22:42
*** otherwiseguy has quit IRC22:45
*** dontalton has quit IRC22:45
*** dhellmann has quit IRC22:46
*** dhellmann has joined #openstack-oslo22:46
*** otherwiseguy has joined #openstack-oslo22:48
*** hogepodge has joined #openstack-oslo22:58
*** sdake_ has joined #openstack-oslo23:00
*** mriedem has joined #openstack-oslo23:02
*** sdake has quit IRC23:04
openstackgerritMerged openstack/futurist: Ensure universal wheels are built  https://review.openstack.org/19571423:06
*** mriedem has quit IRC23:06
*** mriedem has joined #openstack-oslo23:06
harlowjadims_ if u get around to that, maybe 0.2 or 0.1.1 or whatever release :)23:07
*** sdake has joined #openstack-oslo23:07
openstackgerritJoshua Harlow proposed openstack/taskflow: Use module-level on_done_callbacks vs method-local ones  https://review.openstack.org/19578423:09
*** sdake_ has quit IRC23:11
openstackgerritJoshua Harlow proposed openstack/tooz: Update compatibility matrix due to file drivers new abilities  https://review.openstack.org/19578823:18
openstackgerritDoug Hellmann proposed openstack/stevedore: Document the signature for check_func  https://review.openstack.org/19566723:23
openstackgerritDoug Hellmann proposed openstack/oslo.config: Remove oslo namespace package  https://review.openstack.org/18196123:24
openstackgerritDoug Hellmann proposed openstack/oslo.messaging: Remove oslo namespace package  https://review.openstack.org/18197423:37
*** sputnik13 has quit IRC23:39
*** kzaitsev_mb has quit IRC23:52
openstackgerritJoshua Harlow proposed openstack/taskflow: Convert executor proxied engine options into there correct type  https://review.openstack.org/19579623:53
*** sdake has quit IRC23:54
*** sdake has joined #openstack-oslo23:55
*** Rockyg has quit IRC23:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!