Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-08-21
16:29:44 mordred agree
16:30:25 efried mordred cdent My main goal here was to avoid having multiple disparate sources of the official API reference URL.
16:30:26 mordred themost important thing for sta to be the authority on is "this is the service-type name for this project and it used to be called these things"
16:31:33 mordred efried: yup
16:31:42 efried That is, I didn't want sta to be saying "the API reference is <result of some calculation, or this override>" and then api-site to be saying "the API reference is <result of some calculation whose logic might sort of duplicate what's in sta, but might diverge at some point if we forget or aren't careful>"
16:31:53 mordred efried: +1000
16:32:00 mordred efried: we share the same goal :)
16:32:05 efried Sweet.
16:32:06 cdent I’ll see that 1000 and raise 10000
16:32:54 mordred so - I thnk honestly step 1 is to verify whether there are actually any non-compliant api-ref docs
16:33:00 efried mordred Did that.
16:33:09 cdent sta has plenty of examples, yes?
16:33:09 mordred cool! are there?
16:33:18 mordred sta is, I think, out of date
16:33:24 efried mordred The ones that are still in https://review.openstack.org/#/c/495887/ are non-compliant
16:33:46 efried Because I checked the "official" constructed URL for all of those and it's 404.
16:33:48 cdent “out of date” :) some authority ;)
16:34:07 mordred ah. awesome
16:34:39 efried Well, I didn't check the URLs that are in there to make sure they're *not* 404 :)
16:34:48 efried 13 of 'em, I think.
16:37:13 efried They're all good except for https://developer.openstack.org/api-ref/monitoring-events-api/ - which I think we knew was still in the works. (Also, https://git.openstack.org/cgit/openstack/zun/tree/api-ref/source is a source tree.)
16:40:11 efried I can't tell from https://review.openstack.org/#/c/482443/ where those docs are going to land.
16:49:47 mordred efried: - api-ref-jobs:
16:49:49 mordred service: monitoring-events
16:50:06 mordred efried: that's what's in jenkins/jobs/projects.yaml
16:50:26 efried mordred Then we should whack that guy's override from https://review.openstack.org/#/c/495887/
16:51:00 mordred it doens't look like thaturl exists though - so maybe no docs have published since that job definition landed?
16:51:14 efried mordred The above change set hasn't merged yet.
16:51:40 efried Point is, both URLs are 404, but once https://review.openstack.org/#/c/482443/ lands, it'll be the right one, so we should remove the override.
16:52:35 mordred agree
16:52:40 openstackgerrit Eric Fried proposed openstack/service-types-authority master: Generate standard api_reference https://review.openstack.org/495887
16:52:44 efried Cool. ^^
#openstack-sdks - 2017-08-22
00:35:08 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/494775
01:45:39 openstackgerrit liuxiaoyang proposed openstack/python-openstacksdk master: Add doc8 rule and check doc/source files https://review.openstack.org/496058
05:35:19 openstackgerrit Vitalii Kulanov proposed openstack/cliff master: Update doc on Sphinx integration process https://review.openstack.org/493869
08:09:28 openstackgerrit lidong proposed openstack/python-openstacksdk master: Update links in README https://review.openstack.org/496145
08:59:18 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Provide a helper method to get headers from environ https://review.openstack.org/495356
09:45:32 openstackgerrit Shinya Tsunematsu proposed openstack/python-openstackclient master: enable glance client to support client cert https://review.openstack.org/496178
09:55:10 openstackgerrit Shinya Tsunematsu proposed openstack/python-openstackclient master: enable glance client to support client cert https://review.openstack.org/496178
10:17:26 openstackgerrit Stephen Finucane proposed openstack/cliff master: sphinxext: Correct issues with usage formatting https://review.openstack.org/496189
11:34:11 openstackgerrit Merged openstack/python-openstacksdk master: Update links in README https://review.openstack.org/496145
11:53:35 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Import functions from placement https://review.openstack.org/496212
11:53:44 cdent elmiko: ^
12:32:21 elmiko cdent: ack
16:33:30 openstackgerrit Merged openstack/cliff master: sphinxext: Correct issues with usage formatting https://review.openstack.org/496189
17:08:31 openstackgerrit Harry Rybacki proposed openstack/python-openstackclient master: Implied Roles https://review.openstack.org/290253
17:53:21 openstackgerrit Akihiro Motoki proposed openstack/cliff master: Fix regexp for detecting long options https://review.openstack.org/486562
17:53:41 amotoki stephenfin: stevemar: more cliff regexp fix ^^
18:20:58 openstackgerrit Vitalii Kulanov proposed openstack/cliff master: Update doc on Sphinx integration process https://review.openstack.org/493869
19:24:45 cdent efried, mordred : you two may have input on https://review.openstack.org/#/c/496321/ (governance change fixing api-doc links)
19:25:32 efried cdent Looking...
19:31:58 efried cdent Thanks for the heads up. Not sure how that yaml is used, but I assume it's parsed/converted somewhere. And that somewhere ought to use sta to populate those api links.
19:32:25 efried servce-types-authority should be authoritative; swhy it's called 'authority'.
19:32:48 cdent well, yes and no. Originally it was authoritative service types. And that’s it.
19:33:00 cdent It’s only recently that it arrogated authority for lots of other stuff
19:33:10 cdent which is probably good and just, but shouldn’t be assumed (yet)
19:39:56 efried fair enough
20:45:04 cdent edleafe: i’m working on your comments on https://review.openstack.org/#/c/496212/1/microversion_parse/tests/test_extract_version.py,unified and I’m not sure how to make you happy
20:54:17 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Import functions from placement https://review.openstack.org/496212
20:54:17 openstackgerrit Chris Dent proposed openstack/microversion-parse master: Provide a helper method to get headers from environ https://review.openstack.org/495356
21:08:42 edleafe Nothing makes me happy!
21:11:04 edleafe cdent: it's things like 'test_version_is_tuple' that checks that it's equal to '1.5'
21:12:00 edleafe Maybe rename the class 'TestVersion1_5' or something :)
21:12:28 cdent I’m still not grokking what you want instead
21:13:30 edleafe it's the "action at a distance" feeling
21:13:45 edleafe looking at that test, why does it equal 1.5?
21:13:54 cdent cuz magic
21:14:02 edleafe why not 1.6? or 4.2?
21:14:05 cdent cuz unit tests that prove that python works are stupid
21:14:23 edleafe agreed
21:15:52 cdent I went ahead and pushed a new version a few minutes ago without addressing the tests, because I couldn’t think of anything else to do
21:15:53 edleafe I used to write my unit tests so that they always used random values. So every time the tests were run, the actual values would be different
21:16:04 edleafe I wouldn't waste time on it
21:16:14 edleafe It's just one of my pet peeves
21:16:22 edleafe like visual indentation
21:16:26 cdent ONE OF MANY
21:17:01 cdent having opinions is far more intresting than having none
21:18:13 cdent edleafe: just wrote to os-dev about whether to move more stuff in microversion-parse
21:18:22 cdent I think that’s my last act of the day.
21:19:13 edleafe cdent: BTW, I have a doc appt during the API SIG meeting on Thursday, so you get to assign me everything
21:19:25 cdent \o/
21:55:01 openstackgerrit Michael Johnson proposed openstack/python-openstacksdk master: Fix octavia l7rules https://review.openstack.org/496424
22:19:58 openstackgerrit Merged openstack/osc-lib master: Updated from global requirements https://review.openstack.org/494869
22:27:43 openstackgerrit Merged openstack/cliff master: Move comments up in [extras] section of setup.cfg https://review.openstack.org/495460
22:31:42 openstackgerrit Merged openstack/cliff master: Update doc on Sphinx integration process https://review.openstack.org/493869
22:32:39 openstackgerrit Rajath Agasthya proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/494775
#openstack-sdks - 2017-08-23
01:33:00 openstackgerrit liuxiaoyang proposed openstack/python-openstacksdk master: Add tox.ini file. https://review.openstack.org/496058
01:37:30 openstackgerrit lidong proposed openstack/python-openstacksdk master: Fix some typos https://review.openstack.org/496459
01:41:26 openstackgerrit liuxiaoyang proposed openstack/python-openstacksdk master: Add tox.ini file. https://review.openstack.org/496058
02:04:17 openstackgerrit liuxiaoyang proposed openstack/python-openstacksdk master: Add doc8 rule and check doc/source files https://review.openstack.org/496058
03:30:20 openstackgerrit lihaijing proposed openstack/python-openstackclient master: Update image cli doc and fix some typos https://review.openstack.org/496474
04:41:24 openstackgerrit Praveen N proposed openstack/python-openstackclient master: remove the wrong argument https://review.openstack.org/495155
05:03:55 openstackgerrit lihaijing proposed openstack/python-openstackclient master: Fix "openstack image unset" command's help message typo https://review.openstack.org/496486
05:29:43 openstackgerrit Merged openstack/python-openstacksdk master: Fix some typos https://review.openstack.org/496459
06:01:35 openstackgerrit lihaijing proposed openstack/python-openstackclient master: Update image cli doc and fix some typos https://review.openstack.org/496474
06:10:53 openstackgerrit ShangXiao proposed openstack/cliff master: Fix typos and replace http with https for doc links in cliff https://review.openstack.org/495477
09:53:04 openstackgerrit Merged openstack/cliff master: Updated from global requirements https://review.openstack.org/494775
09:53:11 openstackgerrit Merged openstack/cliff master: Update and replace http with https for doc links https://review.openstack.org/485920
09:53:16 openstackgerrit Merged openstack/cliff master: Fix regexp for detecting long options https://review.openstack.org/486562
11:28:32 openstackgerrit Erik Berg proposed openstack/python-openstackclient master: new subcommand for hypervisor: evacuate https://review.openstack.org/490015

Earlier   Later