| *** romcheg has joined #openstack-ironic | 00:00 | |
| *** achanda has joined #openstack-ironic | 00:09 | |
| NobodyCam | nights ya all | 00:14 |
|---|---|---|
| *** romcheg has quit IRC | 00:15 | |
| *** Haomeng|2 has quit IRC | 00:19 | |
| *** anish has left #openstack-ironic | 00:23 | |
| rloo_ | night NobodyCam | 00:23 |
| rloo_ | me too, I'm calling it a night. I'll be back on Tuesday :-) | 00:24 |
| *** rloo_ has quit IRC | 00:24 | |
| *** rloo has joined #openstack-ironic | 00:29 | |
| *** jiangfei|3 has quit IRC | 00:29 | |
| *** jiangfei|3 has joined #openstack-ironic | 00:29 | |
| rloo | hi JayF, JoshNang, jroll: could one of you tell the other ones, maybe JayF, that the cleaning doc mentions clean_step http://docs.openstack.org/developer/ironic/deploy/cleaning.html#what-clean-step-is-running | 00:30 |
| rloo | JayF, JoshNang, jroll: but clean_step is not yet exposed in ironic and thus not available in the client. I think you can see something in driver_internal_info but not sure. also wondering if we should microversion the cleaning stuff. ciao. | 00:32 |
| *** rloo has quit IRC | 00:32 | |
| *** TheJulia has quit IRC | 00:35 | |
| *** TheJulia_ has joined #openstack-ironic | 00:35 | |
| *** ChuckC has quit IRC | 00:44 | |
| *** jiangfei|3 has quit IRC | 00:58 | |
| *** jiangfei|3 has joined #openstack-ironic | 00:59 | |
| *** tteggel_ has quit IRC | 01:00 | |
| *** mordred has quit IRC | 01:01 | |
| *** jgrimm has quit IRC | 01:01 | |
| *** tteggel has joined #openstack-ironic | 01:01 | |
| *** logan2_ has quit IRC | 01:02 | |
| *** gilliard has quit IRC | 01:02 | |
| *** jgrimm has joined #openstack-ironic | 01:02 | |
| *** JoshNang has quit IRC | 01:02 | |
| *** logan2_ has joined #openstack-ironic | 01:02 | |
| *** JoshNang has joined #openstack-ironic | 01:06 | |
| *** mordred has joined #openstack-ironic | 01:06 | |
| *** gilliard has joined #openstack-ironic | 01:07 | |
| *** chenglch has joined #openstack-ironic | 01:23 | |
| *** harlowja is now known as harlowja_away | 01:52 | |
| *** romcheg has joined #openstack-ironic | 01:59 | |
| *** romcheg has quit IRC | 02:04 | |
| *** ijw_ has quit IRC | 02:27 | |
| *** Marga_ has quit IRC | 02:28 | |
| *** jerryz has joined #openstack-ironic | 02:28 | |
| *** spandhe has quit IRC | 02:32 | |
| *** jjohnson2 has joined #openstack-ironic | 02:34 | |
| *** jcoufal has joined #openstack-ironic | 02:39 | |
| *** achanda has quit IRC | 03:03 | |
| *** jcoufal has quit IRC | 03:12 | |
| *** achanda has joined #openstack-ironic | 03:16 | |
| *** achanda has quit IRC | 03:36 | |
| *** links has joined #openstack-ironic | 03:45 | |
| *** achanda has joined #openstack-ironic | 03:45 | |
| *** chlong has joined #openstack-ironic | 03:48 | |
| *** chlong has quit IRC | 03:49 | |
| *** romcheg has joined #openstack-ironic | 03:49 | |
| *** romcheg has quit IRC | 03:54 | |
| *** achanda has quit IRC | 04:02 | |
| *** subscope has quit IRC | 04:07 | |
| *** jjohnson2 has quit IRC | 04:08 | |
| *** Marga_ has joined #openstack-ironic | 04:15 | |
| *** jlvillal has quit IRC | 04:25 | |
| *** jlvillal has joined #openstack-ironic | 04:28 | |
| *** penick has joined #openstack-ironic | 04:41 | |
| *** penick has quit IRC | 04:42 | |
| *** achanda has joined #openstack-ironic | 04:45 | |
| *** penick has joined #openstack-ironic | 04:46 | |
| *** penick has quit IRC | 04:46 | |
| *** aweeks has quit IRC | 04:47 | |
| *** cppforlife_ has quit IRC | 04:47 | |
| *** cppforlife_ has joined #openstack-ironic | 04:49 | |
| *** aweeks has joined #openstack-ironic | 04:50 | |
| *** penick has joined #openstack-ironic | 05:15 | |
| *** penick has quit IRC | 05:15 | |
| *** Marga_ has quit IRC | 05:28 | |
| *** mdbooth has quit IRC | 05:32 | |
| *** penick has joined #openstack-ironic | 05:33 | |
| *** mdbooth has joined #openstack-ironic | 05:38 | |
| *** romcheg has joined #openstack-ironic | 05:40 | |
| *** romcheg has quit IRC | 05:47 | |
| *** penick has quit IRC | 05:56 | |
| *** achanda has quit IRC | 05:57 | |
| *** spandhe has joined #openstack-ironic | 06:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Transifex https://review.openstack.org/169184 | 06:12 |
| *** achanda has joined #openstack-ironic | 06:26 | |
| *** jerryz has quit IRC | 06:28 | |
| *** pas-ha has joined #openstack-ironic | 06:31 | |
| *** spandhe has quit IRC | 06:42 | |
| *** dtantsur|afk is now known as dtantsur | 07:02 | |
| dtantsur | Morning folks, TGIF :) | 07:03 |
| *** takadayuiko has joined #openstack-ironic | 07:09 | |
| *** romcheg has joined #openstack-ironic | 07:21 | |
| *** jistr has joined #openstack-ironic | 07:29 | |
| *** jcoufal has joined #openstack-ironic | 07:32 | |
| *** penick has joined #openstack-ironic | 07:32 | |
| *** achanda has quit IRC | 07:33 | |
| *** achanda has joined #openstack-ironic | 07:33 | |
| *** penick has quit IRC | 07:40 | |
| *** penick has joined #openstack-ironic | 08:02 | |
| *** derekh has joined #openstack-ironic | 08:11 | |
| *** penick has quit IRC | 08:19 | |
| *** lucasagomes has joined #openstack-ironic | 08:20 | |
| *** romcheg has quit IRC | 08:32 | |
| *** romcheg has joined #openstack-ironic | 08:35 | |
| *** ndipanov has joined #openstack-ironic | 08:46 | |
| *** andreykurilin_ has joined #openstack-ironic | 08:50 | |
| *** achanda has quit IRC | 08:55 | |
| *** achanda has joined #openstack-ironic | 08:55 | |
| *** oomichi_ has quit IRC | 08:56 | |
| *** oomichi has joined #openstack-ironic | 08:58 | |
| *** oomichi has quit IRC | 09:01 | |
| *** subscope has joined #openstack-ironic | 09:02 | |
| *** yuanying has quit IRC | 09:03 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Use my_ip as default URL instead of localhost https://review.openstack.org/169965 | 09:04 |
| *** takadayuiko has quit IRC | 09:09 | |
| *** pelix has joined #openstack-ironic | 09:21 | |
| *** achanda has quit IRC | 09:25 | |
| *** achanda has joined #openstack-ironic | 09:28 | |
| *** achanda has quit IRC | 09:28 | |
| *** chenglch has quit IRC | 09:30 | |
| *** achanda has joined #openstack-ironic | 09:30 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add missing test for DB migration 2fb93ffd2af1 https://review.openstack.org/170406 | 09:32 |
| *** bnemec has quit IRC | 09:33 | |
| *** andreykurilin_ has quit IRC | 09:50 | |
| *** dtantsur is now known as dtantsur|lunch | 10:00 | |
| *** achanda has quit IRC | 10:02 | |
| *** bnemec has joined #openstack-ironic | 10:19 | |
| *** romcheg has quit IRC | 10:52 | |
| *** Marga_ has joined #openstack-ironic | 10:52 | |
| *** jcoufal_ has joined #openstack-ironic | 10:55 | |
| *** jcoufal has quit IRC | 10:58 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix formatting issue in install guide https://review.openstack.org/170437 | 11:01 |
| *** dtantsur|lunch is now known as dtantsur | 11:03 | |
| *** romcheg has joined #openstack-ironic | 11:05 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Remove DB tests workarounds https://review.openstack.org/170438 | 11:08 |
| *** romcheg has quit IRC | 11:10 | |
| TheJulia_ | NobodyCam: updated, but didn't have a chance to test it on physical hardware yesterday | 11:24 |
| dtantsur | TheJulia_, morning | 11:24 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Implement basic checks in functest/devstack-test.sh https://review.openstack.org/169776 | 11:33 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix help string for glance auth_strategy option https://review.openstack.org/170439 | 11:36 |
| TheJulia_ | good morning dtantsur | 11:39 |
| *** Marga_ has quit IRC | 11:40 | |
| *** alexpilotti has joined #openstack-ironic | 11:42 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: Use my_ip as default URL instead of localhost https://review.openstack.org/169965 | 11:43 |
| *** Marga_ has joined #openstack-ironic | 11:50 | |
| *** lucasagomes is now known as lucas-hungry | 11:51 | |
| *** ndipanov is now known as ndipanoff | 11:51 | |
| *** Marga_ has quit IRC | 12:05 | |
| *** adam_g is now known as adam_g_out | 12:20 | |
| *** jjohnson2 has joined #openstack-ironic | 12:26 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Document ports creating configuration for in-band inspection https://review.openstack.org/169253 | 12:26 |
| *** jistr_ has joined #openstack-ironic | 12:27 | |
| *** dtantsur_ has joined #openstack-ironic | 12:27 | |
| *** jcoufal has joined #openstack-ironic | 12:27 | |
| *** lsmola_ has joined #openstack-ironic | 12:27 | |
| *** dtantsur has quit IRC | 12:28 | |
| *** jcoufal_ has quit IRC | 12:28 | |
| *** jcoufal_ has joined #openstack-ironic | 12:28 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix help string for glance auth_strategy option https://review.openstack.org/170439 | 12:30 |
| *** jistr has quit IRC | 12:30 | |
| *** lsmola has quit IRC | 12:31 | |
| *** jistr has joined #openstack-ironic | 12:31 | |
| *** dtantsur_ has quit IRC | 12:32 | |
| *** jcoufal has quit IRC | 12:32 | |
| *** lsmola_ has quit IRC | 12:32 | |
| *** jistr_ has quit IRC | 12:32 | |
| *** dtantsur has joined #openstack-ironic | 12:37 | |
| *** lsmola_ has joined #openstack-ironic | 12:44 | |
| *** derekh has quit IRC | 13:06 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties https://review.openstack.org/167952 | 13:22 |
| *** openstackgerrit has quit IRC | 13:36 | |
| *** openstackgerrit has joined #openstack-ironic | 13:36 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: iLO driver documentation for UEFI secure boot https://review.openstack.org/168844 | 13:42 |
| TheJulia_ | it seems.... too quiet | 13:46 |
| *** TheJulia_ is now known as TheJulia | 13:46 | |
| jroll | morning TheJulia :) | 13:47 |
| jroll | some people have today off, maybe? | 13:47 |
| TheJulia | I'm sure some do, I know RedHat has today off | 13:48 |
| TheJulia | At least here in the states | 13:49 |
| jroll | yeah | 13:50 |
| jlvillal | dtantsur: I empathize with getting a Canadian visa. We have applied for a Canadian tourist visa for my wife so she could come up to the summit with me. What a pain! | 13:52 |
| dtantsur | oh yeah | 13:52 |
| dtantsur | also morning jroll, jlvillal :) | 13:52 |
| jlvillal | dtantsur: We thought a US tourist visa was bad ;) | 13:53 |
| lucas-hungry | yeah it's quiet today :) I thought it was holiday there in US | 13:53 |
| jlvillal | Good morning dtantsur | 13:53 |
| jlvillal | lucas-hungry: RedHat has a holiday today. But most companies do not. | 13:53 |
| *** lucas-hungry is now known as lucasagomes | 13:53 | |
| lucasagomes | gotcha | 13:54 |
| * lucasagomes is not sure if redhat in ireland is on holiday O.O | 13:54 | |
| jlvillal | lucas-hungry: Red Hat has "spring holiday" which mysteriously always occurs on Good Friday. Strange coincidence :) | 13:54 |
| dtantsur | lol | 13:54 |
| dtantsur | Red Hat Czech does not have holiday today, one on Mon | 13:54 |
| lucasagomes | yeah monday is holiday here too :-) | 13:55 |
| jroll | morning dtantsur, jlvillal, lucasagomes :) | 13:55 |
| lucasagomes | jroll, yo morning | 13:55 |
| jlvillal | jrolll, lucasagomes, TheJulia and the rest of Ironic: Good mroning | 13:55 |
| jlvillal | s/mroning/morning/ | 13:56 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add ability to save logs received from the ramdisk https://review.openstack.org/170195 | 13:56 |
| dtantsur | lucasagomes, can I use you for review of this patch chain ^^^ instead of Imre who is on PTO? | 13:58 |
| lucasagomes | sure | 13:58 |
| NobodyCam | good morning Ironicers and OFC ..... TGIF | 13:58 |
| lucasagomes | will take a look in a sec | 13:58 |
| dtantsur | NobodyCam, morning, really TGIF | 13:59 |
| NobodyCam | morning dtantsur, lucasagomes, jlvillal and TheJulia | 13:59 |
| jlvillal | NobodyCam: morning | 13:59 |
| NobodyCam | :) | 14:00 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Improve error handling when JSON is not returned by agent https://review.openstack.org/170275 | 14:04 |
| jlvillal | dtantsur: ^^^ addressed your comments. Except for _LE | 14:05 |
| dtantsur | yeah, _LE was my inattentiveness :) | 14:05 |
| jlvillal | dtantsur: I cheated and read the docs yesterday while doing the patch :D | 14:06 |
| BadCub | morning folks | 14:09 |
| lucasagomes | BadCub, morning | 14:09 |
| jlvillal | BadCub: morning | 14:09 |
| dtantsur | BadCub, morning | 14:09 |
| NobodyCam | morning BadCub | 14:09 |
| lucasagomes | dtantsur, the logs are just base64 encoded? Perhaps we should also gzip it to save some space? | 14:10 |
| dtantsur | lucasagomes, right, I plan on it, but I also want to save them gzipped. so it's up to the ramdisk essentially | 14:10 |
| dtantsur | maybe it's a stupid idea, don't know.. | 14:10 |
| NobodyCam | ahh lucasagomes I checked with mordred on the license he said you can add an apache license to a mit licensed pice of code. | 14:10 |
| BadCub | heya and happy Friday lucasagomes , jlvillal , dtantsur , NobodyCam :-) | 14:10 |
| mordred | NobodyCam: aroo? | 14:10 |
| NobodyCam | hehehe morning mordred | 14:11 |
| lucasagomes | NobodyCam, oh that tftp is MIT? | 14:11 |
| BadCub | morning mordred | 14:11 |
| mordred | yes. nothing in the MIT license prevents adding an apache license | 14:11 |
| NobodyCam | yes | 14:11 |
| mordred | morning NobodyCam BadCub | 14:11 |
| lucasagomes | oh right so it's missing the header | 14:11 |
| lucasagomes | mordred, we gotta put the MIT license on the LICENSE file too no? | 14:11 |
| NobodyCam | lucasagomes: the orginal code was found at: http://www.nightmare.com/~ryb/code/tftp.py | 14:12 |
| mordred | lucasagomes: no - I do not think we need to add it there | 14:12 |
| lucasagomes | NobodyCam, right, yeah I'm a bit skeptical actually to have a tftp client implementation in our tree | 14:12 |
| lucasagomes | mordred, https://wiki.openstack.org/wiki/LegalIssuesFAQ#Incorporating_BSD.2FMIT_Licensed_Code | 14:12 |
| lucasagomes | NobodyCam, maybe we should just use a python library that already exist to do the tftp work? | 14:13 |
| dtantsur | ++ or require 'tftp' utility | 14:13 |
| NobodyCam | lucasagomes: they are much larger | 14:14 |
| mordred | lucasagomes: well, it won't hurt to add it to the license file - I disagree with that advice taht it's necessary - but it won't hurt | 14:14 |
| lucasagomes | mordred, right | 14:14 |
| NobodyCam | that adds a bedendicy for a "simple" check ... but I can go any which way | 14:14 |
| NobodyCam | dependicy even | 14:15 |
| * NobodyCam goes back to coffee and reading | 14:15 | |
| dtantsur | typing English is hard :D | 14:15 |
| NobodyCam | esp before the first cup of coffee | 14:16 |
| lucasagomes | hah yeah | 14:16 |
| NobodyCam | :) | 14:16 |
| TheJulia | NobodyCam: I thought using IRC before the first cup of coffee was considered a crime? | 14:16 |
| BadCub | TheJulia: It is in most civilized cultures! | 14:17 |
| NobodyCam | lol | 14:17 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: grub2 bootloader support for uefi boot mode https://review.openstack.org/166192 | 14:18 |
| lucasagomes | dtantsur, the discoverd ramdisk that is saving that logs, is that in the instack version only? | 14:23 |
| dtantsur | lucasagomes, it's not implemented yet | 14:23 |
| lucasagomes | dtantsur, right, ok so it's a dict, the key is the path of the log? | 14:24 |
| dtantsur | lucasagomes, just everything, e.g. 'dmesg' | 14:24 |
| lucasagomes | kike /var/log/message or something | 14:24 |
| lucasagomes | right | 14:24 |
| dtantsur | maybe path, though I don't account for / in names | 14:24 |
| lucasagomes | dtantsur, right, code looks good | 14:26 |
| lucasagomes | but I wonder whether it would be better to just collect all logs into a tar.gz file | 14:27 |
| lucasagomes | base64 that and give it back to the service | 14:27 |
| dtantsur | what's the profit? | 14:27 |
| *** hj-hp has joined #openstack-ironic | 14:27 | |
| *** hj-hp has quit IRC | 14:30 | |
| lucasagomes | dtantsur, just looks nicer implementation wiser, instead of interacting on each key and write a file for each element you could just dump a tar.gz file with all the logs | 14:31 |
| lucasagomes | but not a big deal anyway | 14:31 |
| dtantsur | hmm | 14:31 |
| *** pas-ha has quit IRC | 14:32 | |
| *** pas-ha has joined #openstack-ironic | 14:32 | |
| dtantsur | lucasagomes, makes sense, will change to just dumping one file | 14:32 |
| *** links has quit IRC | 14:33 | |
| lucasagomes | plus being zipped actually save some space | 14:33 |
| lucasagomes | dtantsur, ok, lemme know and I will re-review | 14:33 |
| *** hj-hp has joined #openstack-ironic | 14:33 | |
| *** subscope has quit IRC | 14:35 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add ability to save logs received from the ramdisk https://review.openstack.org/170195 | 14:39 |
| dtantsur | lucasagomes, updated | 14:39 |
| *** openstackgerrit has quit IRC | 14:39 | |
| *** openstackgerrit has joined #openstack-ironic | 14:39 | |
| *** pas-ha has quit IRC | 14:40 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Document ipmi_address in HTTP API https://review.openstack.org/170520 | 14:42 |
| dtantsur | also folks, could you have a look at small documentation update? https://review.openstack.org/#/c/169253/ | 14:47 |
| *** Marga_ has joined #openstack-ironic | 14:49 | |
| NobodyCam | dtantsur: are there other supported options? | 14:53 |
| *** pas-ha has joined #openstack-ironic | 14:55 | |
| NobodyCam | lucasagomes: looks like https://pypi.python.org/pypi/tftpy is also MIT lincensed too | 14:56 |
| lucasagomes | NobodyCam, yup but that's a lib | 14:56 |
| NobodyCam | lucasagomes: so that would need to be added to global requirements? | 14:58 |
| lucasagomes | NobodyCam, yeah, there's also dtantsur ideas to just use the tftp utility | 14:58 |
| lucasagomes | if installed Ironic will use it to check for the tftp | 14:59 |
| jroll | what exactly are we trying to check? | 15:00 |
| NobodyCam | ya I don't think its installed by default in ubuntu land | 15:00 |
| lucasagomes | jroll, the patch is checking if the tftp is working | 15:00 |
| lucasagomes | NobodyCam, right it's fine, like ipmitool right? | 15:00 |
| jroll | would 'tftp is listening' be good enough? | 15:00 |
| NobodyCam | jroll: was trying to come up with a way to check the tftp service | 15:00 |
| jroll | although... it's UDP, you won't get a return if you just send a packet | 15:01 |
| jroll | blah | 15:01 |
| NobodyCam | :) | 15:01 |
| jroll | or I should say you can't just open a connection | 15:01 |
| jroll | btw y'all, CONF.scheduler_use_baremetal_filters is broken right now in nova | 15:02 |
| jroll | https://review.openstack.org/170299 | 15:02 |
| lucasagomes | oh | 15:03 |
| jroll | no idea how long it's been broken :) | 15:03 |
| jroll | but yeah | 15:03 |
| lucasagomes | so those are the exact{ram,cpu,disk} filters right? | 15:05 |
| jroll | yeah, one other one too I think | 15:07 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add ability to save logs received from the ramdisk https://review.openstack.org/170195 | 15:07 |
| dtantsur | lucasagomes, ^^^ and parent patch too please | 15:07 |
| *** EmilienM is now known as EmilienM|afk | 15:07 | |
| *** Marga_ has quit IRC | 15:08 | |
| lucasagomes | dtantsur, just to have a context since the commit message is not saying | 15:20 |
| lucasagomes | why was the error plugin disabled by default before? | 15:20 |
| lucasagomes | and we are now enabling it by default to improve debbuing discoverd in case of failures? | 15:21 |
| dtantsur | lucasagomes, 1. mostly because nobody wasn't using; 2. yes, I'll be putting more effort into it | 15:21 |
| lucasagomes | dtantsur, cool, ok | 15:22 |
| *** jcoufal_ has quit IRC | 15:23 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Add append_processing_hooks options https://review.openstack.org/170535 | 15:23 |
| *** rameshg87 has joined #openstack-ironic | 15:25 | |
| rameshg87 | good morning folks | 15:25 |
| dtantsur | rameshg87, morning | 15:26 |
| *** links has joined #openstack-ironic | 15:27 | |
| BadCub | g'morning rameshg87 | 15:27 |
| rameshg87 | dtantsur: BadCub: o/ | 15:27 |
| rameshg87 | i assumed it was a holiday everywhere, but looks like most of the people are here | 15:28 |
| rameshg87 | it's a holiday here for us in India :) | 15:28 |
| NobodyCam | mornign rameshg87 | 15:28 |
| rameshg87 | NobodyCam: o/ | 15:28 |
| NobodyCam | ya not holiday here :( | 15:28 |
| *** Marga_ has joined #openstack-ironic | 15:31 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: Enable ramdisk_error plugin by default https://review.openstack.org/170168 | 15:36 |
| openstackgerrit | Merged stackforge/ironic-discoverd: Add ability to save logs received from the ramdisk https://review.openstack.org/170195 | 15:37 |
| *** absubram has joined #openstack-ironic | 15:38 | |
| *** EmilienM|afk is now known as EmilienM | 15:39 | |
| lucasagomes | rameshg87, hi there | 15:43 |
| rameshg87 | lucasagomes: hello | 15:47 |
| * NobodyCam gets another cup of coffee | 15:47 | |
| *** penick has joined #openstack-ironic | 15:50 | |
| *** jistr has quit IRC | 15:53 | |
| *** jistr has joined #openstack-ironic | 15:53 | |
| *** pelix has quit IRC | 15:59 | |
| *** pelix has joined #openstack-ironic | 16:00 | |
| *** penick has quit IRC | 16:04 | |
| dtantsur | calling it a week, see you! | 16:09 |
| *** dtantsur is now known as dtantsur|afk | 16:09 | |
| NobodyCam | have a great weekend dtantsur|afk | 16:09 |
| jlvillal | dtantsur|afk: Ciao | 16:10 |
| *** pelix has quit IRC | 16:14 | |
| *** kkoski has joined #openstack-ironic | 16:16 | |
| *** kkoski has quit IRC | 16:17 | |
| *** jistr has quit IRC | 16:17 | |
| *** pas-ha has quit IRC | 16:21 | |
| jlvillal | rameshg87: Good evening. Isn't it late? | 16:36 |
| rameshg87 | jlvillal: hello | 16:37 |
| rameshg87 | jlvillal: yeah it is :) | 16:37 |
| rameshg87 | jlvillal: it's 10PM here. i was just about to get off irc. wanted to address comments on on of my review :) | 16:37 |
| jlvillal | rameshg87: I added one more comment | 16:37 |
| * rameshg87 checks | 16:37 | |
| jlvillal | rameshg87: Basically saying, I said the same thing rloo did a few patches back :P | 16:38 |
| jlvillal | rameshg87: https://review.openstack.org/#/c/169419/6/ironic/drivers/modules/iscsi_deploy.py | 16:39 |
| jlvillal | rameshg87: Not a comment that requires any change by you. | 16:39 |
| rameshg87 | jlvillal: yeah just saw that. my opinion doesn't hold anymore. more people are for it :) | 16:40 |
| jlvillal | rameshg87: :D | 16:40 |
| *** Marga_ has quit IRC | 16:40 | |
| rameshg87 | jlvillal: i don't know may be we cared too much about a vendor passthru that's going to be made only by another ramdisk | 16:41 |
| rameshg87 | jlvillal: but that's fine, more the checks, the better. i agree with that. | 16:41 |
| *** Marga_ has joined #openstack-ironic | 16:41 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 16:42 |
| jlvillal | rameshg87: I prefer error messages that give you some data to help diagnose the problem. | 16:42 |
| rameshg87 | jlvillal: ^^^^ | 16:42 |
| jlvillal | rameshg87: Thanks! | 16:42 |
| rameshg87 | jlvillal: yeah i agree, just added them. please have a look | 16:42 |
| rameshg87 | good night ironic. see you on monday | 16:43 |
| rameshg87 | have a great weekend everyone | 16:43 |
| rameshg87 | bye | 16:43 |
| *** rameshg87 has quit IRC | 16:43 | |
| lucasagomes | I will also call it a day | 16:46 |
| lucasagomes | have a good night everyone! Enjoy the weekend | 16:46 |
| openstackgerrit | Merged openstack/ironic: Improve error handling when JSON is not returned by agent https://review.openstack.org/170275 | 16:48 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 16:51 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: FIX Vm stuck when deploying with pxe_ssh + local boot https://review.openstack.org/170117 | 16:52 |
| NobodyCam | have a great weekend lucasagomes | 16:52 |
| lucasagomes | now going | 16:52 |
| NobodyCam | have a pint for /me | 16:52 |
| lucasagomes | NobodyCam, you too! | 16:52 |
| lucasagomes | NobodyCam, no alcohol today in Ireland | 16:52 |
| lucasagomes | :-( | 16:52 |
| NobodyCam | huh | 16:52 |
| lucasagomes | they don't sell it | 16:52 |
| NobodyCam | really | 16:52 |
| lucasagomes | tomorrow I will have one | 16:52 |
| NobodyCam | oh man | 16:52 |
| lucasagomes | NobodyCam, yeah it's a catholic country | 16:52 |
| BadCub | yep | 16:52 |
| * NobodyCam stock the fridge | 16:52 | |
| lucasagomes | so all the pubs are closed today | 16:52 |
| BadCub | dry day | 16:52 |
| lucasagomes | yup | 16:53 |
| NobodyCam | ieek | 16:53 |
| NobodyCam | :-p | 16:53 |
| lucasagomes | heh they only do it twice in a year | 16:53 |
| lucasagomes | good friday and xmas day | 16:53 |
| lucasagomes | so, it's alright :-) | 16:53 |
| NobodyCam | oh :) | 16:53 |
| BadCub | yeah :-) | 16:53 |
| NobodyCam | have two tomorrow then | 16:53 |
| lucasagomes | NobodyCam, BadCub you guys can have a pint for me! | 16:54 |
| lucasagomes | NobodyCam, yup will do | 16:54 |
| lucasagomes | have a great night and a great weekend! | 16:54 |
| NobodyCam | that I will do (after 5 pm) | 16:54 |
| BadCub | lucasagomes: I'll do an extra martini for ya! | 16:54 |
| lucasagomes | BadCub, +1 | 16:54 |
| *** spandhe has joined #openstack-ironic | 16:54 | |
| *** lucasagomes is now known as lucas-dinner | 16:54 | |
| BadCub | have an awesome weekend lucasagomes :-) make up for lost drinking time tomorrow!!! lol | 16:54 |
| lucas-dinner | BadCub, hah will totally do it!! | 16:59 |
| BadCub | ++ | 17:00 |
| TheJulia | out of curiosity, has anyone actually tried setting logical names? | 17:00 |
| NobodyCam | did we land the update to the name field? | 17:02 |
| BadCub | I think we did | 17:02 |
| TheJulia | Looks like its there | 17:02 |
| *** harlowja_away is now known as harlowja | 17:02 | |
| TheJulia | looks like it should be letting me based on the request that I'm sending to the api, digging more | 17:03 |
| *** Marga_ has quit IRC | 17:05 | |
| NobodyCam | BadCub: not sure your tracking but 169981 is not +a's | 17:05 |
| NobodyCam | +a'd even | 17:05 |
| * BadCub looks | 17:06 | |
| BadCub | nope, not tracking, but thanks for the heads up on it :-) | 17:07 |
| * BadCub updates pad | 17:08 | |
| NobodyCam | j* around today or out on holiday? | 17:10 |
| BadCub | J* is out on company outing | 17:11 |
| NobodyCam | ahh that right. | 17:11 |
| NobodyCam | did we want to land the enable cleaning patch? | 17:11 |
| BadCub | I think the consensus yesterday was to go for it. | 17:12 |
| jroll | NobodyCam: I'm around today | 17:13 |
| jroll | TheJulia: maybe it's a version thing? | 17:13 |
| NobodyCam | hey hey jroll | 17:14 |
| NobodyCam | :) | 17:14 |
| jroll | morning :) | 17:14 |
| *** hj-hp has quit IRC | 17:14 | |
| jroll | random question: what would make the power sync loop not run for a node that is not in maintenance mode? | 17:15 |
| TheJulia | jroll: thats exactly it | 17:15 |
| TheJulia | microversioning has never gone into ironicclient | 17:15 |
| jroll | yeah, I've given up on the client :/ | 17:15 |
| jroll | I feel like master may have it | 17:15 |
| NobodyCam | I was thinking of addressing rloo's comment on 169981 (enable cleaning) but didn't want to lose her +2 | 17:15 |
| * TheJulia goes and checks master | 17:16 | |
| * TheJulia sighs | 17:17 | |
| jroll | NobodyCam: I've got a +2 for you, but let me look at the comment | 17:18 |
| jroll | oh, never mind, you +A's | 17:18 |
| jroll | +A'd even | 17:18 |
| TheJulia | jroll: and a versioned request is required to be able to do anything with a name in the api | 17:19 |
| jroll | yep | 17:19 |
| TheJulia | This is rather... frustrating because I wanted to be able to set the name with the ansible module, and the only way I will really be able to do that is to begin forming direct rest requests. :( | 17:21 |
| jroll | TheJulia: is it not on master? | 17:22 |
| TheJulia | nope | 17:22 |
| jroll | uhhhhhh | 17:22 |
| jroll | huh. | 17:22 |
| TheJulia | at least not as far as I can tell at the moment | 17:22 |
| jroll | we should do that. | 17:22 |
| TheJulia | oh wait | 17:23 |
| TheJulia | hmm | 17:23 |
| TheJulia | it is | 17:23 |
| TheJulia | merged 7 days ago, so yeah, I need master ironicclient | 17:24 |
| TheJulia | much better | 17:25 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 17:26 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Enable cleaning by default https://review.openstack.org/165496 | 17:26 |
| NobodyCam | jroll: ^^^ | 17:26 |
| NobodyCam | brb | 17:26 |
| *** links has quit IRC | 17:27 | |
| jroll | NobodyCam: +2 | 17:27 |
| *** penick has joined #openstack-ironic | 17:28 | |
| *** david-lyle has quit IRC | 17:29 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 17:30 |
| *** Marga_ has joined #openstack-ironic | 17:47 | |
| *** achanda has joined #openstack-ironic | 17:57 | |
| devananda | morning, all | 18:02 |
| NobodyCam | morning devananda | 18:03 |
| NobodyCam | :) | 18:03 |
| TheJulia | good afternoon :) | 18:03 |
| *** Marga_ has quit IRC | 18:11 | |
| *** achanda has quit IRC | 18:20 | |
| *** Marga_ has joined #openstack-ironic | 18:21 | |
| *** achanda has joined #openstack-ironic | 18:24 | |
| NobodyCam | devananda: even thou I posted a patch to the enable cleaning patch (165496) I'm going to +a it as my edit was only to the description string of the conf option, and jroll already has a +2 on it. | 18:24 |
| jroll | NobodyCam: that's fine | 18:24 |
| NobodyCam | bam! | 18:26 |
| jroll | devananda: is there any way to look at the hash ring or something? | 18:27 |
| jroll | devananda: or otherwise debug why a node isn't getting a power sync loop run on it, even when it's not in maintenance? | 18:31 |
| devananda | jroll: huh. no, there isn't. | 18:32 |
| devananda | jroll: driver-list should show which conductor's have which drivers | 18:32 |
| jroll | devananda: yeah, this is bizarre, this node just doesn't want to run | 18:32 |
| jroll | right | 18:32 |
| devananda | jroll: so if no conductor is running the right driver, that would explain it. short of that, yea, :( | 18:32 |
| jroll | devananda: we run the same drivers on all conductors, this node is otherwise working (is deployed, node-update works, etc), it just doesn't get the loop run on it | 18:33 |
| jroll | we do have a latent bug with rebalancing, where a conductor might be out of sync with the hash ring, right? | 18:34 |
| jroll | (this is pretty old ironic code, fwiw, it might be fixed since then) | 18:34 |
| NobodyCam | I thought that was fixed.. a couple months ago | 18:35 |
| NobodyCam | imbw thou | 18:35 |
| jroll | oh, maybe | 18:35 |
| jroll | I'm not terribly concerned until I get prod up to date | 18:35 |
| jroll | we've been bad about rebasing this cycle >.> | 18:36 |
| NobodyCam | :-p *shame*shame* lol | 18:36 |
| devananda | jroll: yea, there was a bug about that | 18:36 |
| devananda | jroll: can you restart the conductors? | 18:36 |
| jroll | devananda: just did a rolling restart, same deal | 18:37 |
| devananda | jroll: if after a simple rolling restart it doesn't get picked up, i'd be very surprised/concerned... | 18:37 |
| jroll | same node even | 18:37 |
| jroll | yeah, tis strange | 18:37 |
| jroll | nothing in any logs for that node ID, at debug level | 18:37 |
| *** alexpilotti has quit IRC | 18:37 | |
| *** igordcard_ has joined #openstack-ironic | 18:41 | |
| NobodyCam | TheJulia: are you working on 168120, is it ready for review? | 18:43 |
| TheJulia | review away, I only marked it as WIP as I wanted to test it on real hardware first | 18:45 |
| NobodyCam | I think yuriyz asked the question I would also ask :-p | 18:58 |
| * TheJulia goes and takes a look since she has been looking at other code all day | 19:00 | |
| *** penick has quit IRC | 19:00 | |
| TheJulia | NobodyCam: and that is the frustrating part | 19:01 |
| NobodyCam | the save and reraise question? | 19:02 |
| TheJulia | and I also tried using self.config() which didn't work :( | 19:02 |
| TheJulia | NobodyCam: yup | 19:02 |
| NobodyCam | humm | 19:03 |
| TheJulia | NobodyCam: so save and reraise seems to be a popular vote thing, I'll do whatever | 19:03 |
| TheJulia | I don't see that much of a difference in the end although save_and_reraise_exception does seem to be a bit more useful, and it drove me towards cleaner logic | 19:04 |
| NobodyCam | I think that is more consistent with other code | 19:04 |
| TheJulia | indeed | 19:05 |
| TheJulia | I guess I'll add it back and change the tests later tonight or sometime tomorrow | 19:09 |
| NobodyCam | okay. :) | 19:10 |
| NobodyCam | TheJulia: any holiday weekend plans? | 19:10 |
| TheJulia | nope | 19:11 |
| TheJulia | When you have almost no family, it really is not a holiday. | 19:12 |
| NobodyCam | :/ | 19:13 |
| *** penick has joined #openstack-ironic | 19:14 | |
| *** achanda has quit IRC | 19:15 | |
| *** ndipanoff has quit IRC | 19:21 | |
| *** Marga_ has quit IRC | 19:26 | |
| *** ijw has joined #openstack-ironic | 19:28 | |
| *** mitz has quit IRC | 19:28 | |
| *** achanda has joined #openstack-ironic | 19:29 | |
| *** lucas-dinner has quit IRC | 19:29 | |
| *** mitz has joined #openstack-ironic | 19:37 | |
| *** krtaylor has quit IRC | 19:51 | |
| *** krtaylor has joined #openstack-ironic | 19:52 | |
| NobodyCam | slow day... I guess that makes it a "Good Friday" :{P | 19:58 |
| BadCub | yep | 19:58 |
| TheJulia | agreed | 20:06 |
| krotscheck | I'm going to make everyone here feel dirty: I have javascript talking to ironic-api. | 20:20 |
| krotscheck | mwa. ha. ha. | 20:20 |
| krotscheck | Time to rewrite everything in node! | 20:20 |
| NobodyCam | krotscheck: thats kewl in a dirty kind of way. What is for? horzion? | 20:24 |
| krotscheck | NobodyCam: Eventually, yes. devananda asked me to see how feasible it is to build a webclient for ironic that is entirely independent, but whose guts can also be included in ironic. | 20:25 |
| krotscheck | urrr | 20:25 |
| krotscheck | can also be included in horizon | 20:25 |
| NobodyCam | :) | 20:25 |
| krotscheck | My brain. It likes to randomly reassign pointers inside of my grammar. | 20:26 |
| NobodyCam | krotscheck: I have packet loss between brain and fingers so I feel your pain :-P | 20:27 |
| TheJulia | So this might sound like crazy question, but is there any wy to get the API to return the driver_info password entries, specifically wanting to compare if it needs to be updated. | 20:28 |
| NobodyCam | TheJulia: wouldn't getting node detail give you that? | 20:29 |
| TheJulia | so I would have to list detail of the signle node, but not via get... joy | 20:34 |
| NobodyCam | TheJulia: would it be bad to use update it all the time? | 20:40 |
| NobodyCam | s/use/just/ | 20:40 |
| TheJulia | NobodyCam: There is value in being able to execute an ansible module and know nothing changed. | 20:45 |
| NobodyCam | ++ ahh | 20:46 |
| TheJulia | just means my change set gets longer for shade :) | 20:46 |
| NobodyCam | :) | 20:47 |
| *** harlowja has quit IRC | 20:53 | |
| *** penick has quit IRC | 20:56 | |
| *** andreykurilin_ has joined #openstack-ironic | 21:00 | |
| TheJulia | NobodyCam: hmm... detail which appears to be an option for when you search up an instance uuid, at least in the client, still has it masked :( | 21:01 |
| devananda | TheJulia: correct - we implemented masking, but did not implement a policy flag to allow some users to not have passwords masked | 21:04 |
| devananda | TheJulia: and even then, if you're using ironic stand-alone, you couldn't turn it off selectively | 21:04 |
| BadCub | heya devananda | 21:05 |
| TheJulia | devananda: That is what I was suspecting | 21:08 |
| TheJulia | devananda: thanks for saving me an hour or so :) | 21:08 |
| *** achanda has quit IRC | 21:08 | |
| *** andreykurilin_ has quit IRC | 21:08 | |
| jroll | TheJulia: see policy.json, you can get the passwords | 21:09 |
| *** andreykurilin_ has joined #openstack-ironic | 21:09 | |
| jroll | oh lame, deva answered that | 21:09 |
| jroll | er no, he didn't | 21:09 |
| jroll | devananda: we have a policy flag | 21:09 |
| jroll | https://github.com/openstack/ironic/blob/master/etc/ironic/policy.json#L3 | 21:10 |
| jroll | should be able to set that to * | 21:10 |
| * TheJulia adds that to the list of things to check | 21:15 | |
| jroll | not sure how it works in noauth mode | 21:16 |
| jroll | but it might! | 21:16 |
| TheJulia | that would be kind of bad() in a sense | 21:17 |
| *** Marga_ has joined #openstack-ironic | 21:21 | |
| NobodyCam | brb | 21:27 |
| *** penick has joined #openstack-ironic | 21:49 | |
| openstackgerrit | Clif Houck proposed openstack/ironic: Add support for image caching for the agent driver https://review.openstack.org/161832 | 21:49 |
| *** igordcard_ has quit IRC | 21:51 | |
| openstackgerrit | Merged openstack/ironic: Enable cleaning by default https://review.openstack.org/165496 | 21:57 |
| NobodyCam | devananda: jroll JayF JoshNang ^^^^^^^^^ bam! | 21:58 |
| BadCub | sweet | 21:58 |
| *** spandhe has quit IRC | 22:03 | |
| devananda | time for me to tag a client release? | 22:04 |
| TheJulia | devananda: a client release would be appreciated :) | 22:05 |
| devananda | any pending super-important changes? | 22:05 |
| TheJulia | afaik no | 22:07 |
| *** achanda has joined #openstack-ironic | 22:08 | |
| krotscheck | Urm, what is the plural of chassis? | 22:11 |
| krotscheck | Oh wait. | 22:11 |
| krotscheck | n/m | 22:11 |
| NobodyCam | :) | 22:12 |
| *** david-lyle has joined #openstack-ironic | 22:12 | |
| *** jiangfei|3 has quit IRC | 22:14 | |
| *** achanda has quit IRC | 22:14 | |
| *** spandhe has joined #openstack-ironic | 22:15 | |
| *** ijw_ has joined #openstack-ironic | 22:19 | |
| *** ijw has quit IRC | 22:19 | |
| *** achanda has joined #openstack-ironic | 22:26 | |
| *** penick has quit IRC | 22:30 | |
| NobodyCam | nice.. jroll 170299 look like its been approved! | 22:31 |
| *** penick has joined #openstack-ironic | 22:35 | |
| *** harlowja has joined #openstack-ironic | 22:40 | |
| *** andreykurilin_ has quit IRC | 22:42 | |
| *** achanda has quit IRC | 22:53 | |
| * NobodyCam thinks he will sneak out a bit early this Good Friday | 23:04 | |
| *** Viswanath has joined #openstack-ironic | 23:31 | |
| *** achanda has joined #openstack-ironic | 23:32 | |
| *** penick has quit IRC | 23:35 | |
| * devananda wonders if tagging a new release friday night, just before a long weekend ,is a good idea ... | 23:35 | |
| *** Viswanath has quit IRC | 23:40 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!