| *** bradjones has joined #openstack-ironic | 00:00 | |
| *** bradjones has quit IRC | 00:00 | |
| *** bradjones has joined #openstack-ironic | 00:00 | |
| *** naohirot has joined #openstack-ironic | 00:00 | |
| *** smoriya has joined #openstack-ironic | 00:03 | |
| openstackgerrit | Merged openstack/ironic-specs: Wake-On-Lan (WOL) power driver https://review.openstack.org/187678 | 00:05 |
|---|---|---|
| *** chlong-zzz has joined #openstack-ironic | 00:17 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Doc: Use --notest for creating venv https://review.openstack.org/189984 | 00:20 |
| *** mtanino has quit IRC | 00:27 | |
| *** saripurigopi has joined #openstack-ironic | 00:31 | |
| *** saripurigopi has quit IRC | 00:33 | |
| *** saripurigopi has joined #openstack-ironic | 00:40 | |
| jroll | bug 1463632 is low hanging fruit if anyone wants some :) | 00:41 |
| openstack | bug 1463632 in Ironic "CoreOS GPG key is expired" [Medium,New] https://launchpad.net/bugs/1463632 | 00:41 |
| jroll | FYI, IPA gate is a little sketchy right now. coreos downloads failing. culprit is https://status.cloud.google.com/incident/storage/16025 | 00:45 |
| *** dprince has joined #openstack-ironic | 00:46 | |
| *** zhenguo has joined #openstack-ironic | 00:47 | |
| *** dprince has quit IRC | 00:48 | |
| *** ijw has quit IRC | 00:52 | |
| *** Guest5484 has quit IRC | 01:01 | |
| *** Sukhdev has quit IRC | 01:03 | |
| *** mdbooth has quit IRC | 01:07 | |
| openstackgerrit | Merged openstack/ironic-specs: Add nodes tagging support https://review.openstack.org/183971 | 01:12 |
| *** mdbooth has joined #openstack-ironic | 01:13 | |
| *** valkyrja has joined #openstack-ironic | 01:16 | |
| TheJulia | oh yay, its a valkyrja | 01:17 |
| valkyrja | TheJulia: why, yes, I am... here. | 01:20 |
| *** thrash is now known as thrash|g0ne | 01:20 | |
| valkyrja | now I just need to put on my shirt that says my mother did not raise a princess, she raised a valkyrie | 01:21 |
| *** zz_natorious is now known as natorious | 01:45 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/187680 | 01:48 |
| *** spandhe has quit IRC | 01:49 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add 'pbr' to the requirements https://review.openstack.org/188204 | 01:57 |
| *** yog__ has joined #openstack-ironic | 02:02 | |
| *** dan_ has joined #openstack-ironic | 02:05 | |
| *** dan_ is now known as Guest19563 | 02:05 | |
| *** saripurigopi has quit IRC | 02:07 | |
| *** yog__ has quit IRC | 02:10 | |
| *** yog__ has joined #openstack-ironic | 02:11 | |
| *** boris-42 has quit IRC | 02:12 | |
| *** spandhe has joined #openstack-ironic | 02:25 | |
| *** kan_ has joined #openstack-ironic | 02:25 | |
| *** natorious is now known as zz_natorious | 02:26 | |
| *** spandhe_ has joined #openstack-ironic | 02:28 | |
| *** spandhe has quit IRC | 02:29 | |
| *** spandhe_ is now known as spandhe | 02:29 | |
| *** ramineni has joined #openstack-ironic | 02:45 | |
| *** bradjones has quit IRC | 02:52 | |
| *** bradjones has joined #openstack-ironic | 02:53 | |
| *** bradjones has quit IRC | 02:53 | |
| *** bradjones has joined #openstack-ironic | 02:53 | |
| *** naohirot has quit IRC | 03:06 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Migrate to oslo_db https://review.openstack.org/181905 | 03:22 |
| *** Nisha has joined #openstack-ironic | 03:37 | |
| *** coolsvap|afk is now known as coolsvap | 03:42 | |
| *** saripurigopi has joined #openstack-ironic | 03:46 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add function to clear lock if nodes are reserved https://review.openstack.org/189587 | 03:47 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Check timeout when nodes stuck on deploying state https://review.openstack.org/188688 | 03:47 |
| *** Sukhdev has joined #openstack-ironic | 03:51 | |
| *** naohirot has joined #openstack-ironic | 04:00 | |
| *** naohirot has quit IRC | 04:00 | |
| *** naohirot has joined #openstack-ironic | 04:01 | |
| *** spandhe has quit IRC | 04:13 | |
| openstackgerrit | Merged openstack/ironic: Add additional required RPMs to dev instructions https://review.openstack.org/185993 | 04:14 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/189899 | 04:14 |
| openstackgerrit | Merged openstack/ironic: Fix DRAC driver job completion detection https://review.openstack.org/189409 | 04:14 |
| *** chlong-zzz is now known as chlong | 04:32 | |
| *** amotoki_ has joined #openstack-ironic | 04:40 | |
| *** amotoki_ has quit IRC | 04:46 | |
| *** amotoki is now known as amotoki_ | 04:46 | |
| *** amotoki_ has quit IRC | 04:49 | |
| *** amotoki has joined #openstack-ironic | 04:49 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Migrate to oslo_db https://review.openstack.org/181905 | 04:56 |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Migrate to oslo_db https://review.openstack.org/181905 | 04:57 |
| *** achanda has joined #openstack-ironic | 04:58 | |
| openstackgerrit | Pratyusha proposed stackforge/proliantutils: Add iSCSI boot support in RIS https://review.openstack.org/187162 | 05:01 |
| *** lazy_prince has joined #openstack-ironic | 05:04 | |
| lazy_prince | o/ | 05:08 |
| saripurigopi | morning all | 05:11 |
| *** rameshg87 has joined #openstack-ironic | 05:19 | |
| rameshg87 | good morning ironic | 05:19 |
| *** pradipta has joined #openstack-ironic | 05:23 | |
| *** saripurigopi has quit IRC | 05:23 | |
| openstackgerrit | Pratyusha proposed stackforge/proliantutils: Add iSCSI boot support in RIS https://review.openstack.org/187162 | 05:23 |
| openstackgerrit | Pratyusha proposed stackforge/proliantutils: Add iSCSI boot support in RIS https://review.openstack.org/187162 | 05:28 |
| *** achanda has quit IRC | 05:32 | |
| *** stendulker has joined #openstack-ironic | 05:37 | |
| *** kalpase has joined #openstack-ironic | 05:40 | |
| *** kalpase has left #openstack-ironic | 05:40 | |
| *** sandhya has joined #openstack-ironic | 05:40 | |
| *** amotoki_ has joined #openstack-ironic | 05:41 | |
| sandhya | Any further reviews for this - https://review.openstack.org/#/c/186285/4 | 05:42 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Activate iLO license https://review.openstack.org/190030 | 05:43 |
| *** amotoki_ has quit IRC | 05:46 | |
| rameshg87 | sandhya: nothing was formally announced, but we agreed during the summit that this is an ideal candidate for a vendor-specific repo on it's own | 05:46 |
| rameshg87 | and not in ironic | 05:46 |
| sandhya | rameshg87: Thank you... then do you say it should be a stackforge project? | 05:48 |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Fix gate failures https://review.openstack.org/190031 | 05:48 |
| rameshg87 | sandhya: there was a mail thread on this | 05:48 |
| * rameshg87 searches | 05:49 | |
| rameshg87 | sandhya: http://lists.openstack.org/pipermail/openstack-dev/2015-June/065832.html | 05:50 |
| rameshg87 | sandhya: nobody much replied on that. | 05:50 |
| rameshg87 | sandhya: but I think your own stackforge repo is the best place | 05:50 |
| rameshg87 | in my opinion | 05:50 |
| *** Sukhdev has quit IRC | 05:51 | |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Fix gate failures https://review.openstack.org/190031 | 05:52 |
| sandhya | rameshg87: okay... Thanks. Will follow up on this. | 05:52 |
| rameshg87 | ramineni: stendulker: ^^^ review | 05:54 |
| rameshg87 | ramineni: stendulker: fixes gate failures in proliantutils | 05:54 |
| ramineni | rameshg87: ack | 05:54 |
| *** ukalifon1 has joined #openstack-ironic | 05:57 | |
| *** saripurigopi has joined #openstack-ironic | 05:59 | |
| lazy_prince | rameshg87: hi.. | 06:04 |
| lazy_prince | regarding https://review.openstack.org/#/c/137567/4/specs/kilo/ilo-virtualmedia-drivers-dhcpless-deploy.rst on L38 | 06:05 |
| lazy_prince | Do you intend to get details using neutron meta data proxy..? | 06:05 |
| openstackgerrit | Merged stackforge/proliantutils: Fix gate failures https://review.openstack.org/190031 | 06:05 |
| *** jcoufal has joined #openstack-ironic | 06:08 | |
| rameshg87 | lazy_prince: hi | 06:08 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Activate iLO license https://review.openstack.org/190030 | 06:08 |
| rameshg87 | lazy_prince: not really | 06:08 |
| rameshg87 | lazy_prince: directly querying the neutron port to get the details like ip address, subnet, gateway, etc | 06:08 |
| rameshg87 | lazy_prince: which could be passed to the ramdisk to apply for a particular interface | 06:08 |
| lazy_prince | aha.. okay so it is a s pointed by ucas.. | 06:09 |
| lazy_prince | s/ucas/lucas/ | 06:09 |
| rameshg87 | lazy_prince: yes | 06:11 |
| rameshg87 | lazy_prince: need to pursue that spec now for liberty. | 06:13 |
| openstackgerrit | Pratyusha proposed stackforge/proliantutils: Add iSCSI boot support in RIS https://review.openstack.org/187162 | 06:13 |
| * rameshg87 will post a new spec patchset today | 06:13 | |
| lazy_prince | rameshg87: is it not based on the fact that it will work only for HP ProLiant severs..? Can we generalise it for Fake baremetal too.. so that we could use it with DevStack witout worrying about BM availability and without disturbing DHCP servers running in our organization.. | 06:18 |
| *** Sukhdev has joined #openstack-ironic | 06:19 | |
| lazy_prince | not sure if it makes sense.. | 06:19 |
| rameshg87 | lazy_prince: I didn't get | 06:21 |
| rameshg87 | lazy_prince: what do you mean by fake baremetal here ? | 06:21 |
| rameshg87 | lazy_prince: this method will work in any case if you can pass the ip information out-of-band to the bare metal (one of the methods to do so being virtual media) | 06:22 |
| lazy_prince | rameshg87: okay.. here is the deal.. Virtual Media is just a CD device connected to a HP ProLiant Server.. We could do the same with VMs too as in we could attach a CD device to VM and have DHCP less devstack setup.. | 06:23 |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials https://review.openstack.org/170782 | 06:23 |
| lazy_prince | rameshg87: i understand that this would mean a ned driver for Fake baremetal driver with Virtual Media support or something.. | 06:24 |
| lazy_prince | s/ned/need/ | 06:24 |
| rameshg87 | lazy_prince: oh okay | 06:24 |
| rameshg87 | lazy_prince: so fake baremetal driver = pxe_ssh driver, is that what you mean ? | 06:24 |
| rameshg87 | lazy_prince: which uses vms to *fake* baremetals ? | 06:25 |
| lazy_prince | yes.. do you know if something like that is avaliable for VMs (with virtual media thing) ? | 06:25 |
| rameshg87 | lazy_prince: nobody has done it, but we could do the same thing for vms too | 06:26 |
| rameshg87 | lazy_prince: we can connect it as virtual media and then it should work | 06:26 |
| *** achanda has joined #openstack-ironic | 06:26 | |
| lazy_prince | rameshg87: apart from using actual Virtual media, it could help rest of the Virtual media testing on the deploy side by simulating the virtual media on VMs.. | 06:27 |
| lazy_prince | makes sense..? | 06:27 |
| *** lsmola has joined #openstack-ironic | 06:27 | |
| lazy_prince | i am planning to put a bp for this.. do you think, it will fly..? | 06:27 |
| rameshg87 | lazy_prince: I had spec on it last time | 06:27 |
| rameshg87 | https://review.openstack.org/#/c/137933/ | 06:27 |
| rameshg87 | lazy_prince: but initially didn't pickup interest from reviewers and I too didn't work on convincing them :D | 06:28 |
| rameshg87 | lazy_prince: then sort of managed to do it with my virtualbox, by faking proliantutils module to actually deal with virtual box instead of ilo. I test all my stuffs first on that. | 06:29 |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Load authenticate token from HTTP header https://review.openstack.org/174231 | 06:30 |
| rameshg87 | lazy_prince: have a patch for it in proliantutils - https://review.openstack.org/#/c/155647/ (of course long time, so it is in merge conflict) | 06:30 |
| rameshg87 | lazy_prince: I meant I already simulate virtual media with virtual box :) | 06:31 |
| lazy_prince | hmm.. exactly.. however, your BP proposes to use shared storage.. while i plan on using ISOs/Floppy connected to VM. And can it be generalised for all virtualizations like kvm..? | 06:33 |
| *** Sukhdev has quit IRC | 06:33 | |
| rameshg87 | lazy_prince: yes it can .. | 06:33 |
| rameshg87 | lazy_prince: I am +1 for it :) | 06:33 |
| lazy_prince | lets revive your bp and push for it.. | 06:34 |
| rameshg87 | lazy_prince: \o/ | 06:34 |
| *** mdbooth has quit IRC | 06:39 | |
| *** edcragg has joined #openstack-ironic | 06:41 | |
| *** amotoki_ has joined #openstack-ironic | 06:41 | |
| *** mdbooth has joined #openstack-ironic | 06:45 | |
| *** amotoki_ has quit IRC | 06:47 | |
| *** romainh has joined #openstack-ironic | 06:49 | |
| *** kalpase has joined #openstack-ironic | 06:51 | |
| *** dtantsur|afk is now known as dtantsur | 06:51 | |
| dtantsur | Morning! | 06:51 |
| stendulker | Morning dtantsur :) | 06:52 |
| dtantsur | o/ | 06:52 |
| *** yuikotakada has joined #openstack-ironic | 06:57 | |
| yuikotakada | Hi, Ironic | 07:05 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: iLO virtual media drivers to deploy without DHCP https://review.openstack.org/137567 | 07:08 |
| rameshg87 | lazy_prince: ^^ | 07:09 |
| rameshg87 | lazy_prince: let me know your thoughts | 07:09 |
| *** Nisha has quit IRC | 07:10 | |
| dtantsur | yuikotakada, hi, g'evening | 07:12 |
| rameshg87 | dtantsur: hi | 07:18 |
| dtantsur | o/ | 07:18 |
| rameshg87 | dtantsur: can you please have a look at inband raid spec - https://review.openstack.org/#/c/173218/ to see if its okay now | 07:18 |
| rameshg87 | (the whole chain needs to be approved before coming to this, but still good to have some reviews on it ready) | 07:19 |
| dtantsur | sure thing, not right now though | 07:20 |
| *** achanda has quit IRC | 07:20 | |
| dtantsur | I'll start review the whole backlog after lunch today :) | 07:20 |
| rameshg87 | dtantsur: thanks | 07:21 |
| yuikotakada | dtantsur, hi, good afternoon...? rather than evening :) and thank you for sending an e-mail to QA team! | 07:22 |
| *** kan_ has quit IRC | 07:23 | |
| dtantsur | ah, afternoon. I started to get up earlier, so it's no longer always evening for you :) | 07:23 |
| dtantsur | np | 07:23 |
| *** kan_ has joined #openstack-ironic | 07:23 | |
| yuikotakada | dtantsur, oh, for your health? It's good for me, too because we can work together longer :) | 07:25 |
| dtantsur | it's pretty hot outside now, so it makes a lot of sense to get up before my room also becomes hot. but yeah, right, I'll try to do it from now on | 07:26 |
| yuikotakada | dtantsur, definitely. | 07:30 |
| *** kan_ has quit IRC | 07:32 | |
| *** kan_ has joined #openstack-ironic | 07:33 | |
| *** viktors|afk is now known as viktors | 07:33 | |
| *** jistr has joined #openstack-ironic | 07:37 | |
| korean101 | hi ironic! | 07:37 |
| korean101 | sorry about my happening, yesterday | 07:38 |
| korean101 | i get succeed deploy with centos 6/7 images | 07:38 |
| korean101 | i failed cause kernel parameter | 07:39 |
| *** dlpartain has joined #openstack-ironic | 07:39 | |
| korean101 | but centos 6 iamge can't all-dhcp-interface | 07:39 |
| korean101 | this is my another problem | 07:39 |
| korean101 | and thank you all of you! | 07:40 |
| *** dlpartain has left #openstack-ironic | 07:40 | |
| *** amotoki_ has joined #openstack-ironic | 07:43 | |
| *** achanda has joined #openstack-ironic | 07:44 | |
| *** amotoki_ has quit IRC | 07:48 | |
| *** edcragg has quit IRC | 07:48 | |
| *** bethelwell has joined #openstack-ironic | 07:49 | |
| *** chlong has quit IRC | 07:51 | |
| *** ifarkas has joined #openstack-ironic | 07:52 | |
| *** athomas has joined #openstack-ironic | 07:52 | |
| *** bethelwell has quit IRC | 07:56 | |
| *** bethelwell has joined #openstack-ironic | 07:56 | |
| *** leopoldj has joined #openstack-ironic | 08:00 | |
| *** kalpase has quit IRC | 08:00 | |
| *** ijw has joined #openstack-ironic | 08:01 | |
| *** yuanying has quit IRC | 08:06 | |
| *** lucasagomes has joined #openstack-ironic | 08:07 | |
| *** MattMan has joined #openstack-ironic | 08:11 | |
| *** ifarkas has quit IRC | 08:14 | |
| *** kalpase has joined #openstack-ironic | 08:16 | |
| *** ifarkas has joined #openstack-ironic | 08:17 | |
| *** edcragg has joined #openstack-ironic | 08:20 | |
| *** smoriya has quit IRC | 08:21 | |
| *** sandhya has quit IRC | 08:25 | |
| *** lsmola has quit IRC | 08:27 | |
| *** dlpartain has joined #openstack-ironic | 08:32 | |
| *** lsmola has joined #openstack-ironic | 08:34 | |
| *** bradjones has quit IRC | 08:40 | |
| *** dlpartain has left #openstack-ironic | 08:41 | |
| *** bradjones has joined #openstack-ironic | 08:42 | |
| *** bradjones has quit IRC | 08:42 | |
| *** bradjones has joined #openstack-ironic | 08:42 | |
| *** amotoki_ has joined #openstack-ironic | 08:44 | |
| *** dguerri` is now known as dguerri | 08:45 | |
| *** amotoki_ has quit IRC | 08:48 | |
| lazy_prince | korean101: can tell us what kernel parameter was missing or causing problem..? | 08:50 |
| *** vishwanathj has quit IRC | 08:51 | |
| *** ijw has quit IRC | 08:57 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add vendor-passthru to attach and boot an ISO https://review.openstack.org/188317 | 09:02 |
| korean101 | nofb <--- | 09:04 |
| korean101 | pxe_append_params = boot_server=192.168.51.114 | 09:04 |
| korean101 | i remove all params | 09:04 |
| korean101 | but boot_server | 09:04 |
| lazy_prince | korean101: thanks.. that helps.. | 09:05 |
| korean101 | and i have an another question... | 09:05 |
| korean101 | sorry... | 09:05 |
| lazy_prince | sure.. | 09:05 |
| korean101 | is it possible create image based on existing image? | 09:05 |
| *** ifarkas has quit IRC | 09:05 | |
| korean101 | i have a centos6 image | 09:06 |
| korean101 | i wanna create DIB image based on my custom centos 6 image | 09:06 |
| korean101 | example | 09:06 |
| lazy_prince | umm.. you mean using images generated by dib to create another set of images..? | 09:06 |
| lazy_prince | or generating images out of your own built images (without using dib).? | 09:07 |
| korean101 | export DIB_LOCAL_IMAGE=my-custom-image.qcow2 ; disk-image-create centos baremetal dhcp-all-interfaces local-config -o centos6-image | 09:07 |
| korean101 | is it possible? | 09:07 |
| lazy_prince | you could always do that as long as your custom image folows the cloud image disk layout.. | 09:08 |
| *** dtantsur is now known as dtantsur|brb | 09:08 | |
| korean101 | but | 09:08 |
| korean101 | i get a error | 09:08 |
| lazy_prince | like the centos element should know which partition on the image contains root, boot, etc.. | 09:08 |
| korean101 | OMG | 09:09 |
| korean101 | can you explain about that? | 09:09 |
| korean101 | something example commands? | 09:09 |
| korean101 | chroot: failed to run command 'bin/tar': No such file or directory <----- i got a error | 09:09 |
| lazy_prince | so centos element uses base cloud image published by centos community as the base image to start with.. | 09:10 |
| korean101 | hmmm | 09:10 |
| lazy_prince | and then it takes all the files from the root fs of the disk image etc.... to a temp folder.. | 09:10 |
| lazy_prince | and then runs update and upgrades and installs necessary tools.. and gen generates the image... | 09:11 |
| korean101 | ok i see | 09:11 |
| korean101 | but i wanna uses based on my custom centos 6 image | 09:11 |
| lazy_prince | all this logic is distributed in centos element and couple of other elements.. | 09:11 |
| korean101 | is it impossible? | 09:12 |
| lazy_prince | you may need to create a fork of centos element and customize it as per your needs.. | 09:12 |
| korean101 | ah... | 09:12 |
| lazy_prince | i could be worn but this is how all distro elements are coded.. | 09:13 |
| *** ifarkas has joined #openstack-ironic | 09:13 | |
| lazy_prince | so, another thing is that what format the centos cloud image is (downloaded from net) is in.. is it qcow2 or tarball etc.. try packaging your image in same format and give it a go.. | 09:14 |
| korean101 | can i change disk layout my custom image? (like centos community image) | 09:14 |
| korean101 | how can? | 09:14 |
| lazy_prince | yes.. you could give that a try.. | 09:14 |
| lazy_prince | assuming your image is in qcow2 disk format, mount your qcow2 locally and then copy contents to another qcow2 in the format needed.. | 09:15 |
| korean101 | CentOS-6-x86_64-GenericCloud-20141129_01.qcow2 <----- change layout my custom image | 09:15 |
| korean101 | guestmount /mnt1 | 09:16 |
| korean101 | guestmount /mnt2 | 09:16 |
| korean101 | and just copy? | 09:16 |
| lazy_prince | well.. just copy may not preserve the user info and permissions.. you may need to use sudo tar to preserve info.. | 09:17 |
| korean101 | ok | 09:17 |
| korean101 | just tar to untar? | 09:17 |
| lazy_prince | or a simple thing would be to deploy the base cloud image on an instance, make the changes to it and use it as a base image for DIB.. | 09:17 |
| lazy_prince | yup.. | 09:18 |
| korean101 | oh | 09:18 |
| korean101 | i see | 09:18 |
| korean101 | hmmm | 09:20 |
| korean101 | thanks | 09:20 |
| korean101 | i will try it | 09:20 |
| korean101 | and about DIB centos element | 09:20 |
| korean101 | centos 6 doesn't working 'dhcp-all-interfaces' | 09:21 |
| korean101 | but centos 7 do working | 09:21 |
| korean101 | can't get ip via dhcp in centos 6 | 09:21 |
| korean101 | cat get ip via dhcp in centos 7 | 09:21 |
| lazy_prince | if you know how to make it work, push a patch.. :) | 09:21 |
| korean101 | ok... | 09:22 |
| korean101 | thanks | 09:22 |
| korean101 | thanks a lot | 09:22 |
| lazy_prince | welcome | 09:22 |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: IPA: Do a soft power off at the end of deployment https://review.openstack.org/185667 | 09:31 |
| *** v92 has quit IRC | 09:32 | |
| *** kan_ has quit IRC | 09:40 | |
| sambetts | Morning ironic | 09:45 |
| *** pelix has joined #openstack-ironic | 09:45 | |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Activate iLO license https://review.openstack.org/190030 | 09:47 |
| yuikotakada | sambetts: o/ | 09:48 |
| sambetts | o/ yuikotakada | 09:48 |
| *** e0ne has joined #openstack-ironic | 09:55 | |
| *** e0ne is now known as e0ne_ | 10:01 | |
| *** naohirot has quit IRC | 10:01 | |
| *** achanda has quit IRC | 10:04 | |
| *** e0ne_ has quit IRC | 10:07 | |
| *** achanda has joined #openstack-ironic | 10:07 | |
| *** boris-42 has joined #openstack-ironic | 10:09 | |
| *** vdrok has joined #openstack-ironic | 10:09 | |
| vdrok | morning ironic | 10:10 |
| *** achanda has quit IRC | 10:13 | |
| *** dtantsur|brb is now known as dtantsur | 10:16 | |
| dtantsur | sambetts, vdrok, morning! | 10:17 |
| vdrok | morning dtantsur | 10:19 |
| dtantsur | yuikotakada, did I answer your question on https://review.openstack.org/#/c/189310/ ? | 10:20 |
| sambetts | morning dtantsur | 10:20 |
| yuikotakada | dtantsur, yeah, perfectly! Thanks alot. I feel sorry/shameful for teaching me how-to-use-assertIs... | 10:22 |
| dtantsur | yuikotakada, no problem at all, we all learned it one day (and I was learning a lot when I stated with Ironic) :) | 10:23 |
| dtantsur | nobody is born with knowledge of unit testing :D | 10:23 |
| *** wuhg has quit IRC | 10:24 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add boot interface in Ironic https://review.openstack.org/166512 | 10:24 |
| Haomeng|2 | saripurigopi: ping | 10:24 |
| *** Haomeng|2 is now known as Haomeng | 10:24 | |
| saripurigopi | Haomeng|2 : could you take a look at https://review.openstack.org/#/c/159734/ | 10:25 |
| Haomeng | saripurigopi: sure | 10:26 |
| saripurigopi | Haomeng: Thank you. | 10:26 |
| Haomeng | saripurigopi: you are welcome:) | 10:27 |
| *** romainh has quit IRC | 10:27 | |
| *** e0ne has joined #openstack-ironic | 10:35 | |
| *** athomas has quit IRC | 10:37 | |
| *** derekh has joined #openstack-ironic | 10:38 | |
| *** yog__ has quit IRC | 10:39 | |
| *** subscope has joined #openstack-ironic | 10:41 | |
| *** athomas has joined #openstack-ironic | 10:43 | |
| *** amotoki_ has joined #openstack-ironic | 10:45 | |
| *** coolsvap is now known as coolsvap|afk | 10:45 | |
| yuikotakada | dtantsur: thank you for your kindness. And also everytime I am suprised at your knowledge | 10:45 |
| dtantsur | oh thanks, I appreciate that :) thank you for asking good questions as well, that's very important | 10:49 |
| *** amotoki_ has quit IRC | 10:50 | |
| bethelwell | Good morning guys! I have some spare time today so if there are any FE code reviews you want me to have a look at I am more than happy to be pinged :) | 10:50 |
| *** amotoki has quit IRC | 10:54 | |
| rameshg87 | yuriyz: hi | 10:58 |
| yuriyz | hi rameshg87 Ironic | 10:59 |
| rameshg87 | yuriyz: I was just checking your comment regarding https://review.openstack.org/#/c/185667/5/ironic/drivers/modules/agent_base_vendor.py | 10:59 |
| *** absubram has joined #openstack-ironic | 10:59 | |
| rameshg87 | yuriyz: if get_power_state() fails repeatedly with exception, then @retrying.retry() will raise the same exception that get_power_state() raised instead of retrying.RetryError | 11:00 |
| rameshg87 | yuriyz: so in effect, we would have tried for timeout seconds | 11:00 |
| *** spandhe has joined #openstack-ironic | 11:01 | |
| yuriyz | rameshg87 there can be immediate exception from _client.power_off(node) also | 11:02 |
| openstackgerrit | Anusha Ramineni proposed stackforge/proliantutils: Activate iLO license https://review.openstack.org/190030 | 11:02 |
| *** spandhe_ has joined #openstack-ironic | 11:02 | |
| yuriyz | rameshg87 but you can leave as is it's good | 11:03 |
| *** ramineni has quit IRC | 11:04 | |
| *** dlpartain has joined #openstack-ironic | 11:06 | |
| *** spandhe has quit IRC | 11:06 | |
| *** spandhe_ is now known as spandhe | 11:06 | |
| rameshg87 | yuriyz: yeah, it can raise exception from older agent ramdisk where command itself is not available | 11:08 |
| *** yuanying has joined #openstack-ironic | 11:08 | |
| *** Haomeng|2 has joined #openstack-ironic | 11:08 | |
| rameshg87 | yuriyz: we can add 2 more except blocks, but I was just thinking if it's necessary as it conveys the information that we failed to "soft power off" the machine | 11:08 |
| rameshg87 | yuriyz: if you think it looks good as it is now, I would rather like to leave it like this | 11:09 |
| *** alexpilotti has joined #openstack-ironic | 11:10 | |
| *** Haomeng has quit IRC | 11:11 | |
| yuriyz | rameshg87 yes I leave a comment | 11:11 |
| *** amotoki has joined #openstack-ironic | 11:12 | |
| *** yuanying has quit IRC | 11:12 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Allow vendor methods to serve static files https://review.openstack.org/189716 | 11:12 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Change return value of [driver_]vendor_passthru to dict https://review.openstack.org/190120 | 11:12 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Refactor node's and driver's vendor passthru to a common place https://review.openstack.org/190121 | 11:12 |
| *** stendulker has quit IRC | 11:14 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: Sync with latest oslo-incubator https://review.openstack.org/189110 | 11:15 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: Sync with latest oslo-incubator https://review.openstack.org/189110 | 11:15 |
| *** yuikotakada is now known as yuikotak_dinner | 11:15 | |
| rameshg87 | yuriyz: thanks .. | 11:17 |
| *** lucasagomes is now known as lucas-hungry | 11:18 | |
| * rameshg87 goes home | 11:18 | |
| *** rameshg87 has quit IRC | 11:18 | |
| openstackgerrit | Sam Betts proposed stackforge/ironic-discoverd: Provide hook into process when a node isn't found https://review.openstack.org/190124 | 11:23 |
| *** amotoki has quit IRC | 11:25 | |
| *** naohirot has joined #openstack-ironic | 11:28 | |
| *** thrash|g0ne is now known as thrash | 11:31 | |
| *** trown|outttypeww is now known as trown | 11:31 | |
| *** saripurigopi has quit IRC | 11:32 | |
| *** Nisha has joined #openstack-ironic | 11:32 | |
| *** Nisha has quit IRC | 11:36 | |
| *** saripurigopi has joined #openstack-ironic | 11:37 | |
| *** dlpartain has left #openstack-ironic | 11:44 | |
| *** jcoufal has quit IRC | 11:44 | |
| *** heyongli- has joined #openstack-ironic | 11:49 | |
| TheJulia | good morning everyone | 11:51 |
| openstackgerrit | Merged stackforge/ironic-discoverd: Add node() and ports() to NodeInfo https://review.openstack.org/189310 | 11:53 |
| openstackgerrit | Merged stackforge/ironic-discoverd: Naming clean up https://review.openstack.org/189329 | 11:53 |
| *** Haomeng has joined #openstack-ironic | 11:54 | |
| dtantsur | sambetts, trown, this ^^^ could give you some merge conflicts, sorry | 11:54 |
| dtantsur | TheJulia, morning! | 11:54 |
| trown | dtantsur: no problem, and good ugt morning | 11:55 |
| dtantsur | trown, morning! | 11:55 |
| *** Haomeng|2 has quit IRC | 11:57 | |
| *** dlpartain has joined #openstack-ironic | 11:59 | |
| *** dprince has joined #openstack-ironic | 11:59 | |
| *** kalpase has left #openstack-ironic | 12:00 | |
| *** romainh has joined #openstack-ironic | 12:00 | |
| *** jcoufal has joined #openstack-ironic | 12:01 | |
| *** saripurigopi has quit IRC | 12:02 | |
| *** pradipta has quit IRC | 12:02 | |
| *** subscope has quit IRC | 12:05 | |
| *** dlpartain has quit IRC | 12:14 | |
| *** dlpartain has joined #openstack-ironic | 12:15 | |
| *** subscope has joined #openstack-ironic | 12:20 | |
| openstackgerrit | Merged openstack/bifrost: Update test-birfrost to limit log lines https://review.openstack.org/187703 | 12:23 |
| *** bradjones has quit IRC | 12:26 | |
| *** bradjones has joined #openstack-ironic | 12:27 | |
| *** bradjones has quit IRC | 12:27 | |
| *** bradjones has joined #openstack-ironic | 12:27 | |
| *** dlpartain has quit IRC | 12:27 | |
| *** chlong has joined #openstack-ironic | 12:29 | |
| *** amotoki has joined #openstack-ironic | 12:30 | |
| sambetts | dtantsur: Not a problem, I'm no where near finished anyway :) | 12:44 |
| *** e0ne is now known as e0ne_ | 12:47 | |
| *** dlpartain has joined #openstack-ironic | 12:47 | |
| *** e0ne_ is now known as e0ne | 12:50 | |
| *** jcoufal has quit IRC | 12:51 | |
| *** jcoufal has joined #openstack-ironic | 12:52 | |
| *** jcoufal is now known as Guest62178 | 12:52 | |
| *** lucas-hungry is now known as lucasagomes | 12:53 | |
| *** Guest62178 has quit IRC | 12:55 | |
| *** jjohnson2 has joined #openstack-ironic | 12:58 | |
| openstackgerrit | Merged openstack/ironic-specs: Bare Metal Trust Using Intel TXT https://review.openstack.org/133902 | 13:00 |
| *** dlpartain has left #openstack-ironic | 13:02 | |
| *** yuikotakada has joined #openstack-ironic | 13:04 | |
| *** saripurigopi has joined #openstack-ironic | 13:04 | |
| *** cdearborn has joined #openstack-ironic | 13:11 | |
| *** rloo has joined #openstack-ironic | 13:16 | |
| *** jcoufal_ has joined #openstack-ironic | 13:19 | |
| NobodyCam | good morning Ironicers | 13:24 |
| dtantsur | NobodyCam, morning! | 13:24 |
| NobodyCam | morning dtantsur :) | 13:25 |
| dtantsur | folks, after discussing dynamic iPXE spec with Lucas, I believe we do need API passthru for interfaces | 13:25 |
| dtantsur | at the summit we agreed that we'd better promote existing misuses of vendor passthru by deploy interface | 13:26 |
| dtantsur | but looks like we're introducing new ones faster than we're promoting anything | 13:26 |
| dtantsur | without interface passthru we won't be able to decouple boot+deploy from vendor passthru any time soon. | 13:26 |
| dtantsur | lucasagomes, NobodyCam, devananda, thoughts on ^^^ ? | 13:27 |
| lucasagomes | NobodyCam, morning | 13:27 |
| dtantsur | jroll, morning, continuing passthru discussion ^^^ | 13:28 |
| *** jgrimm is now known as zz_jgrimm | 13:28 | |
| jroll | morning dtantsur lucasagomes NobodyCam etc :) | 13:28 |
| lucasagomes | dtantsur, we need to first split boot+deploy, then deprecate the DIB ramdisk (which might take 1 cycle more) and promote heartbeat() lookup() | 13:28 |
| lucasagomes | we still need to keep the heartbeat() and lookup() on the vendor passthru deprecated to have compat with old ramdisks | 13:29 |
| lucasagomes | dtantsur, I don't think we are decoupling vendor passthru that soon :-( | 13:29 |
| jroll | dtantsur: so you suggest any implementation of a given interface should be able to expose some vendor passthru method, yes? | 13:29 |
| dtantsur | lucasagomes, I no longer see point in promoting heartbeat and lookup any more... | 13:29 |
| dtantsur | jroll, yes | 13:29 |
| lucasagomes | + extending interfaces to be able to expose vendor passthru sounds simple, we can do it this cycle | 13:29 |
| lucasagomes | dtantsur, why not? | 13:30 |
| jroll | dtantsur: I think that works today, any method can be decorated | 13:30 |
| lucasagomes | jroll, yeah, well we need to add the metaclass to the other drivers class | 13:30 |
| lucasagomes | so it will map those methods | 13:30 |
| lucasagomes | but yeah it's simple | 13:30 |
| lucasagomes | can be done | 13:30 |
| dtantsur | lucasagomes, these are driver specifics. even despite we'll have 1 deploy drive soon | 13:30 |
| lucasagomes | dtantsur, we won't have 1 deploy driver | 13:30 |
| jroll | lucasagomes: yeah, good point | 13:30 |
| dtantsur | then we can't promote heartbeat and lookup... | 13:31 |
| lucasagomes | we probably will have many, iscsi, agent, (boot from volume ?) | 13:31 |
| lucasagomes | dtantsur, we can because both deploy methods uses such endpoints, they are both done using IPA ramdisk | 13:31 |
| lucasagomes | so all drivers so far will be using the endpoints, but they are different deploy methodologies | 13:31 |
| NobodyCam | morning lucasagomes and jroll | 13:32 |
| dtantsur | lucasagomes, again, that's driver specifics. if we have non-agent driver, well probably need different ones | 13:32 |
| *** jcoufal_ has quit IRC | 13:32 | |
| *** teju has joined #openstack-ironic | 13:32 | |
| lucasagomes | dtantsur, right, but the idea is support *only* IPA as the deploy ramdisk right? at least upstream | 13:32 |
| lucasagomes | so they all will use heartbeat and lookup | 13:32 |
| openstackgerrit | Merged openstack/ironic-specs: iPXE dynamic configuration https://review.openstack.org/177726 | 13:33 |
| jroll | ^^ woot | 13:33 |
| *** jcoufal_ has joined #openstack-ironic | 13:33 | |
| NobodyCam | :) | 13:33 |
| dtantsur | lucasagomes, ok I agree. but for your iPXE stuff we still need interface passthru... I'll probably make it part of my driver composition reform spec, wdyt? | 13:33 |
| lucasagomes | dtantsur, yeah for that it still needed. The way we can solve it is by allows other interfaces to create their own passthru methods | 13:34 |
| lucasagomes | like jroll pointed out | 13:34 |
| jroll | dtantsur: seems fine to put it on that spec | 13:34 |
| dtantsur | cool | 13:34 |
| lucasagomes | dtantsur, seems good to put in the spec, tho it can be done separetly | 13:34 |
| jroll | dtantsur: although I'd say we could do it before that work, it isn't a lot of work | 13:34 |
| lucasagomes | I think there's a benefit in doing it by itself, cause we will get rid of those horrible VendorMixIn interface with static mappings | 13:35 |
| dtantsur | hmm... maybe then put it together with API promotion. anyone working on it? jroll, lucasagomes? | 13:35 |
| lucasagomes | dtantsur, haven't started, but yeah I was planning on doing some of it | 13:35 |
| jroll | I would just do a real short spec separately for it, I think | 13:36 |
| *** jjohnson2_ has joined #openstack-ironic | 13:36 | |
| jroll | the only sections you'd need is proposed change and driver api impact, which will both be small | 13:36 |
| jroll | we already have the methods to do this, it can't possibly get contentious | 13:37 |
| dtantsur | jroll, wanna do it? ;) | 13:37 |
| dtantsur | or should I? | 13:37 |
| jroll | dtantsur: go for it, I've got a ton on my plate right now :/ | 13:37 |
| *** vishwanathj has joined #openstack-ironic | 13:37 | |
| dtantsur | ack | 13:37 |
| jroll | happy to review though! | 13:38 |
| *** jjohnson2 has quit IRC | 13:38 | |
| *** vishwana_ has joined #openstack-ironic | 13:39 | |
| *** vishwanathj has quit IRC | 13:42 | |
| NobodyCam | TheJulia: question on: https://review.openstack.org/#/c/188959 looks like the py stuff was removed from tox.ini but is still in setup.cfg. was this the intent? | 13:46 |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic: Add pxe_ucs and agent_ucs drivers to manage Cisco UCS servers https://review.openstack.org/159734 | 13:47 |
| *** gridinv_ has joined #openstack-ironic | 13:48 | |
| *** jcoufal_ has quit IRC | 13:49 | |
| *** e0ne is now known as e0ne_ | 13:51 | |
| TheJulia | NobodyCam: ahh, i see under the metadata. Making note | 13:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: Add "enroll" state to the state machine https://review.openstack.org/179151 | 13:52 |
| dtantsur | lucasagomes, jroll, may I get your attention to this one as well ^^? | 13:52 |
| jroll | dtantsur: shore thing | 13:53 |
| rloo | dtantsur: isn't that enroll spec approved yet?!! | 13:55 |
| lucasagomes | ack | 13:55 |
| rloo | hello ironic-ers :) | 13:55 |
| *** igordcard_ has quit IRC | 13:56 | |
| *** igordcard_ has joined #openstack-ironic | 13:57 | |
| *** gridinv_ has quit IRC | 13:57 | |
| cdearborn | Morning ironic! | 13:57 |
| cdearborn | Morning rloo! | 13:58 |
| rloo | morning cdearborn! | 13:58 |
| cdearborn | I finally got a chance to dig back into that "object is not iterable" error that a couple of us were seeing when running the tests under py34 | 13:58 |
| cdearborn | (I was seeing this on Fedora 21) | 13:59 |
| NobodyCam | morning rloo, and cdearborn | 13:59 |
| cdearborn | This is due to a bug in mock. There is an existing bug for it: http://bugs.python.org/issue23661 | 13:59 |
| *** e0ne_ is now known as e0ne | 13:59 | |
| rloo | morning NobodyCam | 14:00 |
| NobodyCam | :) | 14:01 |
| cdearborn | The bug pops up when you have a test that does "@mock.patch.object(..., autospec=True, ...) and side_effect = some exception on that mock | 14:01 |
| cdearborn | There are 68 instances of this in the ironic repo | 14:01 |
| cdearborn | a work around is to do this: xxx.side_effect = [ some exception ] (make the side_effect iterable) | 14:02 |
| cdearborn | I've hacked my py34 install with the patch to work around this for now | 14:02 |
| cdearborn | but was wondering if we should consider implementing the work around | 14:03 |
| cdearborn | I'm thinking that other distros will start picking up the bug | 14:03 |
| cdearborn | and more people will start seeing this | 14:03 |
| cdearborn | thoughts? | 14:03 |
| *** subscope has quit IRC | 14:04 | |
| cdearborn | An example stack trace: http://paste.openstack.org/show/281394/ | 14:04 |
| cdearborn | The test that causes it: http://paste.openstack.org/show/281395/ | 14:04 |
| cdearborn | Morning NobodyCam! | 14:05 |
| jroll | cdearborn: I think we should poke some python cores, there's a patch up for that bug :P | 14:05 |
| jroll | cdearborn: but until then I have no issues with hacking it in our repo | 14:05 |
| cdearborn | yeah - there is a patch attached to the bug. I'm not sure how long it will take to be accepted tho, and how long it will be before the distros start picking up the fix | 14:06 |
| jroll | indeed, we should probably hack around it | 14:07 |
| cdearborn | k - I'm happy to take that work since I'm in a state where it's easily reproducible - will create a bug for it | 14:08 |
| jroll | awesome, ty! | 14:08 |
| cdearborn | np, glad to help! | 14:08 |
| lucasagomes | dtantsur, just saw your question re writing a spec to allow other interfaces to have passthru methods | 14:08 |
| lucasagomes | dtantsur, I'm happy to write one if you're not on top of it already | 14:08 |
| * lucasagomes is currently refactoring some of the passthru stuff to facilidate the iPXE work | 14:09 | |
| dtantsur | lucasagomes, please do, I have enroll and driver reform specs to bother about. thanks | 14:09 |
| rloo | cdearborn: some of our tests already have that .side_effect = iter([...]) pattern. I thought/think there may be another bug that caused jlvillal to do that to the tests. | 14:09 |
| dtantsur | morning, rloo, cdearborn | 14:09 |
| cdearborn | dtantsur, morning! | 14:10 |
| lucasagomes | dtantsur, ack | 14:10 |
| rloo | hi dtantsur :) | 14:10 |
| *** yuikotakada has quit IRC | 14:11 | |
| *** degorenko has quit IRC | 14:12 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Refactor node's and driver's vendor passthru to a common place https://review.openstack.org/190121 | 14:13 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Allow vendor methods to serve static files https://review.openstack.org/189716 | 14:13 |
| rloo | hi lucasagomes, if you get a chance, I'm not sure what you'd like me to do/change (abandon it?) for https://review.openstack.org/#/c/188188/ | 14:13 |
| lucasagomes | rloo, /me looks | 14:13 |
| lucasagomes | rloo, my only concern about this patch is that it seems to only happens due some programming error | 14:14 |
| lucasagomes | not sure we will see it in run time | 14:14 |
| rloo | lucasagomes: yes, i'm hoping that's true (only programming) | 14:14 |
| lucasagomes | rloo, so, logging it wouldn't actually make it more difficult to the developer to see the error? | 14:15 |
| lucasagomes | instead of exception.InvalidState | 14:15 |
| rloo | lucasagomes: if it happens, it'll get logged regardless, right? | 14:15 |
| lucasagomes | rloo, I bet it will get logging in the conductor yes | 14:16 |
| rloo | lucasagomes: the node.last_error will never get updated to reflect the deploy error | 14:16 |
| rloo | lucasagomes: i think it gets logged at the messaging layer cuz i didn't see anything in conductor handling this exception | 14:16 |
| rloo | lucasagomes: but regardless of where it gets logged, the node.last_error won't get updated. | 14:17 |
| rloo | lucasagomes: so this change logs it sooner and node.last_error gets updated. | 14:17 |
| *** amotoki has quit IRC | 14:17 | |
| rloo | lucasagomes: so I'm not sure what your concern is. | 14:17 |
| *** amotoki has joined #openstack-ironic | 14:17 | |
| lucasagomes | rloo, right so the idea is get the last_error updated | 14:17 |
| *** amotoki has quit IRC | 14:18 | |
| lucasagomes | rloo, I bet it's fine, it's just that when it's only a programmatic error I don't think I would care much about last_error | 14:18 |
| lucasagomes | rloo, I will change my vote there | 14:18 |
| *** degorenko has joined #openstack-ironic | 14:19 | |
| rloo | lucasagomes: true but what if we're wrong and it isn't a program error (something changes in the future, dunno) | 14:19 |
| *** amotoki has joined #openstack-ironic | 14:19 | |
| lucasagomes | rloo, actually I will try to simulate that error see what happens | 14:19 |
| lucasagomes | rloo, yeah, if we are wrong and it happens in a non-programmatic way that's def a nasty bug | 14:19 |
| lucasagomes | rloo, cause the node won't be in error state even tho it will has last_error set | 14:20 |
| *** amotoki has quit IRC | 14:20 | |
| *** amotoki has joined #openstack-ironic | 14:20 | |
| rloo | lucasagomes: right. but it might make it easier to debug if it happens. | 14:21 |
| *** mtanino has joined #openstack-ironic | 14:21 | |
| rloo | lucasagomes: I am fine abandoning it too. I don't want to spend too much time on this. | 14:21 |
| lucasagomes | rloo, I think it would worth investigate if it can happen in runtime, cause then it's a high prio bug. If we can't go from deploying to deployerror there's something wrong with our state machine | 14:22 |
| lucasagomes | the flow is incorrect | 14:22 |
| rloo | lucasagomes: it cannot happen in runtime now, cuz the state machine is correct. | 14:22 |
| lucasagomes | rloo, right, so I don't think we should catch that exception then | 14:23 |
| lucasagomes | I mean I'm ok doing it, but seems just like extra code | 14:23 |
| lucasagomes | if something is not inteded to fail I don't see why safe guarding it | 14:24 |
| cdearborn | for making the side_effect change, anyone have any thought on which pattern would be better? 1: xxx.side | 14:24 |
| cdearborn | xxx.side_effect = [ ex ] | 14:24 |
| cdearborn | or xxx.side_effect = iter( [ex]) | 14:24 |
| rloo | lucasagomes: the docstring shows raises InvalidState... | 14:24 |
| rloo | lucasagomes: if we don't think it should fail, should we mention that it might fail? | 14:25 |
| lucasagomes | :-( | 14:25 |
| jroll | cdearborn: I don't think it actually matters, really, both are iterable | 14:25 |
| * lucasagomes is divided | 14:25 | |
| lucasagomes | rloo, I don't think so :-/ | 14:25 |
| lucasagomes | rloo, anyway, I don't think it's a big deal. I'm happy changing my vote on that patch | 14:25 |
| jroll | lucasagomes: rloo: here's a case where it might fail: temporary database blip | 14:26 |
| jroll | process_event() calls save() iirc | 14:26 |
| jroll | (I may be wrong, but just throwing things out there) | 14:26 |
| lucasagomes | jroll, but it has a lock right? (I hope the saves()'s have a lock) | 14:26 |
| rloo | jroll: i think this exception would happen before the .save is called. but that's what i mean. we don't know what might happen wrt the code in the future. | 14:26 |
| jroll | lucasagomes: I don't think it matters, db could go away after lock is acquired | 14:27 |
| jroll | though my point is invalid, it wouldn't raise InvalidState :) | 14:28 |
| dtantsur | "if something is not inteded to fail I don't see why safe guarding it" I tend to agree... | 14:29 |
| jroll | poking around, but I'm not so sure set_failed_state is only called while it's in DEPLOYING | 14:31 |
| jroll | (unclear if that matters) | 14:31 |
| rloo | jroll: yeah, we don't know when set_failed_state may be called in the future. so it could be a programming error but not caught before it goes in production. | 14:32 |
| jroll | this is also called for any uncaught exception for agent heartbeats | 14:32 |
| jroll | which seems like a good thing to have in last_error, even though it's probably a programming bug | 14:32 |
| jroll | I guess I see this as exposing better info to the user | 14:33 |
| rloo | <how many things are programming bugs. none of course.> | 14:33 |
| jroll | which is never a bad thing | 14:33 |
| rloo | this conversation reminds me of the C days when people would put ASSERTS or whatever, that would cause the program to stop in production. their rationale for these asserts were to catch programming errors and they'd never actually fire in production. | 14:34 |
| *** yuikotakada has joined #openstack-ironic | 14:35 | |
| jroll | rloo: ha | 14:36 |
| *** zz_natorious is now known as natorious | 14:37 | |
| lucasagomes | rloo, lol | 14:38 |
| lucasagomes | rloo, jroll dtantsur so re #188188 I'm ok changing my vote there | 14:40 |
| lucasagomes | I'm +0 on it | 14:40 |
| jroll | heh, right on | 14:40 |
| rloo | thx lucasagomes | 14:41 |
| jroll | I +2'd if you want to land it | 14:41 |
| *** openstackgerrit has quit IRC | 14:41 | |
| rloo | jroll: I think lucasagomes means he isn't against it but he isn't for it | 14:41 |
| lucasagomes | yeah, I mean I can land it no problem | 14:41 |
| jroll | rloo: right, hence "if you want to" | 14:41 |
| jroll | :) | 14:41 |
| *** yuikotakada has quit IRC | 14:41 | |
| *** openstackgerrit has joined #openstack-ironic | 14:41 | |
| lucasagomes | I just find it odd that we we probably can fail to fial | 14:41 |
| lucasagomes | fail to fail* | 14:41 |
| rloo | jroll: gotcha | 14:41 |
| *** mgoddard1 has joined #openstack-ironic | 14:46 | |
| *** mgoddard has quit IRC | 14:47 | |
| lucasagomes | that --notest passed to tox is pretty cool | 14:49 |
| lucasagomes | indeed that echo never worked for me | 14:50 |
| lucasagomes | heh | 14:50 |
| jlvillal | lucasagomes, I wish I had thought of '--notest'. But someone else suggested it in a different patch that I abandoned | 14:50 |
| NobodyCam | morning jlvillal | 14:51 |
| lucasagomes | jlvillal, I see, well still thanks for updating the docs with it | 14:51 |
| jlvillal | lucasagomes, Vladyslav Drok was the person with the smart idea :) vdrok | 14:51 |
| lucasagomes | vdrok, w00t | 14:51 |
| * jlvillal was going to add 'echo' to the whitelist of commands. | 14:51 | |
| vdrok | jlvillal, lucasagomes :D | 14:51 |
| jlvillal | NobodyCam, Thanks | 14:52 |
| jlvillal | Good morning Ironic | 14:52 |
| NobodyCam | jlvillal: which doc did you update? | 14:52 |
| jroll | morning jlvillal and vdrok :) | 14:52 |
| jlvillal | NobodyCam, Developer quickstart | 14:52 |
| jlvillal | NobodyCam, https://review.openstack.org/189984 | 14:53 |
| NobodyCam | ahh TY | 14:53 |
| vdrok | morning jroll, jlvillal lucasagomes NobodyCam rloo | 14:53 |
| NobodyCam | :) morning vdrok | 14:53 |
| rloo | hiya vdrok and jlvillal | 14:54 |
| * jlvillal worries about a feedback loop if he replies ;) | 14:56 | |
| openstackgerrit | Sam Betts proposed stackforge/ironic-discoverd: Provide hook into process when a node isn't found https://review.openstack.org/190124 | 14:58 |
| *** [1]cdearborn has joined #openstack-ironic | 15:04 | |
| *** Nisha has joined #openstack-ironic | 15:06 | |
| *** natorious is now known as zz_natorious | 15:11 | |
| *** naohirot has quit IRC | 15:11 | |
| *** mikedillion has joined #openstack-ironic | 15:16 | |
| *** ifarkas has quit IRC | 15:18 | |
| *** mikedillion has quit IRC | 15:18 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: Driver composition reform https://review.openstack.org/188370 | 15:18 |
| *** cdearborn has quit IRC | 15:18 | |
| dtantsur | jlvillal, hey morning! | 15:19 |
| *** leopoldj has quit IRC | 15:26 | |
| dtantsur | anyone willing to have a look at RAID spec https://review.openstack.org/173214 before it gets approved? | 15:30 |
| dtantsur | oh, it can't be approved, it should be rebased... | 15:30 |
| dtantsur | meh | 15:30 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: Sync with latest oslo-incubator https://review.openstack.org/189110 | 15:31 |
| rloo | dtantsur: yeah, that raid spec depends on zapping :-( | 15:31 |
| dtantsur | right | 15:31 |
| dtantsur | JoshNang, planning to update the zapping spec? | 15:31 |
| rloo | dtantsur: I have wondered whether it might make more sense to put a 'depends-on' tag in the commit msg instead. | 15:32 |
| dtantsur | what will be the difference? we still have to land zapping before landing some zapping steps... | 15:33 |
| *** pradipta has joined #openstack-ironic | 15:33 | |
| rloo | dtantsur: i don't really know what 'rebasing' means in this context. | 15:33 |
| *** lazy_prince has quit IRC | 15:33 | |
| rloo | dtantsur: and the patch could be approved. it just won't merge until zapping is merged. right now, can't really approve. | 15:33 |
| dtantsur | rloo, actually 2 issues here: 1. raid spec is based on outdated patchset of zapping (spec), thuis needs rebasing; 2. zapping spec has -1 | 15:34 |
| *** lucasagomes has quit IRC | 15:39 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Add cookiecutter-generated files for project https://review.openstack.org/188959 | 15:40 |
| *** Sukhdev has joined #openstack-ironic | 15:46 | |
| JoshNang | dtantsur: yes! i will try to get to it today | 15:49 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Wake-On-Lan Power interface https://review.openstack.org/179078 | 15:49 |
| dtantsur | cool! | 15:49 |
| *** zz_jgrimm is now known as jgrimm | 15:52 | |
| *** lucasagomes has joined #openstack-ironic | 15:53 | |
| * lucasagomes connection flicked | 15:56 | |
| lucasagomes | jroll, any updates on the new release model? Did we sort it out how the versioning will work ? | 15:56 |
| jroll | lucasagomes: I've been trying to get back and update it but keep getting distracted | 15:57 |
| jroll | as far as versioning... I guess all other projects are moving to 11.0, 12.0 etc | 15:58 |
| jroll | so I need to talk to ttx or lifeless and figure out how that's going to work | 15:58 |
| lucasagomes | jroll, no problem about it, it looks good? ttx is ok with us doing it and all? | 15:58 |
| lucasagomes | gotcha | 15:58 |
| lucasagomes | a looks good. looking fwd to have a new release model | 15:58 |
| jroll | lucasagomes: yeah, people seem to be cool with it | 15:59 |
| jroll | me too | 15:59 |
| jroll | hoping to update the spec today or tomorrow | 15:59 |
| *** gridinv_ has joined #openstack-ironic | 15:59 | |
| * rloo shudders at the thought of a new release model | 16:02 | |
| *** gridinv_ has quit IRC | 16:04 | |
| * devananda waves from Berlin | 16:06 | |
| NobodyCam | morning devananda | 16:07 |
| devananda | good evening, NobodyCam :) | 16:07 |
| NobodyCam | :) | 16:07 |
| lucasagomes | devananda, hey there, morning | 16:07 |
| *** saripurigopi has quit IRC | 16:07 | |
| lucasagomes | devananda, you should be at a beer garden now :-) | 16:07 |
| jroll | has anyone thought about software raid + full disk images at all? | 16:07 |
| devananda | re: release models & version numbers, yea, every project is going to move to some other numbers, based on when that project was first released | 16:07 |
| BadCub | hiya devananda | 16:07 |
| NobodyCam | how was the flight | 16:08 |
| devananda | so nova might be 12.0, and ironic might be 4.0, or something | 16:08 |
| jroll | lucasagomes: curious if you have thoughts on my question there | 16:08 |
| devananda | that's unrelated to whether we release on-demand or synchronized, fwiw | 16:08 |
| jroll | devananda: yeah, I'm wondering how pbr/pip/etc react to a thing going from 2015.x to 11.x | 16:08 |
| devananda | NobodyCam: too many screaming babies. couldn't sleep at all | 16:09 |
| lucasagomes | jroll, hmm we can create an image with LVM right? | 16:09 |
| BadCub | ack | 16:09 |
| devananda | jroll: yea, let the release mgrs sort that out :) | 16:09 |
| lucasagomes | I think atomic images use LVM by default | 16:09 |
| NobodyCam | :( ugggh :( | 16:09 |
| dtantsur | devananda, morning! | 16:09 |
| jroll | devananda: easy enough :) | 16:09 |
| devananda | jroll: I think they will use "epochs" | 16:09 |
| devananda | jroll: but once that's done, we /could/ start releasing more often. we don't /have/ to, though | 16:09 |
| BadCub | brb | 16:09 |
| jroll | lucasagomes: yeah, so an lvm with multiple partitions or whatever, and lay that down? | 16:10 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Rework processing hook interface for 2.0.0 https://review.openstack.org/189346 | 16:10 |
| lucasagomes | jroll, yeah... lemme investigate the atomic | 16:10 |
| lucasagomes | jroll, but that said... btrfs is ur friend | 16:10 |
| lucasagomes | btrfs with subvolumes rocks, no more partition tables | 16:11 |
| jroll | lucasagomes: my ideal goal is to be able to do raid and non-raid with the same full disk image | 16:11 |
| jroll | heh yeah | 16:11 |
| mariojv | yeah, if the volume the image is put on is configured to be raid - that can work i think | 16:11 |
| * jroll wonders how customers would feel about btrfs being forced on them :P | 16:11 | |
| * dtantsur goes home | 16:11 | |
| lucasagomes | jroll, yeah it's too early still | 16:12 |
| dtantsur | g'night! | 16:12 |
| lucasagomes | dtantsur, night | 16:12 |
| *** dtantsur is now known as dtantsur|afk | 16:12 | |
| jroll | night dtantsur|afk | 16:12 |
| jroll | lucasagomes: anyway, this is just something we've been kicking around, thought you might have some ideas on it... lvm might be a path forward | 16:13 |
| jroll | need to investigate more | 16:13 |
| lucasagomes | jroll, yeah I think even the pos config could update the raid configuration | 16:13 |
| jroll | haven't used it in depth in some time | 16:13 |
| lucasagomes | so you can use the same image for raid and non raid | 16:13 |
| jroll | pos config? | 16:13 |
| lucasagomes | jroll, after the image is copied onto the disk | 16:13 |
| lucasagomes | kinda like installing the bootloader | 16:13 |
| * jlvillal assumes jroll means Point Of Sale :P | 16:14 | |
| lucasagomes | where I copy it first and then I mount it and install | 16:14 |
| jroll | right | 16:14 |
| lucasagomes | you probably can lay the image down and then configure the lvm accordingly | 16:14 |
| *** jistr has quit IRC | 16:14 | |
| devananda | I'm around another 20 minutes, maybe. then gotta run to dinner plans. anything anyone needs now() ? | 16:14 |
| *** Nisha has quit IRC | 16:15 | |
| * jlvillal hunkers down to figure out Oauth2... | 16:15 | |
| jroll | lucasagomes: hrm, I'll investigate | 16:15 |
| lucasagomes | jroll, ack +1 I think it's a good thing to have | 16:16 |
| lucasagomes | jroll, btw, now that we are also splitting the partition bits in the ironic-libs | 16:16 |
| lucasagomes | jroll, it might also worth to investigate whether partitioning and creating lvm before laying down the image is something we might want to do | 16:17 |
| lucasagomes | so you don't need full disk images (tho you guys already use full disk images) | 16:17 |
| lucasagomes | anyway... it's good stuff :-) | 16:17 |
| devananda | lucasagomes: who's releasing ironic-lib packages (or tagging pip versions) ? | 16:17 |
| jroll | lucasagomes: right, "already use full disk images" is a big thing :P | 16:17 |
| jroll | but yeah, really both should work upstream | 16:17 |
| lucasagomes | devananda, I thought you were ? But it's not at the point to do any release yet | 16:18 |
| lucasagomes | devananda, actually it's just starting. We still need to put all the code that can be used by Ironic and other projects (like IPA) there | 16:18 |
| lucasagomes | devananda, I'm happy to keep an eye on it if needed and tag releases | 16:18 |
| lucasagomes | jroll, :-) true | 16:19 |
| devananda | lucasagomes: ah ok. I've been keeping a light eye on it, but yea, it didn't seem nearly ready for a realease yet | 16:19 |
| lucasagomes | yeah it's not | 16:19 |
| *** mgoddard1 has quit IRC | 16:19 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add pluggable credentials storage https://review.openstack.org/186056 | 16:20 |
| *** mgoddard has joined #openstack-ironic | 16:21 | |
| *** saripurigopi has joined #openstack-ironic | 16:24 | |
| *** teju has left #openstack-ironic | 16:24 | |
| *** e0ne is now known as e0ne_ | 16:25 | |
| *** rloo has quit IRC | 16:25 | |
| *** e0ne_ is now known as e0ne | 16:25 | |
| *** lauramoore has joined #openstack-ironic | 16:28 | |
| *** saripurigopi has quit IRC | 16:28 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add pluggable credentials storage https://review.openstack.org/186056 | 16:29 |
| *** Nisha has joined #openstack-ironic | 16:31 | |
| lauramoore | hey Sukhdev | 16:32 |
| *** spandhe has quit IRC | 16:32 | |
| Sukhdev | lauramoore: Hi | 16:32 |
| lauramoore | Are you in the ML2 meeting yet? | 16:33 |
| Sukhdev | lauramoore: did you get my message on the other channel? Yes, in ML2 meeting | 16:34 |
| *** ukalifon1 has quit IRC | 16:36 | |
| *** trown is now known as trown|lunch | 16:46 | |
| *** romainh has left #openstack-ironic | 16:48 | |
| *** bradbeam_ has quit IRC | 16:48 | |
| openstackgerrit | Clif Houck proposed openstack/ironic: Add support for image caching for the agent driver https://review.openstack.org/161832 | 16:55 |
| *** dguerri is now known as dguerri` | 16:56 | |
| *** derekh has quit IRC | 17:04 | |
| *** spandhe has joined #openstack-ironic | 17:06 | |
| cinerama | hi ironic party people | 17:06 |
| openstackgerrit | Ben Nemec proposed openstack/python-ironicclient: Register global Keystone args first https://review.openstack.org/189952 | 17:06 |
| lucasagomes | folks I'm going to call it a day | 17:06 |
| lucasagomes | have a great night everyone! | 17:07 |
| *** achanda has joined #openstack-ironic | 17:07 | |
| *** lucasagomes is now known as lucas-dinner | 17:07 | |
| NobodyCam | night lucas-dinner | 17:10 |
| NobodyCam | have a good one | 17:11 |
| *** e0ne has quit IRC | 17:11 | |
| *** achanda has quit IRC | 17:15 | |
| *** rloo has joined #openstack-ironic | 17:21 | |
| *** pradipta has quit IRC | 17:24 | |
| *** pelix has quit IRC | 17:31 | |
| *** achanda has joined #openstack-ironic | 17:33 | |
| *** saripurigopi has joined #openstack-ironic | 17:35 | |
| *** Haomeng|2 has joined #openstack-ironic | 17:37 | |
| *** krtaylor has quit IRC | 17:37 | |
| *** Haomeng has quit IRC | 17:40 | |
| *** bethelwell has quit IRC | 17:41 | |
| *** edcragg has quit IRC | 17:41 | |
| *** achanda_ has joined #openstack-ironic | 17:43 | |
| *** achanda has quit IRC | 17:45 | |
| *** Sukhdev has quit IRC | 17:46 | |
| *** Sukhdev has joined #openstack-ironic | 17:46 | |
| openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Add record id to SEL data https://review.openstack.org/190275 | 17:46 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Improve IPA logging https://review.openstack.org/190279 | 17:50 |
| *** dguerri` is now known as dguerri | 17:51 | |
| *** lauramoore has quit IRC | 17:53 | |
| *** harlowja has quit IRC | 17:58 | |
| *** krtaylor has joined #openstack-ironic | 17:59 | |
| *** dguerri is now known as dguerri` | 18:00 | |
| *** yuikotakada has joined #openstack-ironic | 18:00 | |
| *** harlowja has joined #openstack-ironic | 18:02 | |
| *** [1]cdearborn has quit IRC | 18:02 | |
| *** bethelwell has joined #openstack-ironic | 18:03 | |
| *** dguerri` is now known as dguerri | 18:05 | |
| *** saripurigopi has quit IRC | 18:11 | |
| *** achanda_ has quit IRC | 18:16 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Improve IPA logging https://review.openstack.org/190279 | 18:20 |
| *** zz_natorious is now known as natorious | 18:21 | |
| *** Sukhdev has quit IRC | 18:22 | |
| *** ijw has joined #openstack-ironic | 18:22 | |
| *** bradjones has quit IRC | 18:23 | |
| *** bradjones has joined #openstack-ironic | 18:24 | |
| *** bradjones has quit IRC | 18:24 | |
| *** bradjones has joined #openstack-ironic | 18:24 | |
| *** Nisha has quit IRC | 18:27 | |
| *** Nisha_away has joined #openstack-ironic | 18:27 | |
| *** amotoki has quit IRC | 18:28 | |
| *** dprince has quit IRC | 18:31 | |
| *** trown|lunch is now known as trown | 18:33 | |
| *** achanda has joined #openstack-ironic | 18:35 | |
| *** Nisha_away has quit IRC | 18:37 | |
| openstackgerrit | Merged openstack/ironic: Doc: Use --notest for creating venv https://review.openstack.org/189984 | 18:42 |
| *** yuikotakada has quit IRC | 18:42 | |
| openstackgerrit | Merged openstack/ironic: Remove auth token saving from iLO driver https://review.openstack.org/187924 | 18:43 |
| *** e0ne has joined #openstack-ironic | 18:52 | |
| openstackgerrit | Merged openstack/bifrost: Add cookiecutter-generated files for project https://review.openstack.org/188959 | 18:56 |
| *** e0ne has quit IRC | 18:57 | |
| *** e0ne has joined #openstack-ironic | 19:00 | |
| *** achanda has quit IRC | 19:02 | |
| *** achanda has joined #openstack-ironic | 19:03 | |
| *** e0ne has quit IRC | 19:05 | |
| *** edcragg has joined #openstack-ironic | 19:08 | |
| *** Sukhdev has joined #openstack-ironic | 19:14 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial inventory module https://review.openstack.org/184084 | 19:15 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial dynamic enrollment role https://review.openstack.org/188176 | 19:17 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Enable agent_ipmitool driver when testing https://review.openstack.org/188623 | 19:18 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Addition of dynamic configuration drives https://review.openstack.org/188624 | 19:18 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add initial dynamic deployment role https://review.openstack.org/188773 | 19:18 |
| *** achanda has quit IRC | 19:21 | |
| *** amotoki has joined #openstack-ironic | 19:29 | |
| *** amotoki has quit IRC | 19:34 | |
| *** achanda has joined #openstack-ironic | 19:34 | |
| *** bethelwell has quit IRC | 19:45 | |
| *** athomas has quit IRC | 19:52 | |
| *** achanda has quit IRC | 19:55 | |
| *** achanda has joined #openstack-ironic | 19:56 | |
| *** e0ne has joined #openstack-ironic | 20:00 | |
| *** achanda has quit IRC | 20:00 | |
| *** jjohnson2_ has quit IRC | 20:07 | |
| *** bethelwell has joined #openstack-ironic | 20:07 | |
| *** bethelwell has quit IRC | 20:18 | |
| *** e0ne has quit IRC | 20:35 | |
| *** hemna is now known as hemnafk | 20:38 | |
| openstackgerrit | Merged stackforge/pyghmi: Add record id to SEL data https://review.openstack.org/190275 | 20:38 |
| *** dguerri is now known as dguerri` | 20:46 | |
| *** dguerri` is now known as dguerri | 20:52 | |
| openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: Rename edeploy plugin https://review.openstack.org/189944 | 20:53 |
| openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: [WIP] Fix edeploy plugin puts too much data in Ironic extra column https://review.openstack.org/189427 | 20:53 |
| openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: Rename edeploy plugin https://review.openstack.org/189944 | 20:57 |
| openstackgerrit | John Trowbridge proposed stackforge/ironic-discoverd: [WIP] Fix edeploy plugin puts too much data in Ironic extra column https://review.openstack.org/189427 | 20:57 |
| *** spandhe has quit IRC | 21:00 | |
| *** trown is now known as trown|outttypeww | 21:02 | |
| *** [1]cdearborn has joined #openstack-ironic | 21:09 | |
| *** Sukhdev has quit IRC | 21:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/190357 | 21:21 |
| *** ijw has quit IRC | 21:22 | |
| NobodyCam | quite today | 21:22 |
| *** v92 has joined #openstack-ironic | 21:26 | |
| TheJulia | quiet is not a bad thing | 21:30 |
| *** amotoki has joined #openstack-ironic | 21:31 | |
| *** bnemec has quit IRC | 21:31 | |
| *** [1]cdearborn has quit IRC | 21:33 | |
| *** amotoki has quit IRC | 21:36 | |
| *** dguerri is now known as dguerri` | 21:42 | |
| *** [1]cdearborn has joined #openstack-ironic | 21:43 | |
| mrda | Morning Ironic | 21:47 |
| NobodyCam | mornign mrda | 21:48 |
| mrda | hey NobodyCam | 21:51 |
| NobodyCam | me | 21:55 |
| NobodyCam | :-p | 21:55 |
| *** spandhe has joined #openstack-ironic | 21:56 | |
| *** lsmola has quit IRC | 22:01 | |
| mrda | oh, I'm looking for another +2 and a +A on a micro-spec "Add ironicclient version caching" (https://review.openstack.org/#/c/188641/) if anyone has the time | 22:04 |
| *** Sukhdev has joined #openstack-ironic | 22:07 | |
| NobodyCam | mrda: they can remove the cached63 | 22:08 |
| NobodyCam | version information manually | 22:08 |
| NobodyCam | not thinking about adding a --recheck or --renegotiation option | 22:09 |
| NobodyCam | know how to edit a file is a great op trick but forcing a user to can get messy (imo) | 22:10 |
| mrda | so they can a) specify a version on the command line, or | 22:10 |
| mrda | b) remove the cache file | 22:10 |
| mrda | it should be transparent to the user | 22:10 |
| mrda | it's a negotiated version, between ironic and ironicclient, that is less than 5 minutes old | 22:11 |
| mrda | so I don't really see many situations when either a) or b) doesn't cover what the user would want | 22:12 |
| mrda | but if you have strong opinions otherwise, please say so :) | 22:12 |
| *** alexpilotti has quit IRC | 22:13 | |
| NobodyCam | humm.. does the 5 minutes timer start after negotiation or after last command sent? | 22:13 |
| mrda | after a version has been negotiated | 22:13 |
| *** alexpilotti has joined #openstack-ironic | 22:19 | |
| NobodyCam | we don't have a really clean way to make the timeout user adjustable do we :( | 22:21 |
| mrda | well, we could introduce another command line flag | 22:22 |
| *** thrash is now known as thrash|g0ne | 22:22 | |
| mrda | but personally, I think that's overkill | 22:22 |
| mrda | OpenStack is opinionated. If we think that 5 minutes is a fair timeout, and we provide a way for it to be overwritten explicitly, I think that's a good user experience | 22:23 |
| NobodyCam | I was looking at the code [7]_ and was thinking I may even bump that to 600 or 900 seconds if my env was nice and stable. or down to 5 or 10 seconds if I was upgrading | 22:23 |
| jroll | since when is openstack opinionated, openstack is the opposite of opinionated | 22:24 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic-specs: Adds spec for creating a key manager with Barbican to Ironic https://review.openstack.org/188609 | 22:24 |
| mrda | jroll: No sane person would choose MySQL or rabbit if they had a choice :-P | 22:25 |
| NobodyCam | lol | 22:25 |
| NobodyCam | cassandra for the win! | 22:25 |
| NobodyCam | lol | 22:25 |
| NobodyCam | *ducks* | 22:25 |
| mrda | :) | 22:25 |
| mrda | NobodyCam: but Java. Eww. | 22:26 |
| NobodyCam | :) | 22:26 |
| NobodyCam | mrda: the reason I see to adjust that is during an upgrade | 22:26 |
| NobodyCam | I want the cache to time out more quickly then.. | 22:27 |
| mrda | NobodyCam: if you want it, add it as a comment to the review | 22:27 |
| mrda | and then I can work out how it could be implemented sanely | 22:27 |
| jroll | eh | 22:27 |
| NobodyCam | is it worth it | 22:27 |
| jroll | do you absolutely need to be on a newer version within 5 minutes of an upgrade? | 22:27 |
| *** harlowja has quit IRC | 22:27 | |
| jroll | *and* not know how to blow the cache | 22:28 |
| NobodyCam | ya | 22:28 |
| jroll | 5 minutes seems fine to me | 22:28 |
| *** [1]cdearborn has quit IRC | 22:28 | |
| mrda | 5 minutes is just a guess from someone who is not on-call | 22:29 |
| mrda | It could be 10 minutes if you want | 22:30 |
| jroll | generally ironicclient is not a large part of my things-are-on-fire workflow | 22:30 |
| jroll | if that helps :) | 22:30 |
| mrda | that's good to know jroll :) | 22:31 |
| jroll | if it is, it's node-show or node-list, not a huge deal | 22:31 |
| *** amotoki has joined #openstack-ironic | 22:32 | |
| *** harlowja has joined #openstack-ironic | 22:32 | |
| *** jgrimm is now known as zz_jgrimm | 22:34 | |
| *** amotoki has quit IRC | 22:36 | |
| NobodyCam | mrda: bam | 22:42 |
| mrda | NobodyCam: Is that North American slang for something I DON'T UNDERSTAND? | 22:42 |
| mrda | :) | 22:43 |
| mrda | ohh I see, thanks NobodyCam :) | 22:43 |
| NobodyCam | lol dont look up "bam" on urbandictionary | 22:44 |
| NobodyCam | :) | 22:44 |
| jroll | lol | 22:45 |
| jroll | I prefer bamf. | 22:45 |
| NobodyCam | lol | 22:45 |
| openstackgerrit | Merged openstack/ironic-specs: Add ironicclient version caching https://review.openstack.org/188641 | 22:46 |
| NobodyCam | jroll: ++ | 22:46 |
| *** lucas-dinner has quit IRC | 22:48 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Improve IPA logging and exception handling https://review.openstack.org/190279 | 22:51 |
| *** krtaylor has quit IRC | 22:54 | |
| *** natorious is now known as zz_natorious | 23:09 | |
| *** ijw has joined #openstack-ironic | 23:09 | |
| *** yuanying has joined #openstack-ironic | 23:15 | |
| *** naohirot has joined #openstack-ironic | 23:24 | |
| *** edcragg has quit IRC | 23:26 | |
| *** edcragg has joined #openstack-ironic | 23:36 | |
| *** krtaylor has joined #openstack-ironic | 23:38 | |
| *** david-lyle has quit IRC | 23:40 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/190357 | 23:44 |
| *** spandhe has quit IRC | 23:49 | |
| *** naohirot has quit IRC | 23:54 | |
| *** harlowja has quit IRC | 23:54 | |
| *** harlowja has joined #openstack-ironic | 23:55 | |
| *** yuanying has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!