Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-03-01
21:20:29 openstackgerrit Nakul Dahiwade proposed openstack/python-openstackclient master: Jsonify meter and meter rule functional tests https://review.openstack.org/439854
21:41:40 openstackgerrit Merged openstack/osc-lib master: Updated from global requirements https://review.openstack.org/439332
#openstack-sdks - 2017-03-02
01:20:42 openstackgerrit Merged openstack/python-openstacksdk master: Update devstack config to point to a valid image https://review.openstack.org/439847
02:18:53 openstackgerrit Rodrigo Duarte proposed openstack/python-openstackclient master: Use *_as_ids instead *_as_list https://review.openstack.org/435972
02:26:11 openstackgerrit Rodrigo Duarte proposed openstack/python-openstackclient master: Use *_as_ids instead *_as_list https://review.openstack.org/435972
03:09:21 openstackgerrit Ankur proposed openstack/python-openstackclient master: Nit: Trivial doc formatting fix for network flavor https://review.openstack.org/439965
04:24:44 reedip_1 ankur-gupta-f1 : 0/
05:06:53 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/440102
05:06:56 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/440103
05:07:30 ankur-gupta-f4 reedip_: hey buddy
05:10:51 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/440033
05:13:38 openstackgerrit OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/439317
05:20:08 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/440084
05:22:00 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/440102
05:22:03 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/440103
05:25:51 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/440033
05:28:17 openstackgerrit OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/439317
05:34:43 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/440084
05:36:35 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/440102
05:36:38 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/440103
06:10:21 ankur-gupta-f4 reedip_: ping as per FIP
06:10:58 reedip_ fixed-ip needs to be a string, right ? ankur-gupta-f4
06:11:09 reedip_ the handling is done at the neutron-server end
06:11:48 ankur-gupta-f4 double checking
06:13:34 reedip_ https://github.com/openstack/neutron-lib/blob/485928f654e5daa742ec93fb7845516b4ebe99a1/neutron_lib/api/validators.py#L429
06:16:13 ankur-gupta-f4 k just wondering
06:16:33 ankur-gupta-f4 but the unset is a valid point.
06:17:00 reedip_ That is taken care in the UT
06:17:13 reedip_ assertNone does that, I guess
06:19:45 ankur-gupta-f4 opinion. I will remove the 01
06:19:45 ankur-gupta-f4 given the complexity of removing/deleting resources (as seen with how many times the test has been rewritten order wise). I think it instead of 'floating ip unset port' coming in in the cleanUp, it should be done like the the other commands where it is run, a fip or port show is done and confirmed that the attachment no longer exists. But that is just an
06:19:47 ankur-gupta-f4 -1
06:20:01 ankur-gupta-f4 thanks for the answers
06:22:53 ankur-gupta-f4 reedip_: headed to bed. talk to you in morning
06:23:22 reedip_ ankur-gupta-f4 : thanks :)
06:24:39 openstackgerrit jiahui.qiang proposed openstack/python-openstackclient master: Functional test for subnet_pool https://review.openstack.org/440150
09:13:47 openstackgerrit Rico Lin proposed openstack/keystoneauth master: Update test requirement https://review.openstack.org/440236
09:20:13 openstackgerrit Merged openstack/python-openstackclient master: Nit: Trivial doc formatting fix for network flavor https://review.openstack.org/439965
12:23:08 openstackgerrit Huanxuan Ao proposed openstack/python-openstackclient master: Fix NoneType error for volume snapshot create command https://review.openstack.org/440497
13:24:23 openstackgerrit Huanxuan Ao proposed openstack/python-openstackclient master: Remove deprecated "ip floating *" commands https://review.openstack.org/440550
13:30:00 openstackgerrit Merged openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/440103
15:01:59 reedip_1 o/
15:05:20 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient master: Remove deprecated "ip floating *" commands https://review.openstack.org/440550
15:45:00 openstackgerrit Nakul Dahiwade proposed openstack/python-openstackclient master: OSC Network Flavor Profile https://review.openstack.org/396783
15:53:31 openstackgerrit Shashank Kumar Shankar proposed openstack/python-openstackclient master: Introduce Neutron DHCP agent commands to OSC https://review.openstack.org/387611
16:22:23 openstackgerrit Merged openstack/api-wg master: Fix typo in the doc https://review.openstack.org/436865
16:39:08 dims stevemar : dtroyer : does this ring a bell?
16:39:10 dims 2017-03-02 16:31:36.894 | Failed to delete floating_ip with name or ID '0759cfd6-2e42-410b-ab70-7ff81d19562c': 'Proxy' object has no attribute 'session'
16:39:10 dims 2017-03-02 16:31:35.234 | + /opt/stack/new/grenade/projects/70_cinder/resources.sh:destroy:205 : openstack floating ip delete 0759cfd6-2e42-410b-ab70-7ff81d19562c
16:40:12 dims could it be from openstacksdk===0.9.14?
16:44:38 openstackgerrit Nakul Dahiwade proposed openstack/python-openstackclient master: Jsonify meter and meter rule functional tests https://review.openstack.org/439854
16:58:20 reedip_1 dims : this issue was occurring with the older version of devstack
16:58:36 reedip_1 I think there was a new patch which fixed it
16:58:41 reedip_1 dont recall it , sorry
17:01:00 dims reedip_1 : ack thanks
17:10:48 dtroyer dims: yes it is from sdk 0.9.14, fixed in an unreleased OSC
17:10:52 dtroyer time for a release?
17:11:52 openstackgerrit Anindita Das proposed openstack/python-openstackclient master: Jsonify network flavor functional tests https://review.openstack.org/440710
17:16:41 dims ack dtroyer, early next week is fine
17:26:59 ssbarnea hi! Am i the first one that is surprised that I cannot delete resources like networks, routers using wildcards/regex?
17:28:21 reedip_1 ssbarnea : can u elaborate ?
17:28:55 ssbarnea reedip_1: I was hoping that I could do:> router delete pref*
17:29:30 ssbarnea glob or regexp patterns, any would have been ok.
17:30:20 reedip_1 umm, no , you cannot use regex for name/id parameter AFAIK
17:30:42 ssbarnea some applies to instances, networks and routers. Now I don't know if this was not implemented for a special reason or just there was no need for it.
17:31:10 ssbarnea reedip_: if nobody cared about that, I would not mind trying to implement it myself.
17:37:11 reedip_1 ssbarnea : do you think that there is a high requirement for regex ?
17:39:02 ssbarnea reedip_: i don't know, in this case I am only speaking about myself: in order to cleanup remaining resources from broken CI jobs, I do want to use a prefix pattern for removing them. We use a naming convention "job-name-number-" which makes easier to do cleanups.
17:39:29 ssbarnea reedip_: ideally jobs do take care of cleanup themselves, but as you know things can always go wrong.
17:39:59 reedip_1 I know , have experienced it :)
17:40:22 reedip_1 ssbarnea : but I do not think all users follow the same patern
17:40:47 ankur-gupta-f4 ssbarnea: would purge satisfy your requirements? Because that is in works
17:40:51 reedip_1 I mean, not all users use some prefix for naming their resources
17:42:15 ssbarnea reedip_1: indeed, but the idea of being able to use a regex or glob pattern seems live a valid use-case. Now I don't know which limitations we have on job names because if people can use "*" in their names glob will not work. but if the names are close to filename patters blog would sound like a reasonable way.
17:42:18 reedip_1 ankur-gupta-f4 : purge is more like a nuclear warhead for a tenant , it clears all ists resources. I think ssbarnea needs something more fine tuned, like one which removes all routers or networks etc . of a specific tenant
17:43:23 reedip_1 ssbarnea : I agree with you , but then again its not a pretty big requirement , IIUC. And it can be easily scripted at user end if required
17:43:35 ssbarnea indeed, purge would not work, there is a must to be able to specify a pattern. Still, this may be an option of purge itself. I am not sure which is the best place the os-client or the purge utility?
17:43:41 dtroyer ssbarnea: there are a couple of reasons globbing resource names is not implemented, not necessarily in any order: a) not all users feel comfortable allowing that (rm -rf / like fear), even with an option; b) it has not been a priority; c) error handling gets tricky (mostly solved in OSC now with the multiple hand-listed deletes)
17:44:31 dtroyer your closest work-around right now is to list resources using a filter, possibly piping thought more filtering, then piping in to a delete command
17:44:35 dtroyer you know, UNIX-style
17:44:44 ssbarnea dtroyer: maybe you are right, i could easily make a bash script that does this.
17:44:52 ankur-gupta-f4 +1
17:44:58 reedip_1 +1
17:45:03 dtroyer the thing I really want here is the ability to do ith in interactive mode
17:45:18 dtroyer so that takes shell out of the option list
17:45:53 dtroyer also, piping lists of commands into interactive mode is fun and definitely a step forward
18:30:26 ssbarnea dtroyer: I had a good progress writing a cleanup script but hit one problem: network delete may fail due to existing ports being assigned. The bad part is that it seems that not all ports are named so I cannot cleanup them using the naming pattern. Is there a way to remove networks with "cascade"?
18:36:40 dtroyer that's what the purge command is intended to do, but at the project level
18:38:29 ssbarnea dtroyer: that's the problem: at project level is useless because it would wipe everything.
18:39:35 dtroyer not useless, but not what you want...
18:40:10 dtroyer etiher way the same tasks need to be done, whether in a script or inside OSC, walk the resources and delete everything required by Neutron for a specific resource
18:51:18 openstackgerrit Merged openstack/os-client-config master: [Fix gate]Update test requirement https://review.openstack.org/440597
18:52:10 ssbarnea dtroyer: well, i will do it in the end and publish the script in came someone else is interested.
18:59:43 dtroyer ssbarnea: thanks. the logic and decision flow of that would be useful to re-use should it get brought into OSC
19:06:45 openstackgerrit Huanxuan Ao proposed openstack/python-openstackclient master: Fix NoneType error for volume snapshot create command https://review.openstack.org/440497
19:26:46 openstackgerrit Huanxuan Ao proposed openstack/python-openstackclient master: Remove deprecated "ip floating (pool)/ip fixed *" commands https://review.openstack.org/440550
19:54:30 dtroyer FWIW, OSC release note job failures are being fixed in the next reno release (2.1.2?) coming after https://review.openstack.org/#/c/440776/ merges.
19:54:52 dtroyer Please do not blindly recheck wihtout having some belief that the job might pass...
19:57:20 ankur-gupta-f1 I believeeee
20:14:40 openstackgerrit Justin A Wilson proposed openstack/python-openstackclient master: Add support for Cinder API 3.3/3.5 into OSC https://review.openstack.org/421585

Earlier   Later