| opendevreview | Takashi Kajinami proposed openstack/oslo.db master: Deprecate iteritems https://review.opendev.org/c/openstack/oslo.db/+/936314 | 01:58 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/oslo.serialization master: Ignore iteritems() and use items() solely https://review.opendev.org/c/openstack/oslo.serialization/+/936315 | 02:02 |
| *** mhen_ is now known as mhen | 02:19 | |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.service master: Introduce backend system in oslo.service https://review.opendev.org/c/openstack/oslo.service/+/935783 | 08:01 |
| *** ralonsoh_ is now known as ralonsoh | 10:07 | |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.service master: Introduce backend system in oslo.service https://review.opendev.org/c/openstack/oslo.service/+/935783 | 12:18 |
| tkajinam | we are supposed to start the meeting, right ? | 13:06 |
| tkajinam | damani[m], ^^^ | 13:06 |
| damani[m] | #startmeeting oslo | 13:08 |
| opendevmeet | Meeting started Wed Nov 27 13:08:42 2024 UTC and is due to finish in 60 minutes. The chair is damani[m]. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:08 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:08 |
| opendevmeet | The meeting name has been set to 'oslo' | 13:08 |
| damani[m] | hberaud, tkajinam, meeting time | 13:09 |
| hberaud[m] | o/ | 13:09 |
| damani[m] | #link https://etherpad.opendev.org/p/epoxy-oslo-meeting-tracking | 13:09 |
| tkajinam | o/ | 13:09 |
| damani[m] | do you have some topic you would like to discuss | 13:12 |
| hberaud[m] | nope | 13:13 |
| tkajinam | give me a few sec | 13:14 |
| damani[m] | #topic add the backend for olso.service | 13:14 |
| damani[m] | https://review.opendev.org/c/openstack/oslo.service/+/935783?usp=dashboard | 13:16 |
| tkajinam | damani[m], will you explain something about it ? | 13:21 |
| hberaud[m] | still there? | 13:21 |
| * tkajinam added a few review links to the etherpad to mention these later | 13:22 | |
| damani[m] | tkajinam, can you eventually take a look to that patch | 13:22 |
| damani[m] | otherwise, nothing more | 13:22 |
| hberaud[m] | tkajinam: I validated your patches, I put one or two comments | 13:23 |
| damani[m] | #topic a few oslo.utils patches need attention | 13:23 |
| tkajinam | I've left a quick comment but will take a look later... though I probably have to look into heat problems first | 13:23 |
| damani[m] | ok i will take a look | 13:24 |
| tkajinam | damani[m], I wonder if you have any update about the implementation coming next ? I think comments from hberaud[m] and me are dependent on it, I guess | 13:24 |
| tkajinam | I think we are agreeing with the overall design of that switch and then we have to see how the switch will be used next | 13:25 |
| hberaud[m] | yes my comments totally depends on the following steps | 13:25 |
| hberaud[m] | For now I've nothing more to add, all i had to say is already said | 13:26 |
| damani[m] | i have answered to the comment | 13:27 |
| hberaud[m] | I'll let Mike and Takashi decide, I'd have prefered not using the NotImplementedError and not introduce the option while we have no available backends, but if they are ok with that, then I'll put my +2 | 13:29 |
| hberaud[m] | I'll put a comment | 13:30 |
| damani[m] | tkajinam, i agree with you and we can have the option already now | 13:30 |
| damani[m] | and i will remove the threading at the moment | 13:30 |
| tkajinam | I think we can continue the discussion in the review. at this stage I believe we don't need huge update from the current state and it's safe to submit the subsequent changes for actual logic implementation | 13:30 |
| damani[m] | hberaud, i have put a comment about the NotImplementedError again this morning | 13:31 |
| damani[m] | but ok i will change it | 13:31 |
| *** iurygregory_ is now known as iurygregory | 13:31 | |
| hberaud[m] | Yeah I see your answer | 13:31 |
| damani[m] | some other topic we need to discuss this week? | 13:31 |
| hberaud[m] | nope | 13:32 |
| damani[m] | #topic action for the next meeting | 13:32 |
| damani[m] | update the meeting with all the dates | 13:32 |
| tkajinam | yeah and we may put the template with on-going work like oslo.service update | 13:33 |
| damani[m] | #topic open discussion | 13:33 |
| damani[m] | tkajinam, yes right | 13:33 |
| damani[m] | something else? | 13:34 |
| damani[m] | otherwise i think we are done for today | 13:34 |
| tkajinam | nothing else. we can discuss details about the patches I put in the doc in gerrit | 13:34 |
| tkajinam | once these are merged I'll proposed a new release. just fyi | 13:34 |
| damani[m] | sounds good | 13:34 |
| damani[m] | tkajinam, hberaud, thanks a lot for your time and your participation | 13:35 |
| tkajinam | :-) | 13:35 |
| damani[m] | #endmeeting | 13:36 |
| opendevmeet | Meeting ended Wed Nov 27 13:36:38 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 13:36 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/oslo/2024/oslo.2024-11-27-13.08.html | 13:36 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/oslo/2024/oslo.2024-11-27-13.08.txt | 13:36 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/oslo/2024/oslo.2024-11-27-13.08.log.html | 13:36 |
| hberaud[m] | thanks damani | 13:36 |
| tkajinam | thank you | 13:37 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.utils master: Add a release note about the new crypt utilities https://review.opendev.org/c/openstack/oslo.utils/+/936371 | 13:37 |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.service master: Introduce backend system in oslo.service https://review.opendev.org/c/openstack/oslo.service/+/935783 | 13:37 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.utils master: Add a release note about the new crypt utilities https://review.opendev.org/c/openstack/oslo.utils/+/936371 | 13:39 |
| damani[m] | code review done | 13:47 |
| damani[m] | about oslo.service i have remove threading from the choices | 13:47 |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.service master: Introduce backend system in oslo.service https://review.opendev.org/c/openstack/oslo.service/+/935783 | 13:50 |
| damani[m] | hberaud, the option is not anymore exposed | 13:50 |
| damani[m] | the code is here but not exposed | 13:51 |
| damani[m] | and about the NotImplementedError as i said if the key is not present that mean a component is not provide by the backend | 13:53 |
| damani[m] | so for me it seems align with the not implemented | 13:53 |
| damani[m] | but if you really want i can change it | 13:53 |
| hberaud[m] | My comments are cosmetic things, lets go through Mike is comment | 14:01 |
| hberaud[m] | His approach will surely decide what we want | 14:02 |
| hberaud[m] | And what we should have | 14:03 |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.service master: Introduce backend system in oslo.service https://review.opendev.org/c/openstack/oslo.service/+/935783 | 14:03 |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.service master: Introduce backend system in oslo.service https://review.opendev.org/c/openstack/oslo.service/+/935783 | 14:05 |
| damani[m] | hberaud, zzzeek, tkajinam, i have updated the patch | 14:05 |
| damani[m] | i think now it's good :) | 14:05 |
| damani[m] | i hope | 14:05 |
| opendevreview | Daniel Bengtsson proposed openstack/oslo.service master: Introduce backend system in oslo.service https://review.opendev.org/c/openstack/oslo.service/+/935783 | 14:15 |
| opendevreview | Merged openstack/oslo.log master: Remove windows support https://review.opendev.org/c/openstack/oslo.log/+/932631 | 14:17 |
| *** iurygregory_ is now known as iurygregory | 14:42 | |
| opendevreview | Merged openstack/oslo.utils master: Fix get_my_ipv6 in absence of IPv6 https://review.opendev.org/c/openstack/oslo.utils/+/936247 | 14:46 |
| opendevreview | Merged openstack/oslo.utils master: Fix get_my_ipv4 in absence of IPv4 https://review.opendev.org/c/openstack/oslo.utils/+/936273 | 15:11 |
| zigo | hberaud[m]: Hi there ! Could we have reviews for https://review.opendev.org/c/openstack/oslo.utils/+/931899 and https://review.opendev.org/c/openstack/oslo.utils/+/935525 so I can have https://review.opendev.org/c/openstack/ironic/+/936357 to pass ? | 15:14 |
| hberaud[m] | zigo: well both path are already reviewed | 15:15 |
| hberaud[m] | gates are processing... | 15:16 |
| zigo | Oh, missed them, this happened this afternoon, sorry for this. | 15:16 |
| hberaud[m] | np | 15:16 |
| zigo | Then I guess the question is: will there be a new oslo.utils release soon then? | 15:16 |
| hberaud[m] | s/path/patches/ | 15:16 |
| hberaud[m] | no idea, that's more a question for damani or tkajinam | 15:17 |
| opendevreview | Merged openstack/oslo.utils master: Add utility to replace crypt.crypt https://review.opendev.org/c/openstack/oslo.utils/+/931899 | 15:23 |
| zigo | :) | 15:24 |
| opendevreview | Merged openstack/oslo.utils master: Add utility to replace crypt.mksalt https://review.opendev.org/c/openstack/oslo.utils/+/935525 | 15:37 |
| tkajinam | hberaud[m], could you review this https://review.opendev.org/c/openstack/oslo.utils/+/936371 ? | 15:53 |
| hberaud[m] | OMW | 15:53 |
| tkajinam | thx | 15:53 |
| tkajinam | once that is merged I'll proposed a release. I noticed there is a breaking change https://review.opendev.org/c/openstack/oslo.utils/+/903924 being included in the new release so I'll adjust the version number then | 15:54 |
| hberaud[m] | done | 15:54 |
| tkajinam | hberaud[m], thx ! | 15:54 |
| hberaud[m] | let me know if you need a review on the release side | 15:54 |
| tkajinam | hberaud[m], will do. thx ! | 15:55 |
| opendevreview | Takashi Kajinami proposed openstack/oslo.utils master: Adjust the version in descriptions https://review.opendev.org/c/openstack/oslo.utils/+/936383 | 15:55 |
| opendevreview | Merged openstack/oslo.utils master: Add a release note about the new crypt utilities https://review.opendev.org/c/openstack/oslo.utils/+/936371 | 15:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!