*** ociuhandu has joined #openstack-ironic | 00:13 | |
*** ociuhandu has quit IRC | 00:24 | |
*** ociuhandu has joined #openstack-ironic | 00:25 | |
*** ociuhandu has quit IRC | 00:29 | |
*** ociuhandu has joined #openstack-ironic | 00:59 | |
*** ociuhandu has quit IRC | 01:06 | |
*** xiaolin has joined #openstack-ironic | 01:20 | |
*** pas-ha_ has joined #openstack-ironic | 02:20 | |
*** ab-a has joined #openstack-ironic | 02:21 | |
*** pas-ha has quit IRC | 02:28 | |
*** andreaf has quit IRC | 02:28 | |
*** pas-ha_ is now known as pas-ha | 02:28 | |
*** bdodd has quit IRC | 02:31 | |
*** mkowalski has quit IRC | 02:32 | |
*** bdodd has joined #openstack-ironic | 02:34 | |
*** jdandrea has quit IRC | 02:37 | |
*** jdandrea has joined #openstack-ironic | 02:38 | |
*** mkowalski has joined #openstack-ironic | 03:00 | |
*** rcernin has quit IRC | 03:08 | |
*** andreaf has joined #openstack-ironic | 03:25 | |
*** rcernin has joined #openstack-ironic | 03:25 | |
*** rcernin has quit IRC | 03:26 | |
*** rcernin has joined #openstack-ironic | 03:26 | |
*** jdandrea has quit IRC | 03:31 | |
*** ociuhandu has joined #openstack-ironic | 03:34 | |
*** ociuhandu has quit IRC | 03:38 | |
*** Qianbiao has joined #openstack-ironic | 04:07 | |
*** xiaolin has quit IRC | 04:47 | |
*** HagunKim has joined #openstack-ironic | 04:53 | |
*** ricolin has joined #openstack-ironic | 05:19 | |
*** k_mouza has joined #openstack-ironic | 05:24 | |
*** k_mouza has quit IRC | 05:28 | |
*** Qianbiao has quit IRC | 06:08 | |
*** Qianbiao has joined #openstack-ironic | 06:09 | |
iurygregory | good morning Ironic | 06:10 |
---|---|---|
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Remove ironic-grenade-dsvm https://review.opendev.org/732095 | 07:01 |
*** HagunKim has quit IRC | 07:02 | |
*** dougsz has joined #openstack-ironic | 07:14 | |
*** yolanda has joined #openstack-ironic | 07:19 | |
*** Qianbiao has quit IRC | 07:29 | |
*** ociuhandu has joined #openstack-ironic | 07:38 | |
*** ociuhandu has quit IRC | 07:43 | |
*** Qianbiao has joined #openstack-ironic | 07:44 | |
*** ociuhandu has joined #openstack-ironic | 07:52 | |
*** Qianbiao has quit IRC | 07:53 | |
*** Qianbiao has joined #openstack-ironic | 07:53 | |
*** ravsingh has joined #openstack-ironic | 08:02 | |
*** ociuhandu has quit IRC | 08:05 | |
*** ociuhandu has joined #openstack-ironic | 08:46 | |
*** rcernin has quit IRC | 08:49 | |
*** k_mouza has joined #openstack-ironic | 09:02 | |
*** ociuhandu has quit IRC | 09:06 | |
*** ociuhandu has joined #openstack-ironic | 09:08 | |
*** ociuhandu has quit IRC | 09:10 | |
*** ociuhandu has joined #openstack-ironic | 09:10 | |
Qianbiao | hello ironic | 09:11 |
Qianbiao | good morning iurygregory. | 09:12 |
*** belmoreira has joined #openstack-ironic | 09:14 | |
iurygregory | Qianbiao, morning o/ | 09:14 |
*** xiaolin has joined #openstack-ironic | 09:16 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic stable/train: Remove iDRAC Redfish boot device known issue https://review.opendev.org/732122 | 09:29 |
*** ricolin_ has joined #openstack-ironic | 09:44 | |
*** xiaolin has quit IRC | 09:46 | |
*** ricolin has quit IRC | 09:47 | |
*** belmoreira has quit IRC | 09:47 | |
Qianbiao | hi iurygregory, weekly meeting as usual today? | 09:50 |
iurygregory | Qianbiao, today we won't have our weekly meeting see http://lists.openstack.org/pipermail/openstack-discuss/2020-May/015070.html =) | 09:52 |
iurygregory | since this week is the PTG we canceled =) | 09:52 |
Qianbiao | oh ok. | 09:53 |
*** xiaolin has joined #openstack-ironic | 09:58 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add api-ref for indicators API https://review.opendev.org/732149 | 10:18 |
*** k_mouza has quit IRC | 10:18 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add api-ref for indicators API https://review.opendev.org/732149 | 10:19 |
pas-ha | hi all, it seems this backport https://review.opendev.org/#/c/727906 broke somewhat drac in train, as now due to how imports work for any drac driver to be loadable sushy is required to be installed | 10:37 |
patchbot | patch 727906 - ironic (stable/train) - redfish: handle hardware that is unable to set per... (MERGED) - 6 patch sets | 10:37 |
*** ericlei has joined #openstack-ironic | 10:37 | |
*** ericlei has left #openstack-ironic | 10:39 | |
*** ociuhandu has quit IRC | 10:43 | |
*** ociuhandu has joined #openstack-ironic | 10:45 | |
iurygregory | ouch | 10:46 |
iurygregory | yeah Dell CI wasn't happy at all with the backport... | 10:47 |
pas-ha | dell ci was failing for other reasons it seems (failed to find some dependencies like oslo.concurrency) but still.. | 10:49 |
iurygregory | I didn't check the CI results hehe | 10:50 |
iurygregory | probably will need a revert + backport with fixes or just submit a fix and backport | 10:50 |
*** ociuhandu has quit IRC | 10:51 | |
pas-ha | this is solely because the sushy.SOMETHING is used as a default in the top-level function definition as a default for a value, so it spits atttibute error NoneType has no attribute ... | 10:51 |
pas-ha | i'll propose a revert | 10:51 |
iurygregory | yeah =( | 10:51 |
iurygregory | the bad things is that we won't have many cores available today and tomorrow the PTG starts | 10:52 |
iurygregory | so ++ for a revert | 10:52 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic stable/train: Revert "redfish: handle hardware that is unable to set persistent boot" https://review.opendev.org/732158 | 10:53 |
*** k_mouza has joined #openstack-ironic | 10:54 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic stable/train: Revert "redfish: handle hardware that is unable to set persistent boot" https://review.opendev.org/732158 | 11:00 |
*** faizy98 has joined #openstack-ironic | 11:04 | |
rpioso | iurygregory, pas-ha: Please see https://review.opendev.org/731644 | 11:06 |
patchbot | patch 731644 - ironic - WIP: Fix Redfish handle no continuous override boo... - 3 patch sets | 11:06 |
rpioso | TL;DR: The fix is coming | 11:06 |
rpioso | Good PTG morning, ironic | 11:08 |
iurygregory | rpioso, good morning, good to know o/ | 11:10 |
pas-ha | yeah, that's master.. unfortunately some people (like us :-) ) live off the stable branch continuously. we've caught that in our downstream CI for train. uissuri and master are unaffected, at least re driver loading (they do not have sushy.BOOT_SOURCE_ENABLED_ONCE as a default value for an kwarg in _set_boot_device) | 11:11 |
rpioso | iurygregory, pas-ha: Please note the original "redfish: handle ..." which was backported seems to have a deficiency. | 11:12 |
pas-ha | and that's one place that fails driver loading due to how modules are imported and parsed | 11:12 |
pas-ha | yep | 11:12 |
rpioso | pas-ha: It will be backported through stable/train. | 11:12 |
rpioso | I understand the issue. | 11:13 |
rpioso | Note its import is correct :-) | 11:13 |
rpioso | _set_boot_device's kwargs do not reference the optional sushy import. | 11:15 |
pas-ha | that line is already differs in train and in ussuri/master, those have enabled=None | 11:21 |
*** Qianbiao has quit IRC | 11:30 | |
openstackgerrit | Richard G. Pioso proposed openstack/ironic master: Fix Redfish handle no continuous override boot src https://review.opendev.org/731644 | 11:38 |
rpioso | iurygregory, pas-ha: Removed WIP from https://review.opendev.org/731644, because all Dell 3d Party CI tests succeeded. | 11:39 |
patchbot | patch 731644 - ironic - Fix Redfish handle no continuous override boot src - 4 patch sets | 11:39 |
*** Qianbiao has joined #openstack-ironic | 11:52 | |
TheJulia | good morning | 12:01 |
*** dmellado has quit IRC | 12:02 | |
*** rh-jelabarre has joined #openstack-ironic | 12:02 | |
iurygregory | good morning TheJulia | 12:02 |
*** dmellado has joined #openstack-ironic | 12:06 | |
*** ociuhandu has joined #openstack-ironic | 12:06 | |
* TheJulia sighs w/r/t idrac sushy | 12:11 | |
* TheJulia needs more coffee first | 12:13 | |
* TheJulia suspects goign back to bed after current meeting | 12:17 | |
*** ricolin_ is now known as ricolin | 12:31 | |
*** rh-jlabarre has joined #openstack-ironic | 12:40 | |
*** rh-jlabarre has quit IRC | 12:41 | |
*** rh-jlabarre has joined #openstack-ironic | 12:41 | |
*** priteau has joined #openstack-ironic | 12:48 | |
*** beekneemech is now known as bnemec | 12:49 | |
*** bnemec has quit IRC | 12:53 | |
*** ricolin has quit IRC | 12:54 | |
*** ricolin has joined #openstack-ironic | 12:55 | |
*** ravsingh has quit IRC | 12:55 | |
*** xiaolin has quit IRC | 13:05 | |
*** jdandrea has joined #openstack-ironic | 13:12 | |
*** uzumaki has joined #openstack-ironic | 13:14 | |
*** rloo has joined #openstack-ironic | 13:15 | |
*** zbitter is now known as zaneb | 13:17 | |
*** Qianbiao has quit IRC | 13:30 | |
*** Qianbiao has joined #openstack-ironic | 13:30 | |
*** bnemec has joined #openstack-ironic | 13:32 | |
*** tzumainn has joined #openstack-ironic | 13:44 | |
*** ociuhandu has quit IRC | 14:02 | |
*** ociuhandu has joined #openstack-ironic | 14:03 | |
TheJulia | well, I'm ready for my ptg outfit tomorrow | 14:04 |
TheJulia | lol | 14:04 |
* TheJulia has curlers in | 14:04 | |
iurygregory | holiday failed | 14:05 |
TheJulia | huh? | 14:05 |
iurygregory | my callendar show kids day in CZ | 14:05 |
iurygregory | I can't stop laughing XD | 14:05 |
TheJulia | actual bank/business closed holdiay? | 14:05 |
iurygregory | nope | 14:05 |
iurygregory | but Gmail shows as a holiday hehehe | 14:05 |
TheJulia | #success I think | 14:06 |
openstackstatus | TheJulia: Added success to Success page (https://wiki.openstack.org/wiki/Successes) | 14:06 |
TheJulia | Oh wow, I forgot about that | 14:06 |
iurygregory | O.o | 14:06 |
iurygregory | WOW | 14:06 |
iurygregory | magic | 14:06 |
TheJulia | so I found a bug last week in the redfish virutal media code | 14:08 |
*** ociuhandu has quit IRC | 14:08 | |
TheJulia | tl;dr, what happens is without swift, the file ends up with 500 permissions | 14:09 |
TheJulia | at least on my test machine | 14:09 |
TheJulia | which means, nobody or anything could touch the file. | 14:09 |
ajya | TheJulia: we have seen this too, patch was pending | 14:09 |
TheJulia | ohh! | 14:09 |
TheJulia | ajya: got a link? | 14:09 |
ajya | no, it's internal | 14:09 |
ajya | rpioso: can shed some light? | 14:09 |
TheJulia | pity, because this seems logical to push upstream | 14:10 |
rpioso | TheJulia: Will push. In PTG now :-) | 14:10 |
ajya | it was definitely to go upstream, just didn't get time | 14:10 |
TheJulia | did you all add a new config parameter?, I ask becaue it seems like it should just chmod the file and the logical thing to do is either force it 755 or user definable... for tftp we do a dir_permission | 14:12 |
iurygregory | wow | 14:13 |
ajya | changed the permission; There were 2 issues. Another was typo for hex vs oct when trying to set permission | 14:13 |
uzumaki | hello ironic o/ | 14:19 |
*** lucasagomes has joined #openstack-ironic | 14:20 | |
iurygregory | hey uzumaki | 14:22 |
uzumaki | How you doing iurygregory ? | 14:22 |
iurygregory | doing good o/ and you? | 14:22 |
uzumaki | I'm well, thanks, just read about your kids day holiday, that's hilarious xD | 14:23 |
TheJulia | ajya: 0x755 instead of 0o755? | 14:24 |
ajya | TheJulia: need to use oct for mkdir, https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/redfish/boot.py#L396-L398 | 14:29 |
TheJulia | yup, I was looking at that and thinking "that may not work" | 14:30 |
ajya | and the other thing was at this point - https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/redfish/boot.py#L550 rpioso added chmod 644 boot_iso_tmp_file | 14:31 |
TheJulia | likely reasonable, I am going to get some breakfast really quickly and then likely hack on two quick fixes | 14:33 |
* uzumaki reads TheJulia talk of breakfast and remembers the cheese omlette | 14:38 | |
TheJulia | I haven't done that in a while | 14:41 |
TheJulia | although this morning was the first morning in a while that I did the first meeting and then disappeared for a whole hour | 14:41 |
TheJulia | of course... curlers in hair.... take a little time when you don't do it often | 14:41 |
uzumaki | hmmm... <insert thoughtful emoji here> | 14:42 |
*** cdearborn has joined #openstack-ironic | 14:45 | |
TheJulia | so looks like the operating umask is what is picking things up and it... | 14:45 |
TheJulia | hmmm | 14:45 |
*** dtantsur|afk is now known as dtantsur | 14:48 | |
dtantsur | good whatever it is, ironic :) | 14:48 |
iurygregory | a wild dtantsur appeared | 14:49 |
uzumaki | Good $placeholder to you as well dtantsur | 14:49 |
iurygregory | a holiday dtantsur* | 14:49 |
uzumaki | iurygregory, haha! I love those pokemon games xD | 14:49 |
dtantsur | :D | 14:49 |
uzumaki | I guess we could try throwing a pokeball at this mysterious and rare species iurygregory | 14:50 |
iurygregory | it's not effective | 14:50 |
*** Qianbiao has quit IRC | 14:50 | |
uzumaki | ah! master ball? | 14:50 |
iurygregory | but you can get reviews if you say you will give whisky, beer, tea, coffee | 14:50 |
iurygregory | hehehe | 14:51 |
*** Qianbiao has joined #openstack-ironic | 14:51 | |
uzumaki | iurygregory, good strategy! un cafe au lait, s'il vous plait xD | 14:51 |
iurygregory | uzumaki, https://www.youtube.com/watch?v=4BO2HgT2p4M hehehe | 14:51 |
iurygregory | a human pokemon parody video | 14:53 |
uzumaki | hahaha! OMG this is hilarious xD xD | 14:53 |
uzumaki | kevin comes out of the pokeball and he's like "please help me" xD | 14:54 |
iurygregory | the second video is better because it's a family battle | 14:55 |
uzumaki | that was insane omg | 14:57 |
*** rh-jelabarre has quit IRC | 14:57 | |
rpioso | dtantsur: Good Monday morning. When time permits, please review https://review.opendev.org/#/c/731644/ | 14:58 |
patchbot | patch 731644 - ironic - Fix Redfish handle no continuous override boot src - 4 patch sets | 14:58 |
dtantsur | rpioso: hey, I'm here solely for the PTG (public holiday), will check tomorrow | 14:59 |
rpioso | dtantsur: Thank you! | 14:59 |
*** Goneri has joined #openstack-ironic | 15:03 | |
*** diurnalist has joined #openstack-ironic | 15:06 | |
*** belmoreira has joined #openstack-ironic | 15:10 | |
*** lucasagomes has quit IRC | 15:12 | |
TheJulia | :) | 15:14 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 15:23 |
* iurygregory fingers crossed | 15:23 | |
* iurygregory wondering why we run metalsmith / bifrost when we change things in api-ref/ | 15:26 | |
TheJulia | seems like it could be excluded | 15:26 |
*** Qianbiao has quit IRC | 15:31 | |
iurygregory | yeah | 15:40 |
iurygregory | I assume adding - ^api-ref/.*$ to irrelevant-files would do the job | 15:45 |
dtantsur | yep | 15:45 |
dtantsur | I guess we missed them because bifrost/metalsmith themselves don't have api-ref | 15:46 |
iurygregory | and they don't use ironic-base job | 15:46 |
iurygregory | because they don't need for obvious reasons =) | 15:46 |
dtantsur | nor can they | 15:47 |
iurygregory | yeah | 15:47 |
dtantsur | ironic-base implies tempest and devstack, bifrost uses neither, metalsmith - only devstack | 15:47 |
iurygregory | yeah that was my point with obvious reasons | 15:49 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/metalsmith master: Avoid running functional jobs for api-ref changes https://review.opendev.org/732409 | 15:56 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/bifrost master: Avoid running functional jobs for api-ref changes https://review.opendev.org/732410 | 15:56 |
*** gyee has joined #openstack-ironic | 15:57 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix redfish-virtual-media file permission https://review.opendev.org/732413 | 16:11 |
*** jtwill98 has joined #openstack-ironic | 16:14 | |
iurygregory | Upgrading Ironic... /me cross all fingers | 16:18 |
* dtantsur wonders how good iurygregory is at crossing toes | 16:18 | |
jtwill98 | #openstack-ironic | 16:19 |
iurygregory | HOLY MOLY | 16:20 |
iurygregory | 2020-06-01 16:18:58.381619 | controller | 2020-06-01 16:18:58.381 | Upgrading ironic... | 16:20 |
iurygregory | 2020-06-01 16:19:39.072150 | controller | 2020-06-01 16:19:39.071 | Upgrading networking-generic-switch... | 16:20 |
iurygregory | OMG | 16:20 |
iurygregory | OMG | 16:20 |
dtantsur | :D | 16:20 |
uzumaki | you'll probably have a heart attack before the upgrade finishes xD | 16:21 |
iurygregory | Running upgrade-tempest \o/ | 16:21 |
iurygregory | only if it works | 16:21 |
uzumaki | \o/ | 16:21 |
iurygregory | ironic-grenade-multinode-multitenant green after 1yr 6months... | 16:22 |
*** dougsz has quit IRC | 16:22 | |
dtantsur | \o/ | 16:22 |
uzumaki | Then let's hope it doesn't | 16:22 |
uzumaki | iurygregory, nice! | 16:22 |
iurygregory | if it gets green I will drink a beer | 16:23 |
iurygregory | Thank troublesome apache -.-' | 16:23 |
uzumaki | now you sound like an Australian/Irish person | 16:23 |
TheJulia | gasp | 16:23 |
iurygregory | uzumaki, lol | 16:25 |
uzumaki | xD | 16:25 |
iurygregory | I can say that I'm crying because the upgrade finished hehe | 16:25 |
uzumaki | wait, that wasn't supposed to happen | 16:26 |
iurygregory | ofc it's | 16:26 |
* uzumaki gasps louder than TheJulia | 16:26 | |
iurygregory | the feeling you have when you fix something XD | 16:26 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix redfish-virtual-media file permission https://review.opendev.org/732413 | 16:26 |
uzumaki | oh yes! I recently fixed the water pump motor at home, so yes, I know the feeling :D | 16:27 |
iurygregory | ofc tempest will fail \o/ | 16:28 |
uzumaki | yayyy | 16:28 |
uzumaki | now you can drink that beer | 16:29 |
iurygregory | no | 16:29 |
iurygregory | only after tempest runs withou problems | 16:29 |
uzumaki | ofc you can, in situations such as these, the outcome doesn't matter, the propensity to drink beer does xD | 16:29 |
iurygregory | but I think I'm using the wrong regex =D | 16:29 |
iurygregory | The specified regex doesn't match with anythingERROR: InvocationError for command /opt/stack/new/tempest/.tox/tempest/bin/tempest run --regex ironic_tempest_plugin.tests.scenario --concurrency=1 --black-regex= (exited with code 1) | 16:29 |
iurygregory | hehehe | 16:29 |
uzumaki | well, in that case, I suppose you'll be sober for life xD | 16:30 |
iurygregory | nah | 16:30 |
iurygregory | well I'm always sober | 16:30 |
iurygregory | I don't get drunk =) | 16:30 |
JayF | anyone else having issues loading https://etherpad.opendev.org/p/Ironic-VictoriaPTG-Planning | 16:31 |
JayF | Trying to make sure I've read up on all the topics for this week, but I can't get that to load | 16:31 |
iurygregory | JayF, nope fine here | 16:31 |
dtantsur | opens for me | 16:31 |
uzumaki | iurygregory, didn't expect anything else from such a sage one as you <bows with hands tied at front> | 16:31 |
JayF | of course, it works now too. As usual as soon as I ask about something it gets fixed :| | 16:31 |
iurygregory | hehe | 16:32 |
uzumaki | JayF, lol | 16:32 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Iso booting via redfish virtual media https://review.opendev.org/727480 | 16:32 |
dtantsur | JayF: Complaint Driven Development | 16:32 |
dtantsur | works for me all the time as well | 16:32 |
TheJulia | JayF: I did copy it to the meetpad | 16:33 |
iurygregory | TheJulia, FYI we may need to talk with the opendev sysadmins http://lists.openstack.org/pipermail/openstack-discuss/2020-June/015171.html since our meetpad has capital letters | 16:33 |
JayF | TheJulia: I'm not sure what "the meetpad" is? | 16:33 |
TheJulia | https://meetpad.opendev.org/Ironic-PTG | 16:33 |
TheJulia | iurygregory: clarkb already chimed in and said it shouldn't be an issue | 16:33 |
clarkb | well it would be an issue if you have preexisting content | 16:34 |
dtantsur | direct etherpad link if someone needs it: https://meetpad.opendev.org/etherpad/p/ironic-ptg | 16:34 |
*** jtwill98_ has joined #openstack-ironic | 16:34 | |
iurygregory | TheJulia, oh awesome =) | 16:34 |
dtantsur | (note Ironic-PTG vs ironic-ptg) | 16:34 |
TheJulia | then again | 16:34 |
TheJulia | I could also just create a new lower case one and copy the content and change the urls | 16:34 |
*** jtwill98_ has left #openstack-ironic | 16:34 | |
clarkb | (we can rename the existing content to the name meetpad will use which is 'ironic-ptg') | 16:34 |
TheJulia | clarkb: sounds like maybe we should just in case? | 16:35 |
uzumaki | I just wonder how many places there will be inconsistencies with this.. broken link fireworks.. | 16:35 |
clarkb | well https://etherpad.opendev.org/p/Ironic-PTG has no content yet | 16:35 |
clarkb | so we don't need to rename the pad in etherpad | 16:36 |
clarkb | but ya I would replace any links to https://etherpad.opendev.org/p/Ironic-PTG with https://etherpad.opendev.org/p/ironic-ptg | 16:36 |
*** dougsz has joined #openstack-ironic | 16:36 | |
clarkb | then meetpad will end up on the same pad | 16:36 |
uzumaki | probably drop the final etherpad/meetpad link in all the other links, allowing anyone with the outdated link to navigate to the right one? | 16:36 |
TheJulia | umm... hmm | 16:36 |
TheJulia | but I put content in the meetpad | 16:37 |
dtantsur | TheJulia: and it ended up in https://meetpad.opendev.org/etherpad/p/ironic-ptg | 16:37 |
dtantsur | (not /Ironic-PTG) | 16:37 |
TheJulia | woot | 16:37 |
clarkb | right what dtantsur said | 16:37 |
dtantsur | which seems fine in the end? | 16:37 |
TheJulia | yeah, that should be fine I think | 16:37 |
clarkb | jitsi meet (meetpad) is normalizing to all lower case | 16:37 |
clarkb | so publishing all lower case urls keeps everything nice and consistent | 16:37 |
uzumaki | interesting.. | 16:38 |
iurygregory | ideas for what tempest regex we should use in ironic-grenade-multinode-multitenant? =) | 16:38 |
uzumaki | not the wrong one iurygregory | 16:38 |
iurygregory | uzumaki, yeah ofc =) | 16:38 |
dtantsur | :D | 16:38 |
TheJulia | https://meetpad.opendev.org/ironic-ptg works, I'll just change the urls | 16:38 |
iurygregory | or no regex LOL | 16:38 |
dtantsur | I don't remember if we use multitenant or normal tests there | 16:39 |
clarkb | this is apparently a limitation of xmpp which jitsi meet is built on. Unfortunately we didn't realize until very recently | 16:39 |
uzumaki | it's ironic how a complicated lexical language would be called "regular" expressions xD | 16:39 |
dtantsur | clarkb: TIL it's built on XMPP | 16:39 |
iurygregory | dtantsur, I was trying ironic_tempest_plugin.tests.scenario, didn't work =) | 16:39 |
dtantsur | iurygregory: define didn't work? | 16:40 |
iurygregory | dtantsur, The specified regex doesn't match with anything | 16:40 |
dtantsur | mmm, the tempest plugin is not installed? | 16:41 |
iurygregory | also wondering that | 16:47 |
iurygregory | service_available.ironic ..... not found | 16:49 |
iurygregory | hehehe | 16:49 |
iurygregory | ops I have a typo in tempest_plugins D: | 16:50 |
iurygregory | that won't help ofc | 16:50 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 16:54 |
iurygregory | fingers crossed again | 16:54 |
*** dtantsur is now known as dtantsur|afk | 17:01 | |
dtantsur|afk | o/ | 17:01 |
*** k_mouza has quit IRC | 17:03 | |
*** dougsz has quit IRC | 17:08 | |
*** belmoreira has quit IRC | 17:10 | |
*** tkajinam has quit IRC | 17:11 | |
*** k_mouza has joined #openstack-ironic | 17:16 | |
*** k_mouza has quit IRC | 17:20 | |
*** donnyd_ has joined #openstack-ironic | 17:28 | |
* TheJulia wonders where stevebaker is hiding | 17:30 | |
*** ociuhandu has joined #openstack-ironic | 17:57 | |
*** ociuhandu has quit IRC | 18:02 | |
*** sshnaidm is now known as sshnaidm|afk | 18:20 | |
*** ricolin has quit IRC | 18:33 | |
*** mkowalski has quit IRC | 18:35 | |
*** mkowalski has joined #openstack-ironic | 18:44 | |
*** tzumainn has quit IRC | 18:44 | |
*** jdandrea has quit IRC | 18:44 | |
*** mkowalski has quit IRC | 18:45 | |
iurygregory | first time I see "tempest.lib.exceptions.Conflict: Conflict with state of target resource Details: {'type': 'SubnetInUse', 'message': 'Unable to complete operation on subnet aa0e9c7e-1bb2-4565-93c1-0c9d5b0aac36: One or more ports have an IP allocation from this subnet.', 'detail': ''}" | 18:46 |
*** dsneddon has joined #openstack-ironic | 18:47 | |
*** jdandrea has joined #openstack-ironic | 18:47 | |
TheJulia | iurygregory: I would guess the port has not finished deleting? | 18:57 |
iurygregory | TheJulia, probably, still waiting for the job to finish so I can look at logs | 18:57 |
iurygregory | at least we are running the tests *-* | 18:57 |
iurygregory | I'm happy | 18:57 |
TheJulia | :) | 19:01 |
*** tzumainn has joined #openstack-ironic | 19:10 | |
dhellmann | is there a way to remove introspection data from ironic-inspector's database through the API? | 19:14 |
dhellmann | I don't see an obvious way to do it from the command line using osc | 19:14 |
*** mkowalski has joined #openstack-ironic | 19:20 | |
iurygregory | I only see delete for introspection rule .-. | 19:22 |
*** diurnalist has quit IRC | 19:29 | |
TheJulia | I'm not sure there really is a mechanism | 19:38 |
TheJulia | dhellmann: what are you hitting that is sending you down this path? | 19:38 |
dhellmann | I was thinking we would use inspection data for discovery, but that was before I realized the inspector created nodes in ironic on its own | 19:39 |
dhellmann | now I'm worried about filling up my disk, but I expect that would take a lot of nodes | 19:39 |
dhellmann | :-) | 19:39 |
TheJulia | yeah, we likely should add a delete mechanism | 19:40 |
dhellmann | I'm trying to make sure I have the details correct in this metal3 proposal: https://docs.google.com/document/d/1hlMSdim53CQtqlYOnx1ptq3qKHKhP9LhDrRxzCutGoM/edit | 19:40 |
TheJulia | internally it does have htem | 19:40 |
dhellmann | it would be nice if deleting a node cleaned up the inspection, too. but a separate API that just cleaned up the inspection database is good as well | 19:40 |
TheJulia | yeah | 19:42 |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/ussuri: Use ussuri ramdisks https://review.opendev.org/732442 | 19:50 |
TheJulia | looks like stable/ussuri is broken, ramdisks boot not realizing they are ramdisks :( I've pushed up ^^ hopefully this is just transitory on master branch, but given branch jobs seem fine, it is a bit weird | 19:57 |
*** uzumaki has quit IRC | 19:58 | |
*** diurnalist has joined #openstack-ironic | 20:02 | |
iurygregory | ouch D: | 20:02 |
iurygregory | I can work on adding an API in ironic-inspector for delete introspection data for a node | 20:03 |
TheJulia | seems like something worthwhile, eventually | 20:05 |
TheJulia | on the list or relative priorites, I wouldn't know where it woudl fall | 20:05 |
iurygregory | yeah | 20:07 |
*** priteau has quit IRC | 20:38 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Remove ironic-grenade-dsvm https://review.opendev.org/732095 | 20:52 |
iurygregory | lol did a wrong rebase D: | 20:52 |
openstackgerrit | Merged openstack/ironic-lib master: Implement Basic HTTP authentication middleware https://review.opendev.org/729070 | 20:57 |
*** stevebaker has joined #openstack-ironic | 21:00 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Remove ironic-grenade-dsvm https://review.opendev.org/732095 | 21:01 |
*** k_mouza has joined #openstack-ironic | 21:04 | |
dhellmann | iurygregory , TheJulia : I'd put it somewhere near the bottom of the priority list :-) | 21:06 |
iurygregory | dhellmann, ack =) | 21:06 |
*** k_mouza has quit IRC | 21:08 | |
*** jdandrea has quit IRC | 21:21 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 21:34 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Merge jobs https://review.opendev.org/726121 | 21:40 |
*** ociuhandu has joined #openstack-ironic | 21:45 | |
*** ociuhandu has quit IRC | 21:50 | |
*** uzumaki has joined #openstack-ironic | 21:55 | |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in drac tests https://review.opendev.org/731700 | 22:06 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in ansible tests https://review.opendev.org/731744 | 22:06 |
*** rh-jlabarre has quit IRC | 22:17 | |
*** rloo has quit IRC | 22:17 | |
*** rh-jlabarre has joined #openstack-ironic | 22:18 | |
*** diurnalist has quit IRC | 22:22 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: Enable Basic HTTP authentication middleware https://review.opendev.org/727467 | 22:33 |
TheJulia | dhellmann: thanks | 22:39 |
TheJulia | stevebaker: o/ | 22:39 |
stevebaker | TheJulia: heyya | 22:39 |
TheJulia | how goes it? | 22:39 |
stevebaker | TheJulia: good, I decided not to attend the ptg last night because there is to much family stuff happening today. I'll give tonight a crack though | 22:42 |
TheJulia | eek, hope everything is okay | 22:44 |
stevebaker | TheJulia: its banal things like kids having a day off school and furniture deliveries - not much opportunity for catch-up sleep ;) | 22:45 |
*** rcernin has joined #openstack-ironic | 22:47 | |
*** tkajinam has joined #openstack-ironic | 22:47 | |
*** diurnalist has joined #openstack-ironic | 22:56 | |
TheJulia | stevebaker: on the wsme patches, a couple files have "All rights reserved" and no other text, that doesn't seem right to me. | 23:03 |
stevebaker | TheJulia: hmm, that sounds like copypasta I didn't fix, I'll to a refresh on that series | 23:05 |
stevebaker | TheJulia: do you think an ironic-lib release will be required for unit tests to pass on this? https://review.opendev.org/#/c/727467/ | 23:09 |
patchbot | patch 727467 - ironic - Enable Basic HTTP authentication middleware - 6 patch sets | 23:09 |
TheJulia | vdrok: question for you on https://review.opendev.org/#/c/728464 when you can context switch over to ti | 23:15 |
patchbot | patch 728464 - python-ironicclient - Allow to pass additional arguments into adapter - 1 patch set | 23:15 |
TheJulia | stevebaker: unit tests, absolutely required | 23:15 |
TheJulia | if they invoke the library, they will do all testing based on constraints | 23:15 |
stevebaker | TheJulia: thats what I figured | 23:16 |
TheJulia | so your welcome to submit a release request to openstack/releases, and I'll +1 it | 23:19 |
stevebaker | TheJulia: ok, thanks | 23:20 |
TheJulia | stevebaker: btw, how is the basic auth stuff going? | 23:30 |
stevebaker | TheJulia: ironic, inspector, and keystoneauth changes are complete and ready to land. My focus for actual development is on jsonrpc basic auth | 23:32 |
TheJulia | stevebaker: any traction on the keystoneauth change? | 23:33 |
stevebaker | TheJulia: yep a +2 from mordred, I'm just waiting for the py35 job to be fixed https://review.opendev.org/#/c/727562/ | 23:34 |
patchbot | patch 727562 - keystoneauth - Implement HTTP Basic client support in keystoneauth1 - 2 patch sets | 23:34 |
TheJulia | \o/ | 23:34 |
stevebaker | TheJulia: here is an ironic-lib release change https://review.opendev.org/#/c/732467/ | 23:36 |
patchbot | patch 732467 - releases - Release 4.3.0 of ironic-lib - 1 patch set | 23:36 |
TheJulia | +1'ed | 23:36 |
*** diurnalist has quit IRC | 23:50 | |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Fix detection of mkisofs https://review.opendev.org/731698 | 23:51 |
openstackgerrit | Merged openstack/bifrost stable/train: Fix detection of mkisofs https://review.opendev.org/731699 | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!