Friday, 2024-08-02

gouthamrashrodri: just noticed your message00:05
gouthamri have these in my bash_aliases:00:05
gouthamralias uuid="python3 -c 'import uuid; print(\"%s\" % uuid.uuid4(), end=\"\")' | pbcopy && pbpaste && echo"00:05
gouthamralias uuidh="python3 -c 'import uuid; print(\"%s\" % uuid.uuid4().hex, end=\"\")' | pbcopy && pbpaste && echo"00:05
VolodymyrBoiko[m]Hi gouthamr... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/ghGWevSDTPgtpIjVMOdDaIWf>)08:17
VolodymyrBoiko[m]s///, s///, s/is working/works/08:18
gouthamrVolodymyrBoiko[m]: can you please lower the requirements in the master branch; it’d make things far easier.. since these are lower bounds, the build system will end up giving you the most recent available version of a package that’s also allowed by the upper bounds in Openstack12:37
gouthamrVolodymyrBoiko[m]: and we’d get to see the CI running too12:38
VolodymyrBoiko[m]gouthamr: Ok. Will prepare in a moment12:38
gouthamrVolodymyrBoiko[m]: btw, was the VAST CI enabled to run against all changes?12:39
gouthamrVolodymyrBoiko[m]: thank you12:39
VolodymyrBoiko[m]gouthamr: not yet. Where I can share code by link? I'd like to show you current zuul config cause I have question12:43
opendevreviewVolodymyr Boiko proposed openstack/manila master: update requirements for VAST manila driver  https://review.opendev.org/c/openstack/manila/+/92557912:50
opendevreviewAshley Rodriguez proposed openstack/manila-tempest-plugin master: Metadata for Share Export Location  https://review.opendev.org/c/openstack/manila-tempest-plugin/+/92562117:35
opendevreviewAshley Rodriguez proposed openstack/manila master: Metadata for Share Export Location  https://review.opendev.org/c/openstack/manila/+/88754317:48
VolodymyrBoiko[m]Hi. gouthamr carloss please check https://review.opendev.org/c/openstack/manila/+/92557918:40
opendevreviewCarlos Eduardo proposed openstack/manila master: WIP APIs for ensure shares  https://review.opendev.org/c/openstack/manila/+/92427919:04
carlossVolodymyrBoiko[m]: sure, just did. Change looks good, but as gouthamr suggested, I believe we'd benefit from a more detailed commit message. Reusing the example provided by gouthamr is totally fine there :D19:44
opendevreviewJonathan Koerber proposed openstack/manila master: Adding JSON Schema to Quota Class Sets  https://review.opendev.org/c/openstack/manila/+/92446820:00
opendevreviewCarlos Eduardo proposed openstack/manila-specs master: Follow-up ensure shares spec comments  https://review.opendev.org/c/openstack/manila-specs/+/92436820:40
opendevreviewCarlos Eduardo proposed openstack/manila-tempest-plugin master: WIP Enable manage/unmanage tests on cephfs drivers  https://review.opendev.org/c/openstack/manila-tempest-plugin/+/92477421:14
*** clarkb is now known as Guest197421:33
opendevreviewGoutham Pacha Ravi proposed openstack/manila-tempest-plugin master: [dnm] debugging share size errors in cephfs-native job  https://review.opendev.org/c/openstack/manila-tempest-plugin/+/92563321:52
gouthamrVolodymyrBoiko[m]: late response on 22:22
gouthamr> gouthamr: not yet. Where I can share code by link? I'd like to show you current zuul config cause I have question 22:22
gouthamrmaybe github, or paste.opendev.org 22:22
VolodymyrBoiko[m]gouthamr: carloss Could you have a look at my current zuul pipeline config please:... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/RRnebTMJCaLDYPbFHNWIzFbA>)22:29
VolodymyrBoiko[m] * gouthamr: carloss Could you have a look at my current zuul pipeline config please:... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/WDKnOdnpDuTTEJztNneZYAST>)22:31
opendevreviewVolodymyr Boiko proposed openstack/manila master: update requirements for VAST manila driver  https://review.opendev.org/c/openstack/manila/+/92557922:39
opendevreviewGoutham Pacha Ravi proposed openstack/manila-tempest-plugin master: [dnm] debugging share size errors in cephfs-native job  https://review.opendev.org/c/openstack/manila-tempest-plugin/+/92563322:40
*** Guest1974 is now known as clarkb22:44
clarkbVolodymyrBoiko[m]: yes those emails are filtering to only trigger those events if the associated email with the event matche sthat regex23:11
clarkbVolodymyrBoiko[m]: as a side note when you do multi line matrix messages it forces us irc users to clikc a link to read them. This is fine for large content but for messages like that I would just do them on a single line or send multiple messages to make it easier for people to converse23:12
gouthamrclarkb +1 23:12
gouthamrVolodymyrBoiko[m]: as clarkb suggests, if you delete the "email" lines, it should take care of the problem23:13
VolodymyrBoiko[m]clarkb: gouthamr 23:15
VolodymyrBoiko[m]Sorry. Didn't know that. I'm using element and it formats messages nicely whatsoever. I'll delete emails23:15
clarkbya the matrix bridge doesn't really give any indication to matrix users when it happens. Would be nice if it did though I'm not sure how they could do that without it being clunky23:16
* gouthamr got used to it with all the people using matrix/element these days :) 23:18
gouthamrVolodymyrBoiko[m]: another thing i'd recommend doing is to set https://zuul-ci.org/docs/zuul/latest/config/job.html#attr-job.branches - so your CI is only run on "master" for now23:19
gouthamrVolodymyrBoiko[m]: or you could have a fancy regex that ignores branches that we have open that the driver doesn't exist in: "unmaintained/*", "stable/2023.*","stable/2024.1"23:20
gouthamrVolodymyrBoiko[m]: generally, we don't need CI running on stable branches.. but we encourage it, there are a lot of bugfixes that are backported on these older stable branches until they are retired; having CI on these will provide confidence we're not breaking anything when we do23:21
gouthamrVolodymyrBoiko[m]: something like this: https://paste.opendev.org/show/bhwGzGxqlkIR1SLB02BB/ 23:24
opendevreviewMerged openstack/manila master: update requirements for VAST manila driver  https://review.opendev.org/c/openstack/manila/+/92557923:25
VolodymyrBoiko[m]Ok. Thx. So you will enable votes for my CI at some point when it is stable right? Cause now I have Verified: 0 everywhere23:26
gouthamrVolodymyrBoiko[m]: verified: 0 is fine; we only let opendev's Zuul set the Verified label23:45

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!