Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2021-02-11
16:01:27 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Drop .json suffix from networking URLs https://review.opendev.org/c/openstack/openstacksdk/+/775200
17:16:03 alpalka Hello
17:16:24 alpalka @stephenfin are you still here?
17:54:17 iurygregory gtema, hey about your comment for the _proxy.py, basically you want all methods using this type of identation https://github.com/openstack/openstacksdk/blob/master/openstack/baremetal/v1/_proxy.py#L130-L131 ?
17:54:36 gtema yes, like that
17:54:40 iurygregory ack =)
17:54:48 iurygregory I will explain to the intern
17:55:53 gtema formatting in https://review.opendev.org/c/openstack/openstacksdk/+/768612/24/openstack/baremetal/v1/_proxy.py#1368 is mostly confusing
17:58:59 mordred ++
18:11:15 vkmc ashrod98[m], :D
18:11:58 ashrod98[m] vkmc: haha hello!
18:12:23 vkmc well done
18:12:24 vkmc :D
19:21:17 openstackgerrit Merged openstack/cliff master: requirements: Uncap PrettyTable https://review.opendev.org/c/openstack/cliff/+/774915
23:31:08 openstackgerrit Goutham Pacha Ravi proposed openstack/openstacksdk master: Add shared file systems support https://review.opendev.org/c/openstack/openstacksdk/+/773556
#openstack-sdks - 2021-02-12
00:26:45 openstackgerrit Merged openstack/openstacksdk stable/ussuri: Add id query parameter to sg rules https://review.opendev.org/c/openstack/openstacksdk/+/772810
11:38:38 stephenfin gtema: I've proposed a new release of cliff here in order to unblock the PrettyTable uncapping. Could you take a look? https://review.opendev.org/c/openstack/releases/+/775367
11:38:48 stephenfin uncapping patch is here, fwiw https://review.opendev.org/c/openstack/requirements/+/774368
11:38:58 gtema oki, will have a look today
11:45:30 stephenfin thanks
11:45:36 gtema wlcm
12:45:10 openstackgerrit Stephen Finucane proposed openstack/openstacksdk master: Remove unnecessary string formatting https://review.opendev.org/c/openstack/openstacksdk/+/775377
13:12:18 gtema thanks stephenfin and dtantsur
13:12:26 dtantsur np
13:12:28 stephenfin yw
13:12:46 dtantsur gtema: should we maybe ask them to follow-up on https://review.opendev.org/c/openstack/openstacksdk/+/768612/ ?
13:13:05 dtantsur to avoid collecting feedback from all ironic folks once again :)
13:13:08 gtema no problem with that either
13:24:18 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
13:28:43 dtantsur could anyone from the ansible side review https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770015 and https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770036 please?
13:28:59 gtema ok, will do so
13:38:44 openstackgerrit Dmitry Tantsur proposed openstack/ansible-collections-openstack master: ironic: stop putting meaningless values to properties https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770036
13:38:46 dtantsur thanks gtema, updated
13:39:36 gtema processed
14:02:43 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
14:50:28 dtantsur mm, we're also broken by old pip https://zuul.opendev.org/t/openstack/build/6aba594c6bd64c20aee92a68dfa3a875/log/job-output.txt#1241
15:06:42 mordred dtantsur: that seems unfun
15:08:30 gtema oh this cryptogarphy folks - they broke the world
15:09:48 dtantsur ikr?
15:09:56 dtantsur fortunately, it is *usually* fixed by newer pip
15:10:02 dtantsur (at least 19.1.1 to be precise)
15:10:36 gtema yes, I had same issue on my zuul jobs and I have fixed that "__only__" by moving jobs from ubuntu into openshift pods
15:20:08 mordred are we not installing latest pip in the jobs?
15:20:29 mordred like, using any pip other than the absolute latest available pip is usually a recipe for disaster :)
15:21:00 gtema I was trying locally and somehow tox was falling back (even after manually updating pip in venv)
15:21:24 gtema but that might be local issue
15:45:01 frickler that job uses distro pip, which is too old on bionic, maybe move the job to focal?
16:03:02 openstackgerrit Merged openstack/openstacksdk master: Drop .json suffix from networking URLs https://review.opendev.org/c/openstack/openstacksdk/+/775200
16:03:33 openstackgerrit Merged openstack/openstacksdk master: Support Deploy Templates for Ironic API https://review.opendev.org/c/openstack/openstacksdk/+/768612
16:05:01 openstackgerrit Merged openstack/openstacksdk master: Remove unnecessary string formatting https://review.opendev.org/c/openstack/openstacksdk/+/775377
#openstack-sdks - 2021-02-13
07:40:25 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
#openstack-sdks - 2021-02-14
02:21:11 openstackgerrit Manuel Rodriguez proposed openstack/openstacksdk master: Expose description parameter when creating a server https://review.opendev.org/c/openstack/openstacksdk/+/775513
02:27:27 openstackgerrit Manuel Rodriguez proposed openstack/ansible-collections-openstack master: Allow description field to be set with os_server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775514
#openstack-sdks - 2021-02-15
10:33:56 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add functionality to change node provision state https://review.opendev.org/c/openstack/ansible-collections-openstack/+/771430
12:02:50 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix CI ansible-test errors https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
12:07:45 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
12:13:32 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix CI ansible-test errors https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
12:14:17 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix CI ansible-test errors https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
14:32:34 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Mark 2.9 ansible-test sanity as non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
15:54:46 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Mark 2.9 ansible-test sanity as non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
16:13:24 openstackgerrit Manuel Rodriguez proposed openstack/openstacksdk master: Expose description parameter when creating a server https://review.opendev.org/c/openstack/openstacksdk/+/775513
18:50:40 openstackgerrit Sebastian Haderecker proposed openstack/ansible-collections-openstack master: Fix some typos in readme https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775696
19:10:54 openstackgerrit Sebastian Haderecker proposed openstack/ansible-collections-openstack master: Guidelines: Fix links and formatting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775700
19:31:57 openstackgerrit Manuel Rodriguez proposed openstack/ansible-collections-openstack master: Allow description field to be set with os_server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775514
19:41:40 openstackgerrit Merged openstack/ansible-collections-openstack master: Mark 2.9 ansible-test sanity as non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775604
19:47:38 ade_lee mordred, fungi hi - could I get some +2s on https://review.opendev.org/c/openstack/openstacksdk/+/767411 please
20:17:35 fungi ade_lee: i went ahead and approved it, stephenfin has +2'd earlier
20:17:54 fungi i think we've discussed the situation at great length by now, in multiple venues
20:18:10 ade_lee fungi, great thanks!
20:18:43 fungi i'm not a usual sdk reviewer, but i was fairly confident in my review of that change
20:19:44 fungi if others disagree, it's easy enough to revert
20:21:06 fungi ade_lee: were you going to propose the followup code comment change stephenfin suggested?
20:21:23 ade_lee fungi, yup - I can do that
20:21:26 fungi just so we don't forget to rip it out once we only support 3.9 and later
20:21:40 ade_lee fungi, will submit shortly
20:21:42 fungi which is years away, so likely easy to forget in that time
20:49:21 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Migrating image_info module from AnsibleModule to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/773687
20:49:39 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Guidelines: Fix links and formatting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775700
21:09:56 openstackgerrit Ashley Rodriguez proposed openstack/openstacksdk master: Add resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/775707
21:53:29 mordred ade_lee, fungi also looks good to me - thanks. I'm glad a solution was reached on this one - the original "md5 bad mmkay" was tiring :)
22:21:02 openstackgerrit Merged openstack/openstacksdk master: encapsulate md5 calls for fips https://review.opendev.org/c/openstack/openstacksdk/+/767411
22:23:27 openstackgerrit Merged openstack/ansible-collections-openstack master: Fix some typos in readme https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775696
23:20:06 openstackgerrit Merged openstack/ansible-collections-openstack master: ironic: deprecate sub-options of driver_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770015
#openstack-sdks - 2021-02-16
00:43:52 openstackgerrit Merged openstack/ansible-collections-openstack master: Migrating image_info module from AnsibleModule to OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/773687
09:41:05 openstackgerrit Merged openstack/ansible-collections-openstack master: Guidelines: Fix links and formatting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775700
10:43:54 openstackgerrit Merged openstack/ansible-collections-openstack master: Allow description field to be set with os_server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775514
11:47:41 openstackgerrit Merged openstack/ansible-collections-openstack master: ironic: stop putting meaningless values to properties https://review.opendev.org/c/openstack/ansible-collections-openstack/+/770036
14:08:45 uzumaki I'm here, dtantsur
14:09:01 dtantsur uzumaki: do you call anything to configure logging?
14:11:15 uzumaki openstack.enable_logging(debug=True)
14:11:28 uzumaki is this for STDOUT logs or file logs? I suppose by default it will target STDOUT
14:11:57 dtantsur I think so. So yeah, you enabled the logs yourselves, maybe remove it? Or use debug=False?
14:12:10 uzumaki will try, thanks a lot =) Guten Tag!
14:13:40 dtantsur :)
16:56:16 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add functionality to change node provision state https://review.opendev.org/c/openstack/ansible-collections-openstack/+/771430
17:06:11 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP install Octavia plugin for testing LB https://review.opendev.org/c/openstack/ansible-collections-openstack/+/772613
17:22:22 openstackgerrit Anuradha proposed openstack/openstacksdk master: Support Introspection rules for ironic API https://review.opendev.org/c/openstack/openstacksdk/+/775878
17:26:20 openstackgerrit Tosin Farai proposed openstack/ansible-collections-openstack master: Add support for node VIFs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/774635
17:34:11 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Release version 1.3.0 of Openstack Collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/775882
18:04:00 openstackgerrit Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add Octavia job for testing LB https://review.opendev.org/c/openstack/ansible-collections-openstack/+/772613

Earlier   Later