Tuesday, 2019-02-19

*** markvoelker has quit IRC00:09
*** markvoelker has joined #openstack-sdks00:09
*** markvoelker has quit IRC00:14
*** slaweq has quit IRC00:41
*** tosky has quit IRC00:57
*** markvoelker has joined #openstack-sdks01:10
*** whoami-rajat has joined #openstack-sdks01:22
*** markvoelker has quit IRC01:44
openstackgerritXU Xiaodan proposed openstack/python-openstackclient master: add optional argument metric to set metric for routes  https://review.openstack.org/63765801:50
*** markvoelker has joined #openstack-sdks02:41
*** markvoelker has quit IRC03:13
*** mgariepy has quit IRC03:33
*** dave-mccowan has joined #openstack-sdks03:45
*** mgariepy has joined #openstack-sdks04:06
*** markvoelker has joined #openstack-sdks04:10
*** Shrews has quit IRC04:17
*** Shrews has joined #openstack-sdks04:19
*** Shrews has quit IRC04:24
*** Shrews has joined #openstack-sdks04:25
openstackgerritDoug Wiegley proposed openstack/python-openstackclient master: Add 'security_group' type support to network rbac commands  https://review.openstack.org/63676004:25
*** markvoelker has quit IRC04:44
*** irclogbot_2 has quit IRC04:55
*** dave-mccowan has quit IRC05:13
*** mgariepy has quit IRC05:29
*** mgariepy has joined #openstack-sdks05:30
*** markvoelker has joined #openstack-sdks05:41
*** markvoelker has quit IRC06:14
*** e0ne has joined #openstack-sdks06:25
*** e0ne has quit IRC06:26
*** Luzi has joined #openstack-sdks06:48
*** slaweq has joined #openstack-sdks07:10
*** markvoelker has joined #openstack-sdks07:11
openstackgerritVishakha Agarwal proposed openstack/python-openstackclient master: Cleanup: Remove deprecated role list command  https://review.openstack.org/63716607:18
*** e0ne has joined #openstack-sdks07:23
*** markvoelker has quit IRC07:43
*** e0ne has quit IRC08:13
*** gtema has joined #openstack-sdks08:23
*** evrardjp_ is now known as evrardjp08:31
*** markvoelker has joined #openstack-sdks08:41
*** tosky has joined #openstack-sdks08:42
openstackgerritArtem Goncharov proposed openstack/python-openstackclient master: Add possibility to filter images using member_status  https://review.openstack.org/63750808:49
*** jpich has joined #openstack-sdks08:52
*** ttsiouts has joined #openstack-sdks08:57
*** jpena|off is now known as jpena08:59
openstackgerritzhongshengping proposed openstack/openstackclient master: add python 3.7 unit test job  https://review.openstack.org/63773909:05
openstackgerritzhongshengping proposed openstack/osc-lib master: add python 3.7 unit test job  https://review.openstack.org/63774309:05
openstackgerritzhongshengping proposed openstack/cliff master: add python 3.7 unit test job  https://review.openstack.org/63776609:06
openstackgerritzhongshengping proposed openstack/python-openstackclient master: add python 3.7 unit test job  https://review.openstack.org/63778509:06
*** ttsiouts has quit IRC09:08
*** ttsiouts has joined #openstack-sdks09:08
*** ttsiouts has quit IRC09:13
*** markvoelker has quit IRC09:14
*** ttsiouts has joined #openstack-sdks09:25
*** slaweq has quit IRC09:31
*** slaweq has joined #openstack-sdks09:32
*** dtantsur|afk is now known as dtantsur09:34
*** ttsiouts has quit IRC09:38
*** ttsiouts has joined #openstack-sdks09:38
*** e0ne has joined #openstack-sdks09:53
*** ttsiouts has quit IRC09:56
*** ttsiouts has joined #openstack-sdks09:58
*** cdent has joined #openstack-sdks10:06
*** markvoelker has joined #openstack-sdks10:11
*** ttsiouts has quit IRC10:40
*** ttsiouts has joined #openstack-sdks10:41
*** ttsiouts_ has joined #openstack-sdks10:42
*** markvoelker has quit IRC10:43
*** ttsiouts has quit IRC10:45
mordredgtema: jinx. we both rechecked that patch just now :)11:08
gtemanice11:09
gtemadouble is better11:09
mordredhope so11:10
gtemayeah.11:11
gtemabtw mordred, I would need designate for functional tests of dns resouces. We currently have it only in `openstacksdk-functional-devstack-networking`11:13
gtemado you think it is sufficient or should we add something else?11:13
mordredgtema: I think it's sufficient ... as long as we have an env where the tests run11:14
gtemaoki, thks11:15
*** markvoelker has joined #openstack-sdks11:40
openstackgerritArtem Goncharov proposed openstack/python-openstackclient master: add multiple attributes to image from APIv2.7  https://review.openstack.org/63752811:46
*** ttsiouts_ has quit IRC11:57
openstackgerritDavid Rabel proposed openstack/python-openstackclient master: Fix help message of image add project  https://review.openstack.org/63782812:06
mordredgtema: comments on https://review.openstack.org/#/c/637538 - sorry, I wrote the yesterday but forgot to click the button)12:13
gtemano problem.12:14
*** markvoelker has quit IRC12:14
gtemaI was already nerveous about "os_hidden" and "is_hidden" ;-)12:14
gtemaI somehow think it might even strenghten confusion12:15
gtemabut ok12:15
mordredgtema: os_hidden is just awesome12:16
gtemamordred: what about "is_os_hidden"?12:17
gtemalol12:17
* mordred checks gtema for illness ...12:17
gtemaindeed flu12:17
mordredgtema: maybe if glance v2 hadn't gotten rid of the properties dict for user-settable properties, there would have been a decent story for glance to add new system properties without needing to prefix them with os_12:17
gtemamordred: yeah, I am also not happy about all that. But anyway, making it "is_hidden", right?12:18
mordredgtema: yeah. we try to avoid such sins as os_hidden in sdk, which may or may not be smart, but then, we also may or may not be smart :)12:19
gtemaok, "I feel it comin" :)12:20
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add image attributes from v2.7  https://review.openstack.org/63753812:20
mordredbeatiful12:21
* mordred really needs to start writing osc v4 'just use sdk' patches - but not enough hours in the day yet12:21
gtemamordred: also started implementing "stage" and "import" for glance yesterday - also very nice things12:21
mordredoh cool - yeah, I've been meaning to work on those12:22
gtemaI also need around 30 hours in a day to be able to work at all12:22
mordredgtema: did I mention aligning the shade and sdk image methods (specifically, replacing the sdk methods with the ones in the shade layer)?12:23
mordredgtema: I started on it here: https://review.openstack.org/#/c/609684/12:23
gtemawrt osc: as I wrote in the maillist I would be doing that. And Glance is the beginning, therefore those patches12:23
mordredbut that's in merge conflict and needs to be updated12:23
mordredgtema: ah - awesome!12:23
gtemaoh great12:23
mordredgtema: sorry, I missed that from the mailing list12:23
gtemano worry12:23
gtemaprobably wrong subject prefix ;)12:24
gtemamordred, have you seen my query in maillist about "*" region for identity?12:25
mordredno - I'll go look for that12:25
gtemamordred: http://lists.openstack.org/pipermail/openstack-discuss/2019-February/002777.html12:25
gtemais somehow a tricky setup in my case which makes some troubles, but there are no strict regulations from OpenStack about that12:26
mordredyeah. I agree - we should make a solution :)12:27
gtemayupp12:27
mordredgtema: I think the 'easy' workaround for the short term is to just register the identity endpoints in the catalog for each region - even if it's the same endpoint12:27
mordredbut I agree completely that this is lame12:27
mordredand we should actually support the thing you're talking about12:28
gtemamordred: I agree this is an "easy" solution from the implementation POV12:28
mordredin a real non-hacky way12:28
gtemabut I need to convince operations about that12:28
gtemaand this is not trivial12:28
mordredyeah. it never is :)12:28
mordredwould an API-SIG document describing it as a 'correct' choice help?12:29
* mordred needs to AFK for a few minutes - brb12:29
gtemayes, I think it would be12:29
*** jpena is now known as jpena|lunch12:30
*** markvoelker has joined #openstack-sdks13:11
*** ttsiouts has joined #openstack-sdks13:11
*** gtema has quit IRC13:15
*** ttsiouts has quit IRC13:18
*** ttsiouts has joined #openstack-sdks13:19
*** jpena|lunch is now known as jpena13:35
openstackgerritMerged openstack/openstacksdk master: Make sure we pick flavors with disk  https://review.openstack.org/63761813:43
dtantsur\o/13:43
*** markvoelker has quit IRC13:43
dtantsurmordred: could you check https://review.openstack.org/#/c/636601/ if you have a minute?13:44
mordreddtantsur: lgtm!13:48
dtantsurcool13:48
*** dave-mccowan has joined #openstack-sdks13:50
*** dulek has joined #openstack-sdks13:54
dulekHi folks! Can I somehow use openstacksdk to just do plain REST calls to OpenStack endpoint, so I won't worry that e.g. there's no support for Octavia tags yet?13:56
dulekBare requests module won't do all the keystone authentication for me, that's why I'm looking for some middle layer here.13:57
dtantsurdulek: I think you need the keystoneauth library13:57
dtantsurdulek: https://docs.openstack.org/keystoneauth/latest/using-sessions.html13:58
mordreddulek: you can totally make REST calls to an openstack endpoint13:58
dtantsurdulek: openstacksdk also provided keystoneauth Adapters for known services, e.g. connection.baremetal is an adapter for Ironic.13:59
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add image attributes from v2.7  https://review.openstack.org/63753813:59
mordreddulek: each service type has a corresponding adapter on the Connection object - so you should be able to do "conn.load_balancer.get('/something/in/octavia') and it'll return a requests.Response13:59
*** gtema has joined #openstack-sdks13:59
dulekdtantsur: Ooooh, this would probably be perfect…13:59
dulekdtantsur: So I won't need to deal with endpoints names and stuff.14:00
mordredyah14:00
dtantsuryep, see mordred's example14:00
dulekmordred, dtantsur: Awesome, thank you guys!14:00
mordred\o/14:00
mordredok. I've been helpful for today - I can go back to bed now14:00
dtantsur++ about the right time14:00
*** ttsiouts has quit IRC14:04
*** ttsiouts has joined #openstack-sdks14:04
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: WIP Move image methods to sdk image proxy  https://review.openstack.org/60968414:06
*** ttsiouts has quit IRC14:09
*** ttsiouts has joined #openstack-sdks14:09
gtemamordred: just rebased your change 609684 on top of master leaving all other related changes off, since there were lots of conflicts14:12
*** markvoelker has joined #openstack-sdks14:41
mordredgtema: sweet. obviously the code needs some cleanup to start using Image resource objects internally - but I think if we can get that passing tests landing it in that state should let us Resource-ify it in smaller patches14:44
gtemagreat14:46
gtemamordred: let's see what the tests show now. Do you want to continue on it, or should I take it?14:47
johnsomdulek: fyi, Octavia tags support has merged. Still waiting on the rest of those patches however.14:56
*** whoami-rajat has quit IRC15:01
dulekjohnsom: Oh, I see, it's even released. :)15:02
*** markvoelker has quit IRC15:14
*** whoami-rajat has joined #openstack-sdks15:18
*** Luzi has quit IRC15:19
Shrewsdtantsur: +A'd https://review.openstack.org/636594 but left a nit there you might want to see15:19
openstackgerritVishakha Agarwal proposed openstack/python-openstackclient master: Cleanup: Remove deprecated role list filters  https://review.openstack.org/63716615:22
dtantsurShrews: thanks! I'll check that and also the place I copy-pasted it from :)15:24
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add image.task resource  https://review.openstack.org/63791315:42
*** e0ne has quit IRC16:00
*** e0ne has joined #openstack-sdks16:03
*** markvoelker has joined #openstack-sdks16:11
openstackgerritMerged openstack/python-openstackclient master: Fix help message of image add project  https://review.openstack.org/63782816:13
mordredgtema: if you haven't started yet - I'm going to take a quick stab at the failures on that WIP image reorg patch16:21
mordredShrews, dtantsur: https://review.openstack.org/#/c/637563/ if you have a sec ... odyssey4me was having a hard time debugging a failure yesterday in nodepool, and it's a failure mode that is always confusing to users when it happens16:22
mordredShrews, dtantsur: it's a behavior change, but I **seriously** doubt it's a behavior change anyone is depending on because I think I might be the only person to have that whole behavior stack in my head :)16:23
mordredgtema: ^^ you might also want to look at that one16:24
dtantsurmordred: any unit tests to update?16:25
mordreddtantsur: no - it worked fine (although that maybe indicates there is a missing unittest)16:25
dtantsurexactly ;)16:25
mordredthe only use case I could see this making harder is if a user has a cloud that only has an unknown verion of a known service - but even in that case  they can set an endpoint_override and a version override and it'll work16:27
mordreddtantsur: after I poke at this image patch I'll see if I can't come up with a unittest16:27
gtemamordred: you should definitely have a look in to the image-reorg. I have managed to resolve some of the issues, but here a knowledge of shade-creator is required to do it properly16:31
openstackgerritMerged openstack/openstacksdk master: baremetal: implement set_node_power_state in the proxy  https://review.openstack.org/63660116:37
openstackgerritMerged openstack/openstacksdk master: baremetal: support for allocation API  https://review.openstack.org/63659416:39
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add image.task resource  https://review.openstack.org/63791316:43
*** markvoelker has quit IRC16:44
*** dtantsur is now known as dtantsur|afk16:48
*** ttsiouts has quit IRC16:56
*** ttsiouts has joined #openstack-sdks16:56
*** ttsiouts has quit IRC17:01
gtemamordred, odyssey4me: with https://github.com/ansible/ansible/pull/37390 a change of "default" port_security_enabled from True (when not given) to False (explicitly) was introduced. Now all openstacksdk-ansible-devel-functional-devstack are failing17:03
gtemamordred, odyssey4me: I do not really understand why so far, but comparing logs from stable-2.6 and master I see that in 2.6 when no port_security_enabled was given - a net with "True" was created, even when in the SDK we have a default of "False"17:08
*** jpich has quit IRC17:11
gtemamordred, odyssey4me: I got it. Ansible calls cloud.create_network and it bypasses Resource completely, so a "deployed default" plays role. I think we need to create a follow-up in Ansible to make default 'None'17:13
Shrewsmordred: 563 lgtm, but will wait for your unittest17:14
Shrewsmordred: also, sorry for delay. still quite sick  :(17:14
mordredShrews: coolio17:14
mordredShrews: no worries! you should stop being sick :(17:14
Shrewsikr?17:14
*** e0ne has quit IRC17:35
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add image.task resource  https://review.openstack.org/63791317:35
*** markvoelker has joined #openstack-sdks17:41
*** markvoelker has quit IRC18:14
*** jpena is now known as jpena|off18:22
*** e0ne has joined #openstack-sdks18:41
*** e0ne has quit IRC18:42
*** gtema has quit IRC18:43
*** gtema has joined #openstack-sdks18:47
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add image attributes from v2.7  https://review.openstack.org/63753818:49
*** e0ne has joined #openstack-sdks18:54
*** gtema has quit IRC18:56
*** markvoelker has joined #openstack-sdks19:11
*** e0ne has quit IRC19:39
*** e0ne has joined #openstack-sdks19:39
*** mgariepy has quit IRC19:40
*** markvoelker has quit IRC19:44
*** e0ne has quit IRC19:45
*** openstackgerrit has quit IRC20:09
*** gtema has joined #openstack-sdks20:19
*** openstackgerrit has joined #openstack-sdks20:20
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add image attributes from v2.7  https://review.openstack.org/63753820:20
*** cdent has quit IRC20:24
*** gtema has quit IRC20:35
*** ttsiouts has joined #openstack-sdks20:36
*** e0ne has joined #openstack-sdks20:37
*** markvoelker has joined #openstack-sdks20:41
*** ttsiouts has quit IRC21:00
*** ttsiouts has joined #openstack-sdks21:01
*** ttsiouts has quit IRC21:05
*** markvoelker has quit IRC21:13
*** whoami-rajat has quit IRC21:21
*** e0ne has quit IRC21:28
*** dave-mccowan has quit IRC22:07
*** ttsiouts has joined #openstack-sdks22:07
*** markvoelker has joined #openstack-sdks22:10
*** ttsiouts has quit IRC22:12
*** markvoelker has quit IRC22:44
*** markvoelker has joined #openstack-sdks23:41
*** tosky has quit IRC23:58

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