| openstackgerrit | Ian Wienand proposed opendev/system-config master: borg-backup: implement saving a stream, use for database backups https://review.opendev.org/c/opendev/system-config/+/771738 | 00:38 |
|---|---|---|
| openstackgerrit | Merged opendev/system-config master: Check that git clone against x/ project succeeds in testing https://review.opendev.org/c/opendev/system-config/+/773447 | 00:50 |
| clarkb | that is being promoted now | 00:50 |
| clarkb | https://hub.docker.com/layers/opendevorg/gerrit/3.2/images/sha256-32eff5a9fcd25caeeeedbc8e1e64a581ff7fb679a567fec3be0c03452cfc318b?context=explore | 00:52 |
| clarkb | infra-root ^ should I go ahead and docker-compose pull ; down ; up -d that on review.o.o? | 00:52 |
| fungi | deploy succeeded | 00:53 |
| clarkb | fungi: yup and I linked the updated image on docker hub just above | 00:53 |
| ianw | clarkb: ++ from me | 00:53 |
| clarkb | ok proceeding now maybe someone else can do a #status log/notice ? | 00:53 |
| fungi | yeah, i don't see any changes about to report for the openstack tenant in queues besides check, so a brief blip should hopefully go unnoticed | 00:54 |
| clarkb | image is pulled and docker image list shows it updated | 00:54 |
| fungi | status notice The Gerrit service on review.opendev.org is being quickly restarted to apply a new securit ypatch | 00:54 |
| fungi | er, security patch | 00:55 |
| clarkb | fungi: the space at the end is in the wrong plcae but that lgtm | 00:55 |
| ianw | ^ lgtm | 00:55 |
| clarkb | I'm proceeding with the down then up -d now | 00:55 |
| fungi | #status notice The Gerrit service on review.opendev.org is being quickly restarted to apply a new security patch | 00:55 |
| openstackstatus | fungi: sending notice | 00:55 |
| -openstackstatus- NOTICE: The Gerrit service on review.opendev.org is being quickly restarted to apply a new security patch | 00:55 | |
| clarkb | it should be coming back up now | 00:55 |
| clarkb | I can load a change in the web ui | 00:57 |
| fungi | as can i | 00:57 |
| clarkb | looking at the error_log the only thing I notice is that it is complaining about not being able to compress the gc logs that we created | 00:58 |
| clarkb | but I expect that is somethign we can look at later | 00:58 |
| clarkb | the reported version looks good to me too | 00:58 |
| openstackstatus | fungi: finished sending notice | 00:58 |
| fungi | yup! | 00:59 |
| fungi | lgtm | 00:59 |
| clarkb | ianw: are you happy with it too? I think this was going to include a zuul status summary fix? | 01:01 |
| ianw | clarkb: yep, working for me. try a hard refresh and you shoul dhave the new status plugin with line highlight | 01:01 |
| clarkb | yup seems to be working for me | 01:02 |
| clarkb | a900e6ca8657 is the image id of what we were running previously if that is important | 01:04 |
| *** mlavalle has quit IRC | 01:05 | |
| clarkb | https://www.gerritcodereview.com/3.2.html#327 has the release notes too (note we use authGroup Anonymous Users in our replication setup so that replication plugin change shouldn't affect us) | 01:06 |
| clarkb | oh and as a final note I was not logged out of gerrit | 01:07 |
| clarkb | (we've had questions about that in the past and figured I'd add my data point from this restart) | 01:08 |
| *** dviroel has quit IRC | 02:04 | |
| *** hamalq has quit IRC | 02:36 | |
| ianw | oh: "Effectively this means that while running backups with append-only ssh keys, no disk space will be recovered in your BorgBase repo with pruning" | 02:47 |
| fungi | so append-only is incompatible with pruning? | 02:52 |
| fungi | or you just have to do the prune with a different key? | 02:52 |
| ianw | fungi: yeah, it's silently incompatible with pruning from the host ... which makes sense once you realise it | 03:07 |
| ianw | well, not so sure on the silent part, but the not being able to prune from the host | 03:08 |
| ianw | there's no point having another key on the host that runs the remote end in not-append mode, since the threat we're protecting against is someone on the host trying to prune the backups | 03:08 |
| ianw | we can prune from the server side i guess. it introduces the possibility of locking issues | 03:09 |
| *** brinzhang has joined #opendev | 03:15 | |
| *** hemanth_n has joined #opendev | 03:28 | |
| *** ykarel has joined #opendev | 04:11 | |
| *** ykarel_ has joined #opendev | 04:15 | |
| *** ykarel has quit IRC | 04:17 | |
| *** ykarel_ is now known as ykarel | 05:16 | |
| *** DSpider has joined #opendev | 05:16 | |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: borg-backup-server: don't use append-only mode https://review.opendev.org/c/opendev/system-config/+/773570 | 05:30 |
| *** priteau has quit IRC | 05:47 | |
| *** ykarel_ has joined #opendev | 05:51 | |
| *** ykarel has quit IRC | 05:53 | |
| *** ykarel_ is now known as ykarel | 06:22 | |
| *** lbragstad_ has joined #opendev | 06:24 | |
| *** lbragstad has quit IRC | 06:24 | |
| *** ysandeep|away is now known as ysandeep | 06:43 | |
| *** ralonsoh has joined #opendev | 06:49 | |
| *** slaweq has joined #opendev | 06:55 | |
| *** sboyron has joined #opendev | 07:02 | |
| *** marios has joined #opendev | 07:06 | |
| *** sboyron_ has joined #opendev | 07:19 | |
| *** sboyron has quit IRC | 07:22 | |
| openstackgerrit | Merged openstack/diskimage-builder master: Set eus repositories if REG_RELEASE is set https://review.opendev.org/c/openstack/diskimage-builder/+/773403 | 07:25 |
| openstackgerrit | Merged openstack/diskimage-builder master: simplify updating python versions in gentoo https://review.opendev.org/c/openstack/diskimage-builder/+/771861 | 07:28 |
| *** eolivare has joined #opendev | 07:32 | |
| *** slaweq has quit IRC | 07:40 | |
| *** slaweq has joined #opendev | 07:42 | |
| *** fressi has joined #opendev | 07:53 | |
| *** ralonsoh has quit IRC | 07:54 | |
| *** fressi has quit IRC | 07:59 | |
| *** ralonsoh has joined #opendev | 08:01 | |
| *** ralonsoh has quit IRC | 08:03 | |
| *** ralonsoh has joined #opendev | 08:05 | |
| *** fressi has joined #opendev | 08:09 | |
| *** hashar has joined #opendev | 08:13 | |
| *** rpittau|afk is now known as rpittau | 08:25 | |
| *** andrewbonney has joined #opendev | 08:27 | |
| *** dtantsur|afk is now known as dtantsur | 08:36 | |
| *** fressi has quit IRC | 08:39 | |
| *** kopecmartin has quit IRC | 08:48 | |
| *** kopecmartin has joined #opendev | 08:50 | |
| *** priteau has joined #opendev | 08:56 | |
| *** jpena|off is now known as jpena | 08:56 | |
| *** brinzhang has quit IRC | 08:56 | |
| *** brinzhang has joined #opendev | 08:59 | |
| *** brinzhang has quit IRC | 08:59 | |
| *** brinzhang has joined #opendev | 08:59 | |
| *** brinzhang has quit IRC | 09:01 | |
| *** brinzhang has joined #opendev | 09:01 | |
| *** tosky has joined #opendev | 09:02 | |
| *** whoami-rajat__ has joined #opendev | 09:04 | |
| *** d34dh0r53 has quit IRC | 09:39 | |
| *** d34dh0r53 has joined #opendev | 09:39 | |
| *** fressi has joined #opendev | 09:45 | |
| *** d34dh0r53 has quit IRC | 09:48 | |
| *** d34dh0r53 has joined #opendev | 09:49 | |
| *** DSpider has quit IRC | 10:10 | |
| *** tosky has quit IRC | 10:33 | |
| *** tosky has joined #opendev | 10:34 | |
| *** zbr1 has joined #opendev | 11:14 | |
| *** dviroel has joined #opendev | 11:15 | |
| *** zbr has quit IRC | 11:16 | |
| *** zbr1 is now known as zbr | 11:16 | |
| *** raildo has quit IRC | 11:19 | |
| *** hemanth_n has quit IRC | 11:23 | |
| guillaumec | zbr, "Making gerrit zuul result comments more readable", handled by https://review.opendev.org/c/opendev/system-config/+/771445 , screenshot from test change: https://948ffe8030bc066a5b79-207bd119fd0c3b58e9c78074b243256d.ssl.cf5.rackcdn.com/771798/9/check/system-config-run-review/257d4b0/bridge.openstack.org/screenshots/gerrit-change-page-1.png | 11:25 |
| *** sshnaidm|ruck is now known as sshnaidm|afk | 11:38 | |
| *** ysandeep is now known as ysandeep|afk | 11:44 | |
| *** brinzhang has quit IRC | 11:45 | |
| *** brinzhang has joined #opendev | 11:46 | |
| *** hemanth_n has joined #opendev | 11:49 | |
| *** lpetrut has joined #opendev | 11:51 | |
| zbr | guillaumec: very beautiful output. how about the comments below? i guess we hide them right? | 11:54 |
| *** iurygregory_ has joined #opendev | 12:09 | |
| *** iurygregory has quit IRC | 12:09 | |
| *** ysandeep|afk is now known as ysandeep | 12:14 | |
| *** eolivare_ has joined #opendev | 12:24 | |
| guillaumec | zbr, the screenshot is from the following test change, where i highlight that style is only set for "autogenerated" tagged comment | 12:25 |
| *** eolivare has quit IRC | 12:26 | |
| *** hashar is now known as hasharLunch | 12:29 | |
| zbr | TBH, I can't wait to see this shipped. | 12:33 |
| *** jpena is now known as jpena|lunch | 12:36 | |
| *** sshnaidm|afk is now known as sshnaidm|ruck | 12:36 | |
| *** ysandeep is now known as ysandeep|mtg | 12:37 | |
| *** valery_t has joined #opendev | 12:42 | |
| *** eolivare_ has quit IRC | 12:46 | |
| valery_t | I've submitted this review https://review.opendev.org/c/openstack/python-openstackclient/+/773649 | 12:47 |
| valery_t | How can I trigger a verification? | 12:48 |
| frickler | valery_t: it happens automatically, it just takes some time. you can watch the progress on https://zuul.opendev.org/t/openstack/status | 12:50 |
| frickler | valery_t: with the current load we have, it will take around 4 hours for nodes to get assigned to your patch | 12:51 |
| *** zoharm has joined #opendev | 12:53 | |
| *** iurygregory_ is now known as iurygregory | 13:04 | |
| *** Tengu has quit IRC | 13:09 | |
| *** Tengu has joined #opendev | 13:10 | |
| *** Tengu has quit IRC | 13:10 | |
| *** Tengu has joined #opendev | 13:10 | |
| *** Tengu has quit IRC | 13:10 | |
| *** Tengu has joined #opendev | 13:11 | |
| *** Tengu has quit IRC | 13:11 | |
| *** Tengu has joined #opendev | 13:18 | |
| *** hasharLunch is now known as hashar | 13:18 | |
| *** mlavalle has joined #opendev | 13:18 | |
| *** eolivare_ has joined #opendev | 13:18 | |
| *** fressi has quit IRC | 13:21 | |
| *** fressi has joined #opendev | 13:23 | |
| *** jpena|lunch is now known as jpena | 13:25 | |
| openstackgerrit | Merged openstack/diskimage-builder master: Fix CentOS Stream 8 base repo in centos element https://review.opendev.org/c/openstack/diskimage-builder/+/771979 | 14:10 |
| sshnaidm|ruck | clarkb, fungi hi, I wonder if there is an option to not allow "recheck" without a reason? | 14:18 |
| sshnaidm|ruck | like making a trigger regexp requiring something after "recheck" | 14:19 |
| *** brinzhang has quit IRC | 14:19 | |
| *** brinzhang has joined #opendev | 14:20 | |
| *** valery_t has left #opendev | 14:36 | |
| *** marios has quit IRC | 14:36 | |
| *** valery_t has joined #opendev | 14:38 | |
| *** valery_t has quit IRC | 14:41 | |
| mnaser | o/ is there a place where we can look at devstack job runtime per cloud? | 14:45 |
| *** valery_t has joined #opendev | 14:45 | |
| openstackgerrit | Slawek Kaplonski proposed zuul/zuul-jobs master: [multi-node-bridge] Add script to configure connectivity https://review.opendev.org/c/zuul/zuul-jobs/+/762650 | 14:50 |
| frickler | mnaser: I don't think we have that aggregated, you'd have to gather data from jobs and group by cloud | 14:58 |
| frickler | mnaser: also I did ping you earlier about ipv6 connectivity to vexxhost, did you see that? | 14:58 |
| mnaser | frickler: ah :( i was hoping to check and see if some of the tripleo issues on our cloud are limited to tripleo or affecting all jobs | 14:58 |
| mnaser | frickler: i dont think so | 14:58 |
| frickler | mnaser: seems connectivity from as3320 and possibly others is broken because there are no IRR records for your /48s | 14:59 |
| mnaser | ah yay, the joy that is IRR, i haven't seen that | 14:59 |
| *** marios has joined #opendev | 14:59 | |
| frickler | mnaser: would be great if you could create some route objects | 15:00 |
| *** hemanth_n has quit IRC | 15:02 | |
| *** brinzhang has quit IRC | 15:06 | |
| *** brinzhang has joined #opendev | 15:06 | |
| *** brinzhang has quit IRC | 15:08 | |
| *** brinzhang has joined #opendev | 15:09 | |
| *** brinzhang has quit IRC | 15:10 | |
| *** brinzhang has joined #opendev | 15:11 | |
| *** brinzhang has quit IRC | 15:12 | |
| *** brinzhang has joined #opendev | 15:12 | |
| *** slaweq has quit IRC | 15:14 | |
| *** brinzhang has quit IRC | 15:14 | |
| *** slaweq has joined #opendev | 15:14 | |
| *** brinzhang has joined #opendev | 15:15 | |
| *** valery_t has quit IRC | 15:18 | |
| *** hashar is now known as hasharAway | 15:27 | |
| *** lpetrut has quit IRC | 15:27 | |
| *** fressi has quit IRC | 15:30 | |
| fungi | sshnaidm|ruck: we had that for some years, and the end result was that it didn't actually provide any useful control because people would just put in whatever. at one point we in fact required that they reference a bug when entering a recheck comment, so people would just find loosely related/unrelated bugs to mention or make up random bug numbers entirely, so the data we got from it was fairly useless | 15:30 |
| sshnaidm|ruck | fungi, I see | 15:31 |
| fungi | the hope was that it would get people to research why builds failed and confirm existing bugs or open new bugs for those problems, but the incentive was just not there | 15:31 |
| fungi | in short, people who are inclined to actually look into failures are apt to record and track them anyway, and there's little way to force people who aren't interested in software quality to do that sort of work when all they care about is their patch merging | 15:32 |
| fungi | other than peer pressure from core review teams, like ignoring reviews from people who chronically abuse the system that way | 15:33 |
| openstackgerrit | Mohammed Naser proposed openstack/project-config master: Switch to using v3-standard-8 flavors https://review.opendev.org/c/openstack/project-config/+/773710 | 15:33 |
| mnaser | frickler, fungi, clarkb: ^ appreciate a review on that if/when you are around | 15:33 |
| frickler | mnaser: ah, great, I've been meaning to ask for some time what would be a "natural flavor" for you, because 8/8 isn't for us | 15:38 |
| mnaser | yeah the above fits much better for us (and honestly, it uses up un-used resources in our cloud) | 15:38 |
| frickler | would be interesting to gather some opinion on that from our other cloud providers, too | 15:38 |
| mnaser | we end up with idle resources on 8/8 | 15:38 |
| frickler | mnaser: how does your cpu overbooking factor look like? | 15:39 |
| mnaser | frickler: 1.25 | 15:39 |
| frickler | o.k., that's pretty low, we vary from 1.5 to 2, but then we don't run too many of those hungry CI jobs ;) | 15:40 |
| *** ysandeep|mtg is now known as ysandeep | 15:41 | |
| mnaser | frickler: so we're pretty much 1 vcpu = 1 thread | 15:41 |
| fungi | in the past when we needed to use flavors with additional memory, we artificially limited it at boot time with a kernel parameter | 15:41 |
| fungi | so that projects wouldn't accidentally merge changes which consume way more memory than our standard node size | 15:42 |
| fungi | though i think that has to be implemented in our dib elements | 15:43 |
| fungi | clarkb: ^ when you're around, maybe last-minute addition to the meeting to at least mention that | 15:46 |
| *** mlavalle has quit IRC | 15:49 | |
| fungi | i know it's been a concern in the past, but it's been a while since we last had to deal with it, and the skew was never this massive (the nodes will have 4x our usual ram) | 15:52 |
| *** ysandeep is now known as ysandeep|away | 15:54 | |
| openstackgerrit | Merged openstack/project-config master: Switch to using v3-standard-8 flavors https://review.opendev.org/c/openstack/project-config/+/773710 | 15:56 |
| clarkb | fungi: mnaser right the concern is that if you merge on a cloud with 32GB of memory then run the next set of jobs on 8GB of memory you can easily fail | 16:02 |
| clarkb | and end up wedged | 16:02 |
| clarkb | I think we need to add the kernel based limiting again if we're going to do that | 16:03 |
| clarkb | or we need to communicate to people that they need to be careful about memory use as they can wedge themselves | 16:03 |
| *** hasharAway is now known as hashar | 16:04 | |
| fungi | zbr: if you're around, can you expound on what https://review.opendev.org/635576 (path-based package declarations in bindep) was supposed to do? i didn't mention it in the release notes because i can't see how it could possibly work | 16:13 |
| fungi | maybe i'm missing some important detail | 16:13 |
| fungi | it seems like the implementation is incomplete, and will always list those packages as not installed | 16:13 |
| zbr | looking now | 16:14 |
| fungi | an easy solution might be to check whether there is a file/link at the listed path and treat that as an "installed" state | 16:14 |
| zbr | ....very long time ago. yep a little bit incomplete. Still, i think is easily fixable: for names starting with "/" it should just do a "which" and consider them as available if they exist. | 16:16 |
| fungi | i was thinking maybe just os.path.exists() | 16:16 |
| zbr | yeah that | 16:17 |
| zbr | is not perfect but compatible with major package managers, and we about the issue you reported. | 16:17 |
| *** ykarel has quit IRC | 16:20 | |
| *** lbragstad_ is now known as lbragstad | 16:21 | |
| fungi | zbr: cool, so i'll stick with the plan for now to not mention it in the release notes, we'll presumably want to either revert that feature or finish it | 16:50 |
| fungi | but doesn't have to happen before releasing | 16:51 |
| fungi | i'll go ahead and merge the release notes change and prep to tag the repo | 16:51 |
| zbr | fungi: thanks, i think is an ok approach, as likely nobody is using this new feature. | 16:51 |
| zbr | in fact I am sure nobody because we did not make a release | 16:51 |
| fungi | right, that's why i wanted to talk through it before we release | 16:53 |
| fungi | it's not mentioned in the docs either, so for now it's not discoverable anyway | 16:54 |
| fungi | otherwise i'd want to merge a quick docs amendment marking it incomplete so people aren't surprised. but it's highly unlikely anyone will find an undocumented and unannounced feature, and if they do and complain it's incomplete, oh well | 16:55 |
| fungi | zbr: in git-review news, after dissecting how zuul's quickstart job solves similar user bootstrapping issues, looks like i should be able to fix it via the rest api using the default "admin" account (present when the service is started in development_become_any_user auth mode) | 16:58 |
| fungi | hoping to give that a shot shortly | 16:58 |
| zbr | fungi: super. sorry for not being more active, a conversion of role to collection got be very busy. use of ansible-test is... a hard to forget experience. | 16:59 |
| fungi | zbr: no worries, and no need to apologize, just trying to keep you updated/included | 17:00 |
| *** marios has quit IRC | 17:04 | |
| openstackgerrit | Merged opendev/bindep master: Add missing release notes https://review.opendev.org/c/opendev/bindep/+/773179 | 17:08 |
| *** zbr1 has joined #opendev | 17:16 | |
| *** zbr has quit IRC | 17:18 | |
| *** zbr1 is now known as zbr | 17:18 | |
| openstackgerrit | Jeremy Stanley proposed opendev/zone-opendev.org master: Update floating IP for airship-citycloud mirror https://review.opendev.org/c/opendev/zone-opendev.org/+/773777 | 17:22 |
| clarkb | fungi: ^ is that new address expected to work? or we're just updating it now so that when networking works again we're ready? | 17:23 |
| fungi | clarkb: ^ if you can check that out quickly, i'll work on testing the mirror out once dns updates | 17:23 |
| clarkb | fungi: +2 from me | 17:23 |
| fungi | it is expected to be functional again (huge thanks to noonedeadpunk for getting teh right folks involved and keeping us in the loop) | 17:23 |
| clarkb | awesome | 17:23 |
| clarkb | fungi: I went ahead and approved it | 17:28 |
| fungi | thanks | 17:30 |
| openstackgerrit | Merged opendev/zone-opendev.org master: Update floating IP for airship-citycloud mirror https://review.opendev.org/c/opendev/zone-opendev.org/+/773777 | 17:34 |
| fungi | once that deploys i'll give the mirror a once-over and then approve 773240 | 17:35 |
| fungi | clarkb: looking at http://cacti.openstack.org/cacti/graph.php?action=view&local_graph_id=26&rra_id=all it doesn't seem like the gerrit patch yesterday made any difference on overall system load average (comparing against values from 24 hours ago) | 17:38 |
| clarkb | ya I wasn't sure if it would or not, just that leaky memory may have contributed to additional gc load | 17:40 |
| fungi | our nameservers are serving the new address for the citycloud mirror now, but i've still got the old one cached for another 35 minutes | 17:41 |
| *** d34dh0r53 has quit IRC | 17:42 | |
| *** ralonsoh has quit IRC | 17:45 | |
| *** d34dh0r53 has joined #opendev | 17:50 | |
| *** jpena is now known as jpena|off | 17:55 | |
| clarkb | fungi: https://mirror01.kna1.airship-citycloud.opendev.org/ loads for me now | 17:59 |
| clarkb | is there a change I can reviwe to reenable the nodepool provider? | 18:00 |
| fungi | clarkb: 773240 per above | 18:00 |
| fungi | i'll un-wip it since you're able to confirm | 18:00 |
| fungi | my cache still has another 15 minutes or so on it | 18:00 |
| clarkb | we can wait for your cache to clear it out if you like | 18:01 |
| clarkb | but ya seems to work for me | 18:01 |
| clarkb | fungi: mnaser: also it looks like the vexxhost flavor change has merged? so we're now at risk of landing changes that can only pass in vexxhost? | 18:04 |
| clarkb | has anyone looked at resurrecting the kernel boot parameter memory limits yet? | 18:04 |
| clarkb | or do we just want to accept that risk and maybe warn peopel of it | 18:05 |
| *** eolivare_ has quit IRC | 18:05 | |
| fungi | it's worth keeping in mind that we have the same situation all over with other resources... jobs might only pass in ovh because they have more available disk, or might pass somewhere else because they have better i/o bandwidth or faster/more cpus | 18:09 |
| clarkb | that is true, though I think with memory we're very likely to trip over it given the amount of swapping jobs do | 18:10 |
| fungi | we've treated ram specially that way in the past, maybe with good reason, but it's about the only thing we've held stable between nodes in different environments | 18:10 |
| openstackgerrit | Sorin Sbârnea proposed opendev/system-config master: Upgrade ansible-lint https://review.opendev.org/c/opendev/system-config/+/773782 | 18:16 |
| fungi | i thought we stopped running ansible-lint against system-config, but maybe that hasn't merged yet | 18:18 |
| zbr | fungi: yep the linter discovered that two files inside playbooks/ folder where in facts tasks, instead of being put inside tasks/ subfolder. | 18:20 |
| *** fbo|off is now known as fbo | 18:22 | |
| fungi | so either that works and the linter is being picky, or we're not using those files and they can be cleaned up? | 18:31 |
| fungi | we do have rather a bit of cruft in there we've been slowly cleaning out as we stumble across it | 18:31 |
| *** sshnaidm|ruck is now known as sshnaidm|afk | 18:34 | |
| *** dtantsur is now known as dtantsur|afk | 18:35 | |
| clarkb | fungi: the two exampels work(ed) fine | 18:35 |
| clarkb | fungi: they are task files for gitea project renames | 18:35 |
| clarkb | ansible will lookup files in the working dir aiui | 18:36 |
| clarkb | fungi: so ya I think a case of the linter being picky and wanting you to confrom to certain structure expectations beyond what the tool will accept | 18:36 |
| *** rpittau is now known as rpittau|afk | 18:39 | |
| *** hashar is now known as hasharDinner | 18:41 | |
| fungi | clarkb: i was going to removed ianw's workflow -1 vote on 773240 so we can merge it, but i guess core reviewers can't remove each other's votes? | 18:44 |
| fungi | er, was going to remove | 18:44 |
| fungi | at this point we can probably just wait for him to return anyway | 18:45 |
| zbr | combining tasks and playbooks inside the same folder is a very bad practice as you will not be able to distinguish between them without looking inside. that practice has more downsides: what if one file is has a fatal error in it and you cannot infer its type? | 18:45 |
| clarkb | fungi: ya I think you need admin? | 18:46 |
| fungi | ahh, yeah i suppose so. anyway, will wait a bit | 18:49 |
| yoctozepto | what did I break if I am getting "error: remote unpack failed: error Missing tree e659d987466862920dce31253ec61b0b63a23592" when doing `git review`? | 18:53 |
| yoctozepto | tbh, I can't see this id neither locally nor remotely | 18:55 |
| yoctozepto | :D | 18:55 |
| *** artom has quit IRC | 18:56 | |
| *** diablo_rojo_phon has joined #opendev | 18:56 | |
| *** artom has joined #opendev | 18:56 | |
| clarkb | yoctozepto: if I clone kolla-ansible git show works on that sha1 | 18:56 |
| clarkb | I cloned from gerrit not opendev too | 18:56 |
| fungi | yoctozepto: maybe you're proposing it to the wrong repo? wrong branch? | 18:57 |
| clarkb | yoctozepto: its happening in org.eclipse.jgit.transport.ReceivePack.receivePackAndCheckConnectivity() | 18:57 |
| clarkb | I think what it is saying is that the pack you've sent isn't complete | 18:57 |
| yoctozepto | well, I haven't changed much | 18:57 |
| yoctozepto | did a `git review -d` | 18:57 |
| yoctozepto | amended a patch | 18:57 |
| yoctozepto | and tried to push back | 18:57 |
| fungi | maybe pulled into a dirty fork of master? | 18:58 |
| fungi | though -d should have recreated everything | 18:58 |
| clarkb | as it apepars to be doing a connectivity check of the pack it is receiving (not of the one it has) | 18:58 |
| yoctozepto | well, this patch is on a top of a stack | 18:58 |
| yoctozepto | https://review.opendev.org/c/openstack/kolla-ansible/+/770621 | 18:58 |
| fungi | you may need to rebase the entire stack on the latest state of the first change in the stack | 18:58 |
| fungi | if you haven't done so already | 18:59 |
| yoctozepto | I haven't | 18:59 |
| yoctozepto | should I always now? | 18:59 |
| yoctozepto | feels painful | 18:59 |
| *** whoami-rajat__ has quit IRC | 18:59 | |
| fungi | no need to rebase on the target branch unless there's a merge conflict somewhere, but altering a change in the middle of a stack will leave later changes which depended on it broken anyway | 18:59 |
| yoctozepto | no, it's just the tip that I'm modifying | 19:00 |
| yoctozepto | I'll try rebasing locally and pushing the stack | 19:00 |
| clarkb | and/or try a git fsck on your local repo? | 19:00 |
| clarkb | maybe something about the local packs | 19:00 |
| fungi | that is an error i would expect if one of the parents of the change you amended is outdated, but i also wouldn't expect that state to be created by a git review -d | 19:00 |
| clarkb | fwiw cloning then git show on that ref works for me | 19:00 |
| fungi | meeting time | 19:00 |
| yoctozepto | clarkb: it works, my bad previously, haven't checked the repo but the current branch | 19:01 |
| yoctozepto | all right, rebasing the whole stack helped | 19:01 |
| yoctozepto | thanks, guys | 19:01 |
| yoctozepto | I hope we merge this monster sooner than later :D | 19:01 |
| clarkb | yoctozepto: not sure I understand " haven't checked the repo but the current branch" | 19:02 |
| yoctozepto | clarkb: checked the commit ids in the current branch, I confused the things | 19:03 |
| *** sboyron_ has quit IRC | 19:03 | |
| yoctozepto | never mind, it's getting late | 19:03 |
| *** andrewbonney has quit IRC | 19:15 | |
| *** rchurch has joined #opendev | 19:26 | |
| *** hasharDinner is now known as hashar | 19:39 | |
| openstackgerrit | Merged openstack/project-config master: Revert "Temporarily stop booting nodes in citycloud-kna1" https://review.opendev.org/c/openstack/project-config/+/773240 | 19:40 |
| *** artom has quit IRC | 19:43 | |
| *** artom has joined #opendev | 19:43 | |
| ianw | fungi: did anyone cleanup the new mirror server i tried to start? | 20:01 |
| fungi | ianw: not yet, wasn't sure if we wanted to continue with it | 20:02 |
| ianw | no particular need if we've already updated the dns, i'll remove it now | 20:02 |
| ianw | it wasn't deployed as such, because we could never talk to it | 20:03 |
| *** stand has joined #opendev | 20:07 | |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: borg-backup: implement saving a stream, use for database backups https://review.opendev.org/c/opendev/system-config/+/771738 | 20:11 |
| *** DSpider has joined #opendev | 20:15 | |
| openstackgerrit | Jeremy Stanley proposed openstack/project-config master: Move bindep to opendev tenant https://review.opendev.org/c/openstack/project-config/+/773793 | 20:24 |
| corvus | sn9 imminent | 20:24 |
| openstackgerrit | Jeremy Stanley proposed opendev/bindep master: Publish release notes https://review.opendev.org/c/opendev/bindep/+/773795 | 20:26 |
| openstackgerrit | Jeremy Stanley proposed opendev/bindep master: Build docs for OpenDev https://review.opendev.org/c/opendev/bindep/+/773796 | 20:26 |
| openstackgerrit | Jeremy Stanley proposed opendev/bindep master: Move all jobs in-repo https://review.opendev.org/c/opendev/bindep/+/773797 | 20:26 |
| johnsom | I there, one of my patches has a job in "paused" (773784) in zuul. Isn't that a manual set state? Anyone know why it's paused? | 20:30 |
| johnsom | Clicking the job link gives an empty console and not details. | 20:31 |
| johnsom | I there->Hi there. sigh | 20:31 |
| corvus | rud | 20:33 |
| fungi | johnsom: that job serves content to other jobs which are running | 20:37 |
| fungi | from what i gather, tripleo builds up a container registry and then serves the container images from it to builds of dependent jobs | 20:38 |
| fungi | this way each job doesn't have to redundantly build the same container images | 20:38 |
| johnsom | Hmm, ok. This is my first patch in tripleo in a long time, so maybe I'm missing something about that job. | 20:39 |
| fungi | it will un-pause and finish up once the dependent builds complete | 20:39 |
| johnsom | So, it built the repo, but is paused until the other patch jobs finish. Makes sense I guess. I will dig in more. | 20:39 |
| johnsom | Zuul is getting so fancy.... lol | 20:39 |
| fungi | yeah, think of that build as an ephemeral server providing content to other running builds | 20:40 |
| *** lamt has joined #opendev | 20:40 | |
| fungi | i believe this is one of tripleo's workarounds for avoiding dockerhub rate limits | 20:40 |
| johnsom | Ah, yeah, those are.... interesting | 20:41 |
| openstackgerrit | Jeremy Stanley proposed opendev/bindep master: Move all jobs in-repo https://review.opendev.org/c/opendev/bindep/+/773797 | 21:14 |
| *** zoharm has quit IRC | 21:14 | |
| openstackgerrit | Jeremy Stanley proposed opendev/bindep master: Build docs for OpenDev https://review.opendev.org/c/opendev/bindep/+/773796 | 21:14 |
| openstackgerrit | Clark Boylan proposed opendev/system-config master: Run gerrit 3.2 and 3.3 functional tests https://review.opendev.org/c/opendev/system-config/+/773807 | 21:29 |
| clarkb | ianw: ^ first attempt at testing with 3.2 and 3.3 gerrits | 21:30 |
| clarkb | no idea if I got that right | 21:30 |
| ianw | looks about right | 21:39 |
| *** hashar has quit IRC | 21:40 | |
| openstackgerrit | Merged opendev/bindep master: Publish release notes https://review.opendev.org/c/opendev/bindep/+/773795 | 21:53 |
| openstackgerrit | Merged openstack/project-config master: Move bindep to opendev tenant https://review.opendev.org/c/openstack/project-config/+/773793 | 22:14 |
| ianw | clarkb/fungi: did you have any particular thoughts on the --append logs (773570). i'm thinking i can actually go ahead with our other changes; the only problem is that the prune isn't actually pruning so we need to continue to keep a close eye on disk-space | 22:29 |
| ianw | i have run a manual prune on vexxhost to the parameters in 771738 and we have 136G free now | 22:30 |
| *** diablo_rojo has joined #opendev | 22:53 | |
| *** slaweq has quit IRC | 22:55 | |
| *** mlavalle has joined #opendev | 22:56 | |
| *** mchc has quit IRC | 23:01 | |
| *** JayF has quit IRC | 23:13 | |
| *** JayF has joined #opendev | 23:17 | |
| openstackgerrit | Ian Wienand proposed opendev/system-config master: borg-backup: implement saving a stream, use for database backups https://review.opendev.org/c/opendev/system-config/+/771738 | 23:41 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
