Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-01-24
14:06:25 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient: Updated from global requirements https://review.openstack.org/424663
14:08:26 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/423277
14:23:03 Dinesh_Bhor briancurtin: Hi, whenever you get time please take a look at it: https://review.openstack.org/#/c/420722/
14:29:06 dtroyer OSC folk, I have a question regarding an option name in https://review.openstack.org/#/c/356219/ (that I should have sorted out a long time ago)… I'd like to change it before release tomorow if we are going to change it…
14:30:32 dtroyer stevemar, huanxuan, reedip, ankur-gupta-f1, RuiChen ^^^^^
15:06:26 openstackgerrit Merged openstack/js-openstack-lib: Typo fix: constistent => consistent https://review.openstack.org/424446
15:07:29 openstackgerrit Artom Lifshitz proposed openstack/python-openstackclient: Use image client for images instead of compute https://review.openstack.org/424688
15:35:45 stevemar briancurtin: apparently trying to get things un-blacklisted in the requirements repo is a PITA
15:35:55 stevemar briancurtin: do you mind releasing a new version of the sdk
15:35:58 stevemar 0.9.13?
15:36:04 stevemar dtroyer: do you agree?
15:36:21 briancurtin stevemar: yeah let me take a look at what we have going on
15:36:37 stevemar briancurtin: ty
15:41:40 stevemar dtroyer: oh my: https://review.openstack.org/#/c/424688/1
15:43:09 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/423277
15:49:44 openstackgerrit Merged openstack/python-openstacksdk: Modified DHCP/Network Resource https://review.openstack.org/423511
15:49:54 openstackgerrit Merged openstack/python-openstacksdk: Updated from global requirements https://review.openstack.org/423686
15:51:10 openstackgerrit Merged openstack/python-openstacksdk: Fix error messages are not displayed correctly https://review.openstack.org/420722
15:52:37 openstackgerrit Merged openstack/python-openstacksdk: Fix Setting Quotas in Neutron https://review.openstack.org/419911
15:53:30 openstackgerrit Merged openstack/python-openstacksdk: Add project ID in QuotaDefault requests https://review.openstack.org/420301
16:17:10 ankur-gupta-f1 dtroyer: You question on that is about the tense of the option names? --allowed-address is the most readable. 'A ports allowed address is <blah>.
16:23:56 dtroyer ankur-gupta-f1: but that is not how we are using it. We use it in a declarative sentence: allow this address on this port
16:39:48 dtroyer stevemar: I'm looking to see what testing we have to cover that change, not sure we have much, if any
16:40:46 stevemar dtroyer: i also think we need a new sdk release, already pinged briancurtin about it -- trying to change the requirements repo is a PITA when something had been blacklisted
16:40:52 ankur-gupta-f1 impartial. Both make sense to me
16:41:16 briancurtin stevemar, dtroyer: i’m putting it together now, just moving things around to get it in order
16:41:57 reedip_ hi
16:44:03 openstackgerrit Merged openstack/python-openstackclient: Functional tests debug support https://review.openstack.org/423154
16:44:47 openstackgerrit Merged openstack/python-openstacksdk: Base for workflow service (mistral) https://review.openstack.org/412079
16:45:13 openstackgerrit Merged openstack/python-openstacksdk: Add workflow service (mistral) https://review.openstack.org/414919
16:46:01 openstackgerrit Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585
16:46:58 openstackgerrit Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585
16:47:59 reedip_ dtroyer : your question is related to naming in https://review.openstack.org/#/c/356219/, right?
16:48:02 reedip_ looking at it
16:48:20 dtroyer reedip: yes
16:51:14 reedip_ dtroyer : when looking at the negation, no-allowed-address-pair makes more sense than no-allow-address-pair
16:51:47 ankur-gupta-f1 +1
16:51:59 dtroyer yes it does. I'm thinking that is secondary here though
16:52:51 reedip_ dtroyer: I understand what you are saying is that the addresses are not yet allowed, then how can it be in the past tense
16:53:20 reedip_ and thats why allow-address makes a bit more sense
16:53:56 openstackgerrit Brian Curtin proposed openstack/python-openstacksdk: Initial docs for bare-metal service https://review.openstack.org/411132
16:54:44 stevemar briancurtin: hehe, dtroyer has skirted the question
16:55:17 stevemar briancurtin: when you're done moving things around, let me know when you're about to release, we can test OSC's functional tests with SDKs master branch now
16:55:20 dtroyer stevemar: which one? I'm juggling 3 things atm :)
16:55:26 stevemar dtroyer: :)
16:55:37 stevemar dtroyer: end-of-release-fun!
16:55:45 stevemar dtroyer: the one about needing a new sdk version
16:55:53 dtroyer whee!!! Plus TC _and_ board meetings this afternoon
16:55:54 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient: Updated from global requirements https://review.openstack.org/423277
16:56:21 dtroyer ya, I think new release is simpler than unblacklisting, even with the risks. question is, how much do we trust our tests? :)
16:56:36 dtroyer and can it all get done in time?
16:57:31 openstackgerrit Artom Lifshitz proposed openstack/python-openstackclient: Use image client for images instead of compute https://review.openstack.org/424688
16:58:26 stevemar dtroyer: its gonna have to!
17:04:10 reedip_ dtroyer : I am more inclined to allowed-address-pair
17:04:54 reedip_ because of negation primarily ( maybe a wrong case )
17:05:26 reedip_ and also maybe because the user already decided to allow the addresses
17:05:27 dtroyer I am looking for consistency, and we have a rule regarding options and negation: add "—no-" to the beginning
17:05:44 dtroyer that overrides English tenses in that regard
17:05:44 reedip_ do we have a rule for the past tense ?
17:06:25 dtroyer I'm not certain it is written, it is in the HIG if so, but out longstanding practice is to make options 'active' or present tesnt: —enable rather than --enabled
17:06:47 dtroyer these are not passive commands
17:06:52 reedip_ makes sense to have active commands
17:07:30 reedip_ I do not have anything against the active command
17:18:34 openstackgerrit Brian Curtin proposed openstack/python-openstacksdk: Add docs for the workflow service https://review.openstack.org/424755
17:39:04 openstackgerrit Merged openstack/python-openstacksdk: Initial docs for bare-metal service https://review.openstack.org/411132
17:44:04 openstackgerrit Brian Curtin proposed openstack/python-openstacksdk: Add docs for the workflow service https://review.openstack.org/424755
18:27:50 openstackgerrit Merged openstack/python-openstacksdk: Add docs for the workflow service https://review.openstack.org/424755
18:45:01 briancurtin stevemar: sdk master is currently at a state i’d like to release. what do you need to do now?
18:48:27 stevemar briancurtin: i need 25 minutes to test it
18:49:16 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient: test sdk https://review.openstack.org/424796
18:49:39 stevemar briancurtin: you can play the jeopardy theme song while looking at the zuul results to ^
19:11:34 ankur-gupta-f1 briancurtin: https://www.youtube.com/watch?v=IkdmOVejUlI here you go.
19:17:45 briancurtin hah
19:46:44 stevemar briancurtin: our fucntional tests pass!
19:46:56 stevemar briancurtin: you are in the clear to release 0.9.13 cc dtroyer
19:47:16 stevemar functional tests pass just fine https://review.openstack.org/#/c/424796/
20:14:24 openstackgerrit Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585
20:33:42 dims briancurtin stevemar : do you need this for ocata?
20:33:54 briancurtin dims: yes
20:34:29 stevemar dims: what is "this"
20:34:41 dims oops https://review.openstack.org/#/c/422069/
20:35:51 dims stevemar : ^
20:36:01 dims briancurtin : i know you filed it :)
20:36:19 dims briancurtin : freeze starts thu (email just hit -dev)
20:40:18 briancurtin dims: is my response to Matthew Thode on that review sufficient to answer your question?
20:43:52 openstackgerrit Justin A Wilson proposed openstack/python-openstackclient: Add support for Cinder API 3.3 into OSC https://review.openstack.org/421585
20:50:03 dims y that helped briancurtin
20:58:51 briancurtin dims: thanks a lot!
22:07:59 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient: change assert_show_fields to not fail on new fields https://review.openstack.org/424433
23:11:46 stevemar dtroyer: heads up for https://review.openstack.org/#/c/418190/ and https://review.openstack.org/#/c/424847/
23:12:51 dtroyer stevemar: I've got one coming too… standby one
23:31:05 openstackgerrit Steve Martinelli proposed openstack/python-openstackclient: change assert_show_fields to not fail on new fields https://review.openstack.org/424433
23:32:11 stevemar dtroyer: you may want to join -nova and talk to mriedem and andreyk about it
23:32:24 dtroyer stevemar: so do we need to get that novaclient review merged still? or do we need to work around that problem yet? I thought it was fixed?
23:32:34 stevemar dtroyer: currently the sdk requirements bump will break them
23:32:43 stevemar dtroyer: i have no effing clue and i'm too tired to think :)
23:32:58 dtroyer sure, because someone still hasn't fixed the real issue, which I'm not sure I know anymore what it is
23:33:21 stevemar all i know is i want the minimum sdk bumped, and nova is stopping us
23:33:25 stevemar i have no idea why
23:39:45 briancurtin how could that even occur?

Earlier   Later