Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-09-22
14:10:32 openstackgerrit Merged openstack/osc-lib master: Fix l-c testing for ubuntu focal https://review.opendev.org/750830
14:33:21 dtantsur gtema: hey, any objections to an sdk release? we still need the unreleased fix for _fix_argv
14:33:39 gtema only after we land some more stuff
14:33:44 gtema which is already there
14:34:07 gtema https://review.opendev.org/#/c/751234/
14:34:11 gtema https://review.opendev.org/#/c/750072/
14:34:31 gtema https://review.opendev.org/#/c/746578/
14:34:35 gtema https://review.opendev.org/#/c/746586/
14:35:02 dtantsur I don't think it's far to hold a serious bug fix for more than 1k loc of features..
14:35:20 gtema I need those to proceed with OSC
14:35:43 gtema and waiting for 2 weeks already
14:37:02 gtema if you promise me to review those this week and cut additional release next week - I am fine
14:37:12 gtema :D
14:37:32 dtantsur I can promise it, although I should let you know that I don't like your approach to this
14:37:48 gtema why?
14:38:42 dtantsur you're hold up issuing a fix for consumer problems based on the features you personally need
14:38:46 dtantsur * holding
14:39:06 gtema me personally? This is a joke, not?
14:39:31 dtantsur no, I'm serious
14:39:38 gtema I am not holding a fix. I just want us to move with features, and not only bugs
14:40:05 dtantsur https://review.opendev.org/#/c/746578/ has an unmerged dependency
14:40:10 gtema everybody complains OSC sucks, but nobody work on it. And nobody even caring doing review for work
14:40:43 gtema that is is what really not good
14:41:22 dtantsur hmm, sure, let's keep a high impact bug, so that ironic folks regret using openstacksdk or...?
14:41:32 gtema didn't noticed one patch is not yet merged
14:41:35 gtema okay
14:42:06 gtema would you create release change or should I do?
14:42:32 dtantsur like, I hear your pain. but now it's the 2 of us and hberaud, right?
14:42:45 dtantsur and I suspect only you is actually paid to work on SDK/OSC
14:42:46 gtema yes
14:42:59 gtema nope, I am not paid for that
14:43:04 dtantsur okay, even worse
14:43:19 dtantsur I'm trying my best with SDK reviews, but I have a lot of stuff happening on both ironic upstream and RH downstream fronts
14:44:01 dtantsur gtema: I can request a release. we can release as often as we want to, realistically. even every week.
14:44:03 gtema I can suggest introducing 2-weeks rule for changes by cores: no objections in 2 weeks - okay for self merge
14:44:06 gtema what do you think?
14:44:30 dtantsur I'd switch to 1x +2 for cores (similar to what is done for stable branches)
14:44:47 gtema but we don't have a single +2
14:44:56 gtema my changes are there for 2 weeks
14:45:46 gtema that is the problem - there are no reviews at all, neither in SDK nor in OSC
14:46:57 gtema just permanently pinging somebody else for a review is like: switch the light off and go home
14:48:39 dtantsur gtema: please let me know if you want to address minor issues in https://review.opendev.org/#/c/751234/ or in a follow-up
14:49:04 gtema followup
14:53:42 dtantsur same for https://review.opendev.org/#/c/750072/
14:53:56 gtema okay, thanks
14:54:04 dtantsur gtema: if you can follow-up today, we roll it all into the release
14:54:20 gtema ok
16:13:35 openstackgerrit Merged openstack/openstacksdk master: Add some compute console operations https://review.opendev.org/751234
16:42:20 openstackgerrit Merged openstack/openstacksdk master: Add additional compute flavor operations https://review.opendev.org/750072
16:42:22 openstackgerrit Merged openstack/openstacksdk master: Add _max_microversion for aggregates https://review.opendev.org/750200
17:05:23 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Follow-up on 751234 and 750072 https://review.opendev.org/753409
#openstack-sdks - 2020-09-23
06:03:18 openstackgerrit MaAoyu proposed openstack/openstacksdk master: Remove install unnecessary packages https://review.opendev.org/753509
06:15:38 openstackgerrit MaAoyu proposed openstack/os-api-ref master: Remove install unnecessary packages https://review.opendev.org/753512
06:40:31 openstackgerrit MaAoyu proposed openstack/osc-lib master: Remove install unnecessary packages https://review.opendev.org/753519
07:16:04 openstackgerrit zhangboye proposed openstack/openstacksdk master: Remove install unnecessary packages https://review.opendev.org/753532
07:54:09 openstackgerrit Merged openstack/osc-lib master: requirements: Drop os-testr https://review.opendev.org/740233
08:45:28 gtema dtantsur thanks. I will request release as soon as change lands
08:45:47 dtantsur ++
08:46:09 dtantsur gtema: thinking of ^^, maybe we need weekly review jams?
08:46:22 gtema maybe
08:46:39 dtantsur allocate an hour when most of us (2? 3?) are here and go through open patches (asynchronously, without video/audio)?
08:46:55 gtema would be perhaps hard to achieve (same time), but if every one does spend at least some time once a week for reviews - great
08:47:30 gtema let's try next week
08:57:21 gtema btw dtantsur, do you know if we can actually request release in Victoria currently?
08:58:49 dtantsur gtema: *thinking slowly* it's no longer victoria, is it? I guess we've branched already? or not..
08:59:30 dtantsur oh, I forgot to backport my fix to victoria \o/
08:59:36 gtema kinda, but there is no wallaby folder in releasees
08:59:39 dtantsur okay, so the dance is a bit different then
08:59:58 dtantsur we cannot release wallaby just yet. we need to backport the fix to victoria and release that. and that will require a requirements FFE
09:00:09 dtantsur I guess I should do it myself then because I know the underlying issue better
09:00:22 gtema okay
09:00:43 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk stable/victoria: Fix a bogus error in config loader when using several args with dashes https://review.opendev.org/753547
09:00:43 gtema but then we currently wait for wallaby to be open to cut a release. Right?
09:00:45 dtantsur gtema: if you could review ^^^ I can take it from there
09:00:49 gtema sure
09:00:50 dtantsur gtema: this is correct
09:00:55 gtema oki
10:02:42 openstackgerrit Merged openstack/openstacksdk master: Remove install unnecessary packages https://review.opendev.org/753509
10:16:07 openstackgerrit Merged openstack/openstacksdk master: Follow-up on 751234 and 750072 https://review.opendev.org/753409
11:27:21 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add server migration list CLI https://review.opendev.org/675304
11:49:27 dtantsur oookay, lower constraints are failing on victoria
11:50:02 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk stable/victoria: Fix l-c testing for ubuntu focal https://review.opendev.org/753590
11:50:32 dtantsur gtema, gmann, this ^^ is bad because it bumps requirements on a stable branch.. but I don't know a way around it
11:53:05 gtema ok
11:59:35 dtantsur I assume we'll need to make a minor release because of that
12:00:00 gtema sure
12:26:05 admin0 hi .. what package provides coe ? openstack: 'coe' is not an openstack command
12:28:41 dtantsur admin0: maybe python-magnumclient?
13:23:17 admin0 dtantsur, thanks
#openstack-sdks - 2020-09-24
02:55:09 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add server migration list CLI https://review.opendev.org/675304
05:37:53 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add server migration list CLI https://review.opendev.org/675304
08:27:16 openstackgerrit Merged openstack/openstacksdk stable/victoria: Fix l-c testing for ubuntu focal https://review.opendev.org/753590
09:10:00 openstackgerrit MaAoyu proposed openstack/python-openstackclient master: Remove install unnecessary packages https://review.opendev.org/754013
09:35:52 openstackgerrit MaAoyu proposed openstack/shade master: Remove install unnecessary packages https://review.opendev.org/754047
10:46:46 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add server migration list CLI https://review.opendev.org/675304
11:46:12 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: WIP Fix linters for new ansible release https://review.opendev.org/754074
13:10:24 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: WIP Fix linters for new ansible release https://review.opendev.org/754074
14:33:31 openstackgerrit Merged openstack/openstacksdk stable/victoria: Fix a bogus error in config loader when using several args with dashes https://review.opendev.org/753547
14:49:36 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/openstacksdk master: Include "fields" to "Port" query parameters https://review.opendev.org/754113
15:11:45 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListPort query https://review.opendev.org/754117

Earlier   Later