| *** rama_y has quit IRC | 00:09 | |
| *** suro-patz has quit IRC | 00:12 | |
| *** rloo has joined #openstack-ironic | 00:18 | |
| *** hoangcx_ has joined #openstack-ironic | 00:24 | |
| *** mtanino_ has joined #openstack-ironic | 00:26 | |
| *** rbudden has quit IRC | 00:27 | |
| *** mtanino has quit IRC | 00:27 | |
| *** keedya has quit IRC | 00:37 | |
| *** praneshp has quit IRC | 00:42 | |
| *** baoli has joined #openstack-ironic | 00:48 | |
| *** praneshp has joined #openstack-ironic | 00:50 | |
| *** baoli has quit IRC | 00:54 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Unify testing scripts https://review.openstack.org/297424 | 01:12 |
|---|---|---|
| *** saripurigopi has quit IRC | 01:13 | |
| *** hoangcx has quit IRC | 01:17 | |
| *** hoangcx_ is now known as hoangcx | 01:17 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Unify testing scripts https://review.openstack.org/297424 | 01:20 |
| *** thrash is now known as thrash|g0ne | 01:27 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Restart consoles on conductor startup https://review.openstack.org/257252 | 01:34 |
| *** baoli has joined #openstack-ironic | 01:34 | |
| *** rloo has quit IRC | 01:50 | |
| *** mtanino_ has quit IRC | 02:02 | |
| *** bapalm has quit IRC | 02:06 | |
| *** baoli has quit IRC | 02:09 | |
| *** baoli_ has joined #openstack-ironic | 02:09 | |
| *** yuanying has quit IRC | 02:12 | |
| *** yuanying has joined #openstack-ironic | 02:12 | |
| *** bapalm has joined #openstack-ironic | 02:13 | |
| *** kbyrne has quit IRC | 02:13 | |
| *** zer0c00l has quit IRC | 02:13 | |
| *** zer0c00l has joined #openstack-ironic | 02:13 | |
| *** baoli_ has quit IRC | 02:15 | |
| *** praneshp has joined #openstack-ironic | 02:19 | |
| *** hoangcx has quit IRC | 02:19 | |
| *** hoangcx has joined #openstack-ironic | 02:24 | |
| *** kbyrne has joined #openstack-ironic | 02:24 | |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic: add new portal_port option for iscsi module https://review.openstack.org/303888 | 02:34 |
| *** yuanying has quit IRC | 02:52 | |
| *** Marga_ has quit IRC | 03:14 | |
| *** links has joined #openstack-ironic | 03:20 | |
| *** rbudden has joined #openstack-ironic | 03:29 | |
| *** rbudden has quit IRC | 03:34 | |
| *** kalpase has joined #openstack-ironic | 03:38 | |
| *** hoangcx has quit IRC | 03:45 | |
| *** hoangcx has joined #openstack-ironic | 03:47 | |
| *** achanda has joined #openstack-ironic | 03:49 | |
| *** yuanying has joined #openstack-ironic | 03:50 | |
| *** Marga_ has joined #openstack-ironic | 04:04 | |
| *** achanda has quit IRC | 04:05 | |
| *** appu103_ has joined #openstack-ironic | 04:06 | |
| *** appu103_ is now known as aparnav | 04:07 | |
| *** saripurigopi has joined #openstack-ironic | 04:07 | |
| *** penick has joined #openstack-ironic | 04:08 | |
| *** vmud213 has joined #openstack-ironic | 04:09 | |
| *** penick_ has joined #openstack-ironic | 04:12 | |
| *** Marga__ has joined #openstack-ironic | 04:12 | |
| *** penick has quit IRC | 04:13 | |
| *** penick_ is now known as penick | 04:13 | |
| *** Nisha has joined #openstack-ironic | 04:15 | |
| *** Marga_ has quit IRC | 04:15 | |
| *** saripurigopi has quit IRC | 04:32 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add support for building ISO for deploy ramdisk https://review.openstack.org/264590 | 04:32 |
| *** penick has quit IRC | 04:33 | |
| *** Marga__ is now known as fragatina | 04:39 | |
| *** Fdaisuke_ has joined #openstack-ironic | 04:42 | |
| *** Fdaisuke has quit IRC | 04:44 | |
| *** hoangcx has quit IRC | 04:47 | |
| *** spandhe has joined #openstack-ironic | 04:49 | |
| *** aparnav has quit IRC | 05:05 | |
| *** appu103_ has joined #openstack-ironic | 05:05 | |
| *** appu103_ is now known as aparnav | 05:05 | |
| *** hoangcx has joined #openstack-ironic | 05:08 | |
| *** itamarl has joined #openstack-ironic | 05:29 | |
| vmud213 | morning ironicers | 05:35 |
| *** suro-patz has joined #openstack-ironic | 05:39 | |
| *** spandhe has quit IRC | 05:40 | |
| Haomeng | vmud213: morning:) | 05:42 |
| *** spandhe has joined #openstack-ironic | 05:42 | |
| vmud213 | morning Haomeng :) | 05:43 |
| Haomeng | vmud213: :) | 05:43 |
| vmud213 | Haomeng: Would you mind taking a look at an IPA patch https://review.openstack.org/#/c/302143 when you get a chance? | 05:50 |
| Haomeng | vmud213: sure, will check, thanks | 05:50 |
| vmud213 | great. Thanks | 05:50 |
| Haomeng | vmud213: dont have more experence about EFI, so just review on common level:) | 05:59 |
| vmud213 | ok...Any comments as such? | 06:00 |
| vmud213 | thanks for reviewing | 06:00 |
| Haomeng | vmud213: reviewing, will add:) | 06:01 |
| *** spandhe has quit IRC | 06:10 | |
| yuikotakada | morning, ironic | 06:14 |
| *** achanda has joined #openstack-ironic | 06:23 | |
| *** tesseract has joined #openstack-ironic | 06:24 | |
| *** tesseract is now known as Guest15115 | 06:24 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config https://review.openstack.org/293876 | 06:30 |
| *** divya_ has joined #openstack-ironic | 06:35 | |
| divya_ | hi all | 06:35 |
| divya_ | ironic node is set to enroll state | 06:35 |
| divya_ | not going to manage state | 06:35 |
| divya_ | how to change it | 06:35 |
| *** suro-patz has quit IRC | 06:51 | |
| *** Fdaisuke has joined #openstack-ironic | 06:58 | |
| *** Fdaisuke_ has quit IRC | 07:01 | |
| *** mgould has joined #openstack-ironic | 07:06 | |
| *** athomas has joined #openstack-ironic | 07:08 | |
| lucasagomes | morning all :-) | 07:12 |
| *** hoangcx has quit IRC | 07:12 | |
| yuikotakada | lucasagomes, morning :) | 07:13 |
| *** ifarkas has joined #openstack-ironic | 07:13 | |
| Haomeng | lucasagomes: lucas, morning, so early today:) | 07:13 |
| Haomeng | yuikotakada: morning:) | 07:13 |
| yuikotakada | Haomeng, hi, morning :) | 07:13 |
| Haomeng | yuikotakada: :) | 07:13 |
| lucasagomes | Haomeng, heh yeah, I'm in czech rep. visiting the office. So I'm 1hr ahead of my normal time | 07:14 |
| Haomeng | lucasagomes: :) | 07:14 |
| mgould | morning Ironic! | 07:15 |
| * mgould is also in .cz and one hour ahead of normal | 07:16 | |
| *** Nisha has quit IRC | 07:22 | |
| vmud213 | morning mgould, lucasagomes,yuikotakada | 07:24 |
| divya_ | good morning mgould | 07:24 |
| *** mkovacik has joined #openstack-ironic | 07:24 | |
| *** praneshp has quit IRC | 07:24 | |
| mgould | morning vmud213 Haomeng yuikotakada divya_! | 07:24 |
| lucasagomes | vmud213, divya_ morning :-) | 07:24 |
| Haomeng | morning mgould: :) | 07:25 |
| vmud213 | lucasagomes: Would you mind looking at https://review.openstack.org/#/c/302143 if you have time | 07:25 |
| lucasagomes | vmud213, hi there, I will take a look. I just need to catch up with some stuff because I was away for a week | 07:26 |
| vmud213 | lucasagomes: no problem | 07:26 |
| vmud213 | mgould: what is this .cz ? | 07:26 |
| yuikotakada | mgould, vmud213, morning :) | 07:27 |
| vmud213 | wondering if you are refering to any timezone | 07:27 |
| mgould | vmud213, the Czech Republic | 07:28 |
| mgould | .cz is the TLD | 07:28 |
| mgould | most of the Red Hat Ironic team is in Brno for a meeting | 07:28 |
| vmud213 | oh..ok | 07:34 |
| *** ndipanov has quit IRC | 07:46 | |
| *** mbound has joined #openstack-ironic | 07:47 | |
| *** Nisha has joined #openstack-ironic | 07:48 | |
| *** e0ne has joined #openstack-ironic | 07:49 | |
| *** daemontool_ has joined #openstack-ironic | 07:50 | |
| *** daemontool has quit IRC | 07:53 | |
| *** ndipanov has joined #openstack-ironic | 07:59 | |
| *** vsaienko has joined #openstack-ironic | 08:00 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Get root device hints from the node object https://review.openstack.org/296539 | 08:03 |
| *** e0ne has quit IRC | 08:04 | |
| *** fragatina has quit IRC | 08:11 | |
| *** moshele has joined #openstack-ironic | 08:11 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config https://review.openstack.org/293876 | 08:20 |
| *** zer0c00l_ has joined #openstack-ironic | 08:21 | |
| *** zer0c00l has quit IRC | 08:22 | |
| *** zer0c00l_ is now known as zer0c00l | 08:22 | |
| *** itamarl has quit IRC | 08:23 | |
| *** zer0c00l has quit IRC | 08:23 | |
| *** itamarl has joined #openstack-ironic | 08:23 | |
| *** zer0c00l has joined #openstack-ironic | 08:23 | |
| *** jistr has joined #openstack-ironic | 08:25 | |
| *** jcoufal has joined #openstack-ironic | 08:28 | |
| *** e0ne has joined #openstack-ironic | 08:33 | |
| *** derekh has joined #openstack-ironic | 08:40 | |
| *** dtantsur|afk is now known as dtantsur | 08:42 | |
| dtantsur | Morning Ironic | 08:42 |
| vmud213 | morning dtantsur | 08:48 |
| divya_ | hi all | 08:49 |
| divya_ | i am facing issue when i set the node-provisioning-state to manage | 08:49 |
| divya_ | its in enroll state | 08:49 |
| divya_ | and getting error when i change to manage | 08:50 |
| divya_ | how to change to manage state | 08:50 |
| vmud213 | divya_: what is the error you are getting? | 08:55 |
| divya_ | could not set the provisioning state to manage when it is enroll state | 08:56 |
| vmud213 | dtantsur: would you mind looking at https://review.openstack.org/#/c/303503 ? | 08:56 |
| vmud213 | dtantsur:i can submit another patch if you think the timeout should be changed | 08:57 |
| *** amotoki has joined #openstack-ironic | 08:59 | |
| divya_ | The requested action "provide" can not be performed on node "c508d15e-92e9-43a4-89ea-d8388c26b956" while it is in state "enroll". (HTTP 400) | 09:00 |
| divya_ | thisis the error vmud213 | 09:00 |
| divya_ | i have set to manage state first..still it is in enroll state | 09:01 |
| vmud213 | ok can you just move it to some other state may be "deleted" | 09:01 |
| vmud213 | from state diagram i only see the valid state as "manage" from "enroll" | 09:01 |
| vmud213 | may be u need to delete the entry in the database ad retry? | 09:02 |
| divya_ | i have tried deleted as well.. not able to execute that command in enroll state | 09:03 |
| *** dims has quit IRC | 09:04 | |
| divya_ | i cleaned the db and re-created the node completely still the issue | 09:04 |
| *** dims has joined #openstack-ironic | 09:05 | |
| vmud213 | When u enroll a node it will be moved to available state..then How did u move that to enroll state?I am just trying to understand? | 09:09 |
| divya_ | vmud213:when i update my node it goes to enroll node | 09:11 |
| divya_ | stack@Ubuntu14:~/devstack$ ironic node-list +--------------------------------------+---------+---------------+-------------+--------------------+-------------+ | UUID | Name | Instance UUID | Power State | Provisioning State | Maintenance | +--------------------------------------+---------+---------------+-------------+--------------------+-------------+ | 17631307-9d57-46d3-86ed-c70962261 | 09:11 |
| divya_ | its not in available state | 09:11 |
| dtantsur | vmud213, I agree that we should wait, but you're waiting for a wrong thing | 09:11 |
| vmud213 | dtantsur: i agree with what you are saying | 09:12 |
| vmud213 | my point is it in either case, it actually relies on the paths /sys/class/net/... though they are different | 09:13 |
| vmud213 | And i observed that it takes almost the same point.. | 09:13 |
| vmud213 | s/point/time/ | 09:13 |
| vmud213 | I'm ok updating it with new patch.. | 09:14 |
| *** openstackgerrit has quit IRC | 09:17 | |
| *** openstackgerrit has joined #openstack-ironic | 09:18 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add tests for cleaning https://review.openstack.org/304469 | 09:18 |
| divya_ | Failed to delete node 17631307-9d57-46d3-86ed-c70962261c05: Node 17631307-9d57-46d3-86ed-c70962261c05 is associated with instance 92fe9bc2-ebe9-4f87-aa17-416a227b4c25. (HTTP 409) | 09:26 |
| divya_ | how to delete this ironic node | 09:27 |
| *** e0ne has quit IRC | 09:29 | |
| *** sambetts|afk is now known as sambetts|afk] | 09:30 | |
| *** sambetts|afk] is now known as sambetts | 09:30 | |
| *** e0ne has joined #openstack-ironic | 09:31 | |
| divya_ | hi sam | 09:32 |
| sambetts | Morning all | 09:32 |
| sambetts | \o/ Successfully published! | 09:35 |
| *** shu-mutou is now known as shu-mutou-AFK | 09:35 | |
| sambetts | TinyIPA that is | 09:35 |
| *** mgould has quit IRC | 09:37 | |
| divya_ | last_error | Failed to validate power driver interface for node 17631307-9d57-46d3 | | | -86ed-c70962261c05. Error: SSH connection cannot be established: Failed | | | to establish SSH connection to host 172.26.24.121. | 09:37 |
| divya_ | Failed to validate power driver interface for node 17631307-9d57-46d3-86ed-c70962261c05. Error: SSH connection cannot be established: Failed to establish SSH connection to host 172.26.24.121 | 09:37 |
| divya_ | how to validate power driver interface | 09:42 |
| *** mgould has joined #openstack-ironic | 09:42 | |
| sambetts | divya_: it tells you what the problem is, you need to fix it by working out why it can't establish an ssh connection, and fixing it | 09:43 |
| divya_ | this is the error in log | 09:44 |
| divya_ | RROR ironicclient.common.http [^[[01;36mreq-7229a4c4-153c-4cb8-a8c7-4904323419ec ^[[00;36madmin admin^[[01;31m] ^[[01;35m^[[01;31mError contacting Ironic server: Node 17631307-9d57-46d3-86ed-c70962261c05 is associated with instance 92fe9bc2-ebe9-4f87-aa17-416a227b4c25. (HTTP 409). Attempt 2 of 2^[[00m 2016-04-12 12:50:43.371 ^[[01;31mERROR nova.compute.manager [^[[01;36mreq-7229a4c4-153c-4cb8-a8c7-4904323419ec ^[[00;36madmi | 09:44 |
| vdrok | morning ironic! | 09:44 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix tinyipa initrd tarballs.openstack.org file name https://review.openstack.org/304483 | 09:45 |
| sambetts | o/ vdrok | 09:46 |
| sambetts | vdrok: Cisco CI hw upgrade didn't go well yesterday, we're now suffering a network outtage on one of our core servers :'( | 09:48 |
| vdrok | morning sambetts ! oh :( | 09:48 |
| dtantsur | aarefiev, welcome to ironic-inspector-core :) | 09:52 |
| sambetts | \o/ | 09:52 |
| vdrok | aarefiev: congrats :) | 09:55 |
| *** openstackstatus has quit IRC | 09:57 | |
| *** openstack has joined #openstack-ironic | 10:00 | |
| *** jcoufal has quit IRC | 10:03 | |
| *** miles has joined #openstack-ironic | 10:03 | |
| *** miles is now known as mgould_ | 10:04 | |
| *** mgould has quit IRC | 10:04 | |
| lucasagomes | divya_, so instance_uuid is set to that node, you can get rid of it by ironic node-update <node uuid> remove instance_uuid | 10:06 |
| *** Nisha has quit IRC | 10:08 | |
| *** achanda has quit IRC | 10:08 | |
| divya_ | when i perform a node update:ironic node-update $n1 add properties/cpus=1 properties/memory_mb=1024 properties/local_gb=10 properties/cpu_arch=x86_64 | 10:09 |
| divya_ | its setting the node to enroll state | 10:09 |
| divya_ | it should be available right? | 10:09 |
| divya_ | stack@Ubuntu14:~/logs$ ironic node-update $n1 add properties/cpus=1 properties/memory_mb=1024 properties/local_gb=10 properties/cpu_arch=x86_64 +------------------------+------------------------------------------------------------------------+ | Property | Value | +------------------------+--------------------------------------------------------- | 10:10 |
| aarefiev | dtantsur, sambetts: thanks, it's an honor to join the team | 10:14 |
| lucasagomes | divya_, what? When you update the node it changes the provision state!? | 10:14 |
| sambetts | aarefiev: :D | 10:14 |
| aarefiev | vdrok: thanks | 10:14 |
| divya_ | when i set the "ironic node-set-provision-state $n1 manage" | 10:17 |
| divya_ | it is still in enroll state | 10:17 |
| divya_ | this error also i could see | 10:19 |
| divya_ | Failed to validate power driver interface for node | 10:19 |
| sambetts | it won't be able to leave enroll if it will not validate | 10:19 |
| divya_ | how to resolve the power driver interface | 10:21 |
| lucasagomes | divya_, so your error is that the SSH driver can't SSH into the ip address you've specified | 10:23 |
| lucasagomes | you have to fix that and then it can go to manageable state | 10:24 |
| lucasagomes | and from there you can move it to available (so it will be read to receive an instance) | 10:24 |
| divya_ | what ssh ip address should i give in the command ". ironic node-update $n1 add driver_info/ssh_address=172.26.24.120 driver_info/ssh_username=other driver_info/ssh_password=password driver_info/ssh_port=22 driver_info/ssh_virt_type=virsh" | 10:25 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config https://review.openstack.org/293876 | 10:30 |
| *** nathan-h has joined #openstack-ironic | 10:35 | |
| *** achanda has joined #openstack-ironic | 10:38 | |
| sambetts | lucasagomes: this patch is really struggling to get through the gate today, with timeout issues and now a mirror issue :( https://review.openstack.org/#/c/304011/3 | 10:39 |
| sambetts | vsaienko: I just wonder if we should say something while that timeout is still there, and remove the message from the help after the deprecation period, I could imagen people setting it to a value e.g. 30 and then wondering why its only taking 15 | 10:47 |
| *** achanda has quit IRC | 10:47 | |
| sambetts | vsaienko: also I wonder if we should make the config file setting overide the 15 if the config value is higher than 15 | 10:47 |
| vsaienko | sambetts: I don't think so, it was hardcoded to 15 seconds and for tests it is more than enough. | 10:49 |
| sambetts | vsaienko: I think we either need to say that for ssh drivers in newton it will always be 15 or allow a user who might have a slow machine who sets it to higher than 15 to have that config picked up instead of overridden | 10:51 |
| sambetts | lucasagomes: WDYT? ^ | 10:51 |
| *** Nisha has joined #openstack-ironic | 10:53 | |
| sambetts | otherwise it might lead to unexpected results for someone trying to use the config variable, and we'll just end up with someone coming on here asking why when they've set the config it still waiting for 15 seconds, and we'll have to explain it to them, because unless they actually did through the code they won't see the deprecation warning | 10:53 |
| *** daemontool_ has quit IRC | 10:54 | |
| *** daemontool has joined #openstack-ironic | 10:54 | |
| sambetts | I think maybe we could even have a LOG.warn('Setting the port timeout to 15 by default, this will be removed in Otaka please set port_setup_time to 15') | 10:55 |
| sambetts | we could also make the if statement, if isinstance(power, ssh.SSHPower) and port_timeout ==0:, so that if the config value is anything but the default the config value is used | 10:56 |
| *** [1]cdearborn has joined #openstack-ironic | 11:00 | |
| divya_ | No compute node record for host, when updating host | 11:06 |
| divya_ | what is this error in n-cpu.log | 11:06 |
| *** jcoufal has joined #openstack-ironic | 11:07 | |
| *** achanda has joined #openstack-ironic | 11:08 | |
| vsaienko | sambetts: do we need to have a deprecation warning message there? The timeout is set to 15 second on dev environment https://review.openstack.org/#/c/293876/9/devstack/lib/ironic | 11:09 |
| vsaienko | we do not deprecating anything, we introducing new parameter | 11:09 |
| sambetts | vsaienko: we should be deprecating removing the 15 second timeout, if we're going to be removing it in Otaka, then the default will change to 0 if they don't set the config variable | 11:11 |
| sambetts | vsaienko: we need to take into account people that might be (even though its not advised) be using an ssh outside of devstack | 11:12 |
| *** achanda has quit IRC | 11:12 | |
| *** thrash|g0ne is now known as thrash | 11:16 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config https://review.openstack.org/293876 | 11:17 |
| openstackgerrit | Merged openstack/ironic: Fix tinyipa initrd tarballs.openstack.org file name https://review.openstack.org/304483 | 11:18 |
| divya_ | when i boot a instance "| {"message": "No valid host was found. There are not enough hosts available.", "code": 500, "details": " File \"/opt/stack/nova/nova/conductor/manager.py\", line 392, in build_instances | |*" | 11:18 |
| divya_ | can anybody help | 11:18 |
| vsaienko | divya_: check nova-compute logs for more details, "No valid host was found" is too general error | 11:19 |
| lucasagomes | sambetts, reading... sorry I was out having lunch | 11:20 |
| sambetts | also check nova hypervisor-stats and make sure that the amount of avaiable cpus and RAM matches what you expect | 11:20 |
| sambetts | lucasagomes: sure :) | 11:20 |
| lucasagomes | sambetts, yeah the mirror thing I saw that happening in another patch :- | 11:20 |
| lucasagomes | :-/ | 11:20 |
| lucasagomes | sambetts, right, yeah I wouldn't mind having a note in the help msg of that config saying that for ssh it will always be 15 | 11:23 |
| *** mgould_ has quit IRC | 11:25 | |
| divya_ | ironic node-validate $n1 says "SSH connection cannot be established: Failed to establish SSH connection to host 172.26.24.158." | 11:26 |
| divya_ | i am frustated | 11:26 |
| divya_ | couldnt figure out the issue | 11:26 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Default shred iterations to 0. Always zeroize. https://review.openstack.org/304101 | 11:27 |
| lucasagomes | divya_, the host you are trying to ssh into has a ssh server up and running ? If so, is it using the port 22 (default) ? | 11:28 |
| *** jaypipes has joined #openstack-ironic | 11:28 | |
| *** e0ne has quit IRC | 11:30 | |
| divya_ | but this is baremetal na.. how to ssh into it | 11:31 |
| *** thrash has quit IRC | 11:32 | |
| lucasagomes | divya_, if it's baremetal you shouldn't be using the ssh driver | 11:32 |
| mat128 | good morning everyone | 11:32 |
| sambetts | morning mat128 | 11:32 |
| vdrok | morning mat128 | 11:33 |
| lucasagomes | divya_, the ssh driver is just a testing driver to mock power operations to be used with VMs | 11:33 |
| divya_ | for deploy into baremetal which driver to be used? | 11:33 |
| lucasagomes | divya_, because at the time we couldn't use baremetal for the tests in gate | 11:33 |
| vdrok | mat128: as for your question about vnc console, yes, novnc from branch feature/ikvm-support was used to make that work | 11:33 |
| mat128 | Ahhhh | 11:33 |
| mat128 | thought so :) | 11:33 |
| lucasagomes | divya_, depends on ur hardware, the generic one is the ipmitool driver | 11:34 |
| lucasagomes | divya_, if you have a BMC that supports the IPMI protocol | 11:34 |
| mat128 | vdrok: thanks for the details | 11:35 |
| vdrok | np :) | 11:35 |
| divya_ | thanks lucasagomes.. let me try ipmi | 11:35 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic: [WIP] Implement the IPMIProxy server https://review.openstack.org/303894 | 11:36 |
| *** thrash has joined #openstack-ironic | 11:38 | |
| *** thrash has quit IRC | 11:38 | |
| *** thrash has joined #openstack-ironic | 11:38 | |
| lucasagomes | divya_, cool... but again it depends on ur hardware. If it's just a commodite machine you are trying to provisioning you may want to take a look at wol or amt | 11:39 |
| lucasagomes | things like that | 11:39 |
| TheJulia | good morning | 11:41 |
| sambetts | Hey TheJulia | 11:41 |
| lucasagomes | TheJulia, good morning | 11:41 |
| mat128 | good morning TheJulia | 11:42 |
| mat128 | lucasagomes: your thoughts on https://review.openstack.org/#/c/304101/ ? | 11:42 |
| lucasagomes | mat128, hi there, will take a look in a min | 11:42 |
| mat128 | I can go ahead and make a new setting in IPA and Ironic (was about to, thought it would be good to confirm since you're online) | 11:42 |
| lucasagomes | mat128, right, yeah I know it's more work to do but I actually think that would be the right way | 11:43 |
| lucasagomes | make the zeroing optional | 11:43 |
| lucasagomes | that's what is adding a new pass to that shred command | 11:44 |
| mat128 | excellent :) I'm never against doing the right thing | 11:44 |
| mat128 | yup | 11:44 |
| lucasagomes | because IMO, setting it to 0 is not actually shredding the disk | 11:44 |
| lucasagomes | shred, at least in my head, is about writing random/shuffled data | 11:44 |
| lucasagomes | not zeroes | 11:44 |
| mat128 | 0 isn't shredding but still wiping the drive | 11:45 |
| mat128 | I think it's just the choice of words | 11:45 |
| lucasagomes | right | 11:45 |
| mat128 | in shred's manpage, they suggest using --zero to hide the fact the device was shredded | 11:45 |
| lucasagomes | mat128, I will add a comment in the patch. But you have a +1 to the idea from me | 11:45 |
| mat128 | good, and for the default | 11:45 |
| mat128 | leave at 1 even if both settings have been splitted? | 11:45 |
| lucasagomes | IMO yes | 11:46 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix API node name updates https://review.openstack.org/300983 | 11:46 |
| mat128 | ok good, that's what configuration options are for anyway :) | 11:46 |
| *** Goneri has quit IRC | 11:49 | |
| lucasagomes | commented, thanks for that work :-) | 11:49 |
| mat128 | np :) | 11:50 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Store ramdisk logs on preprocessing failures as well https://review.openstack.org/300011 | 11:56 |
| *** kalpase has left #openstack-ironic | 12:04 | |
| *** dprince has joined #openstack-ironic | 12:04 | |
| dtantsur | devananda, jroll, hey. I get more and more people affected by https://review.openstack.org/#/c/300548/ in mitaka. I wonder if there's some way to partially backport the fix. WDYT? | 12:04 |
| *** openstack has quit IRC | 12:04 | |
| *** openstack has joined #openstack-ironic | 12:07 | |
| dtantsur | yeah, pretty possible. I want to hear from our other stable cores before I proceed :) | 12:07 |
| *** mgould has joined #openstack-ironic | 12:07 | |
| *** moshele has quit IRC | 12:07 | |
| *** alexpilotti has joined #openstack-ironic | 12:12 | |
| *** hoangcx has joined #openstack-ironic | 12:15 | |
| *** alexpilotti has quit IRC | 12:16 | |
| *** alexpilotti has joined #openstack-ironic | 12:16 | |
| aarefiev | dtantsur: hey, why only on preprocessing failures? https://review.openstack.org/#/c/300011 | 12:18 |
| *** moshele has joined #openstack-ironic | 12:20 | |
| dtantsur | aarefiev, dunno :) I was waiting for someone to ask this | 12:20 |
| aarefiev | lol | 12:21 |
| dtantsur | I don't have an answer and I'm open to storing logs on any failure | 12:21 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add config option for ATA erase fallback in agent https://review.openstack.org/302819 | 12:22 |
| aarefiev | dtantsur: imo it make sense to do this on any failure | 12:22 |
| sambetts | dtantsur, aarefiev: I don't see any reason not to just make it store if any failure, as long as we're not opening ourselves up to DDOS by someone pumping invalid data at us until the machine crashes | 12:24 |
| *** baoli has joined #openstack-ironic | 12:26 | |
| dtantsur | DDOS by logs, lol :) we can setup some log rotation | 12:27 |
| dtantsur | ok got it, will change | 12:27 |
| sambetts | yeah, I can think of a horrible attack vector if someone pumped a huge log file at us and we store it everytime, eventually we run out of disk space and it breaks things | 12:28 |
| sambetts | log rotation works as long as we don't save the log if the error comes from node not found, because otherwise they could just send a unique mac / ip everytime | 12:29 |
| dtantsur | well, it all boils down to protecting /v1/continue API from unauthorized usage | 12:30 |
| dtantsur | otherwise there are many ways of exploiting it | 12:30 |
| aarefiev | yeah, I don't think it's hard to ddos now, even w/o big logs | 12:32 |
| sambetts | yeah its worried me for a little while now, but I've not been sure how to fix it, or if we need to :/ | 12:33 |
| jroll | dtantsur: so actually, the stable branch rules seem to only indicate backwards-incompatible config changes are not allowed | 12:33 |
| jroll | also, good morning dtantsur, lucasagomes, sambetts, mat128, congrats aarefiev :D | 12:33 |
| dtantsur | meaning a new option is fine? | 12:33 |
| dtantsur | morning jroll | 12:33 |
| vdrok | morning jroll :) | 12:33 |
| lucasagomes | jroll, hi there! | 12:34 |
| jroll | heya vdrok | 12:34 |
| dtantsur | sambetts, tenant network isolation is the fix :) | 12:34 |
| aarefiev | jroll: morning, thanks :) | 12:34 |
| *** alexpilo_ has joined #openstack-ironic | 12:34 | |
| sambetts | dtantsur: haha yes | 12:34 |
| jroll | dtantsur: sounds like it? I'm not an expert but ya know | 12:34 |
| jroll | dtantsur: ctrl-f http://docs.openstack.org/project-team-guide/stable-branches.html | 12:34 |
| mat128 | morning jroll | 12:35 |
| sambetts | morning jroll, hows driving further than the size of my country going? | 12:35 |
| jroll | sambetts: it's done \o/ | 12:35 |
| sambetts | \o/! Awesome, getting everything set up at your new place now? | 12:36 |
| *** alexpilotti has quit IRC | 12:36 | |
| *** e0ne has joined #openstack-ironic | 12:36 | |
| jroll | nah, stuff is still in shipping and we're staying with some relatives while we shop around a bit | 12:37 |
| sambetts | Nice :D | 12:37 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added functions to translate between uuid and uri https://review.openstack.org/301877 | 12:39 |
| openstackgerrit | Merged openstack/ironic-python-agent: Make sure that .eggs directory is in gitignore https://review.openstack.org/304011 | 12:39 |
| vdrok | jroll: question about https://review.openstack.org/285852. looking at the last devananda comment - check for cleaning net uuid only if cleaning is enabled and dhcp provider is neutron - maybe it should be changed to only checking dhcp provider? As there is manual clean now, and there is no config option for enabling/disabling it | 12:40 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Allow shred zeroize option to be configured https://review.openstack.org/304578 | 12:40 |
| jroll | vdrok: yeah, sounds like it | 12:41 |
| vdrok | ok, will add that | 12:42 |
| *** keedya has joined #openstack-ironic | 12:43 | |
| *** baoli_ has joined #openstack-ironic | 12:43 | |
| *** alexpilo_ has quit IRC | 12:44 | |
| *** alexpilotti has joined #openstack-ironic | 12:46 | |
| *** baoli has quit IRC | 12:46 | |
| sambetts | jroll, vdrok: hmmm I think the problem here is that the FlatNetworkProvider is a neutron network provider, but it can be abused by changing the dhcp provider to work for non-neutron usecases, I wonder if we should add a true noop network provider for devananda's usecase | 12:48 |
| TheJulia | mat128: configure option instead of changing the default? | 12:48 |
| mat128 | TheJulia: that's it, based on my discussions with Lucas, it's preferable to allow both setting to be configured | 12:49 |
| mat128 | and give a better name to the original setting to remove the ambiguity | 12:49 |
| jroll | sambetts: hmm, maybe | 12:49 |
| TheJulia | awesome | 12:49 |
| mat128 | I'm going to leave the "internal" name as-is, because this is the contract between Ironic and IPA | 12:49 |
| mat128 | but the exposed configuration option will be renamed | 12:50 |
| openstackgerrit | Aline Bousquet proposed openstack/ironic-lib: Add support for metrics https://review.openstack.org/301526 | 12:50 |
| TheJulia | awesome | 12:51 |
| *** fragatina has joined #openstack-ironic | 12:51 | |
| *** links has quit IRC | 12:52 | |
| *** Nisha has quit IRC | 12:53 | |
| vdrok | sambetts jroll but then, flat is still going to be the default, and it will be a breaking change for people not using neutron | 12:53 |
| vdrok | though maybe a release note about it would be ok | 12:54 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added get_server_profile_by_uuid and changed model https://review.openstack.org/302225 | 12:54 |
| jroll | vdrok: also true | 12:54 |
| * jroll still trying to catch up on everything and so hasn't thought about it much | 12:54 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config https://review.openstack.org/293876 | 12:56 |
| sambetts | vdrok, jroll: I would love to see the network driver and dhcp providers merge together, because they are defiantly not mutually exclusive, and we can share dhcp code between the flatnetworkdriver and neutron multitenant driver using a mixin or something \ | 12:57 |
| jlvillal | 'defiantly!' ;) | 12:58 |
| jroll | sambetts: yeah, just hoping that we don't re-architect this again before landing it :) | 12:58 |
| sambetts | I know :( | 12:58 |
| vdrok | morning jlvillal | 12:58 |
| jlvillal | Good morning | 12:58 |
| sambetts | jlvillal: I've never managed to spell that right | 12:58 |
| * jlvillal shouldn't wake up this early | 12:58 | |
| jlvillal | sambetts, :) | 12:59 |
| xavierr | goood morning :) | 13:01 |
| *** Goneri has joined #openstack-ironic | 13:03 | |
| *** vishwanathj has quit IRC | 13:03 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Allow shred's zeroize pass to be configurable https://review.openstack.org/304101 | 13:03 |
| lucasagomes | jroll, quick question... the network isolation bits in neutron, right now, is only implemented to arista gear right? | 13:04 |
| lucasagomes | or am I crazy and it's generic for all neutron drivers ? | 13:04 |
| sambetts | lucasagomes: we've got an implmentaion too | 13:04 |
| lucasagomes | sambetts, right, so cisco and arista? | 13:04 |
| sambetts | there is a generic switch driver for bm that someones written too | 13:05 |
| jroll | lucasagomes: it works with ML2, so whatever vendors provide | 13:05 |
| sambetts | http://git.openstack.org/cgit/openstack/networking-generic-switch/ | 13:05 |
| sambetts | I think thats it ^ | 13:06 |
| jroll | lucasagomes: also welcome back :) | 13:06 |
| sambetts | its weird though because its like they are reimplementing ml2 under ml2 | 13:06 |
| lucasagomes | jroll, sambetts thanks a lot | 13:06 |
| lucasagomes | sambetts, right on | 13:06 |
| *** itamarl has quit IRC | 13:06 | |
| *** jjohnson2 has joined #openstack-ironic | 13:07 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Allow shred's zeroize pass to be configurable https://review.openstack.org/304101 | 13:08 |
| *** dtantsur is now known as dtantsur|mtg | 13:11 | |
| TheJulia | mat128: Rats, you fixed the config before I could comment on it | 13:12 |
| mat128 | :) | 13:12 |
| *** ayoung has quit IRC | 13:12 | |
| TheJulia | downside, our revs for cleaning config conflict, not horribly so :) | 13:14 |
| *** rloo has joined #openstack-ironic | 13:16 | |
| TheJulia | good morning rloo | 13:16 |
| rloo | Hey, morning TheJulia! | 13:16 |
| openstackgerrit | Aline Bousquet proposed openstack/ironic-lib: Add support for metrics https://review.openstack.org/301526 | 13:17 |
| alineb | jlvillal: thanks for the review :) I made the changes if you want to recheck ^ | 13:17 |
| *** mtanino has joined #openstack-ironic | 13:18 | |
| jlvillal | alineb, The unit tests didn't change.... | 13:19 |
| jlvillal | alineb, nvm. I clicked 8 compared to 8 ... | 13:20 |
| *** aparnav has quit IRC | 13:21 | |
| jlvillal | alineb, Looking good. Left a comment about a missing unit test.. | 13:21 |
| jlvillal | alineb, Thanks | 13:22 |
| alineb | jlvillal: sure, i will add that, thanks for noticing it's missing :) | 13:23 |
| *** mtanino has quit IRC | 13:30 | |
| *** raildo is now known as raildo-afk | 13:32 | |
| *** rbudden has joined #openstack-ironic | 13:34 | |
| *** links has joined #openstack-ironic | 13:36 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic-python-agent: Wait for all interfaces to be up before node lookup https://review.openstack.org/303503 | 13:39 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 13:42 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network drivers https://review.openstack.org/285852 | 13:42 |
| vdrok | jroll: sambetts for now added none interface | 13:43 |
| *** ametts has joined #openstack-ironic | 13:43 | |
| sambetts | vdrok: I'd prefer Noop but we'll see if what devananda thinks when he gets on | 13:44 |
| vdrok | sambetts: that was for consistency with dhcp :) but yeah, easy to fix | 13:44 |
| *** mag009__1 has joined #openstack-ironic | 13:45 | |
| *** mag009__1 has left #openstack-ironic | 13:45 | |
| *** mgoddard_ has joined #openstack-ironic | 13:45 | |
| *** vmud213 has quit IRC | 13:47 | |
| *** thiagop has joined #openstack-ironic | 13:47 | |
| thiagop | Good morning, ironicers | 13:48 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 13:49 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 13:49 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 13:49 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 13:49 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 13:49 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 13:49 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 13:49 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 13:49 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 13:49 |
| *** mgoddard has quit IRC | 13:49 | |
| vdrok | morning thiagop | 13:49 |
| openstackgerrit | Aline Bousquet proposed openstack/ironic-lib: Add support for metrics https://review.openstack.org/301526 | 13:49 |
| *** hoangcx has quit IRC | 13:53 | |
| thiagop | hiya vdrok | 13:57 |
| mat128 | good morning thiagop | 13:57 |
| jroll | neat: https://github.com/openstack/microversion-parse | 13:57 |
| jroll | dtantsur|mtg: ^ you'll enjoy that ;) | 13:57 |
| thiagop | hiya too mat128 | 13:57 |
| dtantsur|mtg | jroll, yeah, it's on my way to add to projects :) | 13:59 |
| jroll | \o/ | 13:59 |
| * dtantsur|mtg does not have any time this week though | 13:59 | |
| dtantsur|mtg | so if you want to do that - go for it | 14:00 |
| jroll | nor do I, heh | 14:01 |
| jroll | there's plenty of time, it isn't urgent | 14:01 |
| dtantsur|mtg | yeah.. it's on my todo list | 14:01 |
| *** cdearborn has joined #openstack-ironic | 14:02 | |
| vdrok | >:( stupid pep8 mistake | 14:03 |
| *** alexpilotti has quit IRC | 14:03 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add network drivers https://review.openstack.org/285852 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 14:04 |
| *** alexpilotti has joined #openstack-ironic | 14:07 | |
| *** links has quit IRC | 14:08 | |
| *** mtanino has joined #openstack-ironic | 14:10 | |
| *** alexpilotti has quit IRC | 14:15 | |
| *** alexpilotti has joined #openstack-ironic | 14:16 | |
| *** [1]cdearborn has quit IRC | 14:17 | |
| *** alexpilotti has quit IRC | 14:21 | |
| *** alexpilotti has joined #openstack-ironic | 14:23 | |
| *** spandhe has joined #openstack-ironic | 14:35 | |
| *** spandhe has quit IRC | 14:37 | |
| *** mgoddard_ has quit IRC | 14:43 | |
| *** [1]cdearborn has joined #openstack-ironic | 14:43 | |
| *** mgoddard has joined #openstack-ironic | 14:44 | |
| *** [4]cdearborn has joined #openstack-ironic | 14:45 | |
| *** ppiela has joined #openstack-ironic | 14:46 | |
| *** [1]cdearborn has quit IRC | 14:48 | |
| *** raildo-afk is now known as raildo | 14:48 | |
| *** sinval is now known as sinval-afk | 14:51 | |
| *** MattMan has quit IRC | 14:52 | |
| *** MattMan has joined #openstack-ironic | 14:52 | |
| *** afaranha has quit IRC | 14:57 | |
| *** cdearborn has quit IRC | 14:58 | |
| *** ametts has quit IRC | 14:59 | |
| *** hoangcx has joined #openstack-ironic | 15:06 | |
| NobodyCam | good Morning Ironicers :) | 15:08 |
| NobodyCam | got suckucked to Jira this morning lol :p | 15:08 |
| dtantsur|mtg | morning NobodyCam | 15:09 |
| dtantsur|mtg | I feel sorry for you :) | 15:09 |
| NobodyCam | :) morning dtantsur|mtg :p | 15:09 |
| thiagop | morning NobodyCam | 15:09 |
| NobodyCam | morning thiagop :) | 15:10 |
| lucasagomes | NobodyCam, morning | 15:11 |
| NobodyCam | hey hey morning lucasagomes | 15:11 |
| NobodyCam | how was travel | 15:11 |
| lucasagomes | NobodyCam, good, really good actually | 15:11 |
| jroll | morning NobodyCam :) | 15:11 |
| NobodyCam | morning jroll :) | 15:12 |
| lucasagomes | NobodyCam, actually, I'm travelling at the moment | 15:12 |
| NobodyCam | hows your travel going? | 15:12 |
| NobodyCam | lucasagomes: oh still | 15:12 |
| lucasagomes | I'm in the Brno office | 15:12 |
| lucasagomes | NobodyCam, yeah, but work stuff now | 15:12 |
| NobodyCam | :) nice | 15:12 |
| jroll | NobodyCam: completed last night, now just a mess trying to find a place to live and get our stuff there :P | 15:12 |
| NobodyCam | jroll: lol.. :p | 15:13 |
| lucasagomes | jroll, oh that's cool | 15:13 |
| sambetts | o/ NobodyCam | 15:13 |
| *** hemna_ has joined #openstack-ironic | 15:13 | |
| NobodyCam | morning sambetts :) | 15:14 |
| thiagop | you guys move without finding previously a place to live? o.O | 15:14 |
| NobodyCam | lucasagomes: if you happen to have a free minute: https://review.openstack.org/#/c/275726 | 15:15 |
| NobodyCam | you had a +2 on it before | 15:15 |
| jroll | thiagop: kind of hard to find a place from across the country :/ staying with some relatives for now | 15:15 |
| jroll | so not a big deal | 15:15 |
| *** Sukhdev has joined #openstack-ironic | 15:15 | |
| thiagop | jroll: aw, in that case... | 15:15 |
| thiagop | : | 15:16 |
| thiagop | ) | 15:16 |
| NobodyCam | WIFI takeOver ! | 15:16 |
| NobodyCam | lol | 15:16 |
| jroll | lol | 15:16 |
| JayF | jroll: I'm flying down to Tacoma in mid-may to find a place, you're braver than I am, lol | 15:17 |
| thiagop | lol | 15:17 |
| jroll | JayF: well, I assume you don't have family there :P | 15:17 |
| *** openstackgerrit has quit IRC | 15:18 | |
| JayF | jroll: As my mother frequently reminds me :P | 15:18 |
| jroll | there's plenty of room for us here | 15:18 |
| jroll | heh | 15:18 |
| TheJulia | jroll: you are far braver than I.... of course I think I might be crazy with what I'm pondering :) Oh wait, that goes with my persona. | 15:18 |
| JayF | Charlotte was #2 on our list, but we would've changed up where we moved given what happened there recently | 15:18 |
| TheJulia | :( | 15:18 |
| *** openstackgerrit has joined #openstack-ironic | 15:18 | |
| *** jayc has joined #openstack-ironic | 15:19 | |
| *** ayoung has joined #openstack-ironic | 15:20 | |
| *** baoli_ has quit IRC | 15:22 | |
| *** Sukhdev has quit IRC | 15:24 | |
| *** achanda has joined #openstack-ironic | 15:25 | |
| lucasagomes | NobodyCam, will take a look | 15:26 |
| *** baoli_ has joined #openstack-ironic | 15:26 | |
| NobodyCam | :) awesome :) | 15:26 |
| jlvillal | JayF: If you are bored: https://review.openstack.org/#/c/303620/ :) | 15:32 |
| * jlvillal doubts JayF is actually bored. | 15:33 | |
| JayF | I know better than to ever claim boredom | 15:33 |
| JayF | even if I am | 15:33 |
| JayF | jlvillal: I love test cleanups like that | 15:33 |
| jlvillal | :) | 15:34 |
| *** logan- has quit IRC | 15:34 | |
| JayF | jroll: ^ if you have a sec, ytou can land that for john | 15:34 |
| * jlvillal is being a bad man this morning and having hash browns, biscuit, and gravy over everything for breakfast. | 15:35 | |
| JayF | Mmm. I could have some breakfast. Waffles and sausage and hash browns | 15:36 |
| * JayF has lived off frozen breakfast since moving to wfh | 15:36 | |
| *** yarkot_ has joined #openstack-ironic | 15:38 | |
| *** achanda has quit IRC | 15:38 | |
| jlvillal | JayF: Normally mine is much more boring. Some hard boiled eggs. | 15:38 |
| JayF | Now you have me thinking of biscuits and gravy, and lamenting my lack of them, lol | 15:39 |
| *** logan- has joined #openstack-ironic | 15:40 | |
| jroll | done, thanks jlvillal | 15:40 |
| jlvillal | jroll: JayF Thanks :) | 15:41 |
| jroll | I had toast with avocado and an egg this morning \o/ | 15:41 |
| jlvillal | Mmmm Avocado :) | 15:41 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Store ramdisk logs on all processing failures, not only reported by the ramdisk https://review.openstack.org/300011 | 15:42 |
| dtantsur|mtg | sambetts, aarefiev, updated ^^^ to store logs on all processing failures | 15:42 |
| sambetts | :D awesome | 15:43 |
| *** moshele has quit IRC | 15:44 | |
| *** Nisha has joined #openstack-ironic | 15:45 | |
| *** moshele has joined #openstack-ironic | 15:46 | |
| *** jjohnson2 has quit IRC | 15:46 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Create helper function: create_hdparm_info() https://review.openstack.org/303620 | 15:47 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: OneView client with managers https://review.openstack.org/286803 | 15:47 |
| *** vishwanathj has joined #openstack-ironic | 15:48 | |
| *** saripurigopi has joined #openstack-ironic | 15:50 | |
| *** vishwana_ has joined #openstack-ironic | 15:50 | |
| *** vishwanathj has quit IRC | 15:53 | |
| *** jcoufal has quit IRC | 15:54 | |
| *** dtantsur|mtg is now known as dtantsur|afk | 15:54 | |
| *** athomas has quit IRC | 15:55 | |
| *** mgould has quit IRC | 15:55 | |
| devananda | morning, all | 15:56 |
| jroll | ohai devananda | 15:56 |
| sambetts | Morning devananda | 15:56 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for baremetal subcommands in openstackclient https://review.openstack.org/301687 | 15:58 |
| *** praneshp has joined #openstack-ironic | 15:58 | |
| *** ifarkas has quit IRC | 15:58 | |
| lucasagomes | devananda, morning | 15:59 |
| *** alexpilotti has quit IRC | 15:59 | |
| lucasagomes | folks, I'm calling it a day | 15:59 |
| lucasagomes | see y'all tomorrow | 15:59 |
| sambetts | cya later lucasagomes | 15:59 |
| jroll | later man | 15:59 |
| *** lucasagomes is now known as lucas-brno | 15:59 | |
| vdrok | morning devananda | 16:01 |
| *** mkovacik has quit IRC | 16:02 | |
| jlvillal | lucas-brno: Ciao | 16:05 |
| jlvillal | morning devananda | 16:05 |
| *** saripurigopi has quit IRC | 16:06 | |
| *** alexpilotti has joined #openstack-ironic | 16:08 | |
| *** kromanenko has quit IRC | 16:10 | |
| NobodyCam | good morning devananda | 16:10 |
| NobodyCam | night lucas-brno | 16:11 |
| *** cdearborn has joined #openstack-ironic | 16:14 | |
| *** derekh has quit IRC | 16:20 | |
| *** alex_xu_ has quit IRC | 16:24 | |
| *** achanda has joined #openstack-ironic | 16:25 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: [EXPERIMENTAL] Hardware alerts - DB https://review.openstack.org/304756 | 16:25 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: [EXPERIMENTAL] Hardware alerts - obj https://review.openstack.org/304757 | 16:25 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: [EXPERIMENTAL] Hardware alerts - API https://review.openstack.org/304758 | 16:25 |
| *** [4]cdearborn has quit IRC | 16:28 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Remove deprecated group and option names in deploy_utils.py https://review.openstack.org/303081 | 16:28 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Hardware alerts support https://review.openstack.org/304762 | 16:30 |
| *** achanda has quit IRC | 16:30 | |
| *** alex_xu has joined #openstack-ironic | 16:30 | |
| *** saripurigopi has joined #openstack-ironic | 16:36 | |
| jlvillal | vdrok: ping | 16:36 |
| vdrok | jlvillal: pong | 16:36 |
| jlvillal | vdrok: Question for you. Looking here: http://docs.python-requests.org/en/master/api/ | 16:36 |
| jlvillal | At the 'request' method and the arguments | 16:37 |
| * thiagop sees the responses of Openstack Survey | 16:37 | |
| * thiagop wonders why people dislike using gerrit so much... | 16:37 | |
| jlvillal | vdrok: and then this: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L331-L333 | 16:38 |
| jlvillal | vdrok: never mind! | 16:38 |
| vdrok | heh, np :) | 16:38 |
| jlvillal | vdrok: I mean: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L518-L519 | 16:39 |
| jlvillal | That is confusing | 16:39 |
| jlvillal | Am I missing something. On the order of the arguments? | 16:39 |
| jlvillal | It looks out of order, the first two arguments | 16:39 |
| vdrok | jlvillal: it is an argument to keystoneauth session, not requests iirc | 16:40 |
| jlvillal | Oh, that would make more sense. | 16:40 |
| vdrok | https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L280 | 16:40 |
| jlvillal | vdrok: Thanks! | 16:41 |
| JayF | thiagop: What survey? I could vote for "gerrit is awesome", lol | 16:41 |
| vdrok | jlvillal: anyway, that should be refactored, there is some copypaste from keystoneclient I believe | 16:41 |
| thiagop | JayF: #link http://www.openstack.org/assets/survey/April-2016-User-Survey-Report.pdf | 16:41 |
| vdrok | I planned to remove some pieces there | 16:41 |
| jlvillal | vdrok: I was reviewing: https://review.openstack.org/#/c/292552/ | 16:41 |
| jlvillal | And thus got confused | 16:41 |
| vdrok | jlvillal: yep, it is confusing indeed | 16:42 |
| thiagop | JayF: after learning to use Gerrit and see benefits of using it against pull request, I'm almost deploying a gerrit here at the lab... | 16:42 |
| thiagop | gerrit rocks | 16:42 |
| vdrok | jlvillal: e.g. take a look here https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/adapter.py#L70-L98 and compare with our _http_request method :) | 16:43 |
| *** vishwanathj has joined #openstack-ironic | 16:44 | |
| vdrok | jlvillal: I did not want to include this refactor in the patch, as it will have to be backported | 16:44 |
| jroll | thiagop: imo the gerrit workflow is great, the gerrit UI is terrible :P | 16:44 |
| jlvillal | vdrok: Okay. Your patch looks good. I was thinking about doing a +2 +A unless anything has changed over the last month... | 16:44 |
| thiagop | jroll: I kinda liked the new UI. It has a lot to be worked on, but have some interesting features | 16:45 |
| *** jjohnson2 has joined #openstack-ironic | 16:45 | |
| * sambetts is heading oof | 16:45 | |
| sambetts | off | 16:45 |
| sambetts | cya later | 16:45 |
| vdrok | jlvillal: I think it did not :) endpoint_override is still there in keystoneauth sessions | 16:45 |
| *** sambetts is now known as sambetts|afk | 16:45 | |
| vdrok | night sambetts|afk | 16:46 |
| jlvillal | sambetts: Take oof! :) | 16:46 |
| * jlvillal would maybe add 'you hoser', but sambetts|afk may not be familiar with Bob and Doug McKenzie :) | 16:46 | |
| thiagop | what I dislike more is having to check always if I'm on the last patchset. But removing <code>url.split('/')[-1]</code> works for me | 16:46 |
| *** vishwana_ has quit IRC | 16:47 | |
| thiagop | night sambetts|afk | 16:47 |
| jlvillal | thiagop: You might look into gertty. | 16:47 |
| *** dprince has quit IRC | 16:47 | |
| vdrok | jlvillal: thanks! | 16:47 |
| * jlvillal does not use it but knows dtantsur|afk uses gertty | 16:47 | |
| jlvillal | thiagop: https://github.com/openstack/gertty | 16:48 |
| vdrok | yep, I keep seeing people recommending gertty | 16:48 |
| thiagop | jlvillal: dtantsur|afk already pointed me out to this. Seems interesting, but I'm not that passionate by shell tools | 16:49 |
| thiagop | jlvillal: but that "offline" feature... seems very useful | 16:49 |
| vdrok | thiagop: maybe there is a web interface for it :D | 16:49 |
| *** alexpilotti has quit IRC | 16:51 | |
| jlvillal | vdrok: I also think the name "SessionClient" is terrible for that class :) | 16:51 |
| *** Sukhdev has joined #openstack-ironic | 16:52 | |
| jlvillal | KeystoneSessionClient would have been better. | 16:52 |
| jlvillal | I see Session and I'm thinking a Python requests session. | 16:52 |
| thiagop | vdrok: I intend to try it soon | 16:52 |
| vdrok | jlvillal: I'm thinking about removing httpclient completely and leaving only sessionclient | 16:52 |
| *** moshele has quit IRC | 16:52 | |
| vdrok | jlvillal: admin_token auth method is basically the same as we have in http client - https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/loading/_plugins/admin_token.py | 16:53 |
| jlvillal | vdrok: We would probably need to deprecate it. I think things like shade might use it. But not sure on that. | 16:53 |
| vdrok | ie just use the provided endpoint and token | 16:53 |
| jlvillal | I remember doing a change to keystone client and then TheJulia told me I broke the world. Or at least a small town on the world :) | 16:54 |
| vdrok | jlvillal: well, it needs investigation in terms of exceptions and stuff but functionality is the same | 16:54 |
| * jlvillal is unsure if it was his change or he approved the change. | 16:54 | |
| vdrok | yep, I usually create a do not merge change in bifrost depending on ironicclient to check | 16:55 |
| jlvillal | vdrok: I wonder if those 'clients' should be in http.py? Maybe clients.py? | 16:55 |
| *** Guest15115 has quit IRC | 16:55 | |
| vdrok | jlvillal: that is a historical name :) before it was openstack/http.py | 16:56 |
| jlvillal | If you are thinking about refactoring stuff. | 16:56 |
| vdrok | but can be changed, sure | 16:56 |
| jlvillal | I'm just saying random stuff. Not sure if good or bad idea. | 16:56 |
| *** spandhe has joined #openstack-ironic | 16:57 | |
| *** trown is now known as trown|lunch | 16:57 | |
| *** saripurigopi has quit IRC | 16:57 | |
| *** vishwanathj has quit IRC | 16:58 | |
| *** vishwanathj has joined #openstack-ironic | 16:58 | |
| *** vishwana_ has joined #openstack-ironic | 16:59 | |
| *** alexpilotti has joined #openstack-ironic | 16:59 | |
| *** vishwana_ is now known as vishwanathj__ | 17:00 | |
| *** nathan-h has quit IRC | 17:01 | |
| *** vishwanathj has quit IRC | 17:03 | |
| *** sambetts|afk has quit IRC | 17:04 | |
| *** sambetts has joined #openstack-ironic | 17:06 | |
| *** raildo is now known as raildo-afk | 17:08 | |
| *** mgoddard has quit IRC | 17:11 | |
| *** mgoddard has joined #openstack-ironic | 17:12 | |
| *** hoangcx has quit IRC | 17:12 | |
| NobodyCam | general question: should adding new states be concered a breaking api change? | 17:14 |
| devananda | NobodyCam: most definitely | 17:14 |
| jroll | yep | 17:14 |
| NobodyCam | thats what I thought | 17:14 |
| mat128 | NobodyCam: see the Adopt case | 17:14 |
| TheJulia | That is what I remember the discussion being last time it came up :) | 17:14 |
| devananda | also I'm now very curious what new API states you're referring to | 17:14 |
| NobodyCam | just saw the comment here: https://review.openstack.org/#/c/275766/12/ironic/api/controllers/v1/node.py | 17:15 |
| *** jistr has quit IRC | 17:15 | |
| jroll | heh | 17:15 |
| jroll | hrm | 17:15 |
| JayF | I made up an etherpad for us to be able to sync up who is arriving when and staying where at the summit, so we can keep track of folks | 17:17 |
| JayF | and so nobody is left out of the group hangout chat that always forms | 17:17 |
| JayF | https://etherpad.openstack.org/p/ironic-summit-arrivals | 17:17 |
| JayF | feel free to fill it out :) I'll make a group hangout on sunday before summit with everyone on the etherpad | 17:17 |
| *** saripurigopi has joined #openstack-ironic | 17:18 | |
| *** suro-patz has joined #openstack-ironic | 17:18 | |
| * JayF hands TheJulia a monospaced font | 17:18 | |
| TheJulia | WOOT! | 17:19 |
| TheJulia | lol | 17:19 |
| NobodyCam | lol | 17:19 |
| *** piet has joined #openstack-ironic | 17:19 | |
| * TheJulia wonders why not use sheets.google.com *ducks* | 17:19 | |
| *** Sukhdev has quit IRC | 17:19 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 17:19 |
| JayF | TheJulia: I mean, I totally could instead, but some people get chafed about using nonfree tools | 17:19 |
| TheJulia | True :\ | 17:20 |
| *** dencaval has joined #openstack-ironic | 17:23 | |
| * NobodyCam 's old spamming job warns him agenst adding email addy to public doc | 17:25 | |
| *** Sukhdev has joined #openstack-ironic | 17:27 | |
| JayF | NobodyCam: lol | 17:27 |
| JayF | NobodyCam: I worked for a "legit spammer" (i.e. we were just the engine, it's not OUR fault if you bought a list when we told you not to) | 17:28 |
| *** davideag_ has quit IRC | 17:28 | |
| *** davidlenwell has quit IRC | 17:29 | |
| *** raildo-afk is now known as raildo | 17:29 | |
| * NobodyCam may have worked for a affiliate network.. | 17:30 | |
| JayF | Which one/ | 17:32 |
| JayF | I quit my job at iContact because they had a spammy customer, we proved they were spammy and got them "fired" as a customer despite being our biggest revenue generator ... | 17:32 |
| *** davideagnello has joined #openstack-ironic | 17:32 | |
| JayF | ... then they offered to pay us 2x rack rate for the email sending, the CEO and VP of BizDev overrode the technical and support folks | 17:32 |
| JayF | and 2 months later I worked at Rackspace, lol | 17:33 |
| thiagop | lol | 17:34 |
| *** achanda has joined #openstack-ironic | 17:34 | |
| NobodyCam | hydra media was one of hte networks | 17:34 |
| *** davidlenwell has joined #openstack-ironic | 17:36 | |
| Nisha | Is any one aware of the issue "Slave went offline during the build" for the CI | 17:38 |
| openstackgerrit | Aline Bousquet proposed openstack/ironic-lib: Add support for metrics https://review.openstack.org/301526 | 17:39 |
| openstackgerrit | Merged openstack/python-ironicclient: Set endpoint_override while doing session.request https://review.openstack.org/292552 | 17:40 |
| *** rama_y has joined #openstack-ironic | 17:41 | |
| jlvillal | alineb: Thanks :) | 17:42 |
| *** moshele has joined #openstack-ironic | 17:43 | |
| *** ametts has joined #openstack-ironic | 17:45 | |
| *** xavierr has left #openstack-ironic | 17:47 | |
| *** moshele has quit IRC | 17:48 | |
| *** suro-patz1 has joined #openstack-ironic | 17:50 | |
| *** xavierr has joined #openstack-ironic | 17:50 | |
| *** suro-patz has quit IRC | 17:51 | |
| anteaya | Nisha: when asking those types of questions, having a link to the logs accompany the question can be helpful | 17:53 |
| Nisha | anteaya, the ilo CI actually dont capture console logs because of ip address and login credentials information (security concern) as of now. But i am seeing this issue many times and i have to restart jenkins each time i face this issue | 17:54 |
| Nisha | anteaya, sometimes even that doesnt helps | 17:55 |
| Nisha | anteaya, just now i had to restart the jenkins master.... | 17:55 |
| jlvillal | jroll: Any thoughts on starting to plan for the Newton midcycle? | 17:55 |
| TheJulia | So if your having to restart your jenkins master, wouldn't it be in the jenkins master node's logs potentially? | 17:56 |
| * jlvillal was noticing that Nova is discussing theirs | 17:56 | |
| *** xhku has joined #openstack-ironic | 17:56 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Uplink Set https://review.openstack.org/295981 | 17:56 |
| Nisha | TheJulia, what kind of logs from jenkinsmaster? | 17:57 |
| *** trown|lunch is now known as trown | 17:57 | |
| *** yarkot_ has quit IRC | 17:57 | |
| Nisha | the slave will suddenly go offline while running the job | 17:57 |
| TheJulia | Nisha: I'm just speculating that maybe there is data in jenkins.log that might reveal what your encountering | 17:58 |
| anteaya | Nisha: so you are saying that the ilo CI system is your system? | 17:58 |
| anteaya | TheJulia: it is a good speculation | 17:58 |
| Nisha | anteaya, I work for the iLO drivers and currently enabling the iLO drivers in CI | 17:58 |
| Nisha | TheJulia, ok got ur point. let me see | 17:59 |
| anteaya | Nisha: okay so you are asking about your system | 17:59 |
| anteaya | great, now I understand more | 17:59 |
| Nisha | anteaya, yeah | 17:59 |
| anteaya | great that helps me understand better | 18:00 |
| Nisha | anteaya, if this issue is not there then one of the ilo driver gate works fine with no issues | 18:00 |
| thiagop | anteaya: I think what Nisha is saying is that her CI is having a pretty similar behavior as in the bug | 18:00 |
| Nisha | thiagop, which bug? | 18:00 |
| anteaya | sorry what bug is that, thiagop? | 18:00 |
| Nisha | i do see this https://issues.jenkins-ci.org/browse/JENKINS-31050 on the net | 18:01 |
| thiagop | "Slave went offline during the build" | 18:01 |
| Nisha | but this doesnt has any workaround or solution | 18:01 |
| thiagop | sorry, issue | 18:01 |
| anteaya | thiagop: that is the error message | 18:02 |
| Nisha | anteaya, yeah. | 18:02 |
| Nisha | :( | 18:02 |
| anteaya | Nisha: have you met any other third party operators? | 18:02 |
| anteaya | Nisha: there are quite a few | 18:02 |
| anteaya | have you attended any third party meetings? | 18:02 |
| Nisha | anteaya, No | 18:02 |
| anteaya | or spent any time in the #openstack-infra channel talking with them | 18:03 |
| *** ChubYann has joined #openstack-ironic | 18:03 | |
| anteaya | well they are a great group of people | 18:03 |
| *** mjturek1 has joined #openstack-ironic | 18:03 | |
| Nisha | anteaya, sorry, i didnt know that we can talk there | 18:03 |
| anteaya | and I'm going to bet someone has come across this issue previously | 18:03 |
| anteaya | sure | 18:03 |
| *** moshele has joined #openstack-ironic | 18:03 | |
| anteaya | the biggest thing is to be clear you are asking about your system | 18:03 |
| anteaya | which you are operating | 18:03 |
| anteaya | as a random question about a CI system is interpreted as being about our system | 18:04 |
| Nisha | anteaya, thanks will do | 18:04 |
| anteaya | so clarity right off the start really helps | 18:04 |
| anteaya | thank you | 18:04 |
| thiagop | anteaya: yeah? I thought it was a launchpad bug... Maybe I'm assuming too much :) | 18:04 |
| anteaya | thiagop: it is possible, have you a link to the bug? | 18:05 |
| anteaya | I have to admit I don't look at the launchpad bugs often | 18:05 |
| thiagop | anteaya: nope, just really an assumption | 18:05 |
| thiagop | very bad one, indeed | 18:05 |
| anteaya | not a problem | 18:05 |
| *** jayc has quit IRC | 18:06 | |
| *** hamzy has joined #openstack-ironic | 18:09 | |
| anteaya | Nisha: here is information on the third party meetings: http://eavesdrop.openstack.org/#Third_Party_Meeting | 18:09 |
| anteaya | Nisha: the meetings during the week of summit are cancelled | 18:09 |
| anteaya | they are good meetings to ask questions | 18:10 |
| Nisha | anteaya, thanks will make a note of it | 18:10 |
| anteaya | wonderful, I hope to see you there | 18:10 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Fix get_health with a NextScale FPC https://review.openstack.org/304807 | 18:13 |
| *** Sukhdev has quit IRC | 18:14 | |
| JayF | https://review.openstack.org/#/c/302493 is a proposed stable backport with 1x+2 and 7x+1 if someone with those perms wanna take a look | 18:18 |
| *** jayc has joined #openstack-ironic | 18:19 | |
| JayF | My namespace is being invaded! oh no! | 18:20 |
| JayF | :) | 18:20 |
| *** achanda has quit IRC | 18:23 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Unify testing scripts https://review.openstack.org/297424 | 18:25 |
| JayF | NobodyCam: wdyt about landing https://review.openstack.org/#/c/275726/ -- I know you wanted to give lucas-brno a chance to look at it, but that was 3 (business) days ago :) | 18:28 |
| NobodyCam | I asked this morning for him to look at it. | 18:29 |
| NobodyCam | wanta hit the button | 18:29 |
| JayF | NobodyCam: https://review.openstack.org/#/c/297541/ is one you could probably hit the button on | 18:30 |
| *** baoli_ has quit IRC | 18:30 | |
| NobodyCam | oh looks like I had a +2 already | 18:32 |
| NobodyCam | let me take a look | 18:32 |
| *** moshele has quit IRC | 18:33 | |
| NobodyCam | oh I like the changes | 18:33 |
| NobodyCam | JayF: BAm | 18:34 |
| *** raildo is now known as raildo-afk | 18:41 | |
| openstackgerrit | Merged openstack/pyghmi: Fix get_health with a NextScale FPC https://review.openstack.org/304807 | 18:42 |
| *** [1]cdearborn has joined #openstack-ironic | 18:43 | |
| *** alexpilotti has quit IRC | 18:44 | |
| *** daemontool has quit IRC | 18:45 | |
| *** yarkot_ has joined #openstack-ironic | 18:46 | |
| *** Sukhdev has joined #openstack-ironic | 18:46 | |
| *** Sukhdev has quit IRC | 18:47 | |
| *** Sukhdev has joined #openstack-ironic | 18:48 | |
| *** Sukhdev has quit IRC | 18:49 | |
| *** alexpilotti has joined #openstack-ironic | 18:50 | |
| *** keedya has quit IRC | 18:55 | |
| *** Nisha has quit IRC | 18:55 | |
| *** raildo-afk is now known as raildo | 18:56 | |
| *** cdearborn has quit IRC | 18:57 | |
| *** ametts has quit IRC | 18:58 | |
| *** yuanying has quit IRC | 19:04 | |
| *** achanda has joined #openstack-ironic | 19:05 | |
| *** achanda has quit IRC | 19:05 | |
| *** dprince has joined #openstack-ironic | 19:06 | |
| *** Sukhdev has joined #openstack-ironic | 19:10 | |
| *** piet has quit IRC | 19:12 | |
| openstackgerrit | Merged openstack/ironic: Force iRMC vmedia boot from remotely connected CD/DVD https://review.openstack.org/297541 | 19:12 |
| *** raildo is now known as raildo-afk | 19:14 | |
| *** divya_ has quit IRC | 19:15 | |
| *** praneshp has quit IRC | 19:21 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [glance] https://review.openstack.org/304829 | 19:25 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Allow configuring shred's final overwrite with zeros https://review.openstack.org/304101 | 19:30 |
| jlvillal | JayF: I for one appreciate you highlighting patches for people to look at. Thanks :) | 19:39 |
| NobodyCam | JayF: maybe worth adding your arrival etherpad to teh announcment section of the agenda? | 19:39 |
| * jlvillal loses track of all the patches. | 19:39 | |
| jlvillal | Would it be worthwhile to have a deprecation to-do list for each release? For example we say we are deprecating something in Newton and in should be removed in the next release. Would it be good to have an ocata-deprecation-todo.txt (or something) | 19:45 |
| * jlvillal just thinking out loud | 19:45 | |
| *** Sukhdev has quit IRC | 19:45 | |
| jroll | jlvillal: I've only begun thinking about midcycle, nothing much to share yet | 19:46 |
| jroll | feel free to throw location invitations/suggestions at me though | 19:47 |
| jlvillal | jroll: Okay :) I think Intel would be able to host here in Oregon. | 19:48 |
| jroll | I don't believe we have a need for a separate todo list for deprecations, use the bug tracker luke | 19:48 |
| jroll | :P | 19:48 |
| jroll | awesome! | 19:48 |
| jlvillal | jroll: does that mean each patch that says this config option is deprecated they should also create a bug? | 19:48 |
| *** e0ne has quit IRC | 19:48 | |
| jlvillal | For the future release work? | 19:49 |
| jroll | jlvillal: maybe? or they should add it to their personal todo list | 19:49 |
| jroll | if we forget to remove something, then removing it clearly wasn't very important to us | 19:50 |
| *** daemontool has joined #openstack-ironic | 19:50 | |
| jlvillal | True :) | 19:50 |
| jlvillal | Usually the cruft doesn't hurt anything. | 19:50 |
| rloo | jlvillal: i don't think so -- that file would get out of date. | 19:50 |
| jlvillal | rloo: Probably true. Not a big deal for the most part if we forget to remove stuff a cycle or two late. | 19:51 |
| rloo | jlvillal, i think if it is documented in the code, the deprecation and when it will be removed, we can have trolls, er people going through and doing the needful. | 19:51 |
| jlvillal | heh | 19:52 |
| rloo | jlvillal: in fact, most of those are typically low hanging fruit. so someone could go through at the beginning of the cycle, and open low-hanging-fruit-bugs for each of them :) | 19:52 |
| jlvillal | Oh yeah. Good idea. | 19:53 |
| *** raildo-afk is now known as raildo | 19:53 | |
| * jlvillal has ideas for some of his co-workers... | 19:53 | |
| rloo | jlvillal: by the way, i looked at https://review.openstack.org/#/c/301526/ and didn't quite understand it. | 19:53 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [neutron] https://review.openstack.org/304838 | 19:53 |
| jlvillal | rloo: JayF knows more about it. And I think jroll also. | 19:54 |
| rloo | jlvillal: am going to add a comment and see if someone can clarify for me. | 19:54 |
| jlvillal | rloo: But you brought up a good point on the config options. That is a little confusing. | 19:54 |
| jroll | I what now | 19:54 |
| jroll | oh yeah, I know some things | 19:54 |
| jlvillal | jroll: You know all about https://review.openstack.org/#/c/301526/ :) | 19:54 |
| *** yarkot_ has quit IRC | 19:55 | |
| jlvillal | rloo: Not sure why there are separate options and also the IPA options don't seem to be used in the library... | 19:55 |
| jroll | those are good questions though | 19:55 |
| * jlvillal is getting more confused as he thinks about it. | 19:56 | |
| jroll | so, how this was implemented downstream | 19:56 |
| jroll | is the regular configs are for ironic, and the agent configs are for ironic as well, which it sends down to the agent on /lookup | 19:56 |
| jroll | rloo: ^ | 19:57 |
| *** Goneri has quit IRC | 19:57 | |
| jroll | so honestly, I think the agent configs should likely be dropped here, and it should just be regular configs | 19:57 |
| rloo | jroll: and does ironic and the agent use the code in the library? is that the 'shared' part? | 19:57 |
| jroll | and then ironic can have some agent configs to be sent down to the agent, or something | 19:58 |
| jroll | rloo: yeah | 19:58 |
| rloo | jroll: ok, that makes more sense. the configs don't seem to belong to the library. although i have to admit, i didn't look at any of the code. i started and stopped with the configs. | 19:58 |
| jroll | rloo: I think they do belong to the library, we just also need a way to pass them down to the agent | 19:59 |
| rloo | jroll: are those configs used by any code in the library? if not, they don't belong in the library. | 20:00 |
| JayF | rloo: I can show you | 20:00 |
| jroll | rloo: they are | 20:00 |
| JayF | rloo: I have a patch up for ir-cond that ships those configs to the conductor | 20:00 |
| jroll | (I think?) | 20:00 |
| JayF | s/conductor/agent/ | 20:00 |
| JayF | rloo: https://review.openstack.org/#/c/301919/ is the patch related to those agent_* settings | 20:00 |
| rloo | JayF: right, ironic needs those agent* config values, so it can tell the agent what they are. | 20:01 |
| rloo | JayF: but I don't see why those agent* configs are in the library if the library itself doesn't use them directly. | 20:01 |
| JayF | so you're saying the agent_* configs should be in 301919, and not in ironic-lib? | 20:01 |
| JayF | I can buy that | 20:01 |
| jroll | JayF: that's what I'm saying, yes | 20:01 |
| JayF | I'm OK with that. If you all will -1 the patch with that comment, alineb can fix the ironic-lib patch in the morning and I'll move those configs into my patch | 20:02 |
| jroll | rloo: what I meant before is "the not-agent configs belong in the library, and we need a way to pass what those should be to the agent", sorry | 20:02 |
| rloo | JayF: if some other project uses ironic-lib cuz it wants the metrics classes, what would the agent* configs mean for those other sw? | 20:02 |
| rloo | jroll: good, that jives with what i think. although i don't really know much about it :) | 20:03 |
| JayF | rloo: I guess that's reasonable; I just didn't realize we considered any non-ironic use cases for ironic-lib | 20:03 |
| rloo | JayF: well, what if bifrost or inspector wanted it. | 20:03 |
| JayF | rloo: and this is something the spec was a little squishy on, I have no objection whatsoever to doing it the way you and jim want, and I almost did it that way originally :) | 20:03 |
| JayF | rloo: I guess so; although I think the way we're writing this inspector should get it for free | 20:03 |
| *** jayc has quit IRC | 20:04 | |
| JayF | well, I guess not, because it doesn't lookup to the conductor | 20:04 |
| rloo | JayF: libraries are different in that you have to think of them more generically. | 20:04 |
| JayF | so that makes it make even more sense to factor the agent_* configs out of the lib | 20:04 |
| *** keedya has joined #openstack-ironic | 20:04 | |
| rloo | JayF or some other J*, do you want to add a comment to that review since you're more familiar with it than I am? | 20:05 |
| * rloo looks for an easier review to start off with... | 20:05 | |
| JayF | I mean, I'll gladly do it if you don't want to | 20:05 |
| rloo | JayF: heh, I can do so. Just trying to delegate. I'll do it... | 20:06 |
| JayF | I mean, I'll gladly do it too | 20:06 |
| rloo | JayF: ok, you can do it then :) | 20:06 |
| devananda | catching up on the backscroll, I think I agree with rloo | 20:07 |
| rloo | thx JayF (& devananda) | 20:09 |
| *** jayc has joined #openstack-ironic | 20:10 | |
| *** Haomeng has quit IRC | 20:10 | |
| *** Haomeng has joined #openstack-ironic | 20:10 | |
| *** piet has joined #openstack-ironic | 20:11 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Follow-up for Ib3eadf4172e93add9a9855582f56cbb3707f3d39 https://review.openstack.org/304847 | 20:27 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Allow configuring shred's final overwrite with zeros https://review.openstack.org/304101 | 20:28 |
| jlvillal | TheJulia: For your arrival time I did think of the Commodores song. | 20:28 |
| * jlvillal sings, I'm easy, easy like Sunday morning... | 20:29 | |
| JayF | jlvillal: seriously, until I read that, I thought the lyrics were "I need you like a Sunday morning" | 20:30 |
| TheJulia | jlvillal: it is totally not clicking in my brain right now | 20:30 |
| JayF | jlvillal: and never understood the song | 20:30 |
| JayF | lol | 20:30 |
| TheJulia | lol | 20:30 |
| JayF | hearing problems + song lyrics = hilarious lifelong mistakes | 20:30 |
| jlvillal | lol :) | 20:30 |
| JayF | who is mathieu mitchell in irc? | 20:31 |
| mat128 | me | 20:31 |
| JayF | oh perfect, lol | 20:31 |
| jlvillal | JayF: mat128 | 20:31 |
| mat128 | mmitchell was taken already, so I had to take my regular nickname | 20:31 |
| JayF | was going to ask a question but got the answer first | 20:32 |
| * jlvillal darn too late. | 20:32 | |
| jlvillal | TheJulia: Well the song is from 1977... | 20:32 |
| JayF | mat128: I missed the depends-on at first, for the ipa change, now I see it | 20:32 |
| JayF | mat128: was going to say you needed an ipa change that did, well, what your patch does :P | 20:32 |
| JayF | mat128: +2 on IPA change, +1 on Ironic change. Thanks! | 20:32 |
| mat128 | wasn't sure that was the appropriate syntax (does it really causes a dependency link or it's just docs)? | 20:33 |
| mat128 | thank you :) | 20:33 |
| JayF | it does a dependency link for CI purposes | 20:34 |
| JayF | IDK if it prevents landing if someone +A'd it | 20:34 |
| mat128 | ok good | 20:34 |
| jroll | it does | 20:34 |
| JayF | but in this case, shred doesn't get run in the gate | 20:34 |
| JayF | so your change on either side wouldn't have failed any tests, sadly | 20:34 |
| TheJulia | it will prevent it from merging | 20:34 |
| mat128 | with this change, we could run shredding in the gate with n=0 and final_zero = false | 20:34 |
| mat128 | to save time but still prove the process works | 20:34 |
| *** yarkot_ has joined #openstack-ironic | 20:35 | |
| JayF | hmm. wait a sec. I think something else uses that option you moved | 20:35 |
| *** cdearborn has joined #openstack-ironic | 20:35 | |
| JayF | I think ilo drivers use that erase_devices_iterations value as well | 20:35 |
| JayF | so wondering if your change wil lbreak them | 20:35 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 20:35 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 20:36 |
| JayF | oh that is for the *priority* of erase devices | 20:36 |
| JayF | which is what we already use to disable that cleaning step | 20:36 |
| JayF | hmm | 20:36 |
| mat128 | old name still defined as deprecated, so it will be configured, until we remove it | 20:36 |
| mat128 | but please show me, I can fix it tonight or tomorrow morning | 20:36 |
| JayF | mat128: do you think we should accept n=0 and False as valid values? | 20:36 |
| mat128 | JayF: why not? To me, it would be an optimization if we want n=0 and false to simply avoid the cleaning at all | 20:38 |
| mat128 | it could prove to be useful for testing purposes | 20:38 |
| mat128 | just like we can play with priority | 20:38 |
| JayF | mat128: I just put a -1 on the IPA patch asking for a test of n=0 and zero=False | 20:38 |
| JayF | mat128: my concern is, if a deployer configured this way, but left automated_clean to True, there's an expectation that a cleanup will happen | 20:38 |
| JayF | mat128: if zero=False and iterations=0, the cleanup won't happen, which is potentially a security hole | 20:39 |
| mat128 | ok, no problem | 20:39 |
| *** baoli has joined #openstack-ironic | 20:39 | |
| mat128 | do you think it should raise in IPA or in Ironic? | 20:39 |
| TheJulia | JayF: Wouldn't that check also need to go inside the conductor upon startup? | 20:39 |
| JayF | I don't even know for sure if it should fail ;) | 20:39 |
| JayF | but my hunch says maybe it should | 20:39 |
| JayF | TheJulia: probably, depending on outcome | 20:39 |
| mat128 | this is similar to the priority | 20:39 |
| JayF | TheJulia: for now I just wanted mat128 to make the behavior explicit so we see what is wanted to be done | 20:40 |
| TheJulia | JayF: ++ | 20:40 |
| JayF | mat128: yeah; I guess so, although I think it's ... not obvious that setting those two options that way = no cleaning of disk in some cases | 20:40 |
| JayF | maybe we should just check in conductor and toss a warning in the logs if you're configured in a such a way disks could be left unclean/ | 20:40 |
| TheJulia | If it is a condition that is functionally impossible to run cleaning, then I think the conductor should actually fail to startup, but thats just me | 20:41 |
| JayF | well, it wouldn't make running cleaning impossible | 20:41 |
| TheJulia | well | 20:41 |
| TheJulia | true | 20:41 |
| JayF | it would just give a chance (inconsistently) of not cleaning the disk | 20:41 |
| TheJulia | custom hardware managers and all | 20:41 |
| JayF | well also remember cleaning can be more than erase devices, even though upstream it isn't | 20:41 |
| mat128 | TheJulia: we use priority = 0 in staging to avoid spending time cleaning | 20:41 |
| TheJulia | yup | 20:41 |
| mat128 | because we have a different process to prove cleaning *would* work | 20:41 |
| TheJulia | mat128: :( | 20:41 |
| JayF | cleaning with an erase_devices step that is effectively a noop is still cleaning | 20:42 |
| JayF | heh, we clean everything everywhere with a brillo pad | 20:42 |
| mat128 | remember, we shred dollar bills :) | 20:42 |
| mat128 | takes a while | 20:42 |
| JayF | I don't have the time to go digging into new bugs made by having staging and prod configured differently | 20:42 |
| JayF | hahaha, yeah, I totally get you there | 20:42 |
| mat128 | otherwise we'd do secure erase everywhere | 20:42 |
| * TheJulia wonders which is better, magic erasor or brillo pad.... | 20:42 | |
| mat128 | ok here's my suggestion | 20:42 |
| mat128 | because I have to leave for the day | 20:43 |
| mat128 | put -1 on either of the changes | 20:43 |
| mat128 | lets think about it, discuss it tomorrow morning | 20:43 |
| TheJulia | JayF already put a -1 on the ipa change | 20:43 |
| *** rbrud has joined #openstack-ironic | 20:43 | |
| JayF | yeah, just asking for a test | 20:43 |
| JayF | make the behavior whatever you think it should be, I just wanted to be sure it's explicit | 20:43 |
| mat128 | and we can take a clear decision, worst case I can have a patch downstream to remove the "forbid operator from doing crazy things" limitation | 20:43 |
| JayF | and that we decide what it is vs just having an edge case we don't know what happens | 20:43 |
| *** dprince has quit IRC | 20:43 | |
| JayF | mat128: that won't be needed; we don't wanna make you do that, because I don't want you to make me do that ever | 20:43 |
| JayF | mat128: lol | 20:44 |
| mat128 | JayF: hahaha | 20:44 |
| mat128 | good stuff | 20:44 |
| JayF | mat128: I think a warning in the logs that cleaning disks may be less effective is probably the best middle ground | 20:44 |
| JayF | so folks who accidentally misconfigure can know that they did, but folks who want it that way get it that way | 20:44 |
| mat128 | a big security warning | 20:44 |
| mat128 | hmm | 20:44 |
| mat128 | and a note in the config option description | 20:44 |
| mat128 | yeah, gets clearer | 20:44 |
| mat128 | will do that tomorrow morning | 20:44 |
| JayF | LOG.warning('With X and Y set this way, you're not going to clean your disks in some cases. SECURITY VIOLATION!') | 20:44 |
| JayF | lol | 20:44 |
| mat128 | should we do that for priority=0 too? | 20:44 |
| JayF | I certainly wouldn't be opposed to it | 20:45 |
| JayF | but in that case, it's VERY explicit what you're doing | 20:45 |
| JayF | "setting this to 0 disables this" | 20:45 |
| JayF | vs two settings combining to make ... strange behavior | 20:45 |
| JayF | which is less explicit | 20:45 |
| mat128 | yeah, not a combinason of settings | 20:45 |
| mat128 | agreed | 20:45 |
| mat128 | ok, test to clarify + big warning | 20:45 |
| JayF | +1 awesome, isn't open source fun | 20:45 |
| JayF | I'll review your fixes first thing tomorrow, ty | 20:45 |
| mat128 | in Ironic directly, right? | 20:45 |
| JayF | The warning should be in Ironic directly | 20:46 |
| mat128 | (I know my operators wont look at IPA logs, lol) | 20:46 |
| mat128 | yes, ok | 20:46 |
| JayF | but we still need the test in IPA | 20:46 |
| JayF | the unit test | 20:46 |
| mat128 | sure | 20:46 |
| JayF | that shows 0 + False = don't run shred | 20:46 |
| JayF | in fact, you might wanna specifically implement it as a special case or something | 20:46 |
| JayF | my suspicion reading the code is that 0 + False is an invalid config that would break today in IPA | 20:46 |
| mat128 | `shred --force --verbose --iterations 0 my_block_device` runs happily | 20:47 |
| mat128 | return code 0 | 20:47 |
| JayF | and it just does nothing? | 20:47 |
| mat128 | does nothing | 20:47 |
| JayF | lol | 20:47 |
| mat128 | $ shred --force --verbose --iterations 0 my_block_device | 20:47 |
| mat128 | ubuntu@docker-host:~$ echo $? | 20:47 |
| mat128 | 0 | 20:47 |
| JayF | do you trust it does nothing enough to run it against a disk you care about /jk | 20:47 |
| JayF | man, that's kinda awesome hilarious and a little terrible | 20:47 |
| JayF | all rolled into one | 20:47 |
| TheJulia | ugh | 20:48 |
| mat128 | $ sudo shred --force --verbose --iterations 0 /dev/vda | 20:48 |
| mat128 | still here | 20:48 |
| mat128 | :) | 20:48 |
| JayF | good lord I was kidding | 20:48 |
| JayF | and would've felt horrible if it did anything at all | 20:48 |
| JayF | lol | 20:48 |
| mat128 | but it def. tried to open the device (got Operation not permitted without sudo) | 20:48 |
| JayF | that's kinda ... weird | 20:48 |
| mat128 | cattle vs cat, my docker test host machine isnt all that important | 20:49 |
| JayF | I hope you didn't just nuke your MBR or something | 20:49 |
| JayF | oh, docker host, good | 20:49 |
| mat128 | it's just a vm | 20:49 |
| jroll | please don't shred any cats :/ | 20:49 |
| mat128 | ok, to be safe, we can arrange not to call it when false and n=0 | 20:49 |
| mat128 | + proper unittests | 20:49 |
| mat128 | in IPA, big security warning in ironic | 20:49 |
| JayF | I mean, I'm OK with it if the shred command is | 20:49 |
| JayF | just ... surprised shred command is | 20:49 |
| mat128 | lol | 20:50 |
| *** [1]cdearborn has quit IRC | 20:50 | |
| mat128 | I'll think about it (in a few mins, in my car in traffic :(, but might leave it as-is (except test to clarify) | 20:50 |
| mat128 | I'll submit udpates tomorrow morning | 20:50 |
| mat128 | really gotta go now | 20:50 |
| JayF | o/ | 20:50 |
| mat128 | thanks for your time | 20:50 |
| JayF | have a good day | 20:50 |
| JayF | np anytime | 20:50 |
| *** e0ne has joined #openstack-ironic | 20:52 | |
| *** baoli has quit IRC | 21:03 | |
| *** ametts has joined #openstack-ironic | 21:06 | |
| *** raildo is now known as raildo-afk | 21:06 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 21:07 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 21:07 |
| *** jjohnson2 has quit IRC | 21:10 | |
| *** trown is now known as trown|outtypewww | 21:11 | |
| openstackgerrit | Merged openstack/ironic: deployment vmedia ops should not be run when not deploying https://review.openstack.org/300674 | 21:17 |
| *** e0ne has quit IRC | 21:20 | |
| *** hemna_ has quit IRC | 21:22 | |
| *** e0ne has joined #openstack-ironic | 21:25 | |
| *** e0ne has quit IRC | 21:27 | |
| *** hemna_ has joined #openstack-ironic | 21:29 | |
| mrda | Morning Ironic | 21:32 |
| devananda | g'morning, mrda | 21:32 |
| mrda | hey devananda | 21:32 |
| thiagop | morning mrda | 21:33 |
| JayF | o/ | 21:36 |
| mrda | hey thiagop and JayF | 21:36 |
| *** ravig has joined #openstack-ironic | 21:38 | |
| ravig | Hello All, I have a question regarding ironic architecture - is there a feature that can actually make a node boot of an iscsi device instead of local installation or will that go against design philosophy of ironic | 21:42 |
| JayF | Not at all ravig | 21:43 |
| JayF | it's a good question and a commonly requested feature | 21:43 |
| JayF | it's currently in process of being designed, let me get you a link to the RFE | 21:43 |
| JayF | ravig: https://review.openstack.org/#/c/294995/ is the initial spec, still in review, written by TheJulia | 21:43 |
| JayF | ravig: so we can't do it not but we will at some nebulous point in the future when that feature is done :) | 21:44 |
| ravig | JayF: Thanks for responding back, the way I see this, that could be really useful, I mean we can use the same image for both virtual machines and physical nodes at some point | 21:45 |
| JayF | ravig: I'm actually not a person who cares or knows a lot about that feature; I'd suggest reading that spec and if you have questions, post them there | 21:46 |
| *** ayoung has quit IRC | 21:46 | |
| ravig | JayF: Sure thanks a lot :) | 21:46 |
| JayF | np | 21:46 |
| *** piet has quit IRC | 21:47 | |
| *** piet has joined #openstack-ironic | 21:47 | |
| *** [1]cdearborn has joined #openstack-ironic | 21:48 | |
| *** sukhdev has joined #openstack-ironic | 21:49 | |
| *** fragatina has quit IRC | 21:54 | |
| * NobodyCam is back | 21:57 | |
| openstackgerrit | Merged openstack/ironic: In node_power_action() add node.UUID to log message https://review.openstack.org/301350 | 21:58 |
| *** cdearborn has quit IRC | 22:03 | |
| jlvillal | Midsomer mrda ? | 22:05 |
| jlvillal | mrda on the Orient Express? | 22:06 |
| mrda | jlvillal: lol | 22:06 |
| mrda | jlvillal: a mrda of crows? | 22:06 |
| jlvillal | mrda: :) Just checking if you were up for a Nova bug scrub this week | 22:06 |
| mrda | jlvillal: always ;) | 22:06 |
| NobodyCam | morning mrda | 22:06 |
| jlvillal | Sweet :) | 22:07 |
| mrda | hey NobodyCam | 22:07 |
| NobodyCam | :) | 22:07 |
| *** piet has quit IRC | 22:10 | |
| *** alexpilotti has quit IRC | 22:22 | |
| *** Sukhdev_ has joined #openstack-ironic | 22:23 | |
| *** fragatina has joined #openstack-ironic | 22:24 | |
| *** derekh has joined #openstack-ironic | 22:25 | |
| *** fragatina has quit IRC | 22:30 | |
| *** ayoung has joined #openstack-ironic | 22:31 | |
| *** Marga_ has joined #openstack-ironic | 22:32 | |
| *** derekh has quit IRC | 22:33 | |
| *** Marga_ has quit IRC | 22:35 | |
| *** Marga_ has joined #openstack-ironic | 22:36 | |
| *** ametts has quit IRC | 22:37 | |
| *** rama_y has quit IRC | 22:40 | |
| NobodyCam | JayF: did you want to pull the trigger on 275726? | 22:45 |
| JayF | I mean, I can or you can? | 22:50 |
| * JayF landed it | 22:50 | |
| NobodyCam | you rock JayF | 22:52 |
| NobodyCam | :) | 22:52 |
| openstackgerrit | Merged openstack/ironic-specs: Dynamic allocation of nodes on OneView drivers https://review.openstack.org/275726 | 22:52 |
| *** Sukhdev_ has quit IRC | 22:53 | |
| *** Sukhdev_ has joined #openstack-ironic | 22:54 | |
| *** alexpilotti has joined #openstack-ironic | 23:00 | |
| *** mtanino has quit IRC | 23:03 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 23:06 |
| *** Sukhdev_ has quit IRC | 23:10 | |
| *** Sukhdev_ has joined #openstack-ironic | 23:11 | |
| *** alexpilotti has quit IRC | 23:11 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 23:16 |
| thiagop | JayF NobodyCam: thanks for merging that! | 23:17 |
| * thiagop will buy a beer to celebrate | 23:17 | |
| JayF | thiagop: I'm sure you're glad to have it merged :P | 23:17 |
| NobodyCam | hehe | 23:17 |
| JayF | thiagop: it's OK, we've all had one or seven specs like that, that's why I was glad to land it ;P | 23:17 |
| NobodyCam | I thought you'd away already | 23:17 |
| * JayF remembers back in the giant-agent-driver-patch early days of Ironic | 23:17 | |
| NobodyCam | lol ++++ | 23:18 |
| thiagop | I was here trying to fix a stuff in a parallel project | 23:18 |
| *** yuanying has joined #openstack-ironic | 23:18 | |
| thiagop | now I WILL get home | 23:18 |
| thiagop | see you guys tomorrow | 23:18 |
| NobodyCam | night thiagop and congratz | 23:18 |
| *** thiagop has quit IRC | 23:19 | |
| *** alexpilotti has joined #openstack-ironic | 23:20 | |
| *** rbrud has quit IRC | 23:21 | |
| *** alexpilotti has quit IRC | 23:24 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Pass agent metrics config via conductor https://review.openstack.org/301919 | 23:33 |
| *** yuanying has quit IRC | 23:36 | |
| *** jayc has quit IRC | 23:48 | |
| *** sukhdev has quit IRC | 23:52 | |
| *** baoli has joined #openstack-ironic | 23:54 | |
| *** yarkot_ has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!