Friday, 2019-08-02

*** slaweq has joined #openstack-sdks00:11
*** slaweq has quit IRC00:16
*** goldyfruit has joined #openstack-sdks00:40
*** gtema has joined #openstack-sdks01:35
*** gtema has quit IRC01:40
*** e0ne has joined #openstack-sdks01:48
*** e0ne has quit IRC01:52
*** slaweq has joined #openstack-sdks02:11
*** slaweq has quit IRC02:16
*** gtema has joined #openstack-sdks03:36
*** gtema has quit IRC03:41
*** gkadam has joined #openstack-sdks03:49
*** slaweq has joined #openstack-sdks04:11
*** slaweq has quit IRC04:16
*** whoami-rajat has joined #openstack-sdks04:34
*** Luzi has joined #openstack-sdks05:05
*** gtema has joined #openstack-sdks05:38
*** gtema has quit IRC05:42
*** e0ne has joined #openstack-sdks06:04
*** slaweq has joined #openstack-sdks06:10
*** slaweq has quit IRC06:15
openstackgerritpengyuesheng proposed openstack/osc-lib master: Sync Sphinx requirement  https://review.opendev.org/67419206:26
*** e0ne has quit IRC06:42
*** e0ne has joined #openstack-sdks06:45
openstackgerritpengyuesheng proposed openstack/osc-lib master: Sync Sphinx requirement  https://review.opendev.org/67419206:47
openstackgerritpengyuesheng proposed openstack/osc-lib master: Sync Sphinx requirement  https://review.opendev.org/67419207:02
*** slaweq has joined #openstack-sdks07:05
*** redrobot has quit IRC07:17
*** jpenag is now known as jpena07:31
*** e0ne has quit IRC07:31
*** gtema has joined #openstack-sdks07:38
*** jpich has joined #openstack-sdks07:40
*** gtema has quit IRC07:42
*** cdent has joined #openstack-sdks07:48
*** tosky has joined #openstack-sdks07:48
*** ralonsoh has joined #openstack-sdks08:10
dtantsurthanks kmalloc, this is useful feedback!08:11
kmallocdtantsur: this is where we did the work08:12
kmallochttps://github.com/openstack/keystone/blob/master/keystone/server/flask/application.py#L165-L16708:12
kmallocdtantsur: and our API base https://github.com/openstack/keystone/blob/master/keystone/server/flask/common.py08:13
kmallocas you can see, it's doable, it just takes extra code. each of our paths (e.g. /v3/users) is a blueprint and would be found in keystone.api.users (all resource handlers under /v3/users goes in keystone.api.users even if it's something that historically lived elsewhere)08:14
kmallocsince we lean on json home we  have some extra construction bits for that08:14
kmallockeep in mind the blueprint/resource class (we use flask restful) doesn't exist until it is called, so sometimes self.XXX may not work since you're in a class context.08:16
dtantsurokay, cool!08:28
dtantsurnow my biggest problem is how to gradually replace wsme without doing everything at once08:28
dtantsurmordred: hi! opinion needed on https://github.com/dtantsur/rust-osauth/pull/708:54
*** e0ne has joined #openstack-sdks09:03
*** jangutter has quit IRC09:36
*** gtema has joined #openstack-sdks09:40
*** gtema has quit IRC09:44
*** jangutter has joined #openstack-sdks09:51
cdentdeath to wsme!10:08
dtantsur\o/10:22
*** gtema has joined #openstack-sdks10:24
*** cdent has quit IRC10:50
*** ricolin has quit IRC11:00
*** cdent has joined #openstack-sdks11:01
*** zbr has quit IRC11:25
*** jpena is now known as jpena|lunch11:34
mordreddtantsur: responded11:46
dtantsurthx11:50
mordreddtantsur: also - it makes me sad that there are deployments out that which don't expose those urls properly. it's largely because python-novaclient and friends never used them (hardcoding url manipulations instead) - so nobody noticed11:56
dtantsuryep, pretty sad12:01
dtantsursee, my small and young project is already getting ugly hacks because of nova!12:01
dtantsur:D12:01
*** redrobot has joined #openstack-sdks12:07
*** cdent has quit IRC12:18
*** Luzi has quit IRC12:21
*** ricolin has joined #openstack-sdks12:23
*** gkadam has quit IRC12:26
*** cdent has joined #openstack-sdks12:35
*** zbr has joined #openstack-sdks12:35
*** jpena|lunch is now known as jpena12:39
openstackgerritDmitry Tantsur proposed openstack/openstacksdk master: baremetal.configdrive: tolerate user_data as a string  https://review.opendev.org/67429812:49
*** mriedem has joined #openstack-sdks13:21
*** goldyfruit has quit IRC13:25
*** goldyfruit has joined #openstack-sdks13:25
fricklermordred: seems you commented as opendevadmin instead your personal account, I'm assuming that wasn't intended?13:39
*** gkadam has joined #openstack-sdks14:32
*** gkadam has quit IRC14:38
openstackgerritBence Romsics proposed openstack/openstacksdk master: Add router add/remove route operations  https://review.opendev.org/67432414:44
openstackgerritBence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route  https://review.opendev.org/67432514:47
*** goldyfruit has quit IRC14:50
gtemamordred: Houston - we have a problem. Ansible is not capable in resolving resource objects. Latest merged released change for Keypairs results in Ansible not properly getting keypair attributes15:05
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Sort output of functional test trait  https://review.opendev.org/67432815:05
openstackgerritMatt Riedemann proposed openstack/python-openstackclient master: Add openstack server create --boot-from-volume option  https://review.opendev.org/67411115:17
openstackgerritRiccardo Pittau proposed openstack/openstacksdk master: Sort output of functional test trait  https://review.opendev.org/67432815:28
*** goldyfruit has joined #openstack-sdks15:31
*** factor has joined #openstack-sdks15:33
*** cdent has quit IRC15:37
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Enable ansible module test for keypair to check return data  https://review.opendev.org/67433315:43
*** jpena is now known as jpena|off15:45
*** goldyfruit has quit IRC15:48
*** jangutter has quit IRC15:51
elmikoedleafe, dtantsur, following on from thierry's pr, i put up a new patch to remove the early morning hours16:00
dtantsurthx!16:00
elmikoi tagged you all on it16:00
elmikoi know we didn't have a formal vote about it or anything, but i guess the review can serve as that?16:01
dtantsuryep16:01
dtantsuractually, my vote is enough ;) if I stop attending, the slot won't be attended any more16:02
edleafeNo!! I want a formal vote, in a grand hall with trumpets and such!16:02
dtantsuraccepted!16:04
*** jpich has quit IRC16:05
*** jpich has joined #openstack-sdks16:05
openstackgerritLuigi Toscano proposed openstack/openstacksdk master: DNM Test for zuul-jobs  https://review.opendev.org/67434416:12
*** jpich has quit IRC16:14
elmikohaha, i think Thierry already workflowed it. dtantsur's +1 was enough XD16:18
* dtantsur haz powerrrzzz16:18
* edleafe feels left out <sniffle!>16:20
*** altlogbot_2 has quit IRC16:29
*** irclogbot_2 has quit IRC16:33
*** altlogbot_3 has joined #openstack-sdks16:37
*** dtantsur is now known as dtantsur|afk16:39
* elmiko gives edleafe a hug16:41
*** irclogbot_1 has joined #openstack-sdks16:42
* edleafe is happy again16:43
mordredfrickler: whoops! thanks - I forgot to switch logins after setting up the opendevadmin account yesterday16:43
mordredgtema: that's not awesome. when you say "is not capable in resolving resource objects" - that's because ansible is expecting to be able to return them in the json and that's not working? I thought we'd made resoures subclasses of dict so that this would work :(16:44
gtemayes, I am deeply in debuging Ansible jsonifying Resource16:45
gtemaso basically call exit_json(key=SDK_RESOURCE) is returning nulls16:45
gtemait get the keys, but somehow not the values16:46
gtemahttp://paste.openstack.org/show/755341/16:46
gtemaand with the 0.32 including keypair "fix" it becomes broken16:47
gtemain Ansible json.dumps likely will get exception, since it uses "encoding" and switch to "container_to_text" method16:50
gtemasomewhere here something is missing16:50
mordredthat seems really off - and like there's something missing down deep somewhere16:53
gtemaexact16:54
gtemaand it's terrible to debug it, since no propel logging is possible16:54
*** ricolin has quit IRC17:07
*** openstackgerrit has quit IRC17:22
gtemathat's some mistery. When I am copying all those ansible functions into unit tests to be able to reproduce - it works, under ansible it fails17:35
gtemathe iteritems (iter(d.items(**kw)) return empty values from ansible, but correct from sdk unit tests17:38
*** goldyfruit has joined #openstack-sdks17:47
*** ralonsoh has quit IRC17:51
*** gtema has quit IRC18:09
*** gtema has joined #openstack-sdks18:09
*** goldyfruit has quit IRC18:10
*** e0ne has quit IRC18:11
*** gtema has quit IRC18:13
*** openstackgerrit has joined #openstack-sdks18:16
openstackgerritMatt Riedemann proposed openstack/openstacksdk master: Fix DeprecationWarning for using logger.warn  https://review.opendev.org/67440118:16
mriedemeasy fix for something that's now blowing up the nova functional test job logs ^18:16
mriedemhttps://logs.opendev.org/61/673961/1/gate/nova-tox-functional-py36/a4cb2af/job-output.txt.gz#_2019-08-01_17_51_24_07021918:17
*** goldyfruit has joined #openstack-sdks18:20
*** goldyfruit has quit IRC18:26
*** gtema has joined #openstack-sdks18:26
*** gtema has quit IRC19:05
*** gtema has joined #openstack-sdks19:06
*** e0ne has joined #openstack-sdks19:13
*** gtema has quit IRC19:18
*** gtema has joined #openstack-sdks19:19
*** gtema has quit IRC19:23
*** tosky has quit IRC19:32
*** irclogbot_1 has quit IRC19:39
*** irclogbot_0 has joined #openstack-sdks19:43
*** goldyfruit has joined #openstack-sdks19:50
*** slaweq has quit IRC19:55
efriedmriedem: That UnsupportedServiceVersion thing - is that showing up in devstack tests or only functional?20:02
mriedemlet me ask the magic 8 ball20:02
mriedemi mean logstash20:02
*** whoami-rajat has quit IRC20:04
mriedemboth20:04
mriedemhttp://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22UnsupportedServiceVersion%3A%20Service%20placement%20has%20no%20discoverable%20version.%5C%22&from=7d20:04
efriedmordred: Can we make placement a known, supportes service type at v1.x with an empty Proxy?20:05
mordredefried: absolutely. let me make you a patch20:09
efriedthanks!20:09
mordredefried: does placement need to do version discovery on its root endpoint and/or microversions?20:09
efriedmordred: that'll get rid of those warnings, yah?20:09
mordredyeah20:09
efriedmordred: There's only one major version rn.20:10
efriedBy microversion discovery, you mean "I need an endpoint that can handle at least 1.XX"?20:10
mordredwell - I mean getting the min/max microversion range from the endpoint - so yeah. I guess the question is - does placement do microversions20:11
*** e0ne has quit IRC20:11
efriedPlacement definitely "does" microversions, yeah.20:11
mordredawesome.20:11
*** slaweq has joined #openstack-sdks20:11
mordredis there a placement.example.com/v1 ?20:11
mordredyou know - how about I go read the docs and then ask you follow up questions20:11
* mordred wants to make sure the empty proxy has the appropriate amount of emptyness20:12
efriedmordred: And especially if you discover holes in the docs that don't answer these questions, let us know (or open stories or whatever)20:12
efriedmordred: cdent has been in charge of this kind of stuff, so I'm pretty sure we're following all the rules wrt discovery documents and endpoint construction and whatnot.20:12
mordredcool20:13
efriedlike, I'm 95% sure you won't find snowflake behavior20:13
efriedefried@canuc:~$ curl http://192.168.218.27/placement20:15
efried{"versions": [{"status": "CURRENT", "min_version": "1.0", "max_version": "1.36", "id": "v1.0", "links": [{"href": "", "rel": "self"}]}]}20:15
efriedthat href="" is suspicious20:15
*** slaweq has quit IRC20:15
efriedmordred: Poking at my devstack, it is looking like we do not have versioned endpoints for placement.20:22
efriedSo forget what I said about cdent, and let's burn him at the stake.20:22
*** mriedem has quit IRC20:33
*** slaweq has joined #openstack-sdks21:11
*** slaweq has quit IRC21:15
mordredefried: that's fine - I've already got sdk provisions for this sort of thing :)21:54
*** slaweq has joined #openstack-sdks22:11
*** slaweq has quit IRC22:15
*** goldyfruit has quit IRC22:25
*** slaweq has joined #openstack-sdks23:11
*** slaweq has quit IRC23:16

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