| *** rene_ has joined #openstack-ironic | 00:00 | |
| *** rene_ has quit IRC | 00:02 | |
| *** dsneddon_ has quit IRC | 00:13 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 00:13 | |
| *** dsneddon_ has joined #openstack-ironic | 00:14 | |
| *** AlexeyAbashkin has quit IRC | 00:18 | |
| *** ijw has quit IRC | 00:32 | |
| *** dsneddon_ has quit IRC | 00:37 | |
| openstackgerrit | Merged openstack/ironic stable/queens: Remove too large configdrive for handling error https://review.openstack.org/544877 | 00:38 |
|---|---|---|
| *** dsneddon_ has joined #openstack-ironic | 00:39 | |
| *** dsneddon_ has quit IRC | 00:45 | |
| *** persia has joined #openstack-ironic | 00:54 | |
| *** hamzy has joined #openstack-ironic | 01:02 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 01:13 | |
| *** AlexeyAbashkin has quit IRC | 01:18 | |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic master: DRAC set_boot_device does not work with UEFI mode https://review.openstack.org/545184 | 01:18 |
| *** jlabarre has quit IRC | 01:19 | |
| *** dsneddon_ has joined #openstack-ironic | 01:23 | |
| *** ijw has joined #openstack-ironic | 01:29 | |
| *** dsneddon_ has quit IRC | 01:31 | |
| *** dsneddon_ has joined #openstack-ironic | 01:34 | |
| *** crushil__ has quit IRC | 01:42 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Set 'initrd' to 'rescue_ramdisk' for rescue with iPXE https://review.openstack.org/545186 | 01:43 |
| *** dsneddon_ has quit IRC | 01:49 | |
| *** crushil__ has joined #openstack-ironic | 01:56 | |
| *** dsneddon_ has joined #openstack-ironic | 02:02 | |
| *** khyr0n has quit IRC | 02:05 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Build instance PXE options for unrescue https://review.openstack.org/544278 | 02:13 |
| *** ijw has quit IRC | 02:19 | |
| *** harlowja_ has quit IRC | 03:23 | |
| *** jaianshu has joined #openstack-ironic | 03:46 | |
| *** khyr0n has joined #openstack-ironic | 03:54 | |
| *** khyr0n has quit IRC | 03:54 | |
| *** khyr0n has joined #openstack-ironic | 03:55 | |
| *** dsneddon_ has quit IRC | 04:19 | |
| *** vsaienk0 has joined #openstack-ironic | 04:22 | |
| *** vsaienk0 has quit IRC | 04:32 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Set 'initrd' to 'rescue_ramdisk' for rescue with iPXE https://review.openstack.org/545186 | 04:38 |
| *** dsneddon_ has joined #openstack-ironic | 04:44 | |
| *** harlowja has joined #openstack-ironic | 04:46 | |
| *** dsneddon_ has quit IRC | 04:47 | |
| *** charcol-v2 has quit IRC | 04:57 | |
| *** harlowja has quit IRC | 04:59 | |
| *** jaganathan has quit IRC | 05:15 | |
| *** priteau has joined #openstack-ironic | 05:19 | |
| *** pmannidi has quit IRC | 05:19 | |
| *** harlowja has joined #openstack-ironic | 05:19 | |
| *** priteau has quit IRC | 05:23 | |
| *** early has quit IRC | 05:25 | |
| *** harlowja has quit IRC | 05:27 | |
| *** dpawar has joined #openstack-ironic | 05:28 | |
| *** early has joined #openstack-ironic | 05:30 | |
| *** hoonetorg has quit IRC | 05:40 | |
| *** hoonetorg has joined #openstack-ironic | 05:53 | |
| *** jaganathan has joined #openstack-ironic | 06:19 | |
| *** threestrands has quit IRC | 06:20 | |
| *** marios has joined #openstack-ironic | 06:20 | |
| *** openstackstatus has quit IRC | 06:27 | |
| *** openstack has joined #openstack-ironic | 06:29 | |
| *** ChanServ sets mode: +o openstack | 06:29 | |
| *** rwsu has quit IRC | 07:12 | |
| *** mjura has joined #openstack-ironic | 07:16 | |
| *** etingof has quit IRC | 07:18 | |
| *** threestrands has joined #openstack-ironic | 07:22 | |
| *** threestrands has quit IRC | 07:22 | |
| *** threestrands has joined #openstack-ironic | 07:23 | |
| *** threestrands has quit IRC | 07:23 | |
| *** threestrands has joined #openstack-ironic | 07:23 | |
| *** rwsu has joined #openstack-ironic | 07:25 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:26 | |
| *** etingof has joined #openstack-ironic | 07:32 | |
| pas-ha | morning all | 07:33 |
| *** threestrands has quit IRC | 07:34 | |
| *** rcernin has quit IRC | 07:41 | |
| *** dtantsur|afk is now known as dtantsur | 07:46 | |
| dtantsur | morning folks, happy Friday | 07:46 |
| *** olivierbourdon38 has joined #openstack-ironic | 07:54 | |
| *** vsaienk0 has joined #openstack-ironic | 07:59 | |
| *** MattMan has quit IRC | 08:02 | |
| *** MattMan has joined #openstack-ironic | 08:02 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 08:04 |
| *** yee37934 has quit IRC | 08:07 | |
| *** yee379 has joined #openstack-ironic | 08:07 | |
| *** vsaienk0 has quit IRC | 08:08 | |
| *** tesseract has joined #openstack-ironic | 08:27 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Allow sqalchemy filtering by id and uuid https://review.openstack.org/545041 | 08:30 |
| *** e0ne has joined #openstack-ironic | 08:35 | |
| *** jtomasek has joined #openstack-ironic | 08:40 | |
| *** e0ne has quit IRC | 08:54 | |
| *** yangyapeng has joined #openstack-ironic | 08:57 | |
| *** jaganathan has quit IRC | 09:01 | |
| *** mgoddard_ has joined #openstack-ironic | 09:05 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-specs master: Add service management API for ironic https://review.openstack.org/471217 | 09:10 |
| *** pcaruana has joined #openstack-ironic | 09:11 | |
| *** vabada has quit IRC | 09:25 | |
| *** zshi has joined #openstack-ironic | 09:25 | |
| *** priteau has joined #openstack-ironic | 09:26 | |
| *** derekh has joined #openstack-ironic | 09:29 | |
| *** links has joined #openstack-ironic | 09:47 | |
| *** links has quit IRC | 09:49 | |
| *** vsaienk0 has joined #openstack-ironic | 09:51 | |
| *** stendulker has joined #openstack-ironic | 09:56 | |
| *** priteau has quit IRC | 09:56 | |
| *** priteau has joined #openstack-ironic | 09:57 | |
| *** links has joined #openstack-ironic | 09:58 | |
| *** vsaienk0 has quit IRC | 10:01 | |
| *** priteau has quit IRC | 10:03 | |
| *** links has quit IRC | 10:12 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Update iLO documentation for deprecating classical drivers https://review.openstack.org/542593 | 10:13 |
| *** openstackgerrit has quit IRC | 10:18 | |
| *** vsaienk0 has joined #openstack-ironic | 10:21 | |
| *** priteau has joined #openstack-ironic | 10:23 | |
| *** e0ne has joined #openstack-ironic | 10:34 | |
| *** sambetts|afk is now known as sambetts | 10:41 | |
| *** vsaienk0 has quit IRC | 10:46 | |
| dtantsur | TheJulia, jroll: I see the multinode job passing on https://review.openstack.org/545041 is it your success or ...? | 10:51 |
| patchbot | patch 545041 - ironic - Allow sqalchemy filtering by id and uuid | 10:51 |
| *** stendulker has quit IRC | 11:00 | |
| *** lucas-afk is now known as lucasagomes | 11:03 | |
| *** vsaienk0 has joined #openstack-ironic | 11:03 | |
| *** jaganathan has joined #openstack-ironic | 11:06 | |
| *** openstackgerrit has joined #openstack-ironic | 11:08 | |
| openstackgerrit | Mallikarjuna Reddy Kolagatla proposed openstack/ironic master: Updates boot mode on the baremetal as per `boot_mode` https://review.openstack.org/540793 | 11:08 |
| *** e0ne has quit IRC | 11:12 | |
| *** vsaienk0 has quit IRC | 11:13 | |
| *** e0ne has joined #openstack-ironic | 11:13 | |
| dtantsur | pas-ha, vdrok, please review https://review.openstack.org/#/c/541916/ so that we can release ironic-staging-drivers | 11:13 |
| patchbot | patch 541916 - ironic-staging-drivers - Handle 'timeout' parameter in power methods | 11:13 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Update iLO documentation for deprecating classical drivers https://review.openstack.org/542593 | 11:16 |
| *** AlexeyAbashkin has quit IRC | 11:37 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:40 | |
| *** jaianshu has quit IRC | 11:49 | |
| *** olivierbourdon38 has quit IRC | 11:54 | |
| *** links has joined #openstack-ironic | 12:03 | |
| *** etingof has quit IRC | 12:06 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Clean nodes stuck in CLEANING state when ir-cond restarts https://review.openstack.org/349971 | 12:10 |
| dtantsur | revived this ^^^ | 12:15 |
| *** etingof has joined #openstack-ironic | 12:18 | |
| *** jcoufal has joined #openstack-ironic | 12:20 | |
| *** jaganathan has quit IRC | 12:35 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Switch documentation to hardware types https://review.openstack.org/545285 | 12:43 |
| *** jlabarre has joined #openstack-ironic | 12:47 | |
| *** jaganathan has joined #openstack-ironic | 12:49 | |
| *** dprince has joined #openstack-ironic | 12:51 | |
| *** mrch has joined #openstack-ironic | 12:56 | |
| *** mrch has quit IRC | 13:04 | |
| *** athomas has joined #openstack-ironic | 13:10 | |
| *** r-daneel has joined #openstack-ironic | 13:14 | |
| *** trown|outtypewww is now known as trown | 13:26 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Switch documentation to hardware types https://review.openstack.org/545285 | 13:28 |
| *** hshiina has quit IRC | 13:29 | |
| *** ricardoas has quit IRC | 13:36 | |
| jroll | dtantsur: O_O | 13:36 |
| jroll | I'm not aware of us fixing anything | 13:36 |
| dtantsur | weellllll.. then other patches failed today, so dunno | 13:36 |
| jroll | dtantsur: oh, queens->master is a much smaller difference, maybe it's just pike->queens that is broken now | 13:37 |
| Tengu | hello! is there a way to recover a host with "provisionning state" set to "error"? I'd rather avoid deleting it | 13:41 |
| *** dpawar has quit IRC | 13:41 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Validate scheduling fields in basic ops scenario https://review.openstack.org/545297 | 13:42 |
| dtantsur | Tengu: first of all, check why it ended up in ERROR, see node's last_error field | 13:43 |
| *** ricardoas has joined #openstack-ironic | 13:44 | |
| dtantsur | in any case, it probably means that the node was not provisioned, so I suspect deleting or rebuilding it are your two options | 13:44 |
| *** jaganathan has quit IRC | 13:44 | |
| Tengu | rebuilding? how? | 13:44 |
| Tengu | dtantsur: the issue is simple: vbmc was down for a while, and apparently this made the instance switch to maintenance mode (unset of course) and "error" state :/ | 13:45 |
| dtantsur | Tengu: error power_state or provision_state? | 13:46 |
| Tengu | provisionning_state | 13:47 |
| Tengu | power_state id "running" | 13:47 |
| Tengu | err, power on | 13:47 |
| dtantsur | so you were provisioning it? I don't think an active node can go into error itself, even in this case | 13:47 |
| Tengu | hmm ok. well, I'll re-trash the whole thing, it's "just" the lab. | 13:48 |
| *** jlabarre has quit IRC | 13:54 | |
| *** jcoufal has quit IRC | 13:59 | |
| *** jcoufal has joined #openstack-ironic | 14:01 | |
| pas-ha | dtantsur: merged | 14:01 |
| dtantsur | thanks! | 14:01 |
| dtantsur | pas-ha: I'd like to release once it merges. any objections? | 14:02 |
| pas-ha | not from me | 14:03 |
| *** jaganathan has joined #openstack-ironic | 14:05 | |
| *** edleafe is now known as figleaf | 14:08 | |
| *** olivierbourdon38 has joined #openstack-ironic | 14:12 | |
| *** jcoufal has quit IRC | 14:16 | |
| *** jcoufal has joined #openstack-ironic | 14:17 | |
| *** Goneri has joined #openstack-ironic | 14:17 | |
| *** cdearborn has joined #openstack-ironic | 14:20 | |
| *** jcoufal has quit IRC | 14:23 | |
| *** livelace-link has quit IRC | 14:23 | |
| *** jlabarre has joined #openstack-ironic | 14:28 | |
| *** lucasagomes is now known as lucas-hungry | 14:31 | |
| *** zshi has quit IRC | 14:32 | |
| *** vsaienk0 has joined #openstack-ironic | 14:34 | |
| TheJulia | good morning everyone | 14:38 |
| etingof | o/ | 14:41 |
| TheJulia | dtantsur: jroll: I'm convinced our rolling upgrades scenario is just broken... and maybe we should switch to upgrading JUST ironic living in a venv on a multinode devstack... alternatively we could just change the upgrade list to be ironic, and see what we could do about putting it into a venv | 14:42 |
| TheJulia | but! disabling saving bytecode to disk did help another patch pass last night | 14:43 |
| *** dansmith is now known as superdan | 14:43 | |
| dtantsur | morning TheJulia | 14:43 |
| *** vsaienk0 has quit IRC | 14:44 | |
| dtantsur | TheJulia: did we get any feedback from the nova team on this issue? | 14:45 |
| TheJulia | dtantsur: not really. "python bug, not our problem" is the initial response. We then had a different case where posted updates to the placement service was failing and that raised some interest, but no action as far as I'm aware. | 14:47 |
| TheJulia | this all boils down to our scenario is just not really sane | 14:47 |
| dtantsur | okay, so can we change it without rewriting grenade? | 14:48 |
| TheJulia | I _think_ we need to abandon grenade | 14:48 |
| TheJulia | at least for rolling upgrade testing | 14:48 |
| TheJulia | we might be able to get it to work for now, but there is nothing preventing other issues | 14:48 |
| TheJulia | although, if we turn off bytecode being saved to disk, then we have a MUCH better chance of completing a CI run in a mixed upgrade scenario | 14:49 |
| jroll | I mean... it does seem like a python bug to me. we should figure out how to best inject PYTHONDONTWRITEBYTECODE in the short term. abandoning grenade, if we decide to do it, is gonna take a while, and we need to get testing running. | 14:49 |
| TheJulia | I absolutely agree | 14:50 |
| dtantsur | yep | 14:50 |
| TheJulia | well, not about the python bug. I see it as a feature | 14:50 |
| TheJulia | since it _does_ support dynamic loading and dynamic recompliation | 14:50 |
| * dtantsur releases ironic-staging-drivers | 14:51 | |
| jroll | the bug is that a segv can come from that, not that it's a thing | 14:51 |
| jroll | (is what I got from that conversation) | 14:51 |
| TheJulia | jroll: yeah, that is what it looks like given we get successful-ish runs after upgrading since the python process is not going "oh, let me re-read this file!" | 14:52 |
| *** jistr is now known as jistr|mtg | 14:52 | |
| TheJulia | I propose we update the pike to master job, and the pike -> queens job... that is another story with how services are started | 14:53 |
| TheJulia | I've not pinned down where best to fix that at | 14:53 |
| jroll | honestly I'd ask qa or infra what they would recommend, maybe they have a good way to pass an environment variable everywhere | 14:54 |
| jroll | ditto for the q->master job, we can probably do something cleaner than moving python around | 14:54 |
| * jroll totally forgot the other thing he was going to say after that :( | 14:55 | |
| TheJulia | jroll: I'll update the patch and go prod qa folk for thouhts | 14:57 |
| TheJulia | I'm putting in a verbose note right now | 14:57 |
| *** jcoufal has joined #openstack-ironic | 14:58 | |
| *** r-daneel has quit IRC | 14:58 | |
| jroll | TheJulia: cool, thank you | 14:58 |
| jroll | TheJulia: do we continue getting the same problems talking to placement? I could dig into that further, it's probably semi-related to the hash ring stuff anyhow | 14:59 |
| TheJulia | jroll: I _think_ we may have hit a race or something | 14:59 |
| jroll | that could be too | 14:59 |
| *** vsaienk0 has joined #openstack-ironic | 15:04 | |
| *** trown is now known as trown|brb | 15:11 | |
| *** vsaienk0 has quit IRC | 15:14 | |
| *** jistr|mtg is now known as jistr | 15:15 | |
| *** etingof has quit IRC | 15:18 | |
| *** khyr0n has quit IRC | 15:18 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Disable .pyc files for grenade multindoe https://review.openstack.org/544750 | 15:20 |
| TheJulia | jroll: dtantsur: does ^^^ make sense? | 15:20 |
| *** rpioso|afk is now known as rpioso | 15:23 | |
| rpioso | Good Friday morning :) | 15:23 |
| dtantsur | TheJulia: sigh.. it does if it works. | 15:24 |
| dtantsur | nit: you can use mv instead of cp && rm, I guess | 15:24 |
| dtantsur | morning rpioso | 15:24 |
| jroll | TheJulia: +1. I feel like I want to be pedantic in the commit message but just gonna drop it | 15:25 |
| rpioso | dtantsur: Hey! Thank you for the review earlier today. Awesome! | 15:25 |
| dtantsur | np :) | 15:25 |
| *** livelace-link has joined #openstack-ironic | 15:26 | |
| *** trown|brb is now known as trown | 15:27 | |
| *** lucas-hungry is now known as lucasagomes | 15:28 | |
| dtantsur | jroll: do you remember if there were reasons to not do https://review.openstack.org/349971 long ago? | 15:30 |
| patchbot | patch 349971 - ironic - Clean nodes stuck in CLEANING state when ir-cond r... | 15:30 |
| jroll | dtantsur: not that I remember | 15:31 |
| jroll | not sure why we didn't complete it | 15:31 |
| dtantsur | ok, I'd like to fit this in Queens as well, if possible | 15:31 |
| jroll | ++ | 15:32 |
| * dtantsur looks in ironic take over and HA today | 15:32 | |
| *** etingof has joined #openstack-ironic | 15:33 | |
| *** livelace-link has quit IRC | 15:33 | |
| TheJulia | jroll: I had to resist the urge... big time | 15:34 |
| *** livelace-link has joined #openstack-ironic | 15:35 | |
| TheJulia | dtantsur: mv just changes the reference to the inode, we need to orphan the inode | 15:36 |
| dtantsur | TheJulia: why? if anything is already using python, we're out of luck anyway, no? | 15:37 |
| TheJulia | dtantsur: I believe mv failed because the file is busy due to a lock | 15:38 |
| dtantsur | TheJulia: mm, I thought it should not happen on Linux. and if it does, I wonder why rm works. | 15:39 |
| *** r-daneel has joined #openstack-ironic | 15:40 | |
| TheJulia | dtantsur: -f :) | 15:40 |
| dtantsur | -f, --force | 15:41 |
| dtantsur | ignore nonexistent files and arguments, never prompt | 15:41 |
| dtantsur | I don't think -f removes locks or anything like this | 15:41 |
| TheJulia | I thought force ignored locks as well | 15:42 |
| *** e0ne_ has joined #openstack-ironic | 15:43 | |
| *** e0ne has quit IRC | 15:43 | |
| * TheJulia removes the -f flag | 15:44 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Disable .pyc files for grenade multinode https://review.openstack.org/544750 | 15:44 |
| *** e0ne has joined #openstack-ironic | 15:45 | |
| TheJulia | qa folk prodded | 15:46 |
| * jroll adds to it | 15:46 | |
| *** e0ne_ has quit IRC | 15:47 | |
| TheJulia | so... the question then becomes... pike->queens | 15:48 |
| *** mjturek has joined #openstack-ironic | 15:53 | |
| * jroll gets lunch then will be adding tests to 545041 | 15:53 | |
| * dtantsur hates our API unit tests | 15:54 | |
| *** r-daneel has quit IRC | 15:55 | |
| *** vsaienk0 has joined #openstack-ironic | 15:57 | |
| *** pcaruana has quit IRC | 15:57 | |
| *** mjura has quit IRC | 15:58 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: WIP: Register traits on nodes in devstack https://review.openstack.org/545370 | 16:00 |
| *** bnemec is now known as beekneemech | 16:03 | |
| *** r-daneel has joined #openstack-ironic | 16:03 | |
| *** vsaienk0 has quit IRC | 16:07 | |
| *** khyr0n has joined #openstack-ironic | 16:11 | |
| *** khyr0n has quit IRC | 16:18 | |
| *** khyr0n has joined #openstack-ironic | 16:19 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Add parametrization of Ironic locations https://review.openstack.org/545373 | 16:20 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Fix rare HTTP 400 from port list API https://review.openstack.org/545069 | 16:21 |
| dtantsur | TheJulia, sambetts, should be complete ^^^ | 16:21 |
| TheJulia | awesome | 16:21 |
| sambetts | nice! | 16:21 |
| TheJulia | dtantsur: fyi, pasted a link to the irc log from -qa in the grenade patch, you might find it amusing :) | 16:21 |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Validate scheduling fields in basic ops scenario https://review.openstack.org/545297 | 16:23 |
| dtantsur | TheJulia: yep :) | 16:25 |
| TheJulia | Anyway, going to go dig through devstack to see how it launches the nova stuff in pike | 16:25 |
| *** etingof has quit IRC | 16:29 | |
| TheJulia | hmmmmmm | 16:30 |
| *** AlexeyAbashkin has quit IRC | 16:33 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Validate scheduling fields in basic ops scenario https://review.openstack.org/545297 | 16:33 |
| *** devananda has joined #openstack-ironic | 16:34 | |
| *** olivierbourdon38 has quit IRC | 16:38 | |
| *** vsaienk0 has joined #openstack-ironic | 16:43 | |
| *** vsaienk0 has quit IRC | 16:53 | |
| *** r-daneel has quit IRC | 17:02 | |
| sambetts | TheJulia, dtantsur: I'm on PTO all next week, so I won't be around until we meet up at the PTG | 17:04 |
| *** e0ne has quit IRC | 17:06 | |
| dtantsur | sambetts: ack, enjoy! | 17:07 |
| sambetts | thanks! See you all in Dublin! | 17:08 |
| * sambetts --> goes to pack | 17:08 | |
| *** sambetts is now known as sambetts|afk | 17:08 | |
| *** marios has quit IRC | 17:09 | |
| *** derekh has quit IRC | 17:14 | |
| jroll | \o | 17:15 |
| *** links has quit IRC | 17:18 | |
| devananda | I wish I was going to the PTG ... Dublin is a great city! | 17:20 |
| devananda | hope to see everyone in Vancouver instead :) | 17:20 |
| openstackgerrit | Merged openstack/ironic master: tox: stop validating locale files https://review.openstack.org/543523 | 17:23 |
| dtantsur | devananda: /me not coming to Vancouver, I'm afraid | 17:24 |
| devananda | dang... | 17:24 |
| openstackgerrit | Merged openstack/ironic master: Imported Translations from Zanata https://review.openstack.org/542609 | 17:25 |
| rpioso | dtantsur: May a translation string be used as an exception's error string and passed to LOG.error()? | 17:38 |
| dtantsur | rpioso: yep | 17:38 |
| rpioso | dtantsur: I just wanted to confirm :) | 17:39 |
| rpioso | dtantsur: ty | 17:39 |
| *** rpioso is now known as rpioso|eat | 17:40 | |
| TheJulia | sambetts|afk: See you at the ptg! | 17:46 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Allow sqalchemy filtering by id and uuid https://review.openstack.org/545041 | 17:54 |
| jroll | dtantsur: ^ easy patch to finish your week? :) | 17:54 |
| * dtantsur looks | 17:54 | |
| *** harlowja has joined #openstack-ironic | 17:55 | |
| dtantsur | yep, LGTM | 17:55 |
| *** racedo has quit IRC | 17:57 | |
| *** mgoddard_ has quit IRC | 17:58 | |
| *** racedo has joined #openstack-ironic | 17:58 | |
| *** racedo has quit IRC | 17:58 | |
| *** racedo has joined #openstack-ironic | 18:00 | |
| *** racedo has quit IRC | 18:01 | |
| *** dpawar has joined #openstack-ironic | 18:02 | |
| *** racedo has joined #openstack-ironic | 18:03 | |
| jroll | thank you! | 18:04 |
| *** r-daneel has joined #openstack-ironic | 18:05 | |
| *** racedo has quit IRC | 18:09 | |
| *** cdearborn has quit IRC | 18:10 | |
| *** racedo has joined #openstack-ironic | 18:10 | |
| *** crushil__ has quit IRC | 18:13 | |
| *** racedo has quit IRC | 18:14 | |
| *** racedo has joined #openstack-ironic | 18:15 | |
| *** racedo has quit IRC | 18:18 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:24 | |
| *** tesseract has quit IRC | 18:26 | |
| *** AlexeyAbashkin has quit IRC | 18:28 | |
| *** dtantsur is now known as dtantsur|afk | 18:29 | |
| dtantsur|afk | have a nice weekend! | 18:29 |
| *** trown is now known as trown|lunch | 18:33 | |
| *** r-daneel has quit IRC | 18:37 | |
| *** r-daneel has joined #openstack-ironic | 18:38 | |
| *** etingof has joined #openstack-ironic | 18:39 | |
| *** harlowja has quit IRC | 18:44 | |
| *** harlowja has joined #openstack-ironic | 18:48 | |
| *** jaganathan has quit IRC | 18:48 | |
| *** weshay is now known as weshay|bbiab | 18:50 | |
| *** r-daneel has quit IRC | 18:54 | |
| *** dpawar has quit IRC | 18:55 | |
| *** mgoddard_ has joined #openstack-ironic | 18:56 | |
| *** cdearborn has joined #openstack-ironic | 18:57 | |
| *** r-daneel has joined #openstack-ironic | 18:58 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic-tempest-plugin master: Validate scheduling fields in basic ops scenario https://review.openstack.org/545297 | 19:02 |
| *** rpioso|eat is now known as rpioso | 19:02 | |
| mgoddard_ | anyone seen 'ValueError: Circular reference detected' in JSON decode before? | 19:08 |
| mgoddard_ | http://logs.openstack.org/70/545370/1/check/ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa/d5486a9/logs/screen-ir-cond.txt.gz#_Feb_16_16_36_43_894193 | 19:08 |
| jlvillal | mgoddard_, It rings a bell | 19:08 |
| jlvillal | mgoddard_, I remember we had that when Exceptions were sent to be serialized. | 19:08 |
| jlvillal | mgoddard_, But I thought that had been fixed. | 19:09 |
| mgoddard_ | jlvillal: ok, good info | 19:09 |
| jlvillal | In oslo.serialization | 19:09 |
| mgoddard_ | can't really investigate now, life calls | 19:09 |
| jlvillal | mgoddard_, https://review.openstack.org/#/c/508287/ | 19:10 |
| patchbot | patch 508287 - oslo.serialization - When serializing an exception return its 'repr' (MERGED) | 19:10 |
| jlvillal | mgoddard_, Okay. Go have a life :) | 19:10 |
| *** harlowja has quit IRC | 19:10 | |
| mgoddard_ | nice, thanks :) | 19:10 |
| *** lucasagomes is now known as lucas-afk | 19:12 | |
| mgoddard_ | uh oh | 19:14 |
| mgoddard_ | Feb 16 16:36:43.889351 ubuntu-xenial-rax-iad-0002580271 ironic-conductor[20042]: /usr/local/lib/python2.7/dist-packages/oslo_serialization/jsonutils.py:192: UserWarning: Cannot convert Trait(created_at=2018-02-16T16:34:14Z,node_id='1',trait='CUSTOM_GOLD',updated_at=None) to primitive, will raise ValueError instead of warning in version 3.0 | 19:14 |
| TheJulia | ugh....... fun nova issue with placement again | 19:14 |
| mgoddard_ | Raised a bug for that issue. https://bugs.launchpad.net/ironic/+bug/1750027 | 19:18 |
| openstack | Launchpad bug 1750027 in Ironic "Traits & cleaning issue" [Undecided,New] | 19:18 |
| mgoddard_ | Sorry, need to dash. o/ | 19:18 |
| TheJulia | mgoddard_: thanks! | 19:21 |
| *** sambetts|afk has quit IRC | 19:22 | |
| *** mgoddard_ has quit IRC | 19:23 | |
| *** sambetts_ has joined #openstack-ironic | 19:24 | |
| *** vsaienk0 has joined #openstack-ironic | 19:25 | |
| *** trown|lunch is now known as trown | 19:32 | |
| *** vsaienk0 has quit IRC | 19:35 | |
| *** vsaienk0 has joined #openstack-ironic | 19:39 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: DNM: Attempt to collect more info re grenade failures https://review.openstack.org/545147 | 19:41 |
| *** dprince has quit IRC | 19:43 | |
| *** vsaienk0 has quit IRC | 19:49 | |
| jlvillal | mgoddard, Maybe there needs to be a __repr__() method defined | 19:49 |
| *** weshay|bbiab is now known as weshay | 19:50 | |
| *** harlowja has joined #openstack-ironic | 19:50 | |
| *** etingof has quit IRC | 19:57 | |
| *** dprince has joined #openstack-ironic | 20:02 | |
| *** r-daneel_ has joined #openstack-ironic | 20:02 | |
| *** r-daneel has quit IRC | 20:05 | |
| *** r-daneel_ is now known as r-daneel | 20:05 | |
| *** ijw has joined #openstack-ironic | 20:12 | |
| *** crushil__ has joined #openstack-ironic | 20:12 | |
| *** crushil_ has joined #openstack-ironic | 20:14 | |
| *** crushil__ has quit IRC | 20:18 | |
| *** cdearborn has quit IRC | 20:40 | |
| *** ijw has quit IRC | 20:58 | |
| *** ijw has joined #openstack-ironic | 21:02 | |
| *** sambetts_ has quit IRC | 21:06 | |
| *** r-daneel has quit IRC | 21:10 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Disable .pyc files for grenade multinode https://review.openstack.org/544750 | 21:30 |
| *** trown is now known as trown|brb | 21:33 | |
| *** trown|brb is now known as trown|outtypewww | 21:33 | |
| *** jcoufal has quit IRC | 21:35 | |
| *** etingof has joined #openstack-ironic | 21:36 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Disable .pyc files for grenade multinode https://review.openstack.org/544750 | 21:36 |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: DNM Test disabling .pyc files for Grenade https://review.openstack.org/545089 | 21:48 |
| *** r-daneel has joined #openstack-ironic | 21:48 | |
| TheJulia | jroll: ^^^ that _might_ work *crosses fingers* | 21:50 |
| TheJulia | tl;dr with the weirdness on pike->queens... it makes the most sense for systemd to just to silently inject | 21:50 |
| jroll | yes! | 21:51 |
| jroll | I think we should also do that on master, fwiw | 21:51 |
| jroll | much cleaner | 21:51 |
| jroll | I told someone the other day that this is the first time I've wished for a php feature (/etc/php.ini) in python | 21:51 |
| *** dprince has quit IRC | 21:52 | |
| TheJulia | if it works, I'll update the patch on master | 21:52 |
| jroll | sounds good, thanks | 21:52 |
| TheJulia | much cleaner, more sane, less weirdness | 21:52 |
| jroll | ++ | 21:53 |
| * TheJulia wonders if it comes with drinks | 21:54 | |
| EmilienM | TheJulia: can you please join #rdo please? | 22:04 |
| EmilienM | https://review.openstack.org/#/c/542609/ broke ironic rpm in RDO | 22:05 |
| patchbot | patch 542609 - ironic - Imported Translations from Zanata (MERGED) | 22:05 |
| jroll | O_o | 22:07 |
| jlvillal | TheJulia, Off topic. Would it be useful to have the openstackgerrit bot announce when a patch gets a -2 in the gate? I noticed that is a capability. | 22:07 |
| TheJulia | jlvillal: in the gate as in in the testing pipeline? | 22:11 |
| * jroll feels like oprah today | 22:25 | |
| *** crushil_ has quit IRC | 22:26 | |
| jroll | ironic gets a bug! | 22:26 |
| TheJulia | lol | 22:26 |
| jroll | this key doesn't exist any more: https://github.com/openstack/ironic/blob/master/ironic/objects/base.py#L253 | 22:26 |
| jroll | not sure why I'm hitting it, ran online data migrations (this is a queens-ish devstack) | 22:26 |
| TheJulia | hahaha | 22:26 |
| * TheJulia should have just reverted the change | 22:27 | |
| jroll | ah, I have node traits without a version | 22:27 |
| jroll | ok we're happier now | 22:28 |
| jlvillal | TheJulia, gate as in going to be merged | 22:28 |
| * TheJulia wonders how you pulled that off.... | 22:28 | |
| TheJulia | jlvillal: that would be awesome | 22:28 |
| jroll | TheJulia: might have added traits via api while this was in dev | 22:29 |
| jroll | or maybe even via the db? who knows | 22:29 |
| jlvillal | TheJulia, Okay. I'll propose a change later. Need to run to the store. | 22:29 |
| TheJulia | jroll: you may have also never run the online migration | 22:29 |
| TheJulia | jlvillal: thanks! | 22:29 |
| jroll | TheJulia: I just ran it | 22:30 |
| TheJulia | k | 22:30 |
| *** Goneri has quit IRC | 22:30 | |
| * TheJulia deletes conditional and awaits unit tests | 22:31 | |
| jlvillal | TheJulia, hrmm. It seems like it should already be happening. But I don't recall ever seeing a notice. I will dig in later. | 22:33 |
| *** zzzeek has quit IRC | 22:38 | |
| *** priteau has quit IRC | 22:41 | |
| *** zzzeek has joined #openstack-ironic | 22:42 | |
| *** jlabarre has quit IRC | 22:43 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: DNM Test disabling .pyc files for Grenade https://review.openstack.org/545089 | 22:50 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix default object versioning for Rocky https://review.openstack.org/545480 | 22:52 |
| TheJulia | hmm... that one fell through the CI cracks as it were | 22:52 |
| *** goldyfruit has joined #openstack-ironic | 22:59 | |
| goldyfruit | Hi guys | 22:59 |
| goldyfruit | Is it mandatory to have to use Swift/RGW with Ironic ? | 23:00 |
| goldyfruit | My Glance images are stored in Ceph via RBD not via Swift | 23:00 |
| goldyfruit | So technically I don't need the swift_temp_key | 23:00 |
| devananda | hi goldyfruit ! https://docs.openstack.org/ironic/pike/install/configure-glance-swift.html contains some answers to that question | 23:02 |
| devananda | swift tempurls are used to secure the authentication during deploy-callback | 23:03 |
| goldyfruit | devananda: I check this documentation and the one about RGW too | 23:03 |
| TheJulia | since the call from ipa has no means of authentication, it realistically has to have a temp url key setup if you are using the direct deployment method | 23:04 |
| goldyfruit | devananda: https://docs.openstack.org/ironic/pike/admin/radosgw.html | 23:04 |
| goldyfruit | For Glance API: default_store = swift | 23:04 |
| goldyfruit | If it's just create a user/token in RGW, that's fine but I don't want to change my default store from rbd to swift ^^ | 23:06 |
| jroll | I think there's a way to go without | 23:14 |
| * jroll looking for it | 23:14 | |
| *** weshay is now known as weshay_PTO | 23:14 | |
| goldyfruit | jroll: Thanks :) | 23:15 |
| jroll | ah boo, it wasn't what I thought | 23:17 |
| jroll | goldyfruit: you're using nova, yeah? | 23:17 |
| jroll | note to self: we should allow ironic-conductor to download an image and serve it over http somehow | 23:18 |
| TheJulia | stepping away for a run into town/dinner | 23:19 |
| *** figleaf is now known as edleafe | 23:22 | |
| goldyfruit | jroll: yep | 23:22 |
| goldyfruit | TheJulia: enjoy ! | 23:22 |
| jroll | yeah, I'm not sure I have anything for you then, sorry :/ | 23:22 |
| jroll | I guess with the iscsi deploy driver you don't need swift | 23:22 |
| jroll | but then you have to use the iscsi driver :/ | 23:23 |
| goldyfruit | jroll: At least I don't need to move the images from RBD to RGW ^^ | 23:23 |
| jroll | heh, yeah | 23:24 |
| * jroll should also find dinner | 23:24 | |
| jroll | see y'all monday, and good luck goldyfruit | 23:24 |
| goldyfruit | jroll: Thanks for your help ! | 23:24 |
| devananda | goldyfruit: yeah, I think you need to use iscsi deploy driver instead of agent then ... this will cause conductor to buffer the full image locally and serve over TFTP/HTTP rather than tell the agent to download from glance/swift directly | 23:37 |
| *** goldyfruit has quit IRC | 23:37 | |
| *** crushil_ has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!