Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-09-22
09:45:40 openstackgerrit Merged openstack/os-service-types master: Fix pep8 testing for ubuntu focal https://review.opendev.org/750815
10:58:26 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add server migration list CLI https://review.opendev.org/675304
11:32:19 openstackgerrit Merged openstack/microversion-parse master: Adding pre-commit https://review.opendev.org/742156
11:33:01 openstackgerrit Merged openstack/os-api-ref master: Adding pre-commit https://review.opendev.org/742159
12:19:11 brinzhang0 Hi all, can someone explain what benefits the introduction of openstacksdk can bring us? Compared to appclient
12:48:49 fungi brinzhang0: who is "us" in your question? the primary benefit is to users of openstack deployments, and the idea that they only need one consistently architected library to interface with the 50+ different openstack apis (instead of having to install and keep track of calling into 50+ different libraries)
13:05:02 openstackgerrit Merged openstack/requestsexceptions master: Fix pep8 testing for ubuntu focal https://review.opendev.org/750817
13:15:47 openstackgerrit Jesper Schmitz Mouridsen proposed openstack/ansible-collections-openstack master: MAke it possible to create a health monitor to a pool https://review.opendev.org/753306
13:23:07 brinzhang0 fungi: got it, I think I have got the answer in #openstack-nova channel, thanks too ^^
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

Earlier   Later