goldyfruit___ | thanks eandersson for Designate fixes :) | 00:11 |
---|---|---|
*** factor has quit IRC | 01:23 | |
*** factor has joined #openstack-kolla | 01:25 | |
*** factor has quit IRC | 01:28 | |
*** factor has joined #openstack-kolla | 01:28 | |
*** factor has quit IRC | 01:31 | |
*** factor has joined #openstack-kolla | 01:31 | |
*** factor has quit IRC | 02:03 | |
*** factor has joined #openstack-kolla | 02:04 | |
*** factor has quit IRC | 02:07 | |
*** factor has joined #openstack-kolla | 02:07 | |
*** dannins has joined #openstack-kolla | 02:09 | |
*** factor has quit IRC | 02:10 | |
*** factor has joined #openstack-kolla | 02:10 | |
*** factor has quit IRC | 02:14 | |
*** factor has joined #openstack-kolla | 02:15 | |
*** factor has quit IRC | 03:37 | |
*** factor has joined #openstack-kolla | 03:38 | |
*** factor has quit IRC | 03:41 | |
*** factor has joined #openstack-kolla | 03:41 | |
*** factor has quit IRC | 03:42 | |
*** factor has joined #openstack-kolla | 03:43 | |
*** factor has quit IRC | 03:45 | |
*** factor has joined #openstack-kolla | 03:46 | |
*** gkadam has joined #openstack-kolla | 03:46 | |
*** gkadam has quit IRC | 03:47 | |
*** factor has quit IRC | 04:49 | |
*** factor has joined #openstack-kolla | 04:50 | |
*** factor has quit IRC | 04:52 | |
*** factor has joined #openstack-kolla | 04:53 | |
*** factor has quit IRC | 04:56 | |
*** factor has joined #openstack-kolla | 04:57 | |
*** Luzi has joined #openstack-kolla | 04:59 | |
*** pcaruana has joined #openstack-kolla | 05:11 | |
*** radeks_ has joined #openstack-kolla | 05:22 | |
*** janki has joined #openstack-kolla | 06:01 | |
*** factor has quit IRC | 06:02 | |
*** noxoid has quit IRC | 06:21 | |
*** owalsh has quit IRC | 06:23 | |
*** logan- has quit IRC | 06:23 | |
*** owalsh has joined #openstack-kolla | 06:23 | |
*** noxoid has joined #openstack-kolla | 06:24 | |
*** logan- has joined #openstack-kolla | 06:26 | |
*** cah_link has joined #openstack-kolla | 06:29 | |
*** Blinkiz has quit IRC | 06:30 | |
*** radeks_ has quit IRC | 06:32 | |
*** janki has quit IRC | 06:39 | |
*** radeks has joined #openstack-kolla | 06:53 | |
*** rpittau|afk is now known as rpittau | 06:55 | |
*** tonythomas has joined #openstack-kolla | 06:58 | |
*** janki has joined #openstack-kolla | 07:00 | |
openstackgerrit | Chason Chan proposed openstack/kolla master: Add OVMF package to support UEFI instance https://review.opendev.org/682280 | 07:02 |
yoctozepto | morning | 07:13 |
*** ivve has joined #openstack-kolla | 07:32 | |
*** FlorianFa has quit IRC | 07:36 | |
*** FlorianFa has joined #openstack-kolla | 07:41 | |
hrw | morning | 07:50 |
*** janki has quit IRC | 07:57 | |
*** janki has joined #openstack-kolla | 07:57 | |
*** dougsz has joined #openstack-kolla | 07:59 | |
*** BaptisteGer has joined #openstack-kolla | 08:05 | |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: CI: Add bifrost Ubuntu job https://review.opendev.org/682292 | 08:17 |
*** k_mouza has joined #openstack-kolla | 08:18 | |
*** luksky has joined #openstack-kolla | 08:30 | |
yoctozepto | hrw: https://review.opendev.org/#/c/681694/2/docker/mariadb/Dockerfile.j2@49 | 08:32 |
hrw | yoctozepto: moved from xtrabackup image | 08:33 |
yoctozepto | hrw: understood | 08:33 |
yoctozepto | somehow xtrabackup got into installable state before we merged its removal, creepy | 08:35 |
hrw | yoctozepto: not 'somehow'. | 08:35 |
hrw | yoctozepto: infra ubuntu mirror stopped syncing | 08:35 |
hrw | and several things became uninstallable | 08:35 |
hrw | mirror synced, xtrabackup installs. | 08:35 |
yoctozepto | hrw: yeah, I was referring to our luck though | 08:37 |
hrw | yoctozepto: replied to ppc64le review too | 08:38 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: CI: Add bifrost Ubuntu job https://review.opendev.org/682292 | 08:43 |
yoctozepto | hrw: https://review.opendev.org/682228 | 08:46 |
yoctozepto | trivial but I want independent core eye to look at it | 08:46 |
yoctozepto | :-) | 08:46 |
hrw | +2 | 08:49 |
*** skramaja has joined #openstack-kolla | 08:50 | |
*** ondrejduchon has joined #openstack-kolla | 08:54 | |
yoctozepto | thanks hrw | 08:57 |
mgoddard | morning | 08:58 |
mgoddard | just looking at horizon patch yoctozepto | 08:58 |
openstackgerrit | Ondrej Duchon proposed openstack/kolla-ansible master: Added support for custom logging.conf https://review.opendev.org/679812 | 09:00 |
*** jovial[m] has joined #openstack-kolla | 09:00 | |
*** ondrejduchon has quit IRC | 09:00 | |
*** k_mouza has quit IRC | 09:08 | |
*** k_mouza has joined #openstack-kolla | 09:09 | |
hrw | mgoddard: so, what do we do with ppc64le? | 09:22 |
hrw | mgoddard: basically no one maintains it (I did probably all ppc64le related commits), there are no noticable users | 09:23 |
*** mgoddard has quit IRC | 09:26 | |
hrw | hm. we have some ppc64le changes from ibm folks | 09:26 |
*** mgoddard has joined #openstack-kolla | 09:28 | |
openstackgerrit | Doug Szumski proposed openstack/kolla-ansible master: Update configuration on overriding config files https://review.opendev.org/681301 | 09:30 |
openstackgerrit | Chason Chan proposed openstack/kolla-ansible master: Update "openstack_release" variable to static brach name https://review.opendev.org/681633 | 09:34 |
openstackgerrit | Doug Szumski proposed openstack/kolla-ansible master: Update configuration on overriding config files https://review.opendev.org/681301 | 09:35 |
openstackgerrit | Merged openstack/kolla-ansible master: Fix enforced horizon redirect to https https://review.opendev.org/682228 | 10:03 |
openstackgerrit | Merged openstack/kolla-ansible master: CI: Test accessing dashboard https://review.opendev.org/676412 | 10:03 |
*** gfidente has joined #openstack-kolla | 10:13 | |
*** dougsz has quit IRC | 10:19 | |
yoctozepto | mgoddard: thanks, replied | 10:20 |
yoctozepto | hrw: imho leave it be if it adds no real burden on us | 10:21 |
yoctozepto | declare as not supported by us and that's it | 10:21 |
hrw | ok | 10:22 |
*** wuchunyang has joined #openstack-kolla | 10:24 | |
*** wuchunyang has quit IRC | 10:26 | |
*** wuchunyang has joined #openstack-kolla | 10:30 | |
*** wuchunyang has quit IRC | 10:33 | |
*** dougsz has joined #openstack-kolla | 10:39 | |
yoctozepto | mgoddard: https://review.opendev.org/670690 | 10:47 |
yoctozepto | ^ made more flexible and fixed for dashboard and bifrost testing | 10:48 |
yoctozepto | let me know if you are happy with it so that I can rebase ipv6 on it and happily work on it | 10:49 |
openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Catch errors and changes in kolla_toolbox module https://review.opendev.org/682340 | 10:56 |
mgoddard | yoctozepto: will take a look | 11:04 |
hrw | mgoddard: commented | 11:09 |
openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Catch errors and changes in kolla_toolbox module https://review.opendev.org/682340 | 11:11 |
*** vmixor has joined #openstack-kolla | 11:19 | |
*** kplant has joined #openstack-kolla | 11:22 | |
openstackgerrit | Mark Goddard proposed openstack/kolla-ansible stable/stein: Fix kolla_toolbox error handling https://review.opendev.org/682344 | 11:33 |
openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Catch errors and changes in kolla_toolbox module https://review.opendev.org/682340 | 11:42 |
openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Catch errors and changes in kolla_toolbox module https://review.opendev.org/682340 | 11:43 |
openstackgerrit | Merged openstack/kolla master: Remove unneeded Zuul branch matcher https://review.opendev.org/681975 | 11:50 |
openstackgerrit | Jan Horstmann proposed openstack/kolla master: Remove exemption for non-systemd Ubuntu in bifrost https://review.opendev.org/682349 | 11:51 |
*** gfidente has quit IRC | 11:55 | |
openstackgerrit | Merged openstack/kolla-ansible stable/stein: CI: Test accessing dashboard https://review.opendev.org/677684 | 11:56 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla master: Remove systemd-sysv package from openstack-base for RPM-based https://review.opendev.org/682350 | 12:02 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla master: Add package systemd-sysv to bifrost-deploy image https://review.opendev.org/679936 | 12:05 |
*** dave-mccowan has joined #openstack-kolla | 12:08 | |
*** luksky has quit IRC | 12:09 | |
*** dave-mccowan has quit IRC | 12:12 | |
*** dave-mccowan has joined #openstack-kolla | 12:13 | |
*** goldyfruit___ has quit IRC | 12:15 | |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: CI: Add bifrost Ubuntu job https://review.opendev.org/682292 | 12:25 |
*** gfidente has joined #openstack-kolla | 12:35 | |
openstackgerrit | Jan Horstmann proposed openstack/kolla master: Remove exemption for non-systemd Ubuntu in bifrost https://review.opendev.org/682349 | 12:38 |
openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Update "openstack_release" variable to static brach name https://review.opendev.org/681633 | 12:42 |
openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Add custom filters for checking services https://review.opendev.org/665922 | 12:48 |
*** luksky has joined #openstack-kolla | 12:57 | |
*** mixor has joined #openstack-kolla | 13:01 | |
*** vmixor has quit IRC | 13:01 | |
*** mixor has quit IRC | 13:01 | |
*** mixor has joined #openstack-kolla | 13:02 | |
openstackgerrit | Merged openstack/kolla master: Drop Percona XtraBackup in favour of MariaDB Backup https://review.opendev.org/681694 | 13:14 |
*** KeithMnemonic has joined #openstack-kolla | 13:18 | |
*** spiette has quit IRC | 13:22 | |
*** vmixor has joined #openstack-kolla | 13:22 | |
*** mixor has quit IRC | 13:22 | |
*** wuchunyang has joined #openstack-kolla | 13:23 | |
*** skramaja has quit IRC | 13:27 | |
*** spiette has joined #openstack-kolla | 13:27 | |
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org will be offline briefly starting at 14:00 UTC (that's roughly 30 minutes from now) for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html | 13:28 | |
openstackgerrit | Mark Goddard proposed openstack/kolla-ansible master: Catch errors and changes in kolla_toolbox module https://review.opendev.org/682340 | 13:34 |
*** Luzi has quit IRC | 13:38 | |
*** wuchunyang has quit IRC | 13:43 | |
*** signed8bit has joined #openstack-kolla | 13:44 | |
*** pcaruana has quit IRC | 13:45 | |
*** goldyfruit___ has joined #openstack-kolla | 13:50 | |
openstackgerrit | Doug Szumski proposed openstack/kolla-ansible master: Factor out nova-cell role https://review.opendev.org/675659 | 13:51 |
openstackgerrit | Doug Szumski proposed openstack/kolla-ansible master: Factor out nova-cell role https://review.opendev.org/675659 | 14:00 |
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is offline briefly for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html | 14:04 | |
*** ChanServ changes topic to "The Gerrit service on review.opendev.org is offline briefly for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html" | 14:04 | |
*** pcaruana has joined #openstack-kolla | 14:07 | |
kplant | anyone else run into prometheus_openstack_exporter restarting every minute due to "invalid status (401): {\"error\": {\"message\": \"The request you have made requires authentication.\", \"code\": 401, \"title\": \"Unauthorized\"}}" ? | 14:08 |
*** mflynn has joined #openstack-kolla | 14:08 | |
*** janki has quit IRC | 14:09 | |
kplant | i think it's this: https://github.com/openstack-exporter/openstack-exporter/pull/3 | 14:24 |
kplant | i'm thinking stable/stein needs to build with >= 0.0.8 | 14:26 |
mgoddard | oh kplant I remember priteau had a thing for that | 14:28 |
mgoddard | kplant: https://review.opendev.org/#/c/673487/ | 14:28 |
kplant | ah yeah | 14:29 |
kplant | i have the same problem with 0.0.7 in stable/stein | 14:29 |
kplant | i'm going to try to rebuild with 0.0.9 and see if the problem goes away with no other changes | 14:29 |
kplant | i don't think that fix is complete, that only doesn't include cinder in openstack-exporter if cinder is disabled | 14:30 |
kplant | if you are using cinder, it still doesn't work | 14:30 |
kplant | and then the bug i pasted earlier applies | 14:30 |
kplant | ( i think ) | 14:30 |
mgoddard | oh ok | 14:32 |
kplant | i'll let you know the results either way - would it be appropriate to backport this to stable/stein if that's the case? | 14:32 |
*** BjoernT has joined #openstack-kolla | 14:33 | |
*** ChanServ changes topic to "Topic for #openstack-kolla is: support: ask.openstack.org | New to Kolla: docs @ https://docs.openstack.org/kolla/latest/ | | Kolla IRC meetngs on Wednesdays @ 15:00 UTC - see agenda @ https://goo.gl/OXB0DL - IRC channel is *LOGGED* @ http://goo.gl/3mzZ7b" | 14:36 | |
-openstackstatus- NOTICE: The Gerrit outage portion of the current maintenance is complete and the service is back on line, however reindexing for renamed repositories is still underway and some Zuul job fixes are in the process of being applied | 14:36 | |
*** goldyfruit_ has joined #openstack-kolla | 14:36 | |
*** goldyfruit___ has quit IRC | 14:39 | |
mnasiadka | good afternoon | 14:40 |
mgoddard | hi mnasiadka | 14:53 |
*** wuchunyang has joined #openstack-kolla | 14:58 | |
*** goldyfruit___ has joined #openstack-kolla | 15:03 | |
*** wuchunyang has quit IRC | 15:03 | |
*** wuchunyang has joined #openstack-kolla | 15:04 | |
dougsz | Any friendly cores about to take a quick look at this one? https://review.opendev.org/#/c/619756 | 15:05 |
*** goldyfruit_ has quit IRC | 15:05 | |
*** cah_link has quit IRC | 15:07 | |
*** openstackgerrit has quit IRC | 15:08 | |
mnasiadka | friendly? looking for a +2? ;) | 15:08 |
mnasiadka | don't look at me, there's a policy I can't jump through :) | 15:09 |
mnasiadka | yoctozepto: fancy looking at dougsz's review? :) | 15:09 |
*** BjoernT_ has joined #openstack-kolla | 15:10 | |
*** BjoernT has quit IRC | 15:10 | |
dougsz | mnasiadka: indeed! it has been festering around for some time and would be nice to land it | 15:11 |
hrw | yoctozepto got ansible certificate recently so he is the most qualified | 15:13 |
*** ivve has quit IRC | 15:20 | |
*** k_mouza has quit IRC | 15:28 | |
*** k_mouza_ has joined #openstack-kolla | 15:29 | |
yoctozepto | mnasiadka: what policy? | 15:30 |
mnasiadka | yoctozepto: me and Mark are now from the same company, and Doug is also from the same one - it would be a single company merge - I don't think that's in the spirit of community :) | 15:31 |
yoctozepto | mnasiadka: ah, sure, you should have done b2b then and just 'provide services' | 15:32 |
mnasiadka | yoctozepto: and be a thief in the eyes of polish dictator? no thanks :) | 15:33 |
yoctozepto | kidding obviously, will take a look | 15:33 |
yoctozepto | you mean ducktator obviously | 15:33 |
mnasiadka | you said it :) | 15:34 |
yoctozepto | I think ppl are demonizing them too much, it's kind of christian socialism what they represent today - obviously don't like capitalists too much :-) | 15:38 |
*** k_mouza_ has quit IRC | 15:39 | |
mnasiadka | yoctozepto: no politician talks here please :) (worrying somebody will unleash the Brexit talk) :) | 15:42 |
yoctozepto | mnasiadka: oh noez, that would be nightmarish | 15:42 |
mnasiadka | demonizing is what media does today to everything, but there are some economically frightening things happening :) | 15:43 |
*** wuchunyang has quit IRC | 15:43 | |
*** openstackgerrit has joined #openstack-kolla | 15:44 | |
openstackgerrit | Merged openstack/kayobe-config master: Fix .gitreview after project rename https://review.opendev.org/682373 | 15:44 |
*** luksky has quit IRC | 15:44 | |
yoctozepto | mnasiadka: let's move to this talk to a beer (water) meeting, shall we? | 15:45 |
mnasiadka | Yeah, you plan coming to western Poland?:) | 15:45 |
*** wuchunyang has joined #openstack-kolla | 15:46 | |
openstackgerrit | Merged openstack/kayobe-config-dev master: Fix .gitreview after rename https://review.opendev.org/682374 | 15:49 |
*** wuchunyang has quit IRC | 15:51 | |
yoctozepto | mnasiadka: maybe, you mean like Poznan perhaps? | 15:55 |
yoctozepto | you are invited to visit eastern Poland as well | 15:55 |
*** k_mouza has joined #openstack-kolla | 15:58 | |
openstackgerrit | Viktor Michalek proposed openstack/kolla-ansible master: Fix for deployment Debian Buster Stein based images https://review.opendev.org/680745 | 16:01 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config-dev stable/stein: Fix .gitreview after rename https://review.opendev.org/682395 | 16:05 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config-dev stable/rocky: Fix .gitreview after rename https://review.opendev.org/682396 | 16:06 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config-dev stable/queens: Fix .gitreview after rename https://review.opendev.org/682397 | 16:06 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config-dev stable/pike: Fix .gitreview after rename https://review.opendev.org/682399 | 16:07 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config stable/stein: Fix .gitreview after project rename https://review.opendev.org/682400 | 16:09 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config stable/rocky: Fix .gitreview after project rename https://review.opendev.org/682402 | 16:10 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config stable/queens: Fix .gitreview after project rename https://review.opendev.org/682403 | 16:10 |
openstackgerrit | Merged openstack/kayobe-config-dev stable/pike: Fix .gitreview after rename https://review.opendev.org/682399 | 16:13 |
openstackgerrit | Viktor Michalek proposed openstack/kolla-ansible master: Fix for deployment Debian Buster Stein based images https://review.opendev.org/680745 | 16:14 |
openstackgerrit | Merged openstack/kayobe-config-dev stable/stein: Fix .gitreview after rename https://review.opendev.org/682395 | 16:18 |
openstackgerrit | Merged openstack/kayobe-config stable/stein: Fix .gitreview after project rename https://review.opendev.org/682400 | 16:21 |
yoctozepto | dougsz: replied on https://review.opendev.org/619756 | 16:21 |
yoctozepto | or, well, actually commented | 16:21 |
yoctozepto | mgoddard: let's slap the vxlan patch if you don't mind ;-) | 16:22 |
mgoddard | slap it in the face? | 16:22 |
yoctozepto | mgoddard: if it adds w+1 | 16:23 |
openstackgerrit | Merged openstack/kayobe-config-dev stable/rocky: Fix .gitreview after rename https://review.opendev.org/682396 | 16:23 |
openstackgerrit | Merged openstack/kayobe-config stable/queens: Fix .gitreview after project rename https://review.opendev.org/682403 | 16:27 |
*** goldyfruit_ has joined #openstack-kolla | 16:28 | |
openstackgerrit | Merged openstack/kayobe-config-dev stable/queens: Fix .gitreview after rename https://review.opendev.org/682397 | 16:29 |
dougsz | thanks yoctozepto, much appreciated | 16:29 |
openstackgerrit | Merged openstack/kayobe-config stable/rocky: Fix .gitreview after project rename https://review.opendev.org/682402 | 16:30 |
*** goldyfruit___ has quit IRC | 16:31 | |
openstackgerrit | Mark Goddard proposed openstack/kolla master: Add kayobe to deliverables.yaml https://review.opendev.org/674510 | 16:36 |
openstackgerrit | Mark Goddard proposed openstack/kayobe stable/stein: Fix after x/kayobe rename https://review.opendev.org/682408 | 16:39 |
*** gfidente has quit IRC | 16:39 | |
openstackgerrit | Mark Goddard proposed openstack/kayobe master: Update docs after repo rename x/ to openstack/ https://review.opendev.org/674505 | 16:41 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config master: Update docs for repo rename x/ to openstack/ https://review.opendev.org/674507 | 16:43 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config-dev master: Update links to docs and IRC for kolla governance https://review.opendev.org/674508 | 16:44 |
openstackgerrit | Mark Goddard proposed openstack/kayobe-config master: Update links to docs and IRC for kolla governance https://review.opendev.org/674511 | 16:46 |
*** k_mouza has quit IRC | 16:46 | |
openstackgerrit | Viktor Michalek proposed openstack/kolla master: Fix images based on Debian Buster Stein packages https://review.opendev.org/680447 | 16:54 |
*** JamesBenson has joined #openstack-kolla | 16:56 | |
yoctozepto | hrw: hmm, now that redhat is ibm they obviously like ppc64le | 16:59 |
*** rpittau is now known as rpittau|afk | 16:59 | |
*** jonaspaulo has joined #openstack-kolla | 17:15 | |
*** pomac has quit IRC | 17:16 | |
*** pomac has joined #openstack-kolla | 17:22 | |
kplant | mgoddard: switching to prometheus openstack exporter 0.0.9 did clear up the issue | 17:23 |
kplant | i think a combination of 0.0.9 and priteau's change for when cinder is not deployed is the right approach | 17:24 |
*** goldyfruit_ has quit IRC | 17:28 | |
openstackgerrit | Radosław Piliszek proposed openstack/kolla stable/rocky: Add pciutils package to zun-compute https://review.opendev.org/682436 | 17:29 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla master: Add ceph-common package to zun-compute image https://review.opendev.org/682437 | 17:37 |
*** dougsz has quit IRC | 17:40 | |
openstackgerrit | James Kirsch proposed openstack/kolla-ansible master: Refactor service, endpoint and user registration https://review.opendev.org/587590 | 18:02 |
*** goldyfruit has joined #openstack-kolla | 18:05 | |
*** luksky has joined #openstack-kolla | 18:24 | |
openstackgerrit | Merged openstack/kolla master: Remove exemption for non-systemd Ubuntu in bifrost https://review.opendev.org/682349 | 18:38 |
JamesBenson | git question for anyone who is free... I'm trying to do a bug fix, I've changed my branch to "On branch bug/1837767" But the title is not my bug... | 18:48 |
JamesBenson | Sorry, I fix bugs, but not frequently enough to remember the process well. :-( | 18:48 |
JamesBenson | This is the bug I trying to fix at the moment... https://bugs.launchpad.net/bugs/1837767 | 18:50 |
openstack | Launchpad bug 1837767 in kolla rocky "[centos-binary-rocky-xenapi] openstack command failed in nova-compute container" [Undecided,Incomplete] - Assigned to James (james-o-benson) | 18:50 |
JamesBenson | This is my git-review bug title that was put in there: 'Merge "Drop Percona XtraBackup in favour of MariaDB Backup"' | 18:50 |
*** openstackgerrit has quit IRC | 18:52 | |
yoctozepto | JamesBenson: you need to create your commit | 19:02 |
yoctozepto | when you branch off | 19:02 |
JamesBenson | okay | 19:02 |
yoctozepto | you start at the same commit that was on the branch you branched off | 19:02 |
JamesBenson | ? | 19:02 |
yoctozepto | you stage some files | 19:03 |
yoctozepto | commit changes | 19:03 |
yoctozepto | and send to review | 19:03 |
yoctozepto | branch name is irrelevant - it is set as change topic | 19:03 |
yoctozepto | purely informative | 19:03 |
kplant | ^ right | 19:03 |
kplant | make change, git add, git commit, git-review | 19:04 |
kplant | and iirc that's it | 19:04 |
noxoid | https://docs.openstack.org/infra/manual/developers.html#development-workflow | 19:04 |
noxoid | was very helpful for me | 19:04 |
yoctozepto | JamesBenson: https://git-scm.com/book/en/v2/Git-Branching-Basic-Branching-and-Merging | 19:04 |
yoctozepto | and what noxoid posted | 19:04 |
yoctozepto | (just went to copy that) | 19:04 |
noxoid | :P | 19:05 |
yoctozepto | I believe the git book is well-written for either gaining or refreshing information on git | 19:05 |
JamesBenson | how do I get off this branch and keep changes? | 19:05 |
yoctozepto | and then openstack (gerrit) specific stuff is in there | 19:05 |
yoctozepto | JamesBenson: whatever you commit stays in the repo | 19:05 |
yoctozepto | unless you do some "reset" | 19:06 |
yoctozepto | to change the branch simply checkout another branch | 19:06 |
yoctozepto | I wholeheartedly recommend you read the great git book :-) | 19:06 |
JamesBenson | So I did the fix, then "git commit -a --amend" | 19:07 |
*** goldyfruit_ has joined #openstack-kolla | 19:07 | |
JamesBenson | then just git-review right? | 19:07 |
yoctozepto | JamesBenson: amend only if you already had your commit | 19:07 |
yoctozepto | otherwise you amend something in there | 19:07 |
yoctozepto | which, well, may cause you some real trouble | 19:07 |
JamesBenson | oh man... | 19:07 |
JamesBenson | okay | 19:07 |
yoctozepto | so most likely just | 19:08 |
yoctozepto | git commit -a | 19:08 |
yoctozepto | will work for your case | 19:08 |
yoctozepto | and then | 19:08 |
yoctozepto | git review | 19:08 |
*** goldyfruit has quit IRC | 19:09 | |
JamesBenson | damn it... I think I just submitted it... | 19:09 |
JamesBenson | yeah, nothing to commit, clean working tree | 19:10 |
yoctozepto | then you "did nothing"... according to git at least ;-) | 19:10 |
* JamesBenson so confused! | 19:12 | |
JamesBenson | okay | 19:12 |
*** openstackgerrit has joined #openstack-kolla | 19:14 | |
openstackgerrit | Merged openstack/kayobe master: Fix after x/kayobe rename https://review.opendev.org/682371 | 19:14 |
JamesBenson | okay, next stupid question, why am I getting a "fatal: unable to access 'https://github.com/openstack/kolla.git/': The requested URL returned error" | 19:17 |
JamesBenson | I did a "git push --set-upstream origin bug/1837767" | 19:17 |
JamesBenson | oh nevermind | 19:17 |
JamesBenson | sorry | 19:18 |
yoctozepto | yeah, you cannot push there ;-) | 19:18 |
yoctozepto | no problem, everybody learned git the hard way at some point in their lives | 19:19 |
yoctozepto | I mean current git users obviously | 19:19 |
JamesBenson | merge conflict? | 19:20 |
JamesBenson | should I just do the 'git rebase --abort' and and 'git review -R' | 19:21 |
yoctozepto | oh gosh, you must have done something really bad :D | 19:21 |
JamesBenson | ::shakes head:: | 19:21 |
JamesBenson | All I know is that I had the fix, I cloned the kolla repo, applied the fix... then hell broke out. | 19:22 |
yoctozepto | you have the changes you want to apply stored somewhere safely | 19:22 |
JamesBenson | yeah | 19:22 |
JamesBenson | it's small | 19:22 |
JamesBenson | nuclear option... | 19:22 |
yoctozepto | let me step you trough this then | 19:22 |
JamesBenson | thank you! | 19:23 |
yoctozepto | no need to nuke | 19:23 |
yoctozepto | try now | 19:23 |
yoctozepto | git fetch origin | 19:23 |
yoctozepto | does it seem to work well? | 19:23 |
JamesBenson | yeah | 19:23 |
JamesBenson | "master -> origin/master" | 19:23 |
yoctozepto | ok, let's then abort whatever you have done there | 19:24 |
JamesBenson | This fix is for rocky mind you | 19:24 |
yoctozepto | rocky only? | 19:24 |
yoctozepto | we should start with master really | 19:24 |
JamesBenson | well probably applicable to other version as well, just only tried on Rocky | 19:24 |
JamesBenson | okay | 19:24 |
yoctozepto | git rebase --abort | 19:24 |
yoctozepto | if you have not already | 19:24 |
yoctozepto | and also | 19:24 |
JamesBenson | I've been busting butt trying to get it on Rocky since it's Xen stuff and that's already outdated. | 19:25 |
yoctozepto | git reset --hard | 19:25 |
yoctozepto | gets us to some probably clean state | 19:25 |
JamesBenson | okay I'll issue those commands. | 19:25 |
*** dciabrin has quit IRC | 19:25 | |
JamesBenson | "HEAD is now at 217899cf5 Fixed Xen package issue" | 19:25 |
yoctozepto | git checkout master | 19:25 |
JamesBenson | "Your branch is behind 'origin/master' by 2 commits, and can be fast-forwarded." | 19:25 |
yoctozepto | git reset origin/master | 19:26 |
JamesBenson | Unstaged changes after reset: | 19:26 |
JamesBenson | Mdocker/bifrost/bifrost-deploy/Dockerfile.j2 | 19:26 |
JamesBenson | I didn't modify bifrost stuff! | 19:26 |
yoctozepto | git reset --hard | 19:26 |
yoctozepto | I know | 19:26 |
JamesBenson | "HEAD is now at d5f081a87 Merge "Remove exemption for non-systemd Ubuntu in bifrost"" | 19:26 |
yoctozepto | I should have issued a merge there instead of reset, sorry, but we are there anyway | 19:27 |
yoctozepto | now do | 19:27 |
yoctozepto | git checkout -b bug/that-pretty-number-here | 19:27 |
JamesBenson | hahaha, okay | 19:27 |
yoctozepto | now using editor of your choice apply the fix you want | 19:28 |
JamesBenson | "fatal: A branch named 'bug/1837767' already exists" | 19:28 |
yoctozepto | git branch -D bug/1837767 | 19:28 |
yoctozepto | and it's gone | 19:28 |
yoctozepto | then repeat -b | 19:28 |
JamesBenson | "Switched to a new branch 'bug/1837767" | 19:28 |
yoctozepto | and now the editor then | 19:28 |
JamesBenson | applying fix... one moment. | 19:28 |
yoctozepto | (ed, sed, awk, patch also count) | 19:28 |
yoctozepto | then do | 19:30 |
yoctozepto | git commit -a | 19:30 |
yoctozepto | it will pop up another editor for filling in the commit message | 19:30 |
yoctozepto | you should also include a line at the bottom like | 19:30 |
yoctozepto | Closes-bug: #1837767 | 19:30 |
openstack | bug 1837767 in kolla rocky "[centos-binary-rocky-xenapi] openstack command failed in nova-compute container" [Undecided,Incomplete] https://launchpad.net/bugs/1837767 - Assigned to James (james-o-benson) | 19:30 |
yoctozepto | yeah, thank you openstack | 19:31 |
JamesBenson | that closes-bug should be uncommented right? | 19:32 |
yoctozepto | yeah | 19:32 |
JamesBenson | ok | 19:32 |
JamesBenson | nearly there then | 19:32 |
yoctozepto | the number can be hash-preceded or not too | 19:32 |
yoctozepto | both are accepted | 19:33 |
yoctozepto | anyways, after saving this message file, git will let you do | 19:33 |
yoctozepto | git review | 19:33 |
yoctozepto | which sends the patch to gerrit | 19:33 |
openstackgerrit | James Benson proposed openstack/kolla master: Corrects missing packages from Xen https://review.opendev.org/682470 | 19:34 |
*** dciabrin has joined #openstack-kolla | 19:34 | |
JamesBenson | yay! git review actually works. | 19:34 |
yoctozepto | congrats on your first submission, JamesBenson | 19:34 |
JamesBenson | sad part is that it isn't my first! | 19:34 |
JamesBenson | I'm just naturally horrible with git. | 19:35 |
yoctozepto | not first in gerrit | 19:35 |
yoctozepto | but first to kolla | 19:35 |
yoctozepto | let's keep it that way | 19:35 |
JamesBenson | not first there either! | 19:35 |
yoctozepto | https://review.opendev.org/#/q/owner:%22James+Benson%22+status:open | 19:35 |
yoctozepto | sorry | 19:35 |
yoctozepto | https://review.opendev.org/#/q/owner:%22James+Benson%22 | 19:35 |
yoctozepto | still, first in kolla | 19:35 |
yoctozepto | previous in k-a | 19:36 |
yoctozepto | let's pretend kolla is much different ;-) | 19:36 |
JamesBenson | lol | 19:36 |
JamesBenson | regardless, thank you yoctozepto | 19:36 |
JamesBenson | We need to make an AI chat bot out of this... | 19:37 |
yoctozepto | JamesBenson: yw | 19:37 |
yoctozepto | sure, I make an excellent bot | 19:37 |
JamesBenson | :-) | 19:38 |
JamesBenson | now for Jenkins to freak out... | 19:38 |
JamesBenson | lol | 19:38 |
yoctozepto | we've got Zuul | 19:39 |
JamesBenson | Zuul... my stuff always freaks out.. | 19:39 |
JamesBenson | lol | 19:39 |
yoctozepto | we'll see ;-) | 19:39 |
*** kplant has quit IRC | 19:40 | |
yoctozepto | you've got first comment ;p | 19:42 |
yoctozepto | and I'm going to bed | 19:43 |
yoctozepto | good night | 19:43 |
JamesBenson | What's the difference between "git review" and "git review -R" | 19:43 |
JamesBenson | night | 19:43 |
*** mflynn has quit IRC | 19:44 | |
*** igordc has joined #openstack-kolla | 19:51 | |
openstackgerrit | James Kirsch proposed openstack/kolla-ansible master: Refactor service, endpoint and user registration https://review.opendev.org/587590 | 19:55 |
*** admin0 has left #openstack-kolla | 19:58 | |
*** hjensas has quit IRC | 20:10 | |
*** luksky has quit IRC | 20:16 | |
*** hjensas has joined #openstack-kolla | 20:16 | |
*** luksky has joined #openstack-kolla | 20:17 | |
*** pcaruana has quit IRC | 20:38 | |
*** BjoernT_ has quit IRC | 20:46 | |
openstackgerrit | Viktor Michalek proposed openstack/kolla-ansible master: Fix for deployment Debian Buster Stein based images https://review.opendev.org/680745 | 20:51 |
*** k_mouza has joined #openstack-kolla | 21:03 | |
*** JamesBenson has quit IRC | 21:06 | |
*** k_mouza has quit IRC | 21:07 | |
mnasiadka | yoctozepto: Poznan is between Wroclaw and Szczecin, ask hrw if he wants to come and we can set up something ;) | 21:36 |
mnasiadka | mgoddard: https://twitter.com/centosproject/status/1173652996305170432?s=21 - so now we wait for RDO | 21:37 |
hrw | mnasiadka: sounds like plan | 21:58 |
hrw | yoctozepto: remember that I work at Red Hat. over 6 years now. | 21:59 |
hrw | yoctozepto: otherwise ppc64le support would not be there due to lack of access to hw | 21:59 |
mnasiadka | hrw: good, I see your ppc64le deprecation is not coming in as planned - or maybe you planned people to wake up :) | 21:59 |
hrw | mnasiadka: shake the community was a plan | 21:59 |
mnasiadka | hrw: good, there's even my name in the thread now I see... | 22:00 |
*** jonaspaulo has quit IRC | 22:32 | |
*** goldyfruit_ has quit IRC | 22:50 | |
*** Tengu has quit IRC | 23:11 | |
*** k_mouza has joined #openstack-kolla | 23:31 | |
*** k_mouza has quit IRC | 23:35 | |
*** luksky has quit IRC | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!