gouthamr | ashrodri: just noticed your message | 00:05 |
---|---|---|
gouthamr | i have these in my bash_aliases: | 00:05 |
gouthamr | alias uuid="python3 -c 'import uuid; print(\"%s\" % uuid.uuid4(), end=\"\")' | pbcopy && pbpaste && echo" | 00:05 |
gouthamr | alias 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 |
gouthamr | VolodymyrBoiko[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 Openstack | 12:37 |
gouthamr | VolodymyrBoiko[m]: and we’d get to see the CI running too | 12:38 |
VolodymyrBoiko[m] | gouthamr: Ok. Will prepare in a moment | 12:38 |
gouthamr | VolodymyrBoiko[m]: btw, was the VAST CI enabled to run against all changes? | 12:39 |
gouthamr | VolodymyrBoiko[m]: thank you | 12: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 question | 12:43 |
opendevreview | Volodymyr Boiko proposed openstack/manila master: update requirements for VAST manila driver https://review.opendev.org/c/openstack/manila/+/925579 | 12:50 |
opendevreview | Ashley Rodriguez proposed openstack/manila-tempest-plugin master: Metadata for Share Export Location https://review.opendev.org/c/openstack/manila-tempest-plugin/+/925621 | 17:35 |
opendevreview | Ashley Rodriguez proposed openstack/manila master: Metadata for Share Export Location https://review.opendev.org/c/openstack/manila/+/887543 | 17:48 |
VolodymyrBoiko[m] | Hi. gouthamr carloss please check https://review.opendev.org/c/openstack/manila/+/925579 | 18:40 |
opendevreview | Carlos Eduardo proposed openstack/manila master: WIP APIs for ensure shares https://review.opendev.org/c/openstack/manila/+/924279 | 19:04 |
carloss | VolodymyrBoiko[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 :D | 19:44 |
opendevreview | Jonathan Koerber proposed openstack/manila master: Adding JSON Schema to Quota Class Sets https://review.opendev.org/c/openstack/manila/+/924468 | 20:00 |
opendevreview | Carlos Eduardo proposed openstack/manila-specs master: Follow-up ensure shares spec comments https://review.opendev.org/c/openstack/manila-specs/+/924368 | 20:40 |
opendevreview | Carlos Eduardo proposed openstack/manila-tempest-plugin master: WIP Enable manage/unmanage tests on cephfs drivers https://review.opendev.org/c/openstack/manila-tempest-plugin/+/924774 | 21:14 |
*** clarkb is now known as Guest1974 | 21:33 | |
opendevreview | Goutham 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/+/925633 | 21:52 |
gouthamr | VolodymyrBoiko[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 |
gouthamr | maybe 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 |
opendevreview | Volodymyr Boiko proposed openstack/manila master: update requirements for VAST manila driver https://review.opendev.org/c/openstack/manila/+/925579 | 22:39 |
opendevreview | Goutham 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/+/925633 | 22:40 |
*** Guest1974 is now known as clarkb | 22:44 | |
clarkb | VolodymyrBoiko[m]: yes those emails are filtering to only trigger those events if the associated email with the event matche sthat regex | 23:11 |
clarkb | VolodymyrBoiko[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 converse | 23:12 |
gouthamr | clarkb +1 | 23:12 |
gouthamr | VolodymyrBoiko[m]: as clarkb suggests, if you delete the "email" lines, it should take care of the problem | 23: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 emails | 23:15 |
clarkb | ya 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 clunky | 23:16 |
* gouthamr got used to it with all the people using matrix/element these days :) | 23:18 | |
gouthamr | VolodymyrBoiko[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 now | 23:19 |
gouthamr | VolodymyrBoiko[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 |
gouthamr | VolodymyrBoiko[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 do | 23:21 |
gouthamr | VolodymyrBoiko[m]: something like this: https://paste.opendev.org/show/bhwGzGxqlkIR1SLB02BB/ | 23:24 |
opendevreview | Merged openstack/manila master: update requirements for VAST manila driver https://review.opendev.org/c/openstack/manila/+/925579 | 23: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 everywhere | 23:26 |
gouthamr | VolodymyrBoiko[m]: verified: 0 is fine; we only let opendev's Zuul set the Verified label | 23:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!