*** kumarmn has joined #openstack-tc | 00:03 | |
*** kumarmn has quit IRC | 00:05 | |
*** kumarmn has joined #openstack-tc | 00:05 | |
*** superdan is now known as dansmith | 00:11 | |
*** kumarmn has quit IRC | 00:15 | |
*** kumarmn has joined #openstack-tc | 01:15 | |
*** kumarmn has quit IRC | 01:21 | |
*** liujiong has joined #openstack-tc | 01:39 | |
openstackgerrit | zhurong proposed openstack/governance master: Update policy goal for solum https://review.openstack.org/533440 | 01:41 |
---|---|---|
*** kumarmn has joined #openstack-tc | 03:10 | |
*** liujiong has quit IRC | 03:11 | |
*** liujiong has joined #openstack-tc | 03:15 | |
*** kumarmn has quit IRC | 03:19 | |
*** rosmaita has quit IRC | 03:27 | |
*** liujiong has quit IRC | 03:35 | |
*** liujiong has joined #openstack-tc | 03:36 | |
*** liujiong has quit IRC | 03:41 | |
*** kumarmn has joined #openstack-tc | 03:45 | |
*** liujiong has joined #openstack-tc | 03:49 | |
*** kumarmn has quit IRC | 04:42 | |
-openstackstatus- NOTICE: The logs.openstack.org filesystem has been restored to full health. We are attempting to keep logs uploaded between the prior alert and this one, however if your job logs are missing please issue a recheck. | 04:50 | |
*** ChanServ changes topic to "The logs.openstack.org filesystem has been restored to full health. We are attempting to keep logs uploaded between the prior alert and this one, however if your job logs are missing please issue a recheck." | 04:50 | |
*** ChanServ changes topic to "OpenStack Technical Committee office hours: Tuesdays at 09:00 UTC, Wednesdays at 01:00 UTC, and Thursdays at 15:00 UTC | https://governance.openstack.org/tc/ | channel logs http://eavesdrop.openstack.org/irclogs/%23openstack-tc/" | 04:54 | |
*** kumarmn has joined #openstack-tc | 05:43 | |
*** kumarmn has quit IRC | 05:48 | |
*** kumarmn has joined #openstack-tc | 06:44 | |
*** kumarmn has quit IRC | 06:49 | |
*** kumarmn has joined #openstack-tc | 07:45 | |
*** kumarmn has quit IRC | 07:49 | |
*** liujiong has quit IRC | 07:50 | |
openstackgerrit | Masayuki Igawa proposed openstack/governance master: [WIP] Add Cold upgrades capabilities https://review.openstack.org/533544 | 08:01 |
*** masayukig has joined #openstack-tc | 08:16 | |
*** kumarmn has joined #openstack-tc | 08:46 | |
*** kumarmn has quit IRC | 08:50 | |
*** jpich has joined #openstack-tc | 08:58 | |
*** kumarmn has joined #openstack-tc | 09:46 | |
*** kumarmn has quit IRC | 09:51 | |
*** kumarmn has joined #openstack-tc | 10:47 | |
*** cdent has joined #openstack-tc | 10:47 | |
*** dtantsur is now known as dtantsur|bbl | 10:48 | |
*** kumarmn has quit IRC | 10:53 | |
*** cdent has quit IRC | 11:10 | |
*** cdent has joined #openstack-tc | 11:11 | |
*** cdent has quit IRC | 11:16 | |
*** cdent has joined #openstack-tc | 11:17 | |
*** kumarmn has joined #openstack-tc | 11:48 | |
*** kumarmn has quit IRC | 11:53 | |
*** kumarmn has joined #openstack-tc | 12:49 | |
*** andreaf has quit IRC | 12:53 | |
*** andreaf has joined #openstack-tc | 12:53 | |
*** kumarmn has quit IRC | 12:54 | |
*** dtantsur|bbl is now known as dtantsur | 13:01 | |
*** openstackgerrit has quit IRC | 13:18 | |
*** kumarmn has joined #openstack-tc | 13:50 | |
*** kumarmn has quit IRC | 13:54 | |
*** cdent has quit IRC | 14:28 | |
*** kumarmn has joined #openstack-tc | 14:30 | |
*** kumarmn_ has joined #openstack-tc | 14:30 | |
*** kumarmn has quit IRC | 14:34 | |
*** gcb has joined #openstack-tc | 14:44 | |
*** cdent has joined #openstack-tc | 15:05 | |
*** rosmaita has joined #openstack-tc | 15:12 | |
*** hongbin has joined #openstack-tc | 15:29 | |
*** gcb has quit IRC | 15:54 | |
mnaser | how do we ideally want to handle massive patchsets doing things like bumping openstackdocstheme version bump | 16:32 |
mnaser | puppet has a lot of repos and we've gotten a large # of patchsets to bump it up.. | 16:32 |
mnaser | (i'm proposing drop the requirement version and rely on upper-constraints which will avoid this) | 16:32 |
*** jpich has quit IRC | 17:46 | |
*** dtantsur is now known as dtantsur|afk | 18:16 | |
*** kmalloc has joined #openstack-tc | 18:19 | |
-openstackstatus- NOTICE: Zuul has been restarted and has lost queue contents; changes in progress will need to be rechecked. | 18:23 | |
EmilienM | mnaser: I would abandon them | 18:42 |
dhellmann | mnaser , EmilienM : what's the problem with accepting them? | 19:10 |
EmilienM | dhellmann: 1) the person haven't communicated with the team at all before | 19:11 |
EmilienM | dhellmann: 2) the change doesn't fit with our needs | 19:11 |
EmilienM | dhellmann: 3) we waste CI resources every time this things happen | 19:11 |
EmilienM | last time someone proposed a change in all modules for something we didn't want and it triggered hundred of CI jobs | 19:12 |
mnaser | dhellmann: also, we've worked decided to dropping versions from requirements.txt and avoid pinning things and rely on upper constraints, however, we weren't able to coommunicate that was the change we wanted :< | 19:12 |
EmilienM | so what I would do is 1) engage dialog with this person 2) explain why we reject this patch and what we would rather do 3) make sure the person understands how the community works | 19:12 |
dhellmann | mnaser : you should talk to the requirements team before proceeding with that change. They are working on plans that will stop syncing requirements into project repos that may conflict with not having minimums set in-tree | 19:13 |
mnaser | i am really torn about this because it could very much be someone who is trying to pick up low hanging fruit | 19:13 |
dhellmann | EmilienM, mnaser : it would be good for you to talk to the docs team before just rejecting all of the patches. Some of the things we're doing with documentation publishing will require the latest versions of the theme library to work properly. | 19:14 |
EmilienM | dhellmann: we haven't rejected them yet I think | 19:14 |
EmilienM | I did nothing | 19:14 |
mnaser | we have not indeed | 19:14 |
EmilienM | I just skipped reviewing themm | 19:14 |
dhellmann | yeah, I'm just saying... | 19:14 |
mnaser | i wanted to start a conversation around it | 19:14 |
dhellmann | ok. I'm suggesting some people you should include in that. :-) | 19:14 |
EmilienM | to me, puppet-* modules shouldn't have pined versions of any python project | 19:15 |
EmilienM | we should rely on whatever is in requirements | 19:15 |
mnaser | yeah we don't really depend on anything. we just build docs .. actually, even less, we just build release notes | 19:15 |
EmilienM | true | 19:15 |
dhellmann | EmilienM : in the abstract, I agree. In the concrete world of our CI and doc publishing, we need to make sure everything is going to keep working as expected. | 19:15 |
dhellmann | so you could potentially move those requirements into doc/requirement.txt instead of updating test-requirements.txt | 19:16 |
dhellmann | perhaps melissaml would help with that | 19:16 |
mnaser | thats a good idea, and with irrelevant-files we can drop running our entire suite of tests | 19:16 |
mnaser | for these type of changes | 19:16 |
dhellmann | good point | 19:16 |
EmilienM | we do already | 19:17 |
mnaser | EmilienM: not for requirements.txt changes i think | 19:17 |
EmilienM | what I'm saying is not specific to docs | 19:17 |
EmilienM | ah | 19:17 |
EmilienM | let me check | 19:17 |
EmilienM | ah, just for test-requirements.txt | 19:18 |
mnaser | i honestly dont mind these small changes. i believe they're all done in good faith, i just try to see how we can do better at avoiding wasting resources from donated infra and taking away valuable time from devs | 19:18 |
EmilienM | I'll add requirements.txt | 19:18 |
mnaser | no one likes to wait for the gate :> | 19:18 |
EmilienM | mnaser: is https://review.openstack.org/533765 ok? | 19:20 |
EmilienM | I'm unsure if we touch the file it will actually trigger one job after that | 19:20 |
mnaser | EmilienM: im looking at it looks like the build release notes job installs sphinx itself now | 19:21 |
dhellmann | EmilienM, mnaser : 99cloud is a chinese company, iirc, so it may be challenging to find melissaml on IRC. Email is probably the best way to reach out. | 19:22 |
EmilienM | mnaser: so we wouldn't need that file at all? | 19:23 |
EmilienM | dhellmann: ok, thanks | 19:23 |
mnaser | dhellmann: ack | 19:23 |
dhellmann | mnaser , EmilienM : the release notes job should use doc/requirements.txt for anything extra you need, including the theme | 19:24 |
mnaser | EmilienM: i have to double check the jobs and verify for infra, but for now, adding the requirements exception is probably ok... but i wonder indeed as you said no jobs will run | 19:24 |
mnaser | so maybe ideally we should just have 'openstackdocstheme' in doc/requirements.txt and drop requirements.txt | 19:24 |
mnaser | and the nice thing is by doing that we can skip doc/requirements.txt | 19:25 |
mnaser | https://github.com/openstack-infra/zuul-jobs/blob/d9bd856c59c47bbd2dc1fe68c121e682a9fb273a/zuul.yaml#L251-L282 | 19:25 |
mnaser | and we will still have 1 job run | 19:25 |
EmilienM | we don't have doc/ in puppet repos | 19:25 |
mnaser | oh you're right we have releasenotes/ | 19:26 |
EmilienM | yeah | 19:26 |
mnaser | drop requirements.txt and "openstackdocstheme" in test-requirements.txt (with your patch to avoid running a ton of jobs when we drop it from requirements) | 19:26 |
mnaser | EmilienM: +2 your patch and ill email melissaml to ask for them to drop the requirements.txt file and replace it by "openstackdocstheme" only in test-requirements.txt .. however ill contact after we can get this patch merged to avoid the churn | 19:28 |
persia | while it is laudable to conserve resources, and the decision to contact to discuss is an excellent one, is there a metric that is in place to determine how overloaded the CI resources are, as a means of measuring CI impact? My guess would be that if someone were to submit patches at 4:00 UTC they would generally find the system fairly unloaded. | 19:53 |
*** dansmith has quit IRC | 20:22 | |
*** openstackgerrit has joined #openstack-tc | 20:32 | |
openstackgerrit | Mohammed Naser proposed openstack/governance master: Add gating to verify project access levels in Gerrit https://review.openstack.org/533780 | 20:32 |
openstackgerrit | Mohammed Naser proposed openstack/governance master: Add gating to verify project access levels in Gerrit https://review.openstack.org/533780 | 20:34 |
*** Guest49522 has joined #openstack-tc | 20:36 | |
*** Guest49522 is now known as dansmith | 20:40 | |
openstackgerrit | Mohammed Naser proposed openstack/governance master: Add gating to verify project access levels in Gerrit https://review.openstack.org/533780 | 20:50 |
*** flwang has quit IRC | 21:11 | |
*** flwang has joined #openstack-tc | 21:25 | |
fungi | persia: closest metric we have is probably the "Zuul Job Queue" graph at http://grafana.openstack.org/dashboard/db/zuul-status (when "Waiting" is at or near 0, there is no appreciable backlog) | 21:39 |
*** zaneb has quit IRC | 22:03 | |
*** zaneb has joined #openstack-tc | 22:17 | |
persia | That one works, but perhaps the nature of my day caused me to express myself badly: I don't believe it to be a waste of resources to submit even 1000 parallel minor patches that call CI, so long as "Waiting" remains near zero at the time any more pressing change is submitted to gerrit. While I am in support of dialog to understand motivation, and redirect people who are causing excess effort to other people based on a misunderstanding of how to | 23:03 |
persia | help, I don't believe it is useful to claim "waste of CI resources" as part of the justification for why not to do things. | 23:03 |
persia | That is, unless we can usefully demonstrate that having done so actually blocked someone else: while I'm not absolutely confident, I have doubts that adding several thousand more nodes to the CI pools would actually help the social problem that was mentioned before, but claiming a shortage of CI resources could result in a reaction of infrastructure donation if large organisations thought it would help them meet their goals of having more | 23:04 |
persia | contributors. | 23:04 |
persia | (not that more CI nodes is a bad thing: it isn't: this is more about solving the right problem) | 23:05 |
*** cdent has quit IRC | 23:17 | |
*** kumarmn_ has quit IRC | 23:17 | |
*** kumarmn has joined #openstack-tc | 23:18 | |
*** kumarmn has quit IRC | 23:22 | |
*** hongbin has quit IRC | 23:28 | |
*** kumarmn has joined #openstack-tc | 23:35 | |
*** kumarmn has quit IRC | 23:40 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!