| *** smoriya has quit IRC | 00:04 | |
| *** Vikas has joined #openstack-ironic | 00:05 | |
| *** achanda has quit IRC | 00:06 | |
| Vikas | Hi folks..I am a beginner and soon after i commited changes for review,https://review.openstack.org/#/c/208680/, got "-1" from "Citrix XenServer CI".Can anybody please suggest what is this about and what should be my next step? | 00:06 |
|---|---|---|
| *** smoriya has joined #openstack-ironic | 00:10 | |
| *** jaypipes has quit IRC | 00:18 | |
| *** puranamr has quit IRC | 00:19 | |
| *** cing has joined #openstack-ironic | 00:26 | |
| *** chlong has joined #openstack-ironic | 00:26 | |
| devananda | Vikas: hello! if you haven't already, I highly recommend reading http://docs.openstack.org/infra/manual/developers.html#automated-testing (and in fact everything on that whole page is very helpful) | 00:30 |
| devananda | Vikas: also to note, you're in the channgel for the Ironic project, though your commit was to the Nova project. If you have more specific questions about that project, you will have better luck in #openstack-nova than here | 00:31 |
| jroll | (also, it looks like nova's CI may be having bigger problems :P) | 00:32 |
| jroll | er, no. | 00:32 |
| jroll | double : on the if statement, syntax error | 00:32 |
| *** puranamr has joined #openstack-ironic | 00:33 | |
| *** alexpilo_ has joined #openstack-ironic | 00:40 | |
| *** zz_natorious is now known as natorious | 00:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/208695 | 00:44 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/208696 | 00:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/208697 | 00:45 |
| *** alexpilo_ has quit IRC | 00:47 | |
| *** mtanino has quit IRC | 00:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/208695 | 00:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/208697 | 00:53 |
| *** cdearborn has quit IRC | 01:02 | |
| *** Nisha has quit IRC | 01:03 | |
| *** puranamr has quit IRC | 01:08 | |
| *** praneshp has joined #openstack-ironic | 01:11 | |
| *** thrash is now known as thrash|g0ne | 01:19 | |
| *** natorious is now known as zz_natorious | 01:19 | |
| *** ijw has joined #openstack-ironic | 01:21 | |
| *** ijw_ has quit IRC | 01:23 | |
| *** zhenguo has joined #openstack-ironic | 01:27 | |
| *** puranamr has joined #openstack-ironic | 01:29 | |
| *** chenglch has joined #openstack-ironic | 01:31 | |
| *** puranamr has quit IRC | 01:33 | |
| *** ijw has quit IRC | 01:34 | |
| *** cing has quit IRC | 01:35 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add RPCAPIs for RAID configuration https://review.openstack.org/196006 | 01:39 |
| *** alexpilo_ has joined #openstack-ironic | 01:40 | |
| *** zz_natorious is now known as natorious | 01:41 | |
| *** alexpilo_ has quit IRC | 01:44 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Update neutron integration spec https://review.openstack.org/206487 | 01:44 |
| *** pal has joined #openstack-ironic | 01:45 | |
| *** cing has joined #openstack-ironic | 01:48 | |
| *** natorious is now known as zz_natorious | 01:54 | |
| *** degorenko has quit IRC | 01:54 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add APIs for RAID configuration https://review.openstack.org/196007 | 01:54 |
| *** Haomeng has joined #openstack-ironic | 01:56 | |
| *** puranamr has joined #openstack-ironic | 02:10 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add support for inband raid configuration agent ramdisk https://review.openstack.org/198238 | 02:11 |
| *** puranamr has quit IRC | 02:15 | |
| *** praneshp has quit IRC | 02:20 | |
| *** praneshp has joined #openstack-ironic | 02:21 | |
| *** puranamr has joined #openstack-ironic | 02:22 | |
| *** puranamr has quit IRC | 02:27 | |
| *** pal has quit IRC | 02:40 | |
| *** achanda has joined #openstack-ironic | 02:45 | |
| *** zz_natorious is now known as natorious | 02:50 | |
| *** hakimo has joined #openstack-ironic | 02:52 | |
| *** hakimo_ has quit IRC | 02:55 | |
| *** ramineni has joined #openstack-ironic | 02:58 | |
| *** cing has quit IRC | 03:05 | |
| *** huangkai17 has joined #openstack-ironic | 03:12 | |
| ramineni | JoshNang, hi | 03:29 |
| *** achanda has quit IRC | 03:37 | |
| *** jhova has quit IRC | 03:38 | |
| *** Sukhdev has joined #openstack-ironic | 03:41 | |
| *** coolsvap|away is now known as coolsvap | 03:43 | |
| *** Vikas has quit IRC | 03:46 | |
| *** alexpilo_ has joined #openstack-ironic | 03:47 | |
| *** huangkai17 has quit IRC | 03:48 | |
| *** jhova has joined #openstack-ironic | 03:50 | |
| *** Vikas has joined #openstack-ironic | 03:50 | |
| *** alexpilo_ has quit IRC | 03:51 | |
| *** jhova has quit IRC | 03:52 | |
| *** pal has joined #openstack-ironic | 03:56 | |
| *** natorious is now known as zz_natorious | 03:57 | |
| *** Nisha has joined #openstack-ironic | 03:57 | |
| *** cing has joined #openstack-ironic | 03:58 | |
| *** amotoki has joined #openstack-ironic | 03:59 | |
| *** Vikas has quit IRC | 04:09 | |
| *** achanda has joined #openstack-ironic | 04:21 | |
| *** pal has quit IRC | 04:23 | |
| *** pal has joined #openstack-ironic | 04:30 | |
| *** Nisha_away has joined #openstack-ironic | 04:32 | |
| *** Nisha has quit IRC | 04:32 | |
| *** achanda_ has joined #openstack-ironic | 04:35 | |
| *** achanda has quit IRC | 04:35 | |
| *** cing has quit IRC | 04:40 | |
| *** rameshg87 has joined #openstack-ironic | 04:40 | |
| openstackgerrit | Shivanand Tendulker proposed stackforge/proliantutils: Fix to allow Secure Boot API calls only in UEFI boot mode https://review.openstack.org/208371 | 04:43 |
| *** smoriya has quit IRC | 04:50 | |
| *** smoriya has joined #openstack-ironic | 04:51 | |
| *** stendulker has joined #openstack-ironic | 04:52 | |
| *** Vikas_ has joined #openstack-ironic | 04:53 | |
| openstackgerrit | Shivanand Tendulker proposed stackforge/proliantutils: Fixes tox Warning issue https://review.openstack.org/208797 | 04:54 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Make states end-point discoverable via nodes API https://review.openstack.org/205895 | 04:54 |
| rameshg87 | do we have ironic meeting now ? | 05:02 |
| rameshg87 | Haomeng: mrda: ^^^ ?? | 05:03 |
| pshige_ | no traffic found | 05:04 |
| rameshg87 | pshige_: haven't heard from anyone that meeting is cancelled ? | 05:04 |
| *** meghal has joined #openstack-ironic | 05:06 | |
| *** meghal1 has joined #openstack-ironic | 05:07 | |
| pshige_ | I'd not been feeling well these months so I don't get the situation. | 05:08 |
| *** meghal has quit IRC | 05:10 | |
| *** ishant has joined #openstack-ironic | 05:14 | |
| openstackgerrit | Merged stackforge/proliantutils: Fix to allow Secure Boot API calls only in UEFI boot mode https://review.openstack.org/208371 | 05:16 |
| rameshg87 | smoriya: hi | 05:22 |
| smoriya | hi | 05:22 |
| rameshg87 | smoriya: are you planning to address devananda's comments on https://review.openstack.org/#/c/200496 ? | 05:23 |
| smoriya | rameshg87: Thank you for reviewing :) | 05:23 |
| rameshg87 | smoriya: wc :) | 05:23 |
| rameshg87 | smoriya: if not, I can give a shot at updating that spec with changes at those comments | 05:23 |
| smoriya | rameshg87: I'll add brief implementation for both case. | 05:24 |
| smoriya | rameshg87: oh really? | 05:24 |
| rameshg87 | smoriya: I think you were just working on that as I pinged you | 05:24 |
| smoriya | It is really helpful to me. | 05:24 |
| rameshg87 | smoriya: if you want to go ahead, then it's fine with me, otherwise I can update the spec. | 05:24 |
| smoriya | rameshg87: I don't know how much details should I write about driver implementation | 05:25 |
| *** ishant_ has joined #openstack-ironic | 05:26 | |
| mrda | damn | 05:26 |
| rameshg87 | smoriya: let me give a try filling it up. | 05:26 |
| * mrda was busy working and forgot | 05:26 | |
| rameshg87 | mrda: missed the meeting ? | 05:26 |
| mrda | sorry | 05:26 |
| mrda | :( | 05:26 |
| rameshg87 | mrda: no one was around .. | 05:26 |
| mrda | well, if that's the case, no harm done | 05:26 |
| rameshg87 | yeah | 05:27 |
| mrda | double sorry | 05:27 |
| smoriya | rameshg87: sure. thanks. | 05:27 |
| mrda | rameshg87: it's the week before midcycle, so I guess people are focused on that. | 05:27 |
| rameshg87 | mrda: yeah, may be | 05:28 |
| rameshg87 | mrda: are you going for midcycle ? | 05:28 |
| *** ishant has quit IRC | 05:29 | |
| mrda | yes, I'm headin over | 05:29 |
| mrda | heading | 05:29 |
| mrda | rameshg87: you? | 05:29 |
| rameshg87 | mrda: no :( | 05:30 |
| mrda | rats, would have been nice to hang out | 05:30 |
| *** yuanying has quit IRC | 05:31 | |
| *** yuanying has joined #openstack-ironic | 05:32 | |
| rameshg87 | yeah | 05:32 |
| *** yuanying has quit IRC | 05:36 | |
| *** yuanying has joined #openstack-ironic | 05:39 | |
| *** Sukhdev has quit IRC | 05:41 | |
| *** alexpilo_ has joined #openstack-ironic | 05:45 | |
| *** puranamr has joined #openstack-ironic | 05:45 | |
| *** alexpilo_ has quit IRC | 05:49 | |
| *** puranamr has quit IRC | 05:49 | |
| *** meghal1 has quit IRC | 05:51 | |
| *** PradeepV has joined #openstack-ironic | 05:53 | |
| chenke | h | 06:06 |
| *** ukalifon1 has joined #openstack-ironic | 06:07 | |
| Haomeng | chenke: ping | 06:07 |
| chenke | hi haomeng, yes | 06:27 |
| PradeepV | Hi Haomeng | 06:28 |
| PradeepV | I created a ironic node with boot device as disk | 06:28 |
| *** praneshp has quit IRC | 06:29 | |
| PradeepV | when i spawn up an instance using nova, the instance is going to spawing state and the boot device of the ironic node is changed to pxe again | 06:29 |
| chenke | haomeng, so how can i remove the network card information for deploying it again? | 06:30 |
| Haomeng | let me check | 06:32 |
| Haomeng | chenke: yes, the raw image can be monted | 06:32 |
| Haomeng | s/mnted/mounted | 06:32 |
| *** saripurigopi has joined #openstack-ironic | 06:33 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Transifex https://review.openstack.org/206903 | 06:33 |
| chenke | ok | 06:35 |
| chenke | and then, where should i clean the network card infomation? | 06:35 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Trusted boot doc https://review.openstack.org/208438 | 06:41 |
| Haomeng | chenke: fyi - how to mound disk image - http://forensicswiki.org/wiki/Mounting_Disk_Images | 06:44 |
| *** DaveMitchelll has quit IRC | 06:44 | |
| *** alexpilo_ has joined #openstack-ironic | 06:44 | |
| *** Marga_ has joined #openstack-ironic | 06:46 | |
| *** Marga_ has quit IRC | 06:46 | |
| chenke | yes. i have successful mount the image. i mean what network card info i need to clean in the next | 06:46 |
| *** Marga_ has joined #openstack-ironic | 06:47 | |
| *** alexpilo_ has quit IRC | 06:48 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 06:53 |
| rameshg87 | smoriya: ^^^ | 06:53 |
| smoriya | rameshg87: Thanks, I'll look at it :) | 06:54 |
| *** zz_natorious is now known as natorious | 06:55 | |
| *** natorious is now known as zz_natorious | 06:57 | |
| *** dtantsur|afk is now known as dtantsur | 06:59 | |
| *** achanda_ has quit IRC | 07:01 | |
| *** h00327910__ has joined #openstack-ironic | 07:01 | |
| *** yog__ has joined #openstack-ironic | 07:04 | |
| *** degorenko has joined #openstack-ironic | 07:08 | |
| *** jcoufal has joined #openstack-ironic | 07:09 | |
| *** ijw has joined #openstack-ironic | 07:09 | |
| Haomeng | chenke: you can try with "sudo rm -rf /etc/udev/rules.d/70-persistent-net.rules" to clean the network info | 07:10 |
| chenke | i don't have this one, but 99-dhcp-all-interfaces.rules, i guess it is same one. | 07:12 |
| Haomeng | chenke: and make sure cloudinit is clean, it can be run on next os starting | 07:12 |
| Haomeng | chenke: yes, check it out, there should be some mac address | 07:12 |
| chenke | Haomeng: cloudinit? | 07:13 |
| Haomeng | chenke: yes, if you depend on cloudinit to set hostname or other file injection, cloud-init is required | 07:13 |
| chenke | Haomeng: ok , i will find it. | 07:14 |
| Haomeng | good luck:) | 07:17 |
| *** Marga_ has quit IRC | 07:17 | |
| *** DaveMitchelll has joined #openstack-ironic | 07:26 | |
| PradeepV | Hi Haomeng, I created a ironic node with boot device as disk, when i spawn up an instance using nova, the instance is going to spawing state and the boot device of the ironic node is changed to pxe again | 07:27 |
| PradeepV | can you please leet me know if it is a know issue | 07:28 |
| Haomeng | PradeepV: how is the final status? | 07:28 |
| Haomeng | PradeepV: if you did not enable local boot, it should pxe two times | 07:28 |
| PradeepV | the nova instance is doing pxe boot and going to Active | 07:29 |
| Haomeng | PradeepV: so if we did not enable local boot, it should work as design | 07:30 |
| Haomeng | PradeepV: so what is your concern? | 07:30 |
| Haomeng | PradeepV: how about baremetal side, the os is runing as well? | 07:31 |
| PradeepV | yes, initially the boot_device of the ironic is set to disk, nova boot must boot up the baremetal from local disk right | 07:32 |
| PradeepV | here it is doing pxe boot instead | 07:33 |
| Haomeng | PradeepV: did you check your BIOS, if pxe boot is first boot device, that is always? | 07:34 |
| Haomeng | PradeepV: and there is bug for some ipmi firmware, it dont support to save boot device order | 07:34 |
| PradeepV | yes my first boot device in bios is LAN only, | 07:36 |
| *** ifarkas has joined #openstack-ironic | 07:37 | |
| *** alexpilo_ has joined #openstack-ironic | 07:42 | |
| *** lsmola has joined #openstack-ironic | 07:42 | |
| Haomeng | PradeepV: maybe this patch will help you - https://review.openstack.org/#/c/177642/ | 07:45 |
| *** Marga_ has joined #openstack-ironic | 07:46 | |
| Haomeng | PradeepV: it force to set boot device the ipmi which does not support the command 'chassis bootdev' with 'persistent' option' | 07:47 |
| *** chlong has quit IRC | 07:47 | |
| *** alexpilo_ has quit IRC | 07:48 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-specs: Add Zapping support to iLO drivers https://review.openstack.org/145404 | 07:48 |
| *** lucasagomes has joined #openstack-ironic | 07:48 | |
| *** Nisha_away has quit IRC | 07:49 | |
| *** pal has quit IRC | 07:54 | |
| *** ramineni has quit IRC | 07:56 | |
| *** ramineni has joined #openstack-ironic | 07:56 | |
| *** ramineni has quit IRC | 07:59 | |
| *** ramineni has joined #openstack-ironic | 08:00 | |
| *** Nisha_away has joined #openstack-ironic | 08:04 | |
| *** athomas has joined #openstack-ironic | 08:07 | |
| *** jistr has joined #openstack-ironic | 08:08 | |
| *** romainh has joined #openstack-ironic | 08:11 | |
| rameshg87 | lucasagomes: hi | 08:12 |
| lucasagomes | rameshg87, hi there | 08:12 |
| rameshg87 | lucasagomes: regarding ironic-lib sync https://review.openstack.org/#/c/162162/ , we were discussing in the last patchset about the group for the opt | 08:12 |
| rameshg87 | lucasagomes: while new group names like ironic-lib make sense, how do you see it not breaking the users ? | 08:13 |
| rameshg87 | lucasagomes: if someone had some configured value for these parameters, it will break right ? | 08:13 |
| rameshg87 | lucasagomes: speaking about https://review.openstack.org/#/c/162162/9/ironic_lib/disk_utils.py | 08:14 |
| *** DaveMitchelll has quit IRC | 08:14 | |
| *** jcoufal has quit IRC | 08:14 | |
| rameshg87 | I can see only deprecated_group coming to rescue. can't get any other solution for this. | 08:14 |
| lucasagomes | rameshg87, oh right yeah... hmmmmmmmmm | 08:15 |
| lucasagomes | rameshg87, yeah we should totally use deprecation | 08:16 |
| rameshg87 | lucasagomes: okay..commenting on the patch now.. we can add a comment on top of it why it's required | 08:17 |
| lucasagomes | rameshg87, right, yeah leave a todo to remove later | 08:18 |
| lucasagomes | but it should be pretty simple since the config options has a "deprecated_group" parameter | 08:18 |
| lucasagomes | you can pass deprecated_group='deploy' on the options there | 08:18 |
| rameshg87 | yes | 08:18 |
| *** Marga_ has quit IRC | 08:25 | |
| *** Marga_ has joined #openstack-ironic | 08:26 | |
| *** ijw has quit IRC | 08:29 | |
| *** Marga_ has quit IRC | 08:32 | |
| *** zer0c00l has quit IRC | 08:35 | |
| *** crinkle has quit IRC | 08:35 | |
| *** tteggel_ has quit IRC | 08:35 | |
| *** _fortis has quit IRC | 08:35 | |
| *** egon has quit IRC | 08:35 | |
| *** Madasi has quit IRC | 08:35 | |
| *** devananda has quit IRC | 08:35 | |
| *** yog__ has quit IRC | 08:35 | |
| *** Marga_ has joined #openstack-ironic | 08:37 | |
| *** crinkle has joined #openstack-ironic | 08:37 | |
| *** zer0c00l has joined #openstack-ironic | 08:37 | |
| *** tteggel_ has joined #openstack-ironic | 08:37 | |
| *** _fortis has joined #openstack-ironic | 08:37 | |
| *** egon has joined #openstack-ironic | 08:37 | |
| *** Madasi has joined #openstack-ironic | 08:37 | |
| *** devananda has joined #openstack-ironic | 08:37 | |
| *** yog__ has joined #openstack-ironic | 08:40 | |
| *** alexpilo_ has joined #openstack-ironic | 08:41 | |
| *** Marga_ has quit IRC | 08:42 | |
| *** e0ne has joined #openstack-ironic | 08:44 | |
| *** alexpilo_ has quit IRC | 08:45 | |
| *** pal has joined #openstack-ironic | 08:47 | |
| *** alexpilotti has quit IRC | 08:51 | |
| *** alexpilotti has joined #openstack-ironic | 08:52 | |
| *** dtantsur is now known as dtantsur|brb | 08:53 | |
| chenke | Haomeng, i have still stucked at "renamed network interface eth0 to enp7s0f1" after delete rules in /etc/udev/rules.d | 08:53 |
| *** jcoufal has joined #openstack-ironic | 08:54 | |
| chenke | Haomeng,i also checked this is no "HWADDR" in the ifcfg-enp7s0f1(located in the dd image) | 08:54 |
| *** jcoufal has quit IRC | 08:55 | |
| *** jcoufal has joined #openstack-ironic | 08:56 | |
| Haomeng | chenke: yes, that is odd behavior, did you check system log? | 08:56 |
| smoriya | rameshg87: I added a few comments on https://review.openstack.org/#/c/200496/ | 08:57 |
| chenke | Haomeng,i just checked the start up log, which system log do you refer to? | 08:58 |
| smoriya | rameshg87: and I can update the patch if you can't | 08:58 |
| rameshg87 | smoriya: just saw it, please update it if you got some time. I might be able to get back to it only after a couple of hours. | 08:58 |
| smoriya | rameshg87: okay, thank you | 08:59 |
| chenke | i see it rename eth0 to enp7s0f0 successfule while run deploy, but after restart, it can't pass that step | 08:59 |
| *** jcoufal has quit IRC | 09:07 | |
| *** jcoufal has joined #openstack-ironic | 09:11 | |
| *** Vikas_ has quit IRC | 09:13 | |
| stendulker | lucasagomes: Hi, Good Morning! | 09:15 |
| lucasagomes | stendulker, hi there, morning | 09:15 |
| stendulker | lucasagomes: Regarding defect https://bugs.launchpad.net/ironic/+bug/1480367 | 09:15 |
| openstack | Launchpad bug 1480367 in proliantutils "iLO cleaning error: iLO.0.10.UnsupportedOperationInSystemBIOS" [Undecided,Fix committed] - Assigned to Shivanand Tendulker (shivanand-tendulker) | 09:15 |
| lucasagomes | stendulker, oh yeah I saw the fix for the proliantutils | 09:16 |
| lucasagomes | thanks for that! | 09:16 |
| stendulker | lucasagomes: Were you testing it on Gen9 ? | 09:16 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update docs for Fedora 22 https://review.openstack.org/208924 | 09:16 |
| lucasagomes | stendulker, it was someone else in Red Hat testing it | 09:16 |
| lucasagomes | stendulker, and he reported me that problem so I opened a bug upstream for verification | 09:16 |
| lucasagomes | but yeah he's using a Gen9 with 2.20 firmware version | 09:17 |
| lucasagomes | stendulker, I can tell him to try to test it again with the fix | 09:17 |
| stendulker | lucasagomes: ok. Was just wondering what kind usescase you guys use Gen9? | 09:17 |
| stendulker | lucasagomes: Thanks that would be helpful. | 09:17 |
| lucasagomes | stendulker, I don't know the use case... They were testing the osp-direction (the rh installer) with those machine | 09:18 |
| lucasagomes | director* | 09:18 |
| *** marzif_ has joined #openstack-ironic | 09:18 | |
| stendulker | lucasagomes: ok. Do let us know if they see any other discrepancy wrt Proliant in their testing. | 09:19 |
| lucasagomes | stendulker, +1 will do | 09:19 |
| stendulker | lucasagomes: Thank you :) | 09:19 |
| lucasagomes | stendulker, thank you for the fix ! | 09:19 |
| *** rameshg87 has quit IRC | 09:20 | |
| stendulker | lucasagomes: :) | 09:21 |
| *** boris-42 has quit IRC | 09:30 | |
| *** yog__ has quit IRC | 09:30 | |
| *** rameshg87 has joined #openstack-ironic | 09:31 | |
| *** MattMan has quit IRC | 09:39 | |
| *** yog__ has joined #openstack-ironic | 09:39 | |
| *** MattMan has joined #openstack-ironic | 09:39 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Document configuring ironic-api behind mod_wsgi https://review.openstack.org/207091 | 09:40 |
| *** athomas has quit IRC | 09:42 | |
| *** marzif_ has quit IRC | 09:43 | |
| *** marzif_ has joined #openstack-ironic | 09:43 | |
| *** chenglch has quit IRC | 09:43 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 09:47 |
| *** athomas has joined #openstack-ironic | 09:50 | |
| openstackgerrit | Merged stackforge/proliantutils: Fixes tox Warning issue https://review.openstack.org/208797 | 09:51 |
| *** openstackgerrit_ has joined #openstack-ironic | 09:53 | |
| *** sambetts has quit IRC | 09:55 | |
| *** sambetts has joined #openstack-ironic | 09:57 | |
| yuikotakada | Hi, Ironic :) | 09:57 |
| *** jcoufal has quit IRC | 09:59 | |
| yuikotakada | folks, could you please take a look at https://review.openstack.org/#/c/166386/ ? | 09:59 |
| *** jcoufal has joined #openstack-ironic | 09:59 | |
| *** sambetts has quit IRC | 10:01 | |
| *** sambetts has joined #openstack-ironic | 10:04 | |
| *** romainh has quit IRC | 10:06 | |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic-specs: Add out-of-band RAID configuration support for Cisco Hardware https://review.openstack.org/192142 | 10:08 |
| *** jcoufal has quit IRC | 10:10 | |
| openstackgerrit | Merged openstack/ironic: grub2 bootloader support for uefi boot mode https://review.openstack.org/166192 | 10:10 |
| *** rameshg87 has quit IRC | 10:14 | |
| *** rameshg87 has joined #openstack-ironic | 10:15 | |
| openstackgerrit | Merged openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 10:15 |
| *** Marga_ has joined #openstack-ironic | 10:17 | |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic-specs: Out-of-band hardware inspection support for Cisco B/C/M-series servers https://review.openstack.org/204733 | 10:17 |
| *** yuriyz has quit IRC | 10:18 | |
| *** yuriyz has joined #openstack-ironic | 10:19 | |
| *** dtantsur|brb is now known as dtantsur | 10:19 | |
| *** Marga_ has quit IRC | 10:20 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update docs for Fedora 22 https://review.openstack.org/208924 | 10:20 |
| *** Marga_ has joined #openstack-ironic | 10:20 | |
| *** rameshg87 has quit IRC | 10:21 | |
| *** alexpilo_ has joined #openstack-ironic | 10:29 | |
| *** rvasilets___ has joined #openstack-ironic | 10:30 | |
| *** Marga_ has quit IRC | 10:31 | |
| *** alexpilo_ has quit IRC | 10:33 | |
| *** rameshg87 has joined #openstack-ironic | 10:34 | |
| *** yuanying has quit IRC | 10:35 | |
| *** yuanying has joined #openstack-ironic | 10:36 | |
| *** yuikotakada has quit IRC | 10:38 | |
| *** yuanying has quit IRC | 10:38 | |
| *** yuanying has joined #openstack-ironic | 10:38 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Document API versioning https://review.openstack.org/206999 | 10:43 |
| dtantsur | lucasagomes, rameshg87, ^^ | 10:43 |
| lucasagomes | dtantsur, thanks! | 10:44 |
| rameshg87 | dtantsur: oh great .. :) | 10:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Document API versioning https://review.openstack.org/206999 | 10:48 |
| * dtantsur fixed a typo | 10:48 | |
| lucasagomes | dtantsur, thanks re +2'd after typo fix | 10:50 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Implement optional API versioning https://review.openstack.org/203618 | 10:50 |
| *** jcoufal has joined #openstack-ironic | 10:52 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 10:57 |
| *** lucasagomes is now known as lucas-hungry | 11:01 | |
| *** Nisha_away has quit IRC | 11:03 | |
| TheJulia | Good morning | 11:03 |
| *** degorenko has quit IRC | 11:06 | |
| *** degorenko has joined #openstack-ironic | 11:08 | |
| dtantsur | TheJulia, morning | 11:13 |
| rameshg87 | dtantsur: don't we need to inform about 406 stuff ? | 11:14 |
| *** coolsvap is now known as coolsvap|away | 11:14 | |
| rameshg87 | dtantsur: when they try to use with a lower api version which doesn't support the feature | 11:15 |
| dtantsur | rameshg87, oh, maybe. As part of this patch? I'd like someone with English skills to make a follow-up anyway | 11:15 |
| rameshg87 | dtantsur: +1, I am not that "someone" anyway :) | 11:16 |
| rameshg87 | dtantsur: apart from that part it lgtm. I would like to see the docs thing after the job once as well | 11:16 |
| dtantsur | ack | 11:17 |
| * rameshg87 goes home | 11:19 | |
| *** rameshg87 has quit IRC | 11:19 | |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed openstack/ironic-lib: Merge latest disk partitioning code from ironic https://review.openstack.org/162162 | 11:22 |
| *** Marga_ has joined #openstack-ironic | 11:25 | |
| *** stendulker has quit IRC | 11:26 | |
| *** saripurigopi has quit IRC | 11:31 | |
| *** openstackgerrit has quit IRC | 11:31 | |
| *** openstackgerrit_ is now known as openstackgerrit | 11:31 | |
| *** openstackgerrit has quit IRC | 11:32 | |
| *** openstackgerrit_ has joined #openstack-ironic | 11:32 | |
| *** openstackgerrit_ is now known as openstackgerrit | 11:32 | |
| *** openstackgerrit_ has joined #openstack-ironic | 11:33 | |
| *** ramineni has quit IRC | 11:38 | |
| *** dprince has joined #openstack-ironic | 11:39 | |
| *** thrash|g0ne is now known as thrash | 11:46 | |
| *** romainh has joined #openstack-ironic | 11:50 | |
| *** pal has quit IRC | 11:52 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Expose versioning information on GET / endpoint https://review.openstack.org/209012 | 11:53 |
| *** Marga_ has quit IRC | 11:54 | |
| *** jaypipes has joined #openstack-ironic | 11:59 | |
| *** amotoki has quit IRC | 12:01 | |
| *** lucas-hungry is now known as lucasagomes | 12:05 | |
| lucasagomes | dtantsur, re https://review.openstack.org/#/c/208924/ I don't know if you hit send before finish typing | 12:10 |
| lucasagomes | also worth updating ... ? | 12:10 |
| *** Marga_ has joined #openstack-ironic | 12:11 | |
| dtantsur | lucasagomes, it was in response to "Possibly one more place needs to be updated.", sorry | 12:12 |
| lucasagomes | dtantsur, oh gotcha... yeah I'm updating that one | 12:12 |
| lucasagomes | thanks | 12:12 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update docs for Fedora 22 https://review.openstack.org/208924 | 12:13 |
| *** openstackgerrit has quit IRC | 12:16 | |
| *** openstackgerrit has joined #openstack-ironic | 12:17 | |
| *** alexpilo_ has joined #openstack-ironic | 12:23 | |
| *** alexpilo_ has quit IRC | 12:27 | |
| *** jjohnson2 has joined #openstack-ironic | 12:28 | |
| *** Vikas has joined #openstack-ironic | 12:28 | |
| lucasagomes | hi someone can take a look at https://review.openstack.org/#/c/205033 please? The bits in nova have been merged already | 12:31 |
| jroll | lucasagomes: question inline | 12:34 |
| *** bapalm_ has joined #openstack-ironic | 12:34 | |
| lucasagomes | jroll, hi there | 12:34 |
| jroll | morning :) | 12:34 |
| openstackgerrit | Merged openstack/ironic-specs: Implement Zapping States https://review.openstack.org/185122 | 12:37 |
| lucasagomes | jroll, commented | 12:38 |
| lucasagomes | jroll, also https://review.openstack.org/#/c/204162/ did you mean what dtantsur just explained there? | 12:39 |
| dtantsur | jroll, morning | 12:40 |
| jroll | morning dtantsur :) | 12:41 |
| jroll | lucasagomes: I did! | 12:41 |
| *** chlong has joined #openstack-ironic | 12:41 | |
| jroll | but first, I'm now trying to figure out if we need this 'if node.maintenance' block in this patch | 12:42 |
| jroll | I see that's the old behavior, but I'm not sure I get it :P | 12:42 |
| lucasagomes | jroll, dtantsur right... I have to think about it. But wouldn't that be a more complex change? Like and if I request it to move 3 states and it fails in the second? Should I rollback or something | 12:42 |
| lucasagomes | jroll, oh yeah, I just kept it as before | 12:43 |
| *** aarefiev has joined #openstack-ironic | 12:43 | |
| jroll | yeah, I don't think we need it | 12:43 |
| jroll | actually, maybe we do | 12:44 |
| * jroll drinks coffee and +2s | 12:44 | |
| lucasagomes | jroll, that's just preserving the old behavior | 12:44 |
| lucasagomes | jroll, but yeah once we get nova updated to not send that anymore we probably can just remove it all | 12:44 |
| lucasagomes | (next cycle) | 12:44 |
| jroll | right, I was just wondering if it was necessary, but I see it now | 12:44 |
| lucasagomes | cool thanks | 12:46 |
| jroll | lucasagomes: as far queueing things up, it might be more complex, was just a random thought | 12:46 |
| jroll | might be nice as a more general thing | 12:46 |
| lucasagomes | jroll, right, I just don't have any other use case for that | 12:47 |
| lucasagomes | because there are few states where the operator needs to input | 12:47 |
| lucasagomes | enroll -> manage -> available | 12:47 |
| jroll | I mean, software could use it too | 12:47 |
| lucasagomes | once there just available -> active (unless things fails) | 12:47 |
| jroll | it would work for this -> active -> deleted thing | 12:47 |
| jroll | it would work for "deploy when done cleaning" | 12:48 |
| lucasagomes | right hmm | 12:48 |
| jroll | "drop to manageable after this instance is deleted" <- I just thought of that one but I want it | 12:48 |
| jroll | maybe that one wouldn't work, idk | 12:49 |
| lucasagomes | yeah cause it's racy sine it goes to available before | 12:49 |
| *** ishant_ has quit IRC | 12:49 | |
| jroll | more of like, that one to me is "once the user does nova delete, put it in manageable eventually" | 12:49 |
| lucasagomes | jroll, I will put some thought on it | 12:49 |
| jroll | but there isn't transitions queued | 12:50 |
| jroll | ok | 12:50 |
| jroll | I just don't love the db flag for it :/ | 12:50 |
| jroll | I understand why it's there | 12:50 |
| lucasagomes | jroll, yeah me neither but I can't think about something better than that | 12:50 |
| lucasagomes | jroll, one thing we can do is to have a more generic name for that flag and use for other states | 12:50 |
| lucasagomes | like aborting when cleaning etc | 12:50 |
| jroll | right | 12:50 |
| lucasagomes | (on the next abortable clean step for e.g) | 12:51 |
| lucasagomes | and reuse the same flag for other states, that's more generic than only delete | 12:51 |
| jroll | nod | 12:51 |
| * jroll will think on it too | 12:52 | |
| lucasagomes | jroll, because queue states, it's fine but that will cause the node to always go to active before it actually deletes the instance | 12:52 |
| lucasagomes | where the flag can make it mid operation when the node moves to deploywait | 12:52 |
| lucasagomes | of if it's in deploywait we don't even set the flag and just delete it | 12:52 |
| *** alexpilotti has quit IRC | 12:52 | |
| lucasagomes | jroll, right yeah please give it a thought | 12:52 |
| jroll | mmm, that's a good point | 12:52 |
| *** alexpilotti has joined #openstack-ironic | 12:54 | |
| *** pal has joined #openstack-ironic | 12:55 | |
| *** jaypipes has quit IRC | 12:57 | |
| *** smoriya has quit IRC | 12:59 | |
| *** jaypipes has joined #openstack-ironic | 13:00 | |
| *** rvasilets___ has quit IRC | 13:01 | |
| *** ukalifon1 has quit IRC | 13:02 | |
| *** kkoski has joined #openstack-ironic | 13:06 | |
| *** coolsvap|away is now known as coolsvap | 13:06 | |
| *** Marga_ has quit IRC | 13:08 | |
| *** cdearborn has joined #openstack-ironic | 13:09 | |
| *** rameshg87 has joined #openstack-ironic | 13:13 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add placeholders for all previous specs https://review.openstack.org/208522 | 13:14 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Reorganize specs tree https://review.openstack.org/203628 | 13:14 |
| jroll | ^ should be passing tests and good to go now /cc devananda | 13:14 |
| *** rameshg87 has quit IRC | 13:17 | |
| *** marzif_ has quit IRC | 13:18 | |
| *** marzif_ has joined #openstack-ironic | 13:18 | |
| openstackgerrit | Debayan Ray proposed stackforge/proliantutils: common changes for firmware update https://review.openstack.org/203543 | 13:21 |
| lucasagomes | jroll, any updates re releasing Ironic? | 13:22 |
| *** alexpilo_ has joined #openstack-ironic | 13:22 | |
| lucasagomes | the ML thread seems dead now | 13:22 |
| *** coolsvap is now known as coolsvap_ | 13:23 | |
| jroll | lucasagomes: we still have to check the bug backlog and stuff, but yeah we should do it | 13:23 |
| lucasagomes | ack | 13:24 |
| lucasagomes | yeah there's something I would like to do in nova (clean up the logs by just fetching part of the resources) and it does depend on a release of the client | 13:24 |
| *** mjturek1 has joined #openstack-ironic | 13:25 | |
| *** alexpilo_ has quit IRC | 13:27 | |
| *** chlong has quit IRC | 13:31 | |
| *** puranamr has joined #openstack-ironic | 13:33 | |
| *** mestery_afk is now known as mestery | 13:38 | |
| NobodyCam | good morning Ironicers | 13:44 |
| TheJulia | Good morning NobodyCam | 13:46 |
| *** pal_ has joined #openstack-ironic | 13:47 | |
| dtantsur | morning, NobodyCam | 13:47 |
| *** pal has quit IRC | 13:48 | |
| *** pal has joined #openstack-ironic | 13:49 | |
| lucasagomes | NobodyCam, TheJulia morning | 13:49 |
| NobodyCam | good morning TheJulia, dtantsur lucasagomes sambetts jlvillal jroll and everyone else | 13:50 |
| NobodyCam | :) | 13:50 |
| jroll | hiya NobodyCam :) | 13:51 |
| NobodyCam | :) | 13:51 |
| *** Marga_ has joined #openstack-ironic | 13:51 | |
| *** pal_ has quit IRC | 13:51 | |
| *** Marga_ has quit IRC | 14:01 | |
| *** pal has quit IRC | 14:01 | |
| *** pal has joined #openstack-ironic | 14:02 | |
| *** Marga_ has joined #openstack-ironic | 14:02 | |
| *** PradeepV has quit IRC | 14:04 | |
| openstackgerrit | Merged openstack/ironic-specs: Update neutron integration spec https://review.openstack.org/206487 | 14:06 |
| *** mtanino has joined #openstack-ironic | 14:09 | |
| sambetts | Morning all o/ | 14:11 |
| *** ndipanov has joined #openstack-ironic | 14:15 | |
| *** jcoufal has quit IRC | 14:16 | |
| *** r3mu5 has joined #openstack-ironic | 14:17 | |
| *** pal has quit IRC | 14:20 | |
| *** pal has joined #openstack-ironic | 14:20 | |
| lucasagomes | sambetts, hi there | 14:20 |
| *** Marga_ has quit IRC | 14:21 | |
| NobodyCam | mornign sambetts :) | 14:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/208695 | 14:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/209090 | 14:22 |
| openstackgerrit | Merged openstack/bifrost: Minor testing clean-up https://review.openstack.org/208095 | 14:23 |
| openstackgerrit | Merged openstack/bifrost: Remove CoreOS references in IPA image playbooks https://review.openstack.org/208580 | 14:25 |
| openstackgerrit | Merged openstack/bifrost: Make bifrost folders more friendly to vagrent https://review.openstack.org/208576 | 14:25 |
| openstackgerrit | Merged openstack/bifrost: Update enroll role README https://review.openstack.org/208100 | 14:26 |
| *** pal has quit IRC | 14:26 | |
| *** pal has joined #openstack-ironic | 14:27 | |
| sambetts | hey lucasagomes, NobodyCam | 14:28 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Log look up attributes at INFO level https://review.openstack.org/208544 | 14:29 |
| dtantsur | sambetts, morning! | 14:29 |
| sambetts | o/ dtantsur | 14:29 |
| jroll | morning sambetts :) | 14:29 |
| sambetts | jroll o/ | 14:29 |
| NobodyCam | oh zapping spec landed! | 14:35 |
| NobodyCam | JoshNang: w00t | 14:35 |
| *** jcoufal has joined #openstack-ironic | 14:37 | |
| *** ifarkas has quit IRC | 14:41 | |
| *** kkoski has quit IRC | 14:47 | |
| dtantsur | oh yeah, it took a long time :) | 14:47 |
| NobodyCam | :) | 14:47 |
| *** kkoski has joined #openstack-ironic | 14:49 | |
| *** Marga_ has joined #openstack-ironic | 14:49 | |
| *** kkoski has quit IRC | 14:50 | |
| *** kkoski_ has joined #openstack-ironic | 14:52 | |
| *** achanda has joined #openstack-ironic | 14:52 | |
| *** bradjones has joined #openstack-ironic | 14:54 | |
| *** bradjones has quit IRC | 14:54 | |
| *** bradjones has joined #openstack-ironic | 14:54 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/208697 | 14:55 |
| *** jjohnson2 has quit IRC | 14:57 | |
| *** hemna has joined #openstack-ironic | 15:00 | |
| devananda | morning, all | 15:03 |
| *** achanda has quit IRC | 15:04 | |
| NobodyCam | good morning devananda | 15:05 |
| dtantsur | devananda, morning | 15:08 |
| *** coolsvap_ is now known as coolsvap|away | 15:08 | |
| *** ifarkas has joined #openstack-ironic | 15:08 | |
| *** e0ne has quit IRC | 15:10 | |
| *** alexpilo_ has joined #openstack-ironic | 15:11 | |
| *** e0ne has joined #openstack-ironic | 15:11 | |
| jroll | heya devananda \o | 15:13 |
| *** alexpilo_ has quit IRC | 15:15 | |
| lucasagomes | devananda, morning | 15:17 |
| devananda | heya o/ | 15:17 |
| devananda | looks like we have quorum, lol | 15:17 |
| lucasagomes | jroll, dtantsur devananda eveyrone else! Let's try to ironic out the ENROLL / release of ironic problem while we are all here? | 15:17 |
| lucasagomes | iron out* | 15:17 |
| devananda | is now a good time for us to ... yea, that ^ | 15:17 |
| jroll | oh wow, all of us in one place | 15:17 |
| dtantsur | I still don't see any options, except for, well, release | 15:17 |
| lucasagomes | :-P virtually speaking! | 15:18 |
| * TheJulia locks the doors so people are forced to discuss | 15:18 | |
| NobodyCam | lol | 15:18 |
| lucasagomes | no seriously, we need to try to reach a consensus on this because we have been talking for quite a while already | 15:18 |
| jroll | so, what do we need to decide: | 15:18 |
| * devananda throws pencils at the ceilling | 15:18 | |
| lucasagomes | and the ML seems to have lost traction at this point | 15:18 |
| jroll | 1) let's do a release of the server | 15:18 |
| jroll | 2) let's do a release of the client | 15:18 |
| jroll | 3) what version should client default to? should we add deprecation messages? | 15:19 |
| jroll | 4) what needs to be done to release the things? testing, bug check, ??? | 15:19 |
| jroll | anything I'm missing? | 15:19 |
| devananda | so, I'm going to say it again, the ENROLL change in the server does not need to be mandatory | 15:19 |
| lucasagomes | ok... so let's try to figure out 1,2 and 3 at least | 15:19 |
| devananda | i'm sorry thta that seems unpopular because it gets ignored every time i bring it up | 15:19 |
| jroll | devananda: it does now, in version 1.11 | 15:20 |
| jroll | because it landed. | 15:20 |
| devananda | we can land 1.12 that fixes it | 15:20 |
| devananda | and, ewll, anyone who deployed that can deal | 15:20 |
| dtantsur | one more breaking change? | 15:20 |
| devananda | because there are A LOT MORE people deploying from packages | 15:20 |
| jroll | are there? I know of exactly two ironic deployments :) | 15:20 |
| * jroll should shut up because this is tangential | 15:21 | |
| devananda | jroll: read the user survey recently? | 15:21 |
| devananda | 25% of all responders are running it in some form. 10% in production | 15:21 |
| dtantsur | which packages? our customers probably won't get this release... | 15:21 |
| dtantsur | not sure if debian/ubuntu will package intermediate releases as well | 15:21 |
| devananda | dtantsur: exactly | 15:21 |
| devananda | dtantsur: so we can fix it with 1.12 | 15:22 |
| devananda | and only the folks who deploy from master (ie, rackspace) are going to be impacted | 15:22 |
| devananda | and they can add the optional parameter ahead of rolling out th echange | 15:22 |
| devananda | and they won't even notice it | 15:22 |
| dtantsur | anyway, 1. this was discussed 100 times, 2. we use versioning _exactly_ for this, 3. no, we don't ignore people deploying from master | 15:22 |
| devananda | I'm not ignoring them | 15:22 |
| *** Vikas has quit IRC | 15:23 | |
| jroll | you're treating them differently | 15:23 |
| jroll | it's ok to break master as long as we don't break releases? | 15:23 |
| devananda | can anyone give me one logical reason not to add a parameter to the POST /v1/nodes/ statement that allows the client to determine whether the initial state is ENROLL or AVAILABLE ? | 15:23 |
| lucasagomes | NobodyCam, yuriyz Haomeng (rest of the core team online) do you guys have opnions on this ^ matter? I think that right now what has been said have been already discussed a couple of times by the same people | 15:23 |
| *** Marga_ has quit IRC | 15:23 | |
| devananda | cause no one has | 15:23 |
| dtantsur | devananda, sanity is not one? | 15:24 |
| devananda | dtantsur: what is not sane about it? | 15:24 |
| *** Marga_ has joined #openstack-ironic | 15:24 | |
| dtantsur | devananda, can anyone give me one logical reason not to add a parameter to the POST /v1/nodes/ statement that allows the client to determine whether the initial state is MANAGEABLE? CLEANING? ACTIVE? | 15:24 |
| devananda | dtantsur: so - MANAGEABLE would be a reasonable option too | 15:24 |
| NobodyCam | I actually like the idea of adding a pram to set the initial state of a node being created | 15:24 |
| dtantsur | I'd like to have nodes starting as INSPECTING, can I? | 15:24 |
| devananda | dtantsur: CLEANING is a transitive state. ACTIVE requires a lot of additional actions | 15:25 |
| dtantsur | devananda, and AVAILABLE is a wrong state | 15:25 |
| TheJulia | Ditto, an operator could know everythign they need to deploy | 15:25 |
| devananda | dtantsur: again, INSPECTING is a transitional state | 15:25 |
| dtantsur | so what? I want my node to start with inspection right away? | 15:25 |
| devananda | to play devil's advocate, actually, it's possible for bifrost to already know everything it needs and POST all of it at once | 15:25 |
| devananda | dtantsur: INSPECTING is not a stable state | 15:26 |
| jroll | I don't think it's insane to give a choice; I think it's insane to have two methods of opting in, both of which must be provided to opt in. | 15:26 |
| dtantsur | so that point that you didn't hear is: AVAILABLE is a wrong entry point in our state machine. Just as INSPECTING or CLEANING. | 15:26 |
| dtantsur | the second point is jroll's: we already have insanely complex API versioning to opt-out of features | 15:27 |
| devananda | jroll: version headers are not feature flags, nor are they behavioral flags or query parameters | 15:27 |
| jroll | and I think "we're forcing people to deal with this change to use new features beyond 1.11" is a non-argument; it's an additional 3 lines of code or so. | 15:28 |
| dtantsur | .. which are already in place for some people | 15:28 |
| jroll | right | 15:28 |
| dtantsur | because of inspection or cleaning | 15:28 |
| jroll | now 1.12 with the double opt-in is going to break my enrollment code :) | 15:28 |
| *** praneshp has joined #openstack-ironic | 15:28 | |
| devananda | jroll: if having a choice about the initial state is a reasonable thing, then is it reasonable that a user a year from now might still want to choose the initial state to be AVAILABLE, while also having some other new features that we add in the Mikata release? | 15:29 |
| dtantsur | "having a choice about the initial state is a reasonable thing" dtantsur votes for NO | 15:29 |
| jroll | devananda: I didn't say it was reasonable, I just said it isn't insane | 15:29 |
| devananda | *sigh* | 15:30 |
| jroll | if this was the initial proposal, I would have thought real hard about it rather than rejecting it | 15:30 |
| jroll | however now this is another breaking change on top of 1.11 | 15:30 |
| jroll | so if the point is to avoid breaking changes, this doesn't help | 15:30 |
| NobodyCam | what if we added a new endpoint to "create-node-enrolled" and then added a deperaction message to "create-node" giving folks time to transation | 15:31 |
| dtantsur | why? | 15:31 |
| lucasagomes | yeah I also feel a bit eww to add yet another breaking change :-/ | 15:31 |
| dtantsur | the DO have time for transitions, thanks to API versioning | 15:31 |
| jroll | really by enrolling in AVAILABLE state, the user is just skipping validation | 15:32 |
| jroll | that's the only difference | 15:32 |
| TheJulia | I think the point is, the vast majority of users are not broken yet, it can still be fixed however we decide. | 15:32 |
| devananda | ^^^ yes | 15:32 |
| dtantsur | they won't be broken either | 15:32 |
| devananda | ok, let me try this from a different angle | 15:33 |
| devananda | let's say we were to release the server now, and someone installs that from the package | 15:33 |
| devananda | and we release a client now, too | 15:33 |
| devananda | how do we inform them that, at some point in the future, they'll need to change their tooling, because the initial state is going to be different? | 15:34 |
| lucasagomes | pinning to 1.9 ? (as is on master right now) | 15:34 |
| jroll | release notes? | 15:34 |
| devananda | lucasagomes: sure, let's say for the moment that it is pinned to 1.9 | 15:34 |
| jroll | api docs? | 15:34 |
| dtantsur | devananda, well, they'll figure out once they find out they can't use any new features | 15:34 |
| devananda | jroll: let's assume that the user and the operator don't talk to each other | 15:34 |
| jroll | soooo client release notes? api docs? | 15:35 |
| * jroll points at https://review.openstack.org/#/c/206999/ | 15:35 | |
| dtantsur | they'll learn about versioning, get mad at us, learn it again, then figure out how to use it | 15:35 |
| TheJulia | dtantsur: but that breaks them, that requires them to stop, investigate, and fix | 15:35 |
| dtantsur | TheJulia, that's why we invented versioning: so that they do it at their pace, not at the time of package update | 15:35 |
| devananda | so we pin the client to 1.9 for ever? | 15:36 |
| dtantsur | yes | 15:36 |
| dtantsur | (I'd prefer 1.1 but it's too late) | 15:36 |
| lucasagomes | also, I know it's an ugly interface, but they can use different versions for different actions right? If one uses node-create with --api-version 1.9 it will always start in AVAILABLE | 15:36 |
| TheJulia | so what happens in 3-6 months when we want to add a new workflow change that requires api changes? | 15:36 |
| * NobodyCam thinks tht is not sane either | 15:36 | |
| devananda | yea. in principle i'd prefer 1.1 as well, but too late | 15:36 |
| lucasagomes | other commands can use different versions (I know it's terrible) | 15:36 |
| dtantsur | lucasagomes, ++ | 15:36 |
| devananda | lucasagomes: no that's absolutely terrible | 15:37 |
| jroll | I'm curious why it's a problem for users to add three lines of code to their enrollment software in order to get new features | 15:37 |
| devananda | we can't require that a user (or our CLI) mix versions just to accomplish a task | 15:37 |
| dtantsur | devananda, I believe they'll do it (probably we already do tbh) | 15:37 |
| jroll | it's more like, we're requiring they mix versions to use new features AND not need to add 3 lines of code | 15:37 |
| devananda | lucasagomes: any workflow that we support must be completable within one API version | 15:38 |
| lucasagomes | devananda, yeah I'm all for that too. I just trying to brainstome possible different solutions for the problem | 15:38 |
| jroll | the supported workflow is "enroll, manage, provide" for api >=1.11 | 15:38 |
| jroll | we don't support "enroll, deploy" on >=1.11 | 15:39 |
| dtantsur | the same sequence of commands will work for API 1.4-1.11 | 15:39 |
| dtantsur | I mean, create -> manage -> provide works since mid-kilo | 15:39 |
| jroll | true that | 15:39 |
| devananda | to TheJulia's question, if we add another workflow in 3-6mo that requires a similar change, then our users will probably do one of two things | 15:40 |
| devananda | a) see it coming, stick to < 1.11, and then upgrade a year from now | 15:40 |
| devananda | b) get really annoyed that we've broken their applications twice in a year and find another projet | 15:40 |
| dtantsur | s/upgrade/bump API version/ <- let's be clear | 15:40 |
| TheJulia | devananda: I think (a) + complain is what my operational background expects will occur | 15:40 |
| jroll | I still don't see why a user would be so averse to an extra 2 commands in their "I have new hardware" script | 15:41 |
| jroll | I really don't | 15:41 |
| jroll | especially when they can do them on their own time | 15:41 |
| dtantsur | we fix much more things every release :D | 15:41 |
| jroll | like what kind of broken developer culture must you have to not be able to add that code within a matter of months | 15:41 |
| jroll | or if you're using packages within 6 months | 15:41 |
| TheJulia | jroll: that asumes they have a script, they could have lots of code wrapped around the client, and the workflow change could be compounded easilly | 15:42 |
| dtantsur | but seriously, we consider it's ok to make them understand our versioning, but not ok to add 2 lines? | 15:42 |
| jroll | TheJulia: script/code/whatever | 15:42 |
| jroll | it isn't a massive change | 15:42 |
| jroll | worst case you need some sleep statements | 15:42 |
| devananda | dtantsur: actually, my intent with versioning from the get-go was to hide as much of this as possible from the end user | 15:42 |
| jroll | or some polling | 15:42 |
| *** praneshp_ has joined #openstack-ironic | 15:43 | |
| TheJulia | jroll: I was going to say polling logic :) | 15:43 |
| *** pal has quit IRC | 15:43 | |
| jroll | yeah, sure | 15:43 |
| jroll | regardless, a few hours of work over 6 months doesn't seem unreasonable | 15:43 |
| *** praneshp has quit IRC | 15:43 | |
| *** praneshp_ is now known as praneshp | 15:43 | |
| devananda | dtantsur: allowing developers to iterate without breaking operators or users, but it seems that we can't even do that, because folks reject my proposed changes (1.12, optional initial state) | 15:43 |
| jroll | devananda: your proposal breaks some users in favor of other users. I don't get why that's okay. | 15:44 |
| TheJulia | The needs of the many outweigh the needs of the few | 15:44 |
| dtantsur | oh, c'mon, no | 15:45 |
| jroll | .... | 15:45 |
| lucasagomes | problem is we don't have numbers... like it's assumptions right? | 15:45 |
| jroll | lucasagomes++ | 15:45 |
| devananda | jroll: it favors not impacting the long-tail of users who will install this from distro packages | 15:45 |
| devananda | jroll: over the initial burst of people who are deploying from master and deeply involved in the project | 15:45 |
| lucasagomes | what if we take this 1.11 as lesson... Look next time we see a breaking change we should try real _hard_ to find a a non-breakable way. Unless it's absolutely impossible | 15:46 |
| jroll | it's only people who *already* use kilo from packages, fwiw, it doesn't affect anyone that begins with liberty | 15:46 |
| NobodyCam | jroll: what if we added a pram to create that defaulted to enroll / then those users looking for the legecy support could get it | 15:46 |
| lucasagomes | I don't see we walking forward right now with this conversation. It's been the same arguments perhaps we should pick one way or another | 15:46 |
| devananda | jroll: or anyone who installs a distro package that includes kilo a year from now | 15:46 |
| jroll | NobodyCam: users wanting legacy support can get legacy support by passing 1.10 already | 15:46 |
| lucasagomes | and think how we better handle it | 15:46 |
| devananda | because, guess what, some distros will probably STILL be shipping kilo a year form now | 15:46 |
| NobodyCam | not once we add something they want that is behind a version > .11 | 15:47 |
| jroll | jesus | 15:47 |
| dtantsur | that doesn't change anything. that's the idea of versioning: if you request a new version if API, you're reading to accept whatever is in | 15:47 |
| *** albertoffb has joined #openstack-ironic | 15:47 | |
| dtantsur | NobodyCam, "they want" is the problem. if they want, they have to do some work, it won't magically happen anyway | 15:48 |
| dtantsur | (except for cleaning, heh) | 15:48 |
| devananda | jroll: can you lay out your proposed plan here, then let's take an informal vote on it (because no meetbot) | 15:49 |
| *** pradipta has joined #openstack-ironic | 15:49 | |
| lucasagomes | ++ to vote (it's been discussed for ~25 min now) | 15:50 |
| TheJulia | ++ | 15:50 |
| dtantsur | do we have a quorum for any voting? | 15:50 |
| NobodyCam | lucasagomes: 25 minutes + 3 weeks | 15:50 |
| NobodyCam | :-p | 15:50 |
| * dtantsur is ok with voting, just asking | 15:50 | |
| devananda | dtantsur: 5 cores are present | 15:50 |
| *** HenryG has quit IRC | 15:51 | |
| NobodyCam | o/ | 15:51 |
| jroll | devananda: my plan is: release a server. if we must release a client, let's default the version to 1.6 as the current client is today. | 15:51 |
| jroll | but generally I don't think much about the client and mainly want to release a server. | 15:51 |
| dtantsur | we have 9 cores, so ideally we need 5 votes to make a decision... | 15:51 |
| devananda | dtantsur: fyi, voting rules for the TC are "more than half cores present, 2/3 of those present agree" | 15:52 |
| jroll | so we have quorom as long as we all agree, should be easy amirit :D | 15:52 |
| dtantsur | if we do it on meeting, we'll at least provide some people a chance of knowing it | 15:52 |
| devananda | dtantsur: if we follow that, which I think we do, then we need 3 votes right now to pass | 15:52 |
| dtantsur | ok, but that's not in spirit of open design | 15:52 |
| lucasagomes | ok it's an informal voting | 15:53 |
| devananda | dtantsur: sure it is -- this has been discussed on the ML and in public channels several times | 15:53 |
| *** pal has joined #openstack-ironic | 15:53 | |
| devananda | dtantsur: anyone has been free to reply on the ML to get their voice heard | 15:53 |
| jroll | at the risk of getting nothing done again, I'd prefer other cores to have a chance to vote, at least | 15:53 |
| dtantsur | 4 cores were not aware of voting.. | 15:53 |
| jroll | which they do not because we decided to vote immediately while they are sleeping | 15:53 |
| devananda | why do we need to vote just to land some code? | 15:54 |
| TheJulia | invite only poll for 24-48 hours? | 15:54 |
| devananda | wtf? | 15:54 |
| devananda | I just wnat to know what all of you think ... | 15:54 |
| dtantsur | devananda, because 2 people here would want to -2 it? | 15:54 |
| devananda | jroll: was ^^ the entirity of your plan? | 15:54 |
| devananda | 15:51:14 < jroll> devananda: my plan is: release a server. if we must release a client, let's default the version to 1.6 as the current client is today. | 15:54 |
| devananda | 15:51:33 < jroll> but generally I don't think much about the client and mainly want to release a server. | 15:54 |
| jroll | devananda: yes | 15:55 |
| jroll | I'm fine with an informal vote | 15:55 |
| jroll | and go to the list "this is what we decided, you have three days to object" | 15:55 |
| jroll | I misunderstood the intention of the voting. | 15:55 |
| devananda | works for me. I just want us to get out of this deadlock | 15:56 |
| dtantsur | ack | 15:56 |
| lucasagomes | ++ | 15:56 |
| jroll | so option 1: release a server with current code. release a client defaulted to 1.6. | 15:57 |
| jroll | option 2: ??? | 15:57 |
| jroll | ^ devananda | 15:57 |
| lucasagomes | release a server with current code. release a client defaulted to 1.9. | 15:57 |
| lucasagomes | opt 2 | 15:57 |
| devananda | we're back to options? | 15:57 |
| jroll | ok, surely there needs to be an option 3 with the 1.12 | 15:57 |
| jroll | oh | 15:57 |
| jroll | I mean | 15:57 |
| dtantsur | oh now, let's decide on principal question | 15:57 |
| dtantsur | 1.6 vs 1.9 can be decided separately | 15:57 |
| lucasagomes | I don't get why we want to set 1.6 back in the client because 1.9 is already merged and has no backward imcompatibility change from 1.6 to 1.9 | 15:57 |
| jroll | we can leave it at option 1, with a yes/no | 15:57 |
| jroll | that's fine | 15:58 |
| lucasagomes | yeah | 15:58 |
| lucasagomes | dtantsur, ++ | 15:58 |
| dtantsur | we have a critical question: whether to release as it is and move on or rollback 1.11 by 1.12, right? | 15:58 |
| jroll | ok, better rephrased: should we do a 1.12 api version that allows the user to pick begining provision state in (AVAILABLE, ENROLL), defaulting to AVAILABLE? | 15:58 |
| devananda | jroll: ^^ yes, that's the question before us | 15:58 |
| jroll | that's a yes/no question that I think is the main issue. | 15:58 |
| jroll | aight. | 15:59 |
| jroll | so, begin? | 15:59 |
| * jroll votes no | 15:59 | |
| * dtantsur votes no | 15:59 | |
| * NobodyCam votes yes | 15:59 | |
| * devananda votes yes | 15:59 | |
| lucasagomes | I will keep the same opnion I had on the ML, because it's already merged | 15:59 |
| * lucasagomes votes no | 15:59 | |
| dtantsur | tie break! | 15:59 |
| lucasagomes | 1.11 is already merged* | 15:59 |
| devananda | ok then - thanks everyone :) | 16:00 |
| dtantsur | uffff... that was a loooong flame war :) | 16:00 |
| jroll | thanks all, there's nobody else I'd rather argue with :) | 16:00 |
| *** cinerama has quit IRC | 16:00 | |
| devananda | do ya'll want me to write this up, or is someone eles feeling editorial? :) | 16:00 |
| jroll | I can mail the list | 16:01 |
| *** HenryG has joined #openstack-ironic | 16:01 | |
| devananda | I also need to announce some things about the midcycle, and, oh god, meetings ... | 16:01 |
| devananda | jroll: thanks much | 16:01 |
| jroll | my cat has reminded me I forgot to feed her, so let me do that first :P | 16:01 |
| dtantsur | cat is a priority | 16:01 |
| lucasagomes | heh | 16:01 |
| devananda | cat wins | 16:01 |
| jlvillal | Speaking of mid-cycle. Anyone know a decent priced place to stay? | 16:01 |
| NobodyCam | thanks for the debate... /me really believes this is how we make better software :)_ | 16:01 |
| jroll | jlvillal: airbnb wasn't bad for me, hotels seemed to be outrageous | 16:01 |
| * dtantsur goes home, see you tomorrow | 16:02 | |
| *** dtantsur is now known as dtantsur|afk | 16:02 | |
| jroll | night dtantsur \o | 16:02 |
| *** cinerama has joined #openstack-ironic | 16:02 | |
| jlvillal | jroll: Yeah, there seems to be a big convention at the same time. Checking into Airbnb... | 16:02 |
| TheJulia | jlvillal: I looked at hotels over the weekend because reasons() and didn't see anything near downtown that was reasonable. :( | 16:02 |
| jlvillal | TheJulia: Thanks and :( | 16:02 |
| NobodyCam | night dtantsur|afk thank you for staying up with us !!! | 16:02 |
| lucasagomes | jroll, cool make sure you ask for the opnion of the rest of the people that couldn't vote here | 16:03 |
| TheJulia | jlvillal: a statistitian's convention.... | 16:03 |
| jlvillal | TheJulia: Yeah. I think so. 6000+ people if I remember correctly. | 16:04 |
| TheJulia | clearly we need to inquire what are the odds.... | 16:04 |
| TheJulia | :) | 16:04 |
| jlvillal | :D | 16:04 |
| * jlvillal is thinking about staying in Redmond. Wonders how mass transit is? | 16:04 | |
| TheJulia | jlvillal: so... if you can stay close to the light rail that starts at SEA, it is a relatively quick ride into town, ~45 minutes | 16:05 |
| jlvillal | TheJulia: Thanks! | 16:06 |
| lucasagomes | jlvillal, the transport in seattle is pretty good as far as I remember | 16:06 |
| lucasagomes | you can make a breeze card (I think that's how it's called) | 16:06 |
| NobodyCam | lots of uber and such | 16:06 |
| devananda | Orca card | 16:07 |
| jlvillal | lucasagomes: NobodyCam: Good info. Thank you. | 16:07 |
| TheJulia | jlvillal: from SEA to downtown is like $2.50 on the light rail, just for context | 16:07 |
| *** zz_natorious is now known as natorious | 16:07 | |
| devananda | busses are very good, especially around downtown. Uber/Lyft are also very easy to get | 16:07 |
| jlvillal | Okay. Sounds more reasonable to be a little distance away then. | 16:07 |
| devananda | jlvillal: redmond <-> seattle is icky, in my opinion, but if you rent a car and don't mind commuter traffic .... | 16:08 |
| jlvillal | devananda: I was actually going to drive up to Seattle from Portland. But I was hoping not to have to mess with parking and driving. | 16:08 |
| *** HenryG has quit IRC | 16:08 | |
| *** yog__ has quit IRC | 16:08 | |
| devananda | jlvillal: for context, anything east of lake union is going to require driving in traffic | 16:08 |
| *** jistr has quit IRC | 16:09 | |
| devananda | or dealing with specialized communiter busses like this one: http://www.soundtransit.org/schedules/st-express-bus/545/weekday/inbound | 16:10 |
| *** alexpilo_ has joined #openstack-ironic | 16:10 | |
| * jlvillal looks | 16:10 | |
| *** coolsvap|away is now known as coolsvap | 16:10 | |
| devananda | I have no experirnce with those myself | 16:10 |
| jlvillal | devananda: Bus looks good. Has a drop off at 5th and Pine, which looks close to 711 Pike Street | 16:12 |
| devananda | yes | 16:12 |
| devananda | though, i need to annouce a small change in location (it's still close to there) | 16:13 |
| jlvillal | devananda: Ah, good to know. And also start time would be good. Unless I'm being blind. | 16:13 |
| devananda | 9am | 16:13 |
| * jlvillal assumes around 9am | 16:13 | |
| devananda | folks will trickle inwhen they do | 16:14 |
| devananda | but yea | 16:14 |
| BadCub | the location will be setting up breakfast service at 0900hrs | 16:14 |
| *** alexpilo_ has quit IRC | 16:14 | |
| *** e0ne has quit IRC | 16:17 | |
| *** HenryG has joined #openstack-ironic | 16:18 | |
| jroll | alright, email sent | 16:21 |
| *** natorious is now known as zz_natorious | 16:22 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 16:22 |
| *** Marga_ has quit IRC | 16:23 | |
| *** meghal has joined #openstack-ironic | 16:25 | |
| *** pradipta has quit IRC | 16:26 | |
| *** bapalm_ has quit IRC | 16:27 | |
| *** openstackgerrit_ has quit IRC | 16:29 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Fix logging levels https://review.openstack.org/209169 | 16:31 |
| *** ifarkas has quit IRC | 16:37 | |
| *** HenryG has quit IRC | 16:39 | |
| *** praneshp has quit IRC | 16:51 | |
| openstackgerrit | Merged openstack/ironic: Clean up instance_uuid as part of the node's tear down https://review.openstack.org/205033 | 16:52 |
| *** cinerama has quit IRC | 16:57 | |
| *** romainh has left #openstack-ironic | 16:58 | |
| *** e0ne has joined #openstack-ironic | 17:05 | |
| *** cinerama has joined #openstack-ironic | 17:08 | |
| *** cinerama has quit IRC | 17:09 | |
| *** HenryG has joined #openstack-ironic | 17:10 | |
| *** cinerama has joined #openstack-ironic | 17:10 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add deprecation messages on the bash ramdisk endpoints https://review.openstack.org/209188 | 17:15 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Modify docs to talk about IPA https://review.openstack.org/209189 | 17:15 |
| lucasagomes | folks I'm going to call it a day | 17:16 |
| lucasagomes | have a great night everyone! | 17:16 |
| NobodyCam | have a good night lucasagomes | 17:16 |
| lucasagomes | NobodyCam, g'night! | 17:16 |
| *** lucasagomes is now known as lucas-dinner | 17:16 | |
| sambetts | same, g,night everyone o/ | 17:16 |
| NobodyCam | night sambetts | 17:18 |
| NobodyCam | :) | 17:18 |
| sambetts | night NobodyCam | 17:18 |
| *** pal has quit IRC | 17:18 | |
| NobodyCam | :) | 17:19 |
| *** HenryG has quit IRC | 17:20 | |
| *** achanda has joined #openstack-ironic | 17:22 | |
| *** ndipanov has quit IRC | 17:25 | |
| *** athomas has quit IRC | 17:26 | |
| *** e0ne has quit IRC | 17:26 | |
| *** e0ne has joined #openstack-ironic | 17:32 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix detection of existence of image https://review.openstack.org/206337 | 17:37 |
| *** dprince has quit IRC | 17:37 | |
| TheJulia | SpamapS: ^^^^ Took your original idea and extended it... since I'm hitting a similar issue doing a ramdisk. | 17:38 |
| SpamapS | TheJulia: lurvely! | 17:38 |
| *** athomas has joined #openstack-ironic | 17:39 | |
| SpamapS | TheJulia: I'm currently rebuilding our bifrost and letting crinkle write me some puppet for that, so I'll have to let you know if it works later. | 17:39 |
| TheJulia | ok, are you adding in your own env var overrides, or are you using inventory/group_vars/all ? | 17:40 |
| TheJulia | (informal user survay since I want to get rid of group_vars/all) | 17:40 |
| TheJulia | (and I don't like options that I have at the moment... well... there is another way.) | 17:40 |
| *** marzif_ has quit IRC | 17:41 | |
| crinkle | TheJulia: my idea was to use https://review.openstack.org/#/c/202025/ from rcarrillocruz which uses group_vars/all | 17:41 |
| crinkle | TheJulia: but I am ++ on fixing that | 17:42 |
| rcarrillocruz | -e blah.yaml | 17:42 |
| rcarrillocruz | -e '@blah.yaml' rather | 17:42 |
| rcarrillocruz | that's how i've feed vars to my ansible stuff when i didn't want to mess with group_vars or role vars | 17:43 |
| TheJulia | rcarrillocruz: exactly | 17:43 |
| *** zhenguo has quit IRC | 17:43 | |
| *** praneshp has joined #openstack-ironic | 17:44 | |
| TheJulia | crinkle: I think that works, since we went to a dynamic inventory and distinctly separated baremetal/localhost, I should be able to drop the global defaults to group level files | 17:47 |
| *** achanda_ has joined #openstack-ironic | 17:48 | |
| *** david-lyle has quit IRC | 17:48 | |
| rcarrillocruz | speaking of which, EmilienM just approved the puppet bifrost change | 17:49 |
| *** jjohnson2 has joined #openstack-ironic | 17:50 | |
| *** achanda has quit IRC | 17:50 | |
| TheJulia | awesome | 17:53 |
| *** dprince has joined #openstack-ironic | 17:57 | |
| *** dprince has quit IRC | 17:57 | |
| *** alexpilo_ has joined #openstack-ironic | 17:58 | |
| *** alexpilo_ has quit IRC | 18:03 | |
| *** thiagop has joined #openstack-ironic | 18:04 | |
| thiagop | good ugt morning Ironicers | 18:04 |
| *** sambetts has quit IRC | 18:04 | |
| *** sambetts has joined #openstack-ironic | 18:05 | |
| NobodyCam | good (UGT) morning thiagop | 18:07 |
| *** dan_ has joined #openstack-ironic | 18:09 | |
| thiagop | hi NobodyCam | 18:10 |
| *** dan_ is now known as Guest41574 | 18:10 | |
| NobodyCam | :) | 18:10 |
| * NobodyCam issues vagrant up (again) then steps away for a few minutes | 18:12 | |
| *** Guest41574 is now known as dank_ | 18:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/208695 | 18:15 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-specs: Zapping spec follow up https://review.openstack.org/209207 | 18:16 |
| *** HenryG has joined #openstack-ironic | 18:31 | |
| *** HenryG has quit IRC | 18:32 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 18:33 |
| *** HenryG has joined #openstack-ironic | 18:35 | |
| *** bapalm_ has joined #openstack-ironic | 18:41 | |
| *** HenryG_ has joined #openstack-ironic | 18:46 | |
| *** HenryG has quit IRC | 18:48 | |
| *** cdearborn has quit IRC | 18:50 | |
| jroll | rcarrillocruz: have a link to puppet bifrost change? | 18:51 |
| *** cdearborn has joined #openstack-ironic | 18:51 | |
| *** boris-42 has joined #openstack-ironic | 18:53 | |
| NobodyCam | rcarrillocruz: or crinkle ??? | 18:53 |
| NobodyCam | 17:39 | SpamapS > TheJulia: I'm currently rebuilding our bifrost and letting crinkle write me some puppet for that, | 18:53 |
| * NobodyCam wounders who else he can inenvertently ping | 18:54 | |
| jroll | ooo I see it now | 18:54 |
| jroll | https://review.openstack.org/#/c/202025/ | 18:54 |
| crinkle | jroll: NobodyCam https://review.openstack.org/#/c/202025 | 18:54 |
| jroll | yep. thanks | 18:54 |
| crinkle | jinx | 18:54 |
| NobodyCam | :) | 18:54 |
| *** [1]cdearborn has joined #openstack-ironic | 18:55 | |
| jroll | that's interesting | 18:56 |
| jroll | I'm not sure I understand the why, but it's interesting | 18:56 |
| *** alexpilo_ has joined #openstack-ironic | 18:58 | |
| *** alexpilo_ has quit IRC | 19:02 | |
| *** coolsvap is now known as coolsvap|away | 19:10 | |
| *** cdearborn has quit IRC | 19:10 | |
| *** [1]cdearborn has quit IRC | 19:12 | |
| *** dprince has joined #openstack-ironic | 19:14 | |
| *** penick has joined #openstack-ironic | 19:18 | |
| openstackgerrit | Tom Cocozzello proposed openstack/ironic-python-agent: Activate pep8 check that _ is imported https://review.openstack.org/209225 | 19:19 |
| *** Marga_ has joined #openstack-ironic | 19:21 | |
| *** Marga_ has quit IRC | 19:24 | |
| *** Marga_ has joined #openstack-ironic | 19:25 | |
| openstackgerrit | Merged openstack/ironic: Fix a trusted boot test bug https://review.openstack.org/207319 | 19:27 |
| rcarrillocruz | hey | 19:30 |
| NobodyCam | hay | 19:30 |
| rcarrillocruz | sorry, was my son bath time | 19:30 |
| NobodyCam | :-p | 19:30 |
| rcarrillocruz | i will deploy that internally, the puppet bifrost thing | 19:31 |
| rcarrillocruz | happy to see that landing | 19:31 |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic-specs: Add out-of-band RAID configuration support for Cisco Hardware https://review.openstack.org/192142 | 19:38 |
| *** jcoufal has quit IRC | 19:39 | |
| *** [1]cdearborn has joined #openstack-ironic | 19:40 | |
| thiagop | NobodyCam: quick question | 19:44 |
| thiagop | NobodyCam: our driver naming exceeds the "driver" column on nodes tabels. What do we do? Insert it on the driver change or file a bug? | 19:45 |
| *** enikanorov_ has joined #openstack-ironic | 19:45 | |
| thiagop | table** | 19:45 |
| jroll | thiagop: make the driver name shorter? :) | 19:45 |
| * jroll wouldn't mind proposing an ALTER TABLE to make that field longer | 19:45 | |
| thiagop | jroll: our idea was to put the boot interface reference on the name until that driver composition change arrives | 19:46 |
| *** lsmola has quit IRC | 19:46 | |
| jroll | thiagop: well, there's two routes you can take. make the name shorter or the field longer | 19:47 |
| thiagop | but len("agent_pxe_oneview") > 15 | 19:47 |
| jroll | making the field longer will take another patch and a migration, but I would support it | 19:47 |
| jroll | hmm | 19:47 |
| *** ashuk has joined #openstack-ironic | 19:47 | |
| jroll | agt_pxe_oneview is 15 | 19:48 |
| *** enikanorov has quit IRC | 19:48 | |
| thiagop | would break the "agent_*" pattern be acceptable? | 19:48 |
| NobodyCam | agent_pxe_1view is also 15 char | 19:49 |
| jroll | or you could... ^ that | 19:49 |
| jroll | but branding and all | 19:49 |
| jroll | I'd just say agent_oneview, honestly | 19:49 |
| thiagop | Ok then, I'll keep it simple | 19:51 |
| thiagop | (I'll suggest that later :v) | 19:51 |
| *** alexpilo_ has joined #openstack-ironic | 19:53 | |
| *** ashuk has quit IRC | 19:53 | |
| NobodyCam | just checking anyone played with the --ironic-api-version option in cli.. http://paste.openstack.org/show/gp1TXgTpszPS3CSQUC5R/ | 19:53 |
| jroll | ironic --ironic-api-version command blablabla | 19:54 |
| *** ashuk has joined #openstack-ironic | 19:54 | |
| jroll | just like --debug, has to come before the command | 19:54 |
| NobodyCam | ahhh :-p *facepalm* | 19:55 |
| NobodyCam | doh TY jroll :) | 19:55 |
| jroll | np :) | 19:55 |
| jroll | (I also think it's weird) | 19:55 |
| *** ekarlso has quit IRC | 19:55 | |
| *** alexpilo_ has quit IRC | 19:57 | |
| *** ekarlso has joined #openstack-ironic | 19:57 | |
| *** dprince has quit IRC | 20:07 | |
| *** zz_natorious is now known as natorious | 20:09 | |
| *** marzif_ has joined #openstack-ironic | 20:12 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Move DIB components to support depends-on https://review.openstack.org/208577 | 20:12 |
| *** natorious is now known as zz_natorious | 20:16 | |
| *** Marga_ has quit IRC | 20:21 | |
| *** Nisha has joined #openstack-ironic | 20:23 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Rename install role to bifrost-ironic-install https://review.openstack.org/209245 | 20:29 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Updated self referential text in the install role https://review.openstack.org/209246 | 20:29 |
| *** meghal has quit IRC | 20:31 | |
| *** meghal has joined #openstack-ironic | 20:31 | |
| *** meghal has quit IRC | 20:32 | |
| *** meghal has joined #openstack-ironic | 20:32 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 20:36 |
| *** ukalifon1 has joined #openstack-ironic | 20:43 | |
| *** albertoffb has quit IRC | 20:51 | |
| *** alexpilo_ has joined #openstack-ironic | 20:51 | |
| *** alexpilo_ has quit IRC | 20:56 | |
| *** lucas-dinner has quit IRC | 20:56 | |
| zer0c00l | TheJulia: I remember you talking about adding support for building gpt disks with diskimge-builder | 21:00 |
| zer0c00l | I wrote some code that does the same. | 21:00 |
| zer0c00l | May be i should push it for review? | 21:00 |
| TheJulia | OHHH | 21:00 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add node-create example to Troubleshooting section of install-guide https://review.openstack.org/209251 | 21:01 |
| *** Marga_ has joined #openstack-ironic | 21:01 | |
| *** meghal has quit IRC | 21:01 | |
| NobodyCam | let me know if that (^^^^^^^^) helps at all? | 21:01 |
| *** jjohnson2 has quit IRC | 21:05 | |
| thiagop | NobodyCam: why not in the "Enrollment" section? | 21:05 |
| NobodyCam | ???? | 21:05 |
| NobodyCam | thiagop: i'm jumping in to the project meeting. please leave a comment | 21:06 |
| NobodyCam | :) | 21:06 |
| thiagop | NobodyCam: ok | 21:06 |
| NobodyCam | TY :) | 21:06 |
| *** meghal has joined #openstack-ironic | 21:07 | |
| *** bapalm_ has quit IRC | 21:07 | |
| *** HenryG_ is now known as HenryG | 21:17 | |
| *** Marga_ has quit IRC | 21:21 | |
| *** Marga_ has joined #openstack-ironic | 21:22 | |
| *** Nisha has quit IRC | 21:33 | |
| *** kkoski_ has quit IRC | 21:38 | |
| *** e0ne has quit IRC | 21:44 | |
| *** alexpilo_ has joined #openstack-ironic | 21:51 | |
| mrda | Morning Ironic | 21:52 |
| NobodyCam | morning mrda | 21:54 |
| mrda | hey NobodyCam | 21:54 |
| NobodyCam | :) | 21:55 |
| *** alexpilo_ has quit IRC | 21:55 | |
| *** ukalifon1 has quit IRC | 22:01 | |
| *** meghal has quit IRC | 22:03 | |
| *** zz_natorious is now known as natorious | 22:04 | |
| *** chlong has joined #openstack-ironic | 22:07 | |
| *** [1]cdearborn has quit IRC | 22:16 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add node-create example to Troubleshooting section of install-guide https://review.openstack.org/209251 | 22:20 |
| NobodyCam | thiagop: ^^^^^ | 22:20 |
| *** meghal has joined #openstack-ironic | 22:32 | |
| *** ijw has joined #openstack-ironic | 22:33 | |
| *** thrash is now known as thrash|g0ne | 22:34 | |
| *** jaypipes has quit IRC | 22:35 | |
| *** boris-42 has quit IRC | 22:40 | |
| *** alexpilo_ has joined #openstack-ironic | 22:53 | |
| *** penick has quit IRC | 22:53 | |
| *** alexpilo_ has quit IRC | 22:58 | |
| *** Vikas has joined #openstack-ironic | 23:00 | |
| *** Marga_ has quit IRC | 23:01 | |
| *** ijw_ has joined #openstack-ironic | 23:27 | |
| *** hemna has quit IRC | 23:28 | |
| *** ijw has quit IRC | 23:30 | |
| *** ijw has joined #openstack-ironic | 23:33 | |
| *** ijw_ has quit IRC | 23:36 | |
| *** boris-42 has joined #openstack-ironic | 23:37 | |
| *** mariojv has quit IRC | 23:38 | |
| *** mariojv has joined #openstack-ironic | 23:38 | |
| *** jamielennox|away is now known as jamielennox | 23:50 | |
| *** alexpilo_ has joined #openstack-ironic | 23:53 | |
| *** yuanying has quit IRC | 23:55 | |
| *** alexpilo_ has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!