| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-03 | |||
| 09:36:27 | cmurphy | some authoritative unopinionated guide would be really useful | |
| 10:20:32 | adriant | cmurphy: ++ to something like that. | |
| 10:21:43 | dtantsur | edleafe: what's the problem with previewing? won't 'tox -edocs' work for you locally? | |
| 10:22:00 | dtantsur | mordred: thanks! oh, I should put openstacksdk on my review dashboard? :D | |
| 10:25:13 | adriant | cmurphy: we can use keystoneauth and os-client-config as the reference implementation in such a doc, but the goal ultimately is language agnostic thing that describes the auth processes, and conventions in our official openstack projects. and mrhillsman we can potentially use that doc as the checklist as part of any process to make a library offici | |
| 10:25:13 | adriant | al or one we as a community recommend. | |
| 10:26:54 | adriant | do we maybe want to start an email thread about it in openstack-dev and maybe get the right people involved to have such a thing written as part of Rocky? | |
| 10:35:11 | cmurphy | adriant: an email thread sounds like a good place to start | |
| 12:56:46 | edleafe | dtantsur|brb: I get errors when I run tox -edocs locally: http://paste.openstack.org/show/718285/ | |
| 12:57:15 | edleafe | dtantsur|brb: that's not a page I've changed. | |
| 12:59:39 | edleafe | dtantsur|brb: Ah, I see it's changed in master. | |
| 13:00:48 | edleafe | dtantsur|brb: now it looks like it runs, but I still get this at the end: TypeError: write() argument must be str, not bytes | |
| 13:33:51 | mrhillsman | just seeing this adriant will read back through logs in a few | |
| 14:00:06 | dtantsur | edleafe: you must be having the same problem as me: the default python is 3, but our docs (or rather one of the deps) are not compatible with it | |
| 14:01:59 | edleafe | dtantsur: ick | |
| 14:03:56 | openstackgerrit | Dmitry Tantsur proposed openstack/api-wg master: Run 'tox -edocs' with python 2 for now https://review.openstack.org/558519 | |
| 14:03:57 | dtantsur | edleafe: ^^^ | |
| 14:05:48 | edleafe | dtantsur: ok, will review. Thanks | |
| 14:12:45 | cdent | dtantsur, edleafe: looks like yasfb hasn't worked since sep 2017: http://specs.openstack.org/openstack/api-wg/rss.xml maybe we just kill it? | |
| 14:13:19 | dtantsur | cdent: maybe it does not publish updates for guidelines? | |
| 14:13:34 | dtantsur | I'm fine with killing it, given that we anyway announce stuff on the ML | |
| 14:13:49 | cdent | presumably guideline feed would have been the entire point of having a feed? | |
| 14:14:00 | dtantsur | true :) | |
| 14:24:32 | mordred | dtantsur: yes please (to putting openstacksdk on your watch list) | |
| 14:25:06 | mordred | adriant: I think a spec/doc about envvar variables and clouds.yaml is LONG overdue | |
| 14:33:28 | cdent | dtantsur: dhellmann is doing a new release yasfb that ought to fix the bytes problem | |
| 14:33:38 | dtantsur | \o/ | |
| 14:33:40 | cdent | but we'll need to do some digging to find how to get it actually feed the guidelines | |
| 14:49:03 | cdent | dtantsur: https://github.com/sphinx-contrib/yasfb/pull/4 | |
| 14:51:34 | dtantsur | cool! | |
| 15:54:55 | kmalloc | adriant: i'm the wrong person to write an unopinionated doc (ftr), i think we should be far far far more opinionated across all of openstack. | |
| 15:55:17 | kmalloc | adriant: but i'll totally support and review (unbiased) such a doc if someone is writing/working on it. | |
| 16:06:22 | cdent | opinions++ | |
| 16:20:17 | edleafe | cdent: want to push this through quickly? https://review.openstack.org/#/c/558519/ | |
| 16:21:03 | cdent | edleafe: it's not needed. the new release of yasfb that dhellmann and I worked out today fixes it | |
| 16:21:11 | cdent | do a tox -r -edocs and see if it works? | |
| 16:21:21 | cdent | (we don't upper constraints so it ought to) | |
| 16:21:21 | edleafe | cdent: oh, it's already released? | |
| 16:21:24 | cdent | yes | |
| 16:21:28 | cdent | two new releases today | |
| 16:21:36 | edleafe | Ah, that's better! | |
| 16:22:26 | cdent | it also means the guidelines will show up in the generated feeds, whereas they weren't before | |
| 16:23:57 | edleafe | ok, verified that it worked | |
| 16:24:27 | cdent | woot | |
| 16:58:10 | dhellmann | yay! | |
| 22:17:57 | adriant | kmalloc: I doubt it will ever truly be unopinionated, because the way our auth works itself works is fairly opinionated, and how we use it in keystoneauth1 and how the environment values auth values are read in via os-client-config. | |
| 22:24:17 | kmalloc | that's fair, but i think we need to be way more opinionated. but that is my view | |
| 22:25:11 | adriant | kmalloc: I'm ok with opinionated. In fact I kind of prefer if for something like this. If you don't follow the spec and make your library work how we expect it... then your library is wrong. | |
| 22:25:42 | adriant | Auth and auth values should work the same across all the libraries/languages you use. | |
| 22:26:19 | adriant | Just because I'm writing something in golang to talk to openstack doesn't suddenly change how the API interacts, or how others normally use it. | |
| 22:28:57 | cmurphy | all I meant by unopinionated was "not python" | |
| 22:48:26 | adriant | cmurphy: :P | |
| 22:50:12 | openstackgerrit | Sidharth Surana proposed openstack/openstacksdk master: Do not use the next link for pagination https://review.openstack.org/558461 | |
| 23:27:38 | openstackgerrit | Duc Truong proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |
| #openstack-sdks - 2018-04-04 | |||
| 00:08:10 | openstackgerrit | Duc Truong proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |
| 00:12:50 | ssurana | Hey guys! can I get some reviews for https://review.openstack.org/#/c/558461/2 Thanks! | |
| 00:23:15 | openstackgerrit | Sidharth Surana proposed openstack/openstacksdk master: Do not use the next link for pagination https://review.openstack.org/558461 | |
| 02:15:23 | openstackgerrit | chenyb4 proposed openstack/openstacksdk master: pypy is no longer supported by oslo libraries https://review.openstack.org/558668 | |
| 04:12:11 | openstackgerrit | chenyb4 proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |
| 06:40:21 | eandersson | Anyone got time to take a look at this fix for Heat? https://review.openstack.org/#/c/558710/ | |
| 06:41:02 | eandersson | Qiming_, ^ | |
| 06:45:05 | eandersson | The above could also be a bug in the openstacksdk ^ dunno | |
| 08:56:27 | openstackgerrit | chenyb4 proposed openstack/openstacksdk master: Fix resource not exist the resource.status error https://review.openstack.org/558757 | |
| 10:58:33 | openstackgerrit | Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add dns-domain support to Network object https://review.openstack.org/516701 | |
| 13:43:26 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add support for the DNS domain and name attributes for floating IPs https://review.openstack.org/558820 | |
| 13:49:50 | openstackgerrit | Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add dns-domain support to Network object https://review.openstack.org/516701 | |
| 13:49:50 | openstackgerrit | Jens Harbott (frickler) proposed openstack/python-openstackclient master: WIP: Add DNS support to floating IP commands https://review.openstack.org/558824 | |
| 16:07:56 | kuzko | hum, anyone here doing block-device-mappings on the openstack cli? I'm having trouble, even though I use a block-device-mapping with boot_index=0, the CLI is this asking for this openstack server create: error: one of the arguments --image --volume is required | |
| 16:08:48 | kuzko | the whole point of having a boot_index to 0 is to set this as the boot volume | |
| 17:47:22 | openstackgerrit | Sidharth Surana proposed openstack/openstacksdk master: Strip the version prefix from the next link for pagination https://review.openstack.org/558461 | |
| 18:55:14 | frickler | kuzko: can you show the exact command that you are trying to use? it doesn't look to me like specifying the boot_index would be supported in the cli | |
| 19:59:18 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910 | |
| #openstack-sdks - 2018-04-05 | |||
| 11:14:11 | dtantsur | edleafe: hey, did you get any progress with https://review.openstack.org/#/c/554234/ ? I can help today, I guess | |
| 12:35:51 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: Add support for DNS attributes for floating IPs https://review.openstack.org/558820 | |
| 12:42:16 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add functional tests for Neutron DNS extension https://review.openstack.org/559068 | |
| 12:54:36 | edleafe | dtantsur: no, sorry, I've been buried with nova work | |
| 12:55:01 | dtantsur | edleafe: should I take a look? | |
| 13:06:29 | edleafe | dtantsur: if you have the cycles. | |
| 13:06:35 | dtantsur | yup | |
| 13:07:07 | edleafe | dtantsur: I can build the docs, but I don't have an easy way to view them. My dev machine is a remote Linux vm | |
| 13:07:14 | dtantsur | ah | |
| 13:07:19 | dtantsur | I have linux all around me :) | |
| 13:07:26 | dtantsur | incl. my wife's laptop :) | |
| 13:07:39 | dtantsur | edleafe: btw, did we have a new yasfb release? | |
| 13:07:41 | edleafe | me too. Local machines at least have a GUI | |
| 13:09:37 | dtantsur | edleafe: also, how did you overcome SEVERE: Title level inconsistent? | |
| 13:09:43 | dtantsur | I guess you did get past it? | |
| 13:12:08 | edleafe | dtantsur: I think it was fixed in master. | |
| 13:12:45 | dtantsur | lemme try rebasing your patch | |
| 13:21:59 | cdent | dtantsur: yes there's been a new yasfb release | |
| 13:22:04 | dtantsur | ack | |
| 13:24:55 | dtantsur | edleafe: btw, inserting a reference is the beginning of a document to use it with :ref: is kinda antipattern. any document can be references as a whole with :doc: | |
| 13:28:56 | edleafe | dtantsur: ok, good to know | |
| 13:39:04 | openstackgerrit | Dmitry Tantsur proposed openstack/api-wg master: Break up the HTTP guideline into smaller documents https://review.openstack.org/554234 | |
| 13:39:05 | dtantsur | edleafe: wdyt ^^^? | |
| 13:49:02 | edleafe | dtantsur: looks good | |
| 13:49:44 | edleafe | dtantsur: on the main guidelines page, it shows both the HTTP Header guidelines as well as the HTTP Guidelines links | |
| 13:51:41 | dtantsur | edleafe: yep, because they were there previously | |
| 13:51:55 | dtantsur | I can move them, but the links people may have bookmarked will go away | |
| 13:52:23 | edleafe | ah. Don't break stuff. | |