| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-01-18 | |||
| 08:37:09 | RuiChen | https://www.irccloud.com/pastebin/xFXl6TFa/ | |
| 08:37:09 | RuiChen | https://www.irccloud.com/pastebin/H5yOdoYA/ | |
| 08:39:49 | reedip | Ok, now it worked ! | |
| 08:40:33 | RuiChen | \o/ | |
| 08:41:14 | RuiChen | thank you mention the quota set case, that make the patch better | |
| 08:42:01 | reedip | RuiChen but what about the Default Quota now | |
| 08:42:09 | reedip | I need to remove the ID from URI | |
| 08:42:11 | reedip | :) | |
| 08:43:41 | RuiChen | in default quota case, project_id is not the resource id, it's attribute | |
| 08:43:59 | RuiChen | the resource base_path is base_path = '/quotas/%(project)s/default' | |
| 08:44:17 | reedip | So how would that work ? | |
| 08:44:45 | RuiChen | uri = self.base_path % self._uri.attributes in resource class | |
| 08:44:54 | RuiChen | in _prepare_request() | |
| 08:46:31 | reedip | so you are stating that we do not need to change anything for DefaultQuotas?> | |
| 08:47:48 | RuiChen | only to change the def get_quota-default(), pass id and requires_id=False | |
| 08:48:23 | reedip | yes, thats what I was thinking | |
| 08:48:36 | reedip | so I do not need to change much for the defaultquotas patch | |
| 08:48:42 | reedip | from the defaultquotas class | |
| 08:50:02 | openstackgerrit | Reedip proposed openstack/python-openstacksdk: Fix Setting Quotas in Neutron https://review.openstack.org/419911 | |
| 08:50:19 | reedip | RuiChen vv | |
| 08:53:00 | reedip | RuiChen vv | |
| 08:53:16 | openstackgerrit | Reedip proposed openstack/python-openstacksdk: Add project ID in QuotaDefault requests https://review.openstack.org/420301 | |
| 08:54:37 | openstackgerrit | Reedip proposed openstack/python-openstacksdk: Add project ID in QuotaDefault requests https://review.openstack.org/420301 | |
| 08:57:12 | RuiChen | reedip: I had added my +1 for the two patches, thank you | |
| 08:58:14 | reedip | Thanks RuiChen :) | |
| 11:37:56 | openstackgerrit | Chris Dent proposed openstack/api-wg: [WIP] Refactor and re-validate api change guidelines https://review.openstack.org/421846 | |
| 15:48:16 | mordred | dtroyer, stevemar: did we release occ already? | |
| 15:49:09 | mordred | ah - looks like we have | |
| 15:49:16 | dtroyer | https://review.openstack.org/#/c/421469/ yes | |
| 15:49:25 | dtroyer | need another? | |
| 15:49:41 | mordred | well - we got this submitted: https://review.openstack.org/#/c/421668/ which seems both easy and friendly - but also not _super_ important | |
| 15:51:28 | dtroyer | +A, the deadline is tomorrow if we want that in stable | |
| 15:51:30 | dtroyer | wait | |
| 15:51:36 | dtroyer | o-c-c- doesn't do stable | |
| 15:52:09 | dtroyer | so its just a matter of when it would be released, today/tomorrow or next month | |
| 15:54:42 | mordred | oh - neat. cool - no rush then | |
| 15:59:26 | openstackgerrit | Dean Troyer proposed openstack/golang-client: Revert "Add crud operation of volume API v3" https://review.openstack.org/422055 | |
| 16:01:12 | ankur-gupta-f2 | dtroyer: similar note. Does OSC do a feature freeze? | |
| 16:02:59 | dtroyer | ankur-gupta-f: yes, next week. we will have one more release before then, early next week | |
| 16:03:24 | dtroyer | osc-lib freezes tomorrow, its to-become-stable-brans is the current release | |
| 16:03:33 | dtroyer | s/brans/branch/ | |
| 16:03:52 | stevemar | dtroyer: we'll have to get more strict about feature freeze and such soon | |
| 16:04:26 | dtroyer | stevemar: soon == now for the new bits (volume v3, etc). | |
| 16:08:45 | rosmaita | cdent: would be interested in your opinion of the discussion happening on https://review.openstack.org/#/c/420038/ , especially in light of your email this morning about the new tag proposal | |
| 16:09:02 | rosmaita | elmiko: ^^ etoews: ^^ | |
| 16:09:25 | ankur-gupta-f2 | dtroyer: Okay. I have told our team that any reviews coming in on any of these https://etherpad.openstack.org/p/osc-transition-priorities to be addressed ASAP. Thanks | |
| 16:09:30 | cdent | rosmaita: thanks, will look. I'm experiencing "stability" fatigue | |
| 16:09:52 | rosmaita | cdent: ha! well, i will warn you in advance, that this will not help :) | |
| 16:11:38 | elmiko | rosmaita: i'll try to take a look, i'm putting out fires at the moment :/ | |
| 16:11:50 | dtroyer | ankur-gupta-f: at this point new commands (not already in-flight) are probably doubtful | |
| 16:13:00 | rosmaita | elmiko: best wishes for your firefighting! | |
| 16:13:00 | ankur-gupta-f2 | dtroyer: most have been in flight and are nearing completion. Just waiting for the final few reviews | |
| 16:13:41 | cdent | dtroyer: yeah, I have labeling fatigue too | |
| 16:13:56 | elmiko | rosmaita: cheers! | |
| 16:15:39 | cdent | dtroyer: I tend to think of consistency on the spatial (amongst apis) dimension and some term (maybe stability) is need for the time (within an api) dimension | |
| 16:17:48 | dtroyer | cdent: they both make me tired and make me wonder why in the world this is all so hard | |
| 16:21:41 | cdent | my eyes are fuzzy enough that I ready dtroyer as saying "makes me wonder why I'm in this world" | |
| 16:22:42 | openstackgerrit | Dean Troyer proposed openstack/golang-client: Fix unit tests https://review.openstack.org/422080 | |
| 16:23:20 | cdent | I've withheld my opinion on this topic because at least for the time being it is probably too poorly informed to be useful, but yeah it all makes me pretty tired | |
| 16:26:26 | dtroyer | here is mine: one place that a BDFL is useful is in keeping at least some of the cats in the same order everywhere. I've had to say no so many times to OSC command stuff just to be as close to consistent as we are (still a ways to go). There is no way in h3ll that we'd have accomplished that with teams doing their own thing, even if they agreed on the idea. Even now, many of the OSC plugins slide past the guidelines (I d | |
| 16:26:36 | dtroyer | That is just the CLI, and "CLI's are easy" | |
| 16:27:46 | dtroyer | the only way we will get a clean, consistent REST API for OpenStack is to have a single unit (team, working group, whatever) build it <cough>oaktree</cough> | |
| 16:28:08 | dtroyer | </rant> | |
| 16:28:49 | cdent | I can get behind that (modulo corporate enterprisey nonsense) | |
| 16:30:05 | dtroyer | I think it would take a corporate-type sponsor without an API to defend/enhance to sponsor it to ever get anywhere | |
| 16:30:29 | dtroyer | and those are hard to find that also have $$$ to spend "for the good of OpenStack users" | |
| 16:30:51 | dtroyer | dang, Wednesday isn't supposed to be "grumpy day" | |
| 16:31:18 | openstackgerrit | zhiyong.dai proposed openstack/python-openstackclient: Support "--no-property" option in volume snapshot set https://review.openstack.org/416182 | |
| 16:45:53 | openstackgerrit | Shashank Kumar Shankar proposed openstack/python-openstackclient: OSC Network Flavor https://review.openstack.org/382530 | |
| 16:45:54 | openstackgerrit | Shashank Kumar Shankar proposed openstack/python-openstackclient: Introduce neutron flavor associate, disassociate to OSC https://review.openstack.org/403907 | |
| 16:52:51 | openstackgerrit | Dean Troyer proposed openstack/golang-client: Fix unit tests https://review.openstack.org/422080 | |
| 17:08:25 | openstackgerrit | Shashank Kumar Shankar proposed openstack/python-openstackclient: OSC Network Flavor https://review.openstack.org/382530 | |
| 17:31:03 | openstackgerrit | Merged openstack/golang-client: Fix unit tests https://review.openstack.org/422080 | |
| 17:32:53 | openstackgerrit | Merged openstack/os-client-config: Add support for Murano https://review.openstack.org/421668 | |
| 17:34:13 | stevemar | dtroyer: are we waiting for anything in particular before releasing 3.8.0? | |
| 17:34:23 | stevemar | oh i guess we have til next week | |
| 17:34:25 | stevemar | ignore me | |
| 17:35:53 | dtroyer | stevemar: even so, the only release blocker is the broken quota set command, once that is covered the rest is nice to have | |
| 17:36:24 | dtroyer | that gets us in a position for SDK 0.9.13 to be released when it is ready and not be rushed | |
| 17:36:42 | dtroyer | (not that it would be, it just doesn't add any pressure) | |
| 17:37:54 | stevemar | dtroyer: mm, ok, the quota set command is fixed though? | |
| 17:38:14 | stevemar | dtroyer: why not release it sooner and give andrey a chance to review it? we can let the dust settle a bit | |
| 17:38:20 | dtroyer | maybe? I saw conversation in the overnight scrollback that I haven't followed up on yet… I'll be back to OSc this afternoon | |
| 17:38:24 | stevemar | (random thoughts, you can tell me to buzz off) | |
| 17:38:45 | dtroyer | then we'll release twice more | |
| 17:39:02 | dtroyer | I want SDK 0.9.12 to be what we use for stable | |
| 17:39:41 | openstackgerrit | Sindhu Devale proposed openstack/python-openstackclient: OSC Extension Show https://review.openstack.org/382023 | |
| 17:45:43 | openstackgerrit | Sindhu Devale proposed openstack/python-openstackclient: OSC Extension Show https://review.openstack.org/382023 | |
| 18:49:13 | cdent | elmiko, edleafe, etoews : reminder that I won't be around tomorrow to run api-wg meeting. I added the compatibility/stability thing to the agenda, just as a marker | |
| 18:53:06 | edleafe | cdent: cool, I can handle running the meeting | |
| 18:53:31 | cdent | sweet^wsolid^wthanks | |
| 19:10:56 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: DNM: OSC+SDK check https://review.openstack.org/422173 | |
| 19:33:10 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Fix functional test for creating subnet https://review.openstack.org/421310 | |
| 19:35:51 | openstackgerrit | Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585 | |
| 19:37:22 | openstackgerrit | Sindhu Devale proposed openstack/python-openstackclient: OSC Extension Show https://review.openstack.org/382023 | |
| 19:58:23 | openstackgerrit | Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585 | |
| 20:06:48 | openstackgerrit | OpenStack Release Bot proposed openstack/keystoneauth: Update reno for stable/ocata https://review.openstack.org/422208 | |
| 20:17:29 | briancurtin | stevemar: if (when, rather) sdk raised DeprecationWarning when certain calls were used, would you catch them in any tests? | |
| 20:17:41 | briancurtin | dtroyer: ^ | |
| 20:18:50 | dtroyer | briancurtin: I don't see that we catch that exception specifically anywhere | |