Wednesday, 2018-05-16

*** bobh has joined #openstack-sdks00:06
*** linhnm has joined #openstack-sdks00:33
*** bobh has quit IRC00:41
*** bobh has joined #openstack-sdks00:41
*** bobh has quit IRC00:46
*** gildub has quit IRC00:57
*** dave-mcc_ has quit IRC00:59
*** fabian_ has joined #openstack-sdks01:05
*** fabian_ is now known as chenyb401:14
*** edmondsw has joined #openstack-sdks01:18
*** edmondsw has quit IRC01:22
*** shu-mutou has joined #openstack-sdks01:57
*** dave-mccowan has joined #openstack-sdks02:01
*** annp has quit IRC02:03
*** annp has joined #openstack-sdks02:03
*** linhnm has quit IRC02:03
*** dave-mccowan has quit IRC02:32
*** gkadam has joined #openstack-sdks02:36
*** gkadam has quit IRC03:07
*** annp has quit IRC03:20
*** Shilpa has joined #openstack-sdks03:54
*** gildub has joined #openstack-sdks04:09
*** gildub has quit IRC04:23
*** gildub has joined #openstack-sdks04:29
*** gildub has quit IRC04:29
*** gildub has joined #openstack-sdks04:30
*** edmondsw has joined #openstack-sdks04:32
*** dtantsur|afk has quit IRC04:38
*** gildub has quit IRC04:47
*** gildub has joined #openstack-sdks05:33
*** annp has joined #openstack-sdks05:34
*** gildub has quit IRC06:09
*** edmondsw has quit IRC06:49
*** slaweq has quit IRC06:56
*** slaweq has joined #openstack-sdks06:58
openstackgerritSlawek Kaplonski proposed openstack/python-openstackclient master: Make max_burst_kbps option as optional for bw limit QoS rule  https://review.openstack.org/56782807:11
*** gtema has joined #openstack-sdks07:42
*** jpena|off is now known as jpena07:50
*** dims has quit IRC07:59
*** dims has joined #openstack-sdks08:02
*** dims has quit IRC08:07
*** dims has joined #openstack-sdks08:07
*** edmondsw has joined #openstack-sdks08:20
*** edmondsw has quit IRC08:25
*** gkadam has joined #openstack-sdks08:53
*** cdent has joined #openstack-sdks08:57
fricklerOSC cores: in addition to slaweq's request, another reminder for these patches waiting in queue for a long time already: https://review.openstack.org/516701 https://review.openstack.org/558824 https://review.openstack.org/540633 https://review.openstack.org/55974209:23
fricklerdtroyer: amotoki: dhellmann: ^^09:26
*** gildub has joined #openstack-sdks09:33
*** dtantsur has joined #openstack-sdks09:43
openstackgerritSlawek Kaplonski proposed openstack/python-openstackclient master: Make max_burst_kbps option as optional for bw limit QoS rule  https://review.openstack.org/56782809:48
*** shu-mutou has quit IRC10:05
*** edmondsw has joined #openstack-sdks10:08
*** edmondsw has quit IRC10:13
*** corvus has quit IRC10:29
*** corvus has joined #openstack-sdks10:30
*** e0ne has joined #openstack-sdks10:39
*** dave-mccowan has joined #openstack-sdks10:41
*** chenyb4 has quit IRC10:47
*** christi3k has quit IRC10:49
*** christi3k has joined #openstack-sdks10:51
*** cdent has quit IRC10:51
*** pooja_jadhav has quit IRC10:54
*** Shilpa has quit IRC11:22
*** cdent has joined #openstack-sdks11:33
*** gkadam has quit IRC11:43
*** gkadam has joined #openstack-sdks11:43
*** jpena is now known as jpena|lunch11:47
*** pooja_jadhav has joined #openstack-sdks11:54
*** edmondsw has joined #openstack-sdks11:57
*** edmondsw has quit IRC12:01
*** edmondsw has joined #openstack-sdks12:07
*** edmondsw has quit IRC12:07
*** edmondsw has joined #openstack-sdks12:36
dtantsurfolks, who knows what magic to add to keystoneauth Session to make it work with custom SSL certificates that are installed in the system?12:42
mordreddtantsur: morning!12:43
dtantsurI know that openstackclient, ironicclient, etc work correctly, but a naively written application does not..12:43
dtantsurhey mordred12:43
*** jpena|lunch is now known as jpena12:43
dtantsur(it's not impossible that tripleo sets up certificates in a weird way, but everything seems okay with it...)12:44
mordreddtantsur: verify and cert are the arguments you want12:44
dtantsurmordred: okay, but what do I set them to?12:44
mordreddtantsur: they take weird argugments - one sec12:44
mordreddtantsur: https://docs.openstack.org/keystoneauth/latest/api/keystoneauth1.html#keystoneauth1.session.Session12:45
mordreddtantsur: verify takes either True or False of the path to a CA bundle12:45
mordreddtantsur: cert is either a path to a single bundle with cert and key, or a tuple of (cert_path, key_path)12:46
dtantsurhere's a funny thing: just using plain requests works, various CLI work, my application trying to pass a Session to ironicclient does not...12:46
dtantsurso yeah, I know this one, but I don't know where to get these parameters.. should they come from CLI args?12:47
dtantsurlike where in https://review.openstack.org/#/c/568688/8/metalsmith/_cmd.py do I wire them in?12:47
mordreddtantsur: oh - you're using openstack.config (good) - one sec12:47
dtantsurmordred: I tried before that patch (with directly using Session) and with it12:48
dtantsurmaybe ditching ironicclient will help, dunno :)12:48
mordredit will - but let's get this one working :)12:48
dtantsuranother fun fact: it Works On DevStack (tm)12:49
mordred:)12:50
mordreddtantsur: it should be --os-cacert --os-cert and --os-key12:51
dtantsurokay, but what I don't understand is *why* I have to provide it12:52
dtantsurthe certificate is (apparently) in the system store, and devstack is fine with what I'm doing12:52
* dtantsur blames black magic12:52
mordredI would say that it's not actually in the system store12:52
mordredby default with no parameters everything should be set up to look for CAs in the system store12:53
dtantsurright, so I wonder what magic tripleo uses12:53
mordredit's possible this is related to the weirdness with requests bundling certs12:55
dtantsurmmm, possibly. because I try metalsmith from a venv12:55
mordredI can't remember all the details, but in some combos the 'system' ca bundles requests looks for are not the system ca bundles but are instead the built-in vendored ca bundles in requests12:55
mordredbecause *bong*12:56
dtantsurAHA!12:56
dtantsurI tried 'ironic' command from venv and it failed as well12:56
mordredyah. it'll be the vendored bundles thing12:56
mordredI always just put a path to the ca into my clouds.yaml just to be sure12:56
dtantsurso I guess no one is to blame, I'm just unlucky?12:56
mordredwell - I think someone is to blame ... but blaming them won't fix anything :)12:56
dtantsurfair12:57
mordredapparently "finding system installed ca bundles on all platforms is too hard"12:57
dtantsurokay, I guess my patch is ready to land (unless you want to review it)12:57
mordredneat - lemme look just for fun12:57
mordredyour commit message makes me happy12:58
dtantsurfollow-up patches will make you even happier :)12:58
*** fabian_ has joined #openstack-sdks12:59
dtantsurmordred: something in openstacksdk confuses me. we have connection.update_machine (for example) and connection.baremetal.update_node13:02
mordreddtantsur: those all look great! there's a stack of sdk patches that should put us in a good position to support what you need from microversions for ironic13:02
dtantsurwhich one is supposed to be used? connection.baremetal does not have many eatures13:02
dtantsurcool!13:02
mordreddtantsur: yah - so - connection.update_machine is the shade method that TheJulia wrote13:02
mordredconnection.baremetal is the support for ironic that was in the sdk before the merge - we still have work to do to merge the two of them into a single piece of sanity13:03
dtantsurmordred: and which one will be the final?13:04
mordredbut if there is a method directly on connection, I'd use it - it'll be the same code that backs up the ansible modules/bifrost13:04
dtantsursure, it's just confusing. and I don't want to start relying on something that is pending deprecation13:04
*** chenyb4 has joined #openstack-sdks13:05
mordredyah - you should be safe with the connection methods- they're not going to get deprecated13:05
mordredit'll either remain different or might become just a convenience wrapper around connection.baremetal - but it won't go away13:06
dtantsurneat! while we're here: are you open to adding things like "deploy_machine", "provide_machine", etc? essentially, convenience wrappers around node_set_provision_state, similar to inspect_machine?13:07
mordredabsolutely!13:07
dtantsurokay, I may be able to carve some time for these13:09
dtantsuractually, I'd rename node_set_provision_state to something, because consistency13:10
mordredwoot!13:10
mordredmachine_set_provision_state perhaps?13:10
mordredor, honestly, I think set_machine_provision_state is more in keeping with the verb_noun pattern13:12
*** fabian_ has quit IRC13:12
*** fabian_ has joined #openstack-sdks13:12
dtantsuryeah, I like the latter more13:17
*** chenyb4 has quit IRC13:21
*** fabian_ is now known as chenyb413:29
* dtantsur fixes tripleo to set OS_CACERT correctly13:30
fricklerdtantsur: mordred: this might be related to the ca issue, but also doesn't have a solution yet https://storyboard.openstack.org/#!/story/175672313:30
dtantsurnot wow13:31
mordredfrickler: that's awesome13:34
mordredfrickler: so - I think that's going to be an issue with parameter exchange between osc and sdk ... and should be a thing we can fix as part of the rework of how the config is processed and objects get created in osc 413:35
*** gtema has quit IRC13:36
fricklermordred: so when should this "osc 4" thing happen? I'm having a hard time already finding reviewers for small patches to current OSC13:42
mordredfrickler: I think it's a thing dtroyer has already started in on  ... are you going to be in vancouver next week?13:44
fricklernope13:45
mordreddarn. well - after I finish debugging this morning's sdk patch, lemme see if I can spot an easier path to fixing yourbug13:46
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor  https://review.openstack.org/55793513:47
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type  https://review.openstack.org/55804913:47
*** bobh has joined #openstack-sdks14:12
*** gtema has joined #openstack-sdks14:20
*** chenyb4 has quit IRC14:39
*** chenyb4 has joined #openstack-sdks14:47
*** chenyb4 has quit IRC15:12
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Collect timing information for API calls  https://review.openstack.org/56887715:16
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add oslo.config option for split-loggers  https://review.openstack.org/56887815:16
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Return the endpoint_override from EndpointData  https://review.openstack.org/49194715:30
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Stop using pbr autodoc  https://review.openstack.org/56888915:56
*** edmondsw has quit IRC16:15
*** gtema has quit IRC16:15
*** edmondsw_ has joined #openstack-sdks16:18
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Allow registering additional plugin loaders  https://review.openstack.org/33312616:20
*** edmondsw_ has quit IRC16:23
*** gkadam has quit IRC16:46
*** dtantsur is now known as dtantsur|afk16:58
*** ralonsoh has joined #openstack-sdks17:04
*** ralonsoh has quit IRC17:04
*** jpena is now known as jpena|off17:11
openstackgerritColleen Murphy proposed openstack/keystoneauth master: [DNM] See if latest pbr is causing our docs problems  https://review.openstack.org/56890917:33
*** bobh has quit IRC18:04
*** bobh has joined #openstack-sdks18:05
*** edmondsw has joined #openstack-sdks18:57
*** gtema has joined #openstack-sdks19:00
*** e0ne has quit IRC19:05
*** e0ne has joined #openstack-sdks19:05
*** gtema has quit IRC19:07
*** dtroyer has quit IRC19:28
*** dtroyer has joined #openstack-sdks19:28
*** slaweq has quit IRC19:52
*** slaweq has joined #openstack-sdks19:52
*** slaweq_ has joined #openstack-sdks19:57
*** slaweq has quit IRC19:57
*** e0ne has quit IRC20:18
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Collect timing information for API calls  https://review.openstack.org/56887720:44
openstackgerritMonty Taylor proposed openstack/keystoneauth master: Add oslo.config option for split-loggers  https://review.openstack.org/56887820:44
openstackgerritMerged openstack/keystoneauth master: Fix logging of encoded headers  https://review.openstack.org/56836520:45
*** cdent has quit IRC20:56
*** slaweq_ has quit IRC20:59
*** slaweq has joined #openstack-sdks20:59
*** dave-mccowan has quit IRC21:32
*** slaweq has quit IRC21:33
*** slaweq has joined #openstack-sdks21:34
*** d0ugal_ has joined #openstack-sdks21:39
*** dave-mccowan has joined #openstack-sdks21:41
*** d0ugal has quit IRC21:41
*** dave-mccowan has quit IRC21:48
*** dave-mcc_ has joined #openstack-sdks21:49
openstackgerritMerged openstack/keystoneauth master: Return the endpoint_override from EndpointData  https://review.openstack.org/49194721:52
*** slaweq has quit IRC22:21
*** bobh has quit IRC22:27
*** bobh has joined #openstack-sdks22:28
*** bobh has quit IRC22:32
*** d0ugal__ has joined #openstack-sdks22:40
*** d0ugal_ has quit IRC22:41
*** edmondsw has quit IRC22:50
*** slaweq has joined #openstack-sdks23:10
*** edmondsw has joined #openstack-sdks23:11
*** slaweq has quit IRC23:15
*** bobh has joined #openstack-sdks23:58

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