Monday, 2015-08-31

*** baoli has quit IRC00:02
*** microphex has quit IRC00:03
*** achanda has joined #openstack-ironic00:06
*** alexpilotti has quit IRC00:20
*** shadower has quit IRC00:23
*** shadower has joined #openstack-ironic00:23
*** meghal has joined #openstack-ironic00:31
*** meghal1 has joined #openstack-ironic00:33
*** alexpilotti has joined #openstack-ironic00:35
*** meghal has quit IRC00:36
*** alexpilotti has quit IRC00:40
openstackgerritNaohiro Tamura proposed openstack/ironic: Add hardware inspection module for iRMC driver  https://review.openstack.org/19648000:40
*** achanda has quit IRC00:44
*** alexpilotti has joined #openstack-ironic00:47
openstackgerritNaohiro Tamura proposed openstack/ironic: Fix docker package name in Ubuntu 14.04 in Install Guide  https://review.openstack.org/21869400:49
*** chlong has joined #openstack-ironic00:55
*** sambetts has joined #openstack-ironic00:57
*** shoutm has quit IRC01:01
*** shoutm has joined #openstack-ironic01:09
*** baoli has joined #openstack-ironic01:13
*** harshs has joined #openstack-ironic01:13
*** valkyrja has quit IRC01:17
*** valkyrja has joined #openstack-ironic01:17
*** baoli has quit IRC01:19
*** zhenguo has joined #openstack-ironic01:24
*** baoli has joined #openstack-ironic01:33
*** marzif__ has joined #openstack-ironic01:33
*** Haomeng|2 has joined #openstack-ironic01:35
*** Haomeng has quit IRC01:35
*** marzif_ has quit IRC01:37
*** achanda has joined #openstack-ironic01:37
*** Haomeng|2 has quit IRC01:41
*** shoutm has quit IRC01:42
*** shoutm has joined #openstack-ironic01:42
*** baoli has quit IRC01:46
*** alexpilotti has quit IRC01:47
*** chlong has quit IRC01:49
*** baoli has joined #openstack-ironic02:00
*** alexpilotti has joined #openstack-ironic02:03
*** meghal1 has quit IRC02:07
*** alexpilotti has quit IRC02:08
*** alexpilotti has joined #openstack-ironic02:15
*** Haomeng has joined #openstack-ironic02:18
*** achanda has quit IRC02:22
*** achanda has joined #openstack-ironic02:24
*** chenglch has joined #openstack-ironic02:39
*** hakimo_ has joined #openstack-ironic02:52
*** harshs has quit IRC02:52
*** hakimo has quit IRC02:54
openstackgerritZhenguo Niu proposed openstack/ironic: When boot option is not persisted, set boot on next power on  https://review.openstack.org/17764202:57
*** alexpilotti has quit IRC03:07
*** alexpilotti has joined #openstack-ironic03:22
*** praneshp_ has quit IRC03:24
*** eglute has quit IRC03:25
*** praneshp has joined #openstack-ironic03:25
*** zz_natorious has quit IRC03:25
*** alexpilotti has quit IRC03:26
*** jroll has quit IRC03:26
*** saripurigopi has joined #openstack-ironic03:31
*** zz_natorious has joined #openstack-ironic03:31
*** jroll has joined #openstack-ironic03:31
*** eglute has joined #openstack-ironic03:31
*** zz_natorious is now known as natorious03:31
openstackgerritZhenguo Niu proposed openstack/ironic: Check image size before provisioning for agent driver  https://review.openstack.org/21525403:32
*** alexpilotti has joined #openstack-ironic03:34
*** links has joined #openstack-ironic03:40
*** garthb has joined #openstack-ironic03:55
*** rameshg87 has joined #openstack-ironic04:01
*** shoutm has quit IRC04:04
*** shoutm has joined #openstack-ironic04:06
*** baoli has quit IRC04:07
*** VikasC has joined #openstack-ironic04:36
*** sonuk has joined #openstack-ironic04:40
*** Nisha has joined #openstack-ironic04:55
*** Sukhdev has joined #openstack-ironic04:56
*** ig0r__ has joined #openstack-ironic04:59
*** ig0r_ has quit IRC04:59
*** VikasC has quit IRC05:03
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Support multiple deploy mechanisms simultaneously  https://review.openstack.org/20982105:03
*** puranamr has joined #openstack-ironic05:05
openstackgerritRamakrishnan G proposed openstack/ironic: Add new method validate_raid_config to RAIDInterface  https://review.openstack.org/21694605:07
openstackgerritRamakrishnan G proposed openstack/ironic: Add RPC APIs for RAID configuration  https://review.openstack.org/19600605:08
*** VikasC has joined #openstack-ironic05:08
*** garthb has quit IRC05:35
*** baoli has joined #openstack-ironic05:38
openstackgerritRamakrishnan G proposed openstack/ironic: Add APIs for RAID configuration  https://review.openstack.org/19600705:39
*** baoli has quit IRC05:42
*** harshs has joined #openstack-ironic05:43
openstackgerritNisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers  https://review.openstack.org/21706805:50
*** shoutm has quit IRC05:54
openstackgerritNisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers  https://review.openstack.org/21706805:56
*** vvb has joined #openstack-ironic06:15
*** davideagnello has joined #openstack-ironic06:16
*** ishant has joined #openstack-ironic06:18
*** vvb has quit IRC06:21
*** vvb has joined #openstack-ironic06:21
*** harshs has quit IRC06:22
openstackgerritNisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers  https://review.openstack.org/21706806:25
*** davideagnello has quit IRC06:29
saripurigopirameshg87: could you take a look @ https://review.openstack.org/#/c/192142/, https://review.openstack.org/#/c/204733/ when you get time.?06:39
*** VikasC has quit IRC06:40
rameshg87saripurigopi: sure. I can only give a +1, but it looked good to me a long back. :)06:40
rameshg87saripurigopi: I will update my vote06:40
saripurigopirameshg87: sure thank you.06:41
*** shoutm has joined #openstack-ironic06:43
*** praneshp has quit IRC06:54
*** dlpartain has joined #openstack-ironic06:56
*** puranamr has quit IRC06:58
*** shoutm_ has joined #openstack-ironic06:59
*** Sukhdev has quit IRC07:00
*** shoutm has quit IRC07:00
*** dlpartain has left #openstack-ironic07:03
*** natorious is now known as zz_natorious07:03
*** achanda has quit IRC07:05
*** ukalifon has joined #openstack-ironic07:06
openstackgerritNisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers  https://review.openstack.org/19865607:07
openstackgerritNisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers  https://review.openstack.org/21706807:17
*** mgoddard has joined #openstack-ironic07:19
*** VikasC has joined #openstack-ironic07:23
*** romainh has joined #openstack-ironic07:27
*** mgoddard has quit IRC07:34
*** vvb has left #openstack-ironic07:34
*** mgoddard has joined #openstack-ironic07:34
*** mgoddard has quit IRC07:34
*** vvb has joined #openstack-ironic07:34
*** ndipanov has joined #openstack-ironic07:47
*** Nisha_away has joined #openstack-ironic07:48
*** Nisha has quit IRC07:48
*** Nisha_away has quit IRC07:52
*** alexpilotti has quit IRC07:56
*** alexpilotti has joined #openstack-ironic07:57
*** baoli has joined #openstack-ironic07:58
*** alexpilotti has joined #openstack-ironic07:59
*** MattMan has joined #openstack-ironic07:59
*** baoli has quit IRC08:03
*** ifarkas has joined #openstack-ironic08:03
*** alexpilotti has quit IRC08:04
*** Nisha_away has joined #openstack-ironic08:04
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI  https://review.openstack.org/18670008:05
*** jistr has joined #openstack-ironic08:15
*** lucasagomes has joined #openstack-ironic08:15
*** derekh has joined #openstack-ironic08:15
*** alexpilotti has joined #openstack-ironic08:17
*** stendulker has joined #openstack-ironic08:18
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI  https://review.openstack.org/18670008:21
aarefievJayF: hi, thanks Jay, yea I'm new, I'm glad to join ironic folks, hope I'll be useful for community.08:21
*** alexpilotti has quit IRC08:27
*** yog_ has joined #openstack-ironic08:28
*** shoutm_ has quit IRC08:36
saripurigopiNobodyCam, lucasagomes, jlvillal: could you have a look at @ https://review.openstack.org/#/c/192142/, https://review.openstack.org/#/c/204733/ when you get time.?08:38
lucasagomessaripurigopi, hi there, will do sure08:40
lucasagomesaarefiev, welcome!08:40
aarefievlucasagomes: hi, thanks)08:41
Haomenglucasagomes: good morning, Lucas:)08:41
Haomengaarefiev: welcome:)08:41
lucasagomesHaomeng, hi buddy, good mornign!08:41
Haomenglucasagomes: :)08:41
aarefievHaomeng: thanks:)08:41
*** amotoki has joined #openstack-ironic08:41
Haomengamotoki: :)08:41
aarefievthanks guys:)08:41
*** alexpilotti has joined #openstack-ironic08:42
Haomenglucasagomes: I want to add bulk-power function for our api, add the filters to check nodes out and bulk-power on/off, do you think if this is good idea or not?08:43
Haomenglucasagomes: and will add client options supports08:43
lucasagomesHaomeng, yeah it should be all good, the power_state is already indexable in the database08:43
saripurigopilucasagomes: o/ thank you08:43
lucasagomesso you can do it the same way we do for things like provision state08:44
lucasagomesHaomeng, +108:44
Haomenglucasagomes: how about to add bulk action on chassis level?08:44
Haomenglucasagomes: thanks:)08:44
lucasagomesHaomeng, chassis has an /nodes endpoint08:44
Haomenglucasagomes: I just think about some days, which should be common bulk operations supporting08:44
lucasagomesso once added to the nodes the same endpoing is shared on the chassis so yes08:44
Haomenglucasagomes: maybe more commands need bulk-operation supporting for enduser to easy control *bulk* devices08:45
Haomenglucasagomes: so what do you think how can we add such bulk-operation on common level?08:45
Haomenglucasagomes: or just add on special operations such as power on/off, set boot device08:46
lucasagomesHaomeng, what you want is something like https://review.openstack.org/#/c/196529/ ?08:46
lucasagomesoh you want to set the power state08:46
lucasagomesgotcha08:46
Haomenglucasagomes: k, let me check08:46
lucasagomeshmm I haven't thought about it, usually we do some bulk stuff in the client08:46
lucasagomesnot sure if it's perfect tho08:46
lucasagomesbut for example you can say ironic node-delete <uiud> <uuid> <uuid>08:46
*** alexpilotti has quit IRC08:46
lucasagomesetc08:47
lucasagomesand the client loops through the api and issue the commands, perhaps we would need to support something like that in the API08:47
Haomenglucasagomes: yes, it is better that is filter based for user , that will be easy to use08:47
* lucasagomes haven't thought much about it08:47
Haomenglucasagomes: yes, thanks lucas:)08:48
Haomenglucasagomes: nice day, I will go home:)08:48
lucasagomesHaomeng, cool, but in any case yeah it seems useful08:48
lucasagomesHaomeng, oh yeah, forgot it's late there08:48
lucasagomeshave a good evening!08:48
lucasagomessafe trip home08:48
Haomenglucasagomes: when you checkin on morning, I will go home:)08:48
Haomenglucasagomes: thank you:)08:48
lucasagomesyeah08:48
Haomenglucasagomes: :)08:48
lucasagomesTZ's are difficult08:48
Haomenglucasagomes: today you are early:)08:48
lucasagomes:-)08:49
lucasagomesjust a bit08:49
Haomenglucasagomes: ok, bye, good luck:)08:49
lucasagomesbye bye08:49
Haomenglucasagomes: :)08:49
*** alexpilotti has joined #openstack-ironic08:49
*** VikasC has quit IRC08:51
*** e0ne has joined #openstack-ironic08:58
*** e0ne has quit IRC09:09
*** e0ne has joined #openstack-ironic09:09
*** alexpilotti has quit IRC09:24
openstackgerritViktor Serhieiev proposed openstack/ironic: Use oslo.messaging serializers  https://review.openstack.org/18870109:26
*** pelix has joined #openstack-ironic09:27
*** alexpilotti has joined #openstack-ironic09:32
*** alexpilotti has quit IRC09:36
*** ig0r_ has joined #openstack-ironic09:37
*** marzif__ has quit IRC09:38
*** marzif__ has joined #openstack-ironic09:38
*** ig0r__ has quit IRC09:39
openstackgerritNisha Agarwal proposed openstack/ironic: ilo drivers powers off the node in ACTIVE state  https://review.openstack.org/21878509:41
*** VikasC has joined #openstack-ironic09:47
*** alexpilotti has joined #openstack-ironic09:48
Nisha_awaylucasagomes, good morning09:49
*** romcheg has joined #openstack-ironic09:50
Nisha_awaylucasagomes, please look at https://review.openstack.org/198656 https://review.openstack.org/217068 https://review.openstack.org/201904 https://review.openstack.org/21878509:50
*** romainh has quit IRC09:50
*** naohirot has quit IRC09:56
lucasagomesNisha_away, morning! akc09:57
lucasagomesack*09:57
*** Nisha_away is now known as Nisha09:57
Nishalucasagomes, thanks....09:57
Nishasince next ironic tagging is expected to happen by this month end, started pinging again and again09:58
Nishameans Sept end09:58
*** degorenko has joined #openstack-ironic10:08
*** marzif__ has quit IRC10:12
*** getvasanth has joined #openstack-ironic10:15
getvasanthHi All, i got the image booted, but unfortunately i get an error saying "Couldn't determine the UUID of the root partition or the disk identifier after deploying"10:18
getvasanthcan someone point me how i can specify the disk id to image while booting10:19
*** shoutm has joined #openstack-ironic10:27
*** Nisha has quit IRC10:31
rcarrillocruzHaomeng: heya, thanks for the review on https://review.openstack.org/#/c/218561/ . Mind if I investigate the static routes on a follow-up patch?10:35
*** vvb has quit IRC10:37
*** vvb has joined #openstack-ironic10:37
*** vvb has quit IRC10:42
*** marzif__ has joined #openstack-ironic10:47
lucasagomesgetvasanth, the image you're using is a full disk image? Or a partition image?10:50
lucasagomesfull disk image = an image with a partition table10:50
getvasanthlucasagomes: not sure, i built it with DIB and uploaded it, how to verify that image has full disk image10:56
lucasagomesgetvasanth, did you use the "vm" element?10:56
*** shoutm has quit IRC10:56
lucasagomesgetvasanth, you always can mount the image on a loop device and see if it has a partition table10:56
lucasagomes$ qemu-nbd -c /dev/loop0 <image>10:56
getvasanthlucasagomes: No i didnt use the vm module10:57
lucasagomes$fdisk -l /dev/loop010:57
lucasagomesgetvasanth, right, and it's a normal fedora/ubuntu image?10:57
lucasagomesodd10:57
getvasanthlucasagomes: yes, ubuntu image10:57
lucasagomestry to mount that in a loop ^ see if fdisk says something about it please10:57
getvasanthlucasagomes: yes, will mount and see10:58
*** jaypipes has joined #openstack-ironic10:58
lucasagomes(those commands are for qcow images, if it's raw you just need to use losetup10:58
lucasagomesack10:58
lucasagomesmake sure you are root as well and have the ndb module loaded10:58
*** alexpilotti has quit IRC11:05
*** e0ne has quit IRC11:10
openstackgerritWilliam Stevenson proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration  https://review.openstack.org/20614411:11
*** shoutm has joined #openstack-ironic11:19
*** alexpilotti has joined #openstack-ironic11:21
*** rameshg87 has quit IRC11:23
*** alexpilotti has quit IRC11:25
*** ishant has quit IRC11:25
*** jcoufal has joined #openstack-ironic11:28
openstackgerritWilliam Stevenson proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration  https://review.openstack.org/20614411:28
*** lucasagomes is now known as lucas-hungry11:32
*** alexpilotti has joined #openstack-ironic11:33
Haomengrcarrillocruz: sure, thanks:)11:34
Haomengrcarrillocruz: so I did not -1:)11:34
*** jaypipes has quit IRC11:35
rcarrillocruzheh, yeah :P11:35
*** saripurigopi has quit IRC11:35
*** jaypipes has joined #openstack-ironic11:36
*** lazy_prince has joined #openstack-ironic11:38
*** amotoki has quit IRC11:39
*** amotoki has joined #openstack-ironic11:39
*** stendulker has quit IRC11:40
*** marzif__ has quit IRC11:42
*** trown|outttypeww is now known as trown11:44
*** chenglch has quit IRC11:44
*** openstackgerrit has quit IRC11:46
*** openstackgerrit has joined #openstack-ironic11:47
Haomengrcarrillocruz: :)11:47
*** thrash|wknd has quit IRC11:58
*** thrash has joined #openstack-ironic11:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Use vendor mixin in IPMITool drivers  https://review.openstack.org/21882111:59
*** e0ne has joined #openstack-ironic11:59
TheJuliagood morning #openstak-ironic12:01
getvasanthlucasagomes: having trouble in getting the nbd server running, will fix and post you12:03
getvasanthlucasagomes: what is the difference between providing the "vm" option while creating the image?12:06
*** dprince has joined #openstack-ironic12:08
*** romainh has joined #openstack-ironic12:09
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/21656212:12
TheJuliagetvasanth: vm element for disk image builder applies a bootloader, or at least attempts to12:12
*** dlpartain has joined #openstack-ironic12:15
getvasanthTheJulia: thanks for the information, I am rebuilding the image without the vm option12:15
*** dlpartain has left #openstack-ironic12:16
*** zz_natorious is now known as natorious12:16
*** natorious is now known as zz_natorious12:17
*** trown is now known as trown|relocating12:19
*** vvb has joined #openstack-ironic12:25
*** baoli has joined #openstack-ironic12:29
*** baoli_ has joined #openstack-ironic12:31
*** karimb has joined #openstack-ironic12:31
*** baoli_ has quit IRC12:31
*** baoli_ has joined #openstack-ironic12:32
*** alexpilotti has quit IRC12:33
*** baoli has quit IRC12:34
*** krtaylor has quit IRC12:34
openstackgerritJulia Kreger proposed openstack/bifrost: Change playbook to assert ironic owns the boot.ipxe file  https://review.openstack.org/21883312:36
*** VikasC has quit IRC12:37
*** smoriya has quit IRC12:42
*** krtaylor has joined #openstack-ironic12:47
*** ifarkas has quit IRC12:48
*** alexpilotti has joined #openstack-ironic12:48
*** ifarkas has joined #openstack-ironic12:49
*** lucas-hungry is now known as lucasagomes12:50
lucasagomesgetvasanth, sorry was having lunch. Yeah as TheJulia pointed out the vm element creates an image with a partition table and a bootloader12:50
lucasagomesyou still can deploy that via Ironic, but when you upload the image to glance you should not associate a kernel+ramdisk in the glance metadata12:51
lucasagomesand Ironic will deploy it as a full disk image12:51
*** zz_natorious is now known as natorious12:52
TheJuliaNobodyCam: When you get up, 21829012:52
*** alexpilotti has quit IRC12:53
openstackgerritJulia Kreger proposed openstack/bifrost: Correct log file storage location  https://review.openstack.org/21884112:56
rcarrillocruzTheJulia: thanks for 218833, i hit that today12:56
rcarrillocruzi was wondering why +w was needed :/12:56
*** natorious is now known as zz_natorious12:57
*** ukalifon has quit IRC12:57
getvasanthlucasagomes: I have a weird problem, I recreated the image without having the vm option, and booted with nova, nova list says the instance is active, but when i see the ilo, it got rebooted once and stuck in "random: nonblocking pool is initialized"12:58
TheJuliarcarrillocruz: looks like the pxe refactor stuff if I had to guess12:58
TheJuliarcarrillocruz: also looking at some weirdness with url generation in files12:58
TheJuliakernel http://192.168.122.1:8080http://192.168.122.1:8080//a8cb6624-0d9f-c882-affc-046ebb96ec01/deploy_kernel12:59
getvasanthlucasagomes: i have associated the kernel & ramdisk with it12:59
*** alexpilotti has joined #openstack-ironic13:00
*** ukalifon2 has joined #openstack-ironic13:00
TheJuliarcarrillocruz: looks like a defect in the agent config template has been fixed, I'm going to try pushing up a change for bifrost and we might see things mostly clear up shortly13:00
rcarrillocruz++13:00
*** afaranha has joined #openstack-ironic13:02
lucasagomesgetvasanth, right, so this error is in the iLO initialization?13:04
lucasagomesit's not a boot issue is it?13:04
lucasagomescan you tell me if the machine loaded the kernel+ramdisk ?13:04
*** mjturek1 has joined #openstack-ironic13:05
getvasanthlucasagomes: yes the machine loaded the kernel+ramdisk13:05
lucasagomesgetvasanth, and it hangs forever on that message?13:06
getvasanthlucasagomes: but how does the nova still says the instance is in running state13:06
lucasagomesgetvasanth, Nova just remap the provision_state in Ironic13:07
getvasanthlucasagomes: yes, it is hung there13:07
lucasagomesIronic marked the node as ACTIVE and that gets passed to nova13:07
lucasagomesbut Ironic doesn't have any control after the boot starts13:07
getvasanthlucasagomes: ah!13:07
lucasagomesso it's either some kernel issue or some BIOS misconfiguration13:07
lucasagomesthat is causing the boot to hang13:08
openstackgerritJulia Kreger proposed openstack/bifrost: Update iPXE agent template  https://review.openstack.org/21884313:08
TheJuliarcarrillocruz: ^^13:08
getvasanthlucasagomes: BIOS configuration: can u tell me what needs to be checked?13:08
getvasanthlucasagomes: do u have any valid image which boots without any issue13:09
lucasagomesgetvasanth, it may vary from hardware to hardware13:09
lucasagomesI had a couple of problems with iSCSI HBA13:10
lucasagomeswhen it was on it was causing some instances to not find the local disk13:10
getvasanthlucasagomes: I tried the same image in openstack, it booted without any issue.13:10
lucasagomesbut hey, hardware can be tricky sometimes :-(13:10
getvasanthlucasagome: its too tricky now :(13:10
lucasagomesyeah VMs are pretty consistent it easy to get it right. But sometimes hardware sucks... You can try to boot the same version of the OS you created on that machine13:11
lucasagomesusing a livecd13:11
lucasagomescheck the kernel options it passes see if you can spot something there13:11
lucasagomesin Ironci theres a "pxe_append_params" configuration options that is used to set kernel options on the machines13:11
lucasagomes(in the PXE template)13:11
lucasagomesperhaps maybe some of the iLO guys upstream may help you better with iLO stuff13:12
lucasagomesramesh, Nisha (they are offline now :-( )13:12
getvasanthlucasagomes: i can see there is a id added in the pxe-config UUID has been added13:13
lucasagomesyeah in ur tftp root you'll see13:13
lucasagomes /tftpboot/<node uuid>/config13:13
rcarrillocruzTheJulia: +113:13
rcarrillocruzalso13:13
rcarrillocruzhttps://review.openstack.org/#/c/218561/13:13
lucasagomesthat's the PXE config and the kernel cmdlines are there13:13
lucasagomesso you can tweak it and reboot the node13:14
*** rloo has joined #openstack-ironic13:14
lucasagomesbtw googling there's a bunch of topics related to machines getting stuck at that stage13:14
getvasanthlucasagomes: yeah, i am checking that13:14
rcarrillocruzwith that, i force default route the same as the one i define on the static interface, regardless if the dhcp interface runs afterwards and pulls default route from dnsmasq13:14
*** jcoufal has quit IRC13:17
*** kbyrne has quit IRC13:19
NobodyCamgood morning Ironicers...13:21
NobodyCamTheJulia: I'll take a look here is in a few... need coffee first13:21
rcarrillocruzmorning13:21
lucasagomesNobodyCam, TheJulia rcarrillocruz morning ugt13:22
*** e0ne has quit IRC13:22
*** Nisha has joined #openstack-ironic13:22
rloohi lucasagomes, morning rcarrillocruz, NobodyCam, TheJulia, getvasanth13:22
TheJuliaNobodyCam: yes, yes, coffee, lots of coffee.13:23
lucasagomesrloo, hi there! good morning13:24
TheJuliarcarrillocruz: on a good note, 218843 + its parent result in my local test job passing13:24
rcarrillocruzsweet13:25
NobodyCammorning rcarrillocruz lucasagomes rloo sambetts jlvillal jroll(back?) JoshNang JayF TheJulia BadCub getvasanth and anyone else I'm missed in my pre-coffee state13:26
rcarrillocruzheh13:26
TheJuliaIt may still be dark there, you might be able to say good morning to the moon :)13:26
*** e0ne has joined #openstack-ironic13:29
*** mtanino has joined #openstack-ironic13:30
*** links has quit IRC13:30
*** jcoufal has joined #openstack-ironic13:30
getvasanthrloo: good morning to all13:31
NobodyCamTheJulia: yep it is still dark here13:31
*** jcoufal has quit IRC13:31
*** kbyrne has joined #openstack-ironic13:32
*** jcoufal has joined #openstack-ironic13:32
*** trown|relocating is now known as trown13:35
NobodyCamlucasagomes: looking at 218290 ... lgtm, just a question, would you see any value in diff'ing the file that would written to the file on disk to make sure there are not any changes?13:36
*** dims has joined #openstack-ironic13:36
*** ukalifon2 has quit IRC13:39
*** Nisha has quit IRC13:39
*** Nisha has joined #openstack-ironic13:39
lucasagomesNobodyCam, hmm well the idea is also to enable people to do changes to the file13:40
lucasagomesand test it quicker than having to change the ironic config13:40
lucasagomeslike we have to the PXE config files13:40
openstackgerritWilliam Stevenson proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624413:40
NobodyCamahh ok :) that makes sense :)13:41
lucasagomesso I would say testing the existence of the file seems enough (at least for the use case I saw)13:41
lucasagomeswhen I had to change it to test with some old iPXE rooms13:41
lucasagomesROMs*13:41
NobodyCamyep.. +a'd :) Ty lucasagomes :)13:42
lucasagomeso/ thanks much!13:42
openstackgerritWilliam Stevenson proposed openstack/ironic: Add portgroups to support LAG interfaces - net  https://review.openstack.org/20624513:44
openstackgerritWilliam Stevenson proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968713:45
*** Nisha has quit IRC13:51
openstackgerritWilliam Stevenson proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326213:52
lazy_princeI am having some issues with https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/post-install.d/80-ironic-agent#L2013:53
lazy_princeit errors out with No Such File Found error.. Any Clue..?13:54
openstackgerritMerged openstack/ironic: Use automaton's converters/pydot  https://review.openstack.org/20672613:54
*** karimb has quit IRC13:54
*** Nisha has joined #openstack-ironic13:54
openstackgerritMerged openstack/ironic: Add minimum and maximum on port option  https://review.openstack.org/21701913:55
*** karimb has joined #openstack-ironic13:56
TheJuliahmm.. arent-agent dib is still not working with debian :(13:59
lazy_princeThat was more of like curl issue.. this is more of like systemctl issues..13:59
NobodyCamlazy_prince: but this line didn't fail? https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/install.d/ironic-agent-source-install/60-ironic-agent-install#L3214:01
*** ukalifon1 has joined #openstack-ironic14:01
TheJuliaI'm getting an error that leads me to think the rm is failing, but it could be something else.  Seems like debugging is needed14:01
TheJuliaso trown set that line in place14:01
TheJuliaand the source install enables it14:02
TheJuliaso its a fedora/redhatism14:02
lazy_princeNobodyCam: yes.. that line did not fail.. On commenting https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/post-install.d/80-ironic-agent#L20, build completes.. not sure though if the image is usable..14:02
TheJuliahttps://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/install.d/ironic-agent-source-install/60-ironic-agent-install14:02
lazy_princeThe https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/install.d/ironic-agent-source-install/60-ironic-agent-install did not fail.14:03
TheJuliaI know, but look at git blame, then look at service names14:03
NobodyCamlazy_prince: are you doing source install or package?14:04
TheJuliamake two post-install scripts? one for source, one for package?14:04
lazy_princeI am not sure how to alter between both... is there a switch or something...?14:04
*** bnemec has joined #openstack-ironic14:05
lazy_princeThis is how I executed "ramdisk-image-create -x -o $image_name $LinuxDistro  ironic-agent"14:06
lazy_princeI believe, its downloading sources for IPA and building it..14:06
NobodyCamlazy_prince: maybe try setting DIB_DEFAULT_INSTALLTYPE14:07
NobodyCamthat should be the default14:07
lazy_princeNobodyCam: to..?14:07
TheJuliaby default, that element should end up going the source route14:07
trownTheJulia: without that line, how does the service get enabled?14:08
* trown was worried about how to name that14:08
TheJuliatrown: the source based install installs it as ironic-python-agent14:08
TheJuliaand enabled the service14:08
lazy_princemaybe, we should name them properly..14:08
TheJuliaso either the package needs to enable the service, or we need to deliniate in the post steps if this was a source install, or package install14:09
*** jroll has quit IRC14:09
*** jroll has joined #openstack-ironic14:09
trownTheJulia: I am working on the Fedora packaging today, so I will just make the package enable the service14:10
trownTheJulia: I will put up a patch to remove that from post-install14:10
lazy_princeaha.. then we do not need it in the post step..14:10
lazy_princeshall I log a bug on this then..?14:11
trownsorry for the breakage14:11
*** boris-42 has joined #openstack-ironic14:11
TheJuliatrown: instead of having the rpm do it by deault and somebody possibly do something bad to their system14:11
TheJuliatrown: why not just place a script to do the same job in intall.d/ironic-agent-package/install14:11
trownTheJulia: hmm, ya that is probably better14:12
TheJuliaand you could put up the change quicker, and then we could pester dib cores to land it when they have had some coffee :)14:12
trownalthough there would have to be a few misteps to cause a problem just by enabling the agent service14:12
TheJuliatrown: how so?14:12
trowni.e. the machine would have to be registered to Ironic14:13
trownotherwise it does nothing14:13
TheJuliaBut we don't know if that will change one day :)14:13
jrollgood morning all \o14:13
TheJuliagood morning jroll14:13
trownya, I like not having the package auto-enable, so I will put up the dib patch to do it14:14
TheJuliaawesome, ping us when you have the review up :)14:14
lazy_princehttps://bugs.launchpad.net/diskimage-builder/+bug/149058414:14
openstackLaunchpad bug 1490584 in diskimage-builder "ramdisk-image-create ironic-agent fails when using source-install" [Undecided,New]14:14
trownthanks lazy_prince, I will reference that14:14
NobodyCammorning jroll :) welcome back14:14
jrollthanks NobodyCam :)14:14
lazy_princecool.. hope to have the fix land soon...14:15
lucasagomeslazy_prince, I think that for the ironic-agent we don't use ramdisk-image-create14:15
lucasagomeswe use disk-image-create14:15
lucasagomescause we need a full OS with systemd and so on14:15
lucasagomesramdisk-image-create just lay down a filesystem with busybox on it14:15
trownlucasagomes: I think I have been testing with ramdisk-image-create14:15
lucasagomesreally? Ok14:16
lucasagomeswell, I've always tested it with disk-image-create14:16
lazy_princelucasagomes: umm.. just wondering how is the full disk image passed during pxe boot..?14:16
lucasagomesFWIW trown lazy_prince http://docs.openstack.org/developer/ironic/deploy/install-guide.html#buildingdibbaseddeployramdisk14:16
lucasagomeslazy_prince, the same, at the end of the process we cpio and zip the contents14:17
lazy_princehmm.. looks like the doc needs to be corrected...14:17
trownlucasagomes: this is the patch I have been using to build the agent with RDO: https://review.gerrithub.io/#/c/241872/14:17
*** vvb has quit IRC14:18
lucasagomesI see, yeah well I haven't tried it with ramdisk-image-create14:18
lucasagomeslazy_prince, trown https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/cleanup.d/99-ramdisk-create14:18
TheJuliaramdisk-image-create has never worked with the ironic-agent element because the element causes the very last step to bomb out of ramdisk creation, since it does it independently :)14:19
TheJuliathat!14:19
lucasagomesat L21 you can see that it builds the ramdisk14:19
lazy_princewell.. It completed for me.. :)14:19
*** Nisha has quit IRC14:19
trownah...that is probably why I was ending up with both an initramfs, and an initrd file, and if you used the wrong one it fails14:20
trownlucasagomes: awesome, I will change to disk-image-create14:20
NobodyCam:)14:20
lucasagomestrown, cool14:20
*** alexpilotti has quit IRC14:21
*** r-daneel has joined #openstack-ironic14:22
*** thiagop has joined #openstack-ironic14:24
*** zz_natorious is now known as natorious14:28
*** baoli_ has quit IRC14:29
*** baoli has joined #openstack-ironic14:30
trownTheJulia: lazy_prince https://review.openstack.org/21887914:32
lazy_princeokay.. so I verified, both ramdisk-image-create and disk-image-create works and generates the images of same size.. md5 checksum differ (probably due to timestamps difference in the initramfs.14:32
thiagopGood morning Ironicers14:32
openstackgerritJacob McCann proposed openstack/ironic-python-agent: Update qemu-utils package from testing  https://review.openstack.org/21888214:33
lazy_princetrown: so you are assuming that the package install will work only with systems running systemd..?14:33
trownlazy_prince: AFAIK I am the only one working on a package...so yes14:34
lazy_princeshould we get it documented in the README.md..?14:34
TheJuliatrown: comments posted14:35
lazy_princetrown: me too.. :)14:37
TheJulialazy_prince: I +1'ed your other comment14:40
*** lazy_prince has quit IRC14:43
trownTheJulia: updated14:45
TheJuliaMerci14:46
*** garthb has joined #openstack-ironic14:47
*** alexpilotti has joined #openstack-ironic14:48
*** Nisha has joined #openstack-ironic14:49
*** achanda has joined #openstack-ironic14:49
*** dims has quit IRC14:51
TheJuliatrown: +1, but two nitpicks, one you'll likely wnat to fix before a dib core reviews it14:54
trownah ya, commit message for sure, and I am fine with the other too14:55
TheJulia:)14:55
*** jistr is now known as jistr|call14:57
*** praneshp has joined #openstack-ironic15:00
openstackgerritRuby Loo proposed openstack/ironic: Add release notes for 4.0.0  https://review.openstack.org/21684315:01
*** baoli has quit IRC15:01
*** baoli has joined #openstack-ironic15:02
*** dims has joined #openstack-ironic15:03
*** Nisha has quit IRC15:04
*** Nisha has joined #openstack-ironic15:04
*** bnemec has quit IRC15:04
*** achanda has quit IRC15:05
*** shoutm has quit IRC15:05
*** bnemec has joined #openstack-ironic15:09
*** praneshp_ has joined #openstack-ironic15:09
*** praneshp has quit IRC15:11
*** praneshp_ is now known as praneshp15:11
*** vvb has joined #openstack-ironic15:11
*** vvb has quit IRC15:15
*** vvb_ has joined #openstack-ironic15:17
*** vvb_ has quit IRC15:17
*** vvb has joined #openstack-ironic15:18
*** vvb has quit IRC15:20
*** vvb has joined #openstack-ironic15:20
*** vvb has quit IRC15:21
*** vvb has joined #openstack-ironic15:21
*** vvb_ has joined #openstack-ironic15:22
*** vvb has quit IRC15:22
*** vvb_ has quit IRC15:22
*** vvb has joined #openstack-ironic15:22
*** vvb has quit IRC15:25
*** VikasC has joined #openstack-ironic15:25
*** vvb has joined #openstack-ironic15:26
*** vvb_ has joined #openstack-ironic15:26
*** jaypipes has quit IRC15:26
*** ukalifon1 has quit IRC15:26
*** vvb_ has quit IRC15:27
*** vvb has quit IRC15:27
*** vvb has joined #openstack-ironic15:29
*** david-ly_ has joined #openstack-ironic15:31
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/21890615:31
openstackgerritImre Farkas proposed openstack/ironic: DRAC BIOS vendor_passthru: enable rebooting the node  https://review.openstack.org/16307115:32
*** david-ly_ is now known as david-lyle_15:33
*** david-lyle has quit IRC15:33
*** david-lyle_ is now known as david-lyle15:35
*** puranamr has joined #openstack-ironic15:35
*** ifarkas has quit IRC15:37
*** yog_ has quit IRC15:37
*** stendulker has joined #openstack-ironic15:40
BadCubMorning Folks15:40
thiagopmorning BadCub15:41
BadCubHiya thiagop :)15:41
stendulkerBadCub: Good Morning :)15:41
BadCubhiya stendulker :)15:41
stendulkerjroll: Hi15:43
jrollhi stendulker15:44
stendulkerjroll: Had a query on config drive creation in dual bootable (bios + uefi) CoreOS images15:44
jrollstendulker: feel free to ask, but that's nothing I have experience with afaik15:45
stendulkerjroll: config dive creation fails on these images in Bios boot mode.15:45
stendulkerjroll: oh...15:46
TheJuliatrown: youre revision looks great, I've poked two cores on the diskimage-builder repo, you may wish to poke any that you know since it seems like we should just go ahead and get it landed asap.15:46
trownTheJulia: cool, I will ping it out in #tripleo15:46
stendulkerjroll: It fails by giveing warning indicating 'fix the GPT to use all of the space'.15:46
TheJuliadanke15:46
jrollstendulker: our deployment does have coreos images, and boots in BIOS mode, with configdrives... so I'm not sure why it wouldn't work unless our images are special15:46
jrollstendulker: is your drive larger than 2TB?15:46
stendulkerjroll: no15:47
TheJuliastendulker: do you mean repair the gpt table?15:47
jrollhmm15:47
stendulkerjroll: its abt 250GB15:47
stendulkerTheJulia: Yes15:47
jrollyeah, this isn't something I've ever seen15:47
*** vvb2 has joined #openstack-ironic15:47
*** vvb has quit IRC15:47
*** vvb2 has quit IRC15:47
stendulkerI did repair GPT and config drive partition creation through python agent script goes through15:47
TheJuliajroll: basically, if we write a gpt image, we need to repair the gpt table since gpt knows where the end of the disk is15:47
*** vvb has joined #openstack-ironic15:48
*** linggao has joined #openstack-ironic15:48
TheJuliastendulker: Did you post it as a change?15:48
jrollTheJulia: sure, that makes sense, now I'm wondering why our coreos images are special :)15:48
TheJuliajroll: magic?15:48
TheJuliajroll: pizza and magic?15:48
stendulkerWhen instance image is booted, it does not boot up says unable to fien ROOT partition15:48
jroll¯\_(ツ)_/¯15:48
TheJuliasounds like a different image then :(15:49
stendulkerjroll, TheJulia : They do not boot up in BIOS or UEFI mode15:49
thiagophey, jroll is back!15:49
*** vvb has quit IRC15:49
jrolliirc this is the bug where coreos has 4 partitions and so it can't add a configdrive15:49
jrollohai thiagop :)15:49
TheJuliaspeaking of pizza, I need pizza15:49
stendulkerThey have partition tabel like this https://coreos.com/os/docs/latest/sdk-disk-partitions.html15:49
*** viktors has joined #openstack-ironic15:49
stendulkerjroll: Oh is it..15:50
*** vvb has joined #openstack-ironic15:50
*** amotoki has quit IRC15:50
jrollTheJulia: stendulker our images are the coreos_production_rackspace_onmetal* files here http://stable.release.core-os.net/amd64-usr/current/15:50
stendulkerrepair of GPT partition was helpful and config drive creation works with OpenSuse images15:50
jrollif you want to check them out15:50
*** VikasC has quit IRC15:50
jrollI don't think we modify them from there15:51
stendulkerjroll: I used http://stable.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz215:51
stendulkerjroll: I think they are same15:51
jrollstendulker: I don't know anything else about those images, just showing you what we use :)15:51
*** Sukhdev has joined #openstack-ironic15:51
TheJuliajroll: I thinks stendulker is talking about modifying the scripts in IPA...15:52
jrollthey're almost certainly not the same15:52
stendulkerjroll: but you used rackspace and i tried openstack one15:52
stendulkerTheJulia: Correct15:52
jrollTheJulia: sure. I know nothing about what needs to happen to make that image work. the one I use works.15:52
jrollI hate to say it, but patches welcome15:52
stendulkercopy_configdrive_to_disk.sh15:52
TheJuliajroll: yah15:53
*** jistr|call is now known as jistr15:53
stendulkerjroll: I'm kind of stuck...15:53
TheJuliastendulker: is there a nice and detailed bug that has been filed for IPA on this?15:53
stendulkerjroll: Had raised a patch https://review.openstack.org/#/c/215097/15:53
jrollstendulker: I am not an expert on partitioning or filesystems, I'd love to help but this is beyond my knowledge15:53
stendulkerTheJulia: https://launchpad.net/bugs/148688715:53
openstackLaunchpad bug 1486887 in Ironic "Config drive creation fails in UEFI boot mode for whole disk image " [High,In progress] - Assigned to Shivanand Tendulker (stendulker)15:53
*** saripurigopi has joined #openstack-ironic15:53
stendulkerjroll, TheJulia : This patch will not help CoreOS images which are dual bootable15:54
jrollstendulker: you said that patch makes BIOS mode fail to boot though, right?15:54
jrollyeah15:54
JayFTheJulia: stendulker: jroll: FWIW the _onmetal* CoreOS images are incredibly specific to our network configurations and unlikely to work properly in other configurations.15:54
stendulkerIt helps UEFI only images15:54
jrollJayF: again, all I'm doing is showing which image we use that magically works.15:54
jrollbeyond that I know nothing15:54
* TheJulia thinks we're not on the same page but kind of gets what stendulker is running into15:55
jrollheh15:55
stendulkerjroll: anyone in Ironic you could suggest can be consulted for this?15:55
JayFjroll: I actaully do know somethings about our image though :) I helped build it!15:55
*** alexpilotti has quit IRC15:55
jrollstendulker: JayF might know things, lucasagomes might know things, idk15:55
JayFIF it's about UEFI vs BIOS bootloaders15:56
TheJuliaI really need to go find food so I'll be someplace for the meeting a little later15:56
jrollit isn't a CoreOS thing, it's an "images with lots of partitions" thing it seems15:56
* lucasagomes reads15:56
stendulkerTheJulia: The bug I pasted covers only UEI part of the issue. The patch helps in resolving it15:56
JayFI know a lot less than I should :(15:56
TheJuliabbiab15:56
stendulkerTheJulia: But this patch does not help BIOS+UEFI disk images15:56
lucasagomesjroll, morning15:56
jrollhi lucasagomes :)15:56
lucasagomesjroll, hey welcome back15:56
stendulkerlucasagomes: Help needed :)15:56
jrollthanks!15:57
stendulkerlucasagomes: When I IPA ramdisk tries to create config drive for CoreOS disk image, it fails with GPT warning15:57
lucasagomesstendulker, I'm a little confused. If you're using BIOS mode how you are deploying an image with GPT?15:58
stendulkerlucasagomes: Upon fixing GPT warning and creating the config drive it fails to boot. All done in BIOS boot mode15:58
stendulkerIts dual bootable image15:58
stendulkerThey supply like that. This image boots up if no config drive is created15:58
stendulkerlucasagomes: They have this layout https://coreos.com/os/docs/latest/sdk-disk-partitions.html15:59
*** mitchjameson has joined #openstack-ironic15:59
stendulkerlucasagomes: Images are hosted here http://stable.release.core-os.net/amd64-usr/current/coreos_production_openstack_image.img.bz215:59
* lucasagomes looks16:00
lucasagomesstendulker, right I'm not 100% sure tho16:00
stendulkerlucasagomes: This is patch to fix GPT warnings. These warning are emitted both in BIOS and UEFI mode https://review.openstack.org/#/c/215097/16:00
lucasagomeswhat you see in boot time? any error specific?16:00
*** puranamr has quit IRC16:01
stendulkerlucasagomes: In BIOS mode, it says unable to get ROOT16:01
jrollwhat bootloader is this using?16:01
jrolllegacy grub doesn't support GPT16:01
stendulkergrub216:01
jrollnor LILO16:01
jrollhm16:01
stendulkerthese images does boot in Bios and UEFI when no config drive is created.16:01
stendulkerso I think image is fine16:02
stendulkerproblem arises when we try to create config drive through IPA ramdisk16:02
lucasagomesstendulker, have you tried to dump the partition table and device labels before and after the configdrive creation?16:03
lucasagomessee if something changed there?16:03
lucasagomesalso in the kernel command line how is it looking for the root?16:03
lucasagomesis it just pointing to the partition? root=/dev/blah ? UUID ? root=UUID=<blah> ? device mapper ? root=/dev/mapper/<blah>16:04
lucasagomesor label16:04
lucasagomesand so on16:04
JayFstendulker: that's likely because config drive partition creation in IPA16:04
JayFstendulker: explicitly doesn't support GPT today16:04
stendulkerlucasagomes: Before onfig drive creation it has partition nos as 1,2,3,4,6,916:04
JayFstendulker: only reason CoreOS works in onmetal is bceause they precreate a GPT partition labelled config-216:04
lucasagomesyeah I wonder if it's trying to write to the protected MBR from the GPT table16:04
*** baoli has quit IRC16:04
lucasagomeswhich will cause it to screw up things16:04
JayFyeah, that's what I suspect16:04
* JayF brb16:04
stendulkerlucasagomes: When config drive cerated, it is given partition id 516:04
*** trown is now known as trown|lunch16:04
*** baoli has joined #openstack-ironic16:05
*** romcheg has quit IRC16:05
lucasagomesRight, and before the image has any extended partition ?16:05
*** baoli has quit IRC16:05
lucasagomescause MBR only support 4 primary partitions16:05
*** praneshp has quit IRC16:06
*** praneshp has joined #openstack-ironic16:06
lucasagomesyou can use one of the slots for primary to be an extended partition so you can add more partitions to it16:06
lucasagomesI wondering if it's somehow overwritting it16:06
lucasagomeshttps://en.wikipedia.org/wiki/Extended_boot_record16:06
stendulkerlucasagomes: we pass label as ROOT. But wil have to capture those details16:06
lucasagomesyeah maybe the MBR you are trying to write too already have 4 primery partitions (one being an extended)16:07
lucasagomesor no extended just 4 primaries normal partition16:07
*** lazy_prince has joined #openstack-ironic16:07
lucasagomesstendulker, yeah please dump the partion layout before and after so we can investigate16:07
jrollso, like JayF said...16:08
jrollthe agent doesn't support configdrive creation on GPT today16:08
stendulkerlucasagomes: Yes, will capture this data16:08
lucasagomesthanks16:08
jrollI should have mentioned that before, my brain isn't 100% functional yet16:08
stendulkerjayF, jroll: There isn't much of a difference in creation of config drive for GPT. The same set of commands work if the image is GPT only image16:09
stendulkerThis problem is arising with images that support GPT+MB to make it dual bootable16:10
NobodyCamjroll: that you who added the 4.0.0 and 4.1.0 to the agenda?16:10
jrollNobodyCam: haven't touched the agenda, maybe deva or rloo16:10
NobodyCamdeva out until the 8th :)16:10
rloojroll, NobodyCam: oh, it was me. sorry, forgot to put '[rloo]'16:10
stendulkerjayF, jroll : Only fix required for agent is to fix the GPT warning before creating config drive . Rest of teh commands are same16:11
NobodyCamlol all good... just wanted to know who to ping16:11
NobodyCam:)16:11
jrollNobodyCam: vacation doesn't preclude someone from adding things to the agenda :P16:11
rlooNobodyCam: sec, let me update it16:11
*** natorious is now known as zz_natorious16:11
NobodyCamjroll: I not sure there are internets where he is16:11
NobodyCam:-p16:12
jrollI'm mostly kidding16:12
jrollbut16:12
NobodyCam:)16:12
jrollnext time I go on vacation I'm going to add lots of absurd things to the agenda16:12
rlooNobodyCam: updated.16:13
*** jistr has quit IRC16:13
rlooNobodyCam: you're welcome to talk about the releases too. I just wanted to make sure we mentioned them.16:14
stendulkerlucasagomes: Here is the partition data http://paste.openstack.org/show/435098/16:16
stendulkerlucasagomes: /dev/sda5 is the config drive created by agent16:17
*** vishwanathj has joined #openstack-ironic16:18
NobodyCam:)16:18
stendulkerMoning NobodyCam :)16:19
stendulkerMorning :)16:20
*** praneshp has quit IRC16:20
*** e0ne has quit IRC16:20
*** jaypipes has joined #openstack-ironic16:22
TheJuliaI thought vacation was just a workday from a tropical location???16:26
lucasagomesstendulker, looking16:26
stendulkerlucasagomes: Thank you. This data is after config drive creation.. Re-deploying to get initial snapshot16:26
lucasagomesthanks, stendulker well for the initial you probably can just mount it in a loop device16:27
lucasagomesI mean the image16:27
lucasagomesthat's actually a great way to test it. Mimic'ing what IPA is doing on a loop device16:27
lucasagomesand trying to boot the image on a VM16:28
*** viktors is now known as viktors|afk16:28
lucasagomes(faster than deploying via Ironic at least)16:28
stendulkerlucasagomes: Yes, I can do that as well. Was bringing up so that i could get more details if anything is required while creating the config drive16:28
*** alexpilotti has joined #openstack-ironic16:28
lucasagomesstendulker, gotcha16:28
NobodyCammorning stendulker :)16:29
*** tideline_ has joined #openstack-ironic16:32
*** alexpilotti has quit IRC16:33
*** killer_prince has joined #openstack-ironic16:33
*** lifeless has quit IRC16:34
*** lazy_prince has quit IRC16:35
*** tideline has quit IRC16:35
*** karimb has quit IRC16:36
*** killer_prince is now known as lazy_prince16:36
jlvillalGood morning all.  I hope everyone enjoyed their weekend.16:36
*** vvb has quit IRC16:37
stendulkerlucasagomes: Fdisk data on image is http://paste.openstack.org/show/435142/16:37
*** stendulker has quit IRC16:38
lucasagomesstendulker, parted -l16:38
lucasagomesfdisk only works with MBR16:38
lucasagomesor gdisk -l16:38
*** alexpilotti has joined #openstack-ironic16:39
NobodyCammornign jlvillal :)16:39
*** vvb has joined #openstack-ironic16:39
jlvillalHey NobodyCam. Did you get much smoke up where you are?  Weekend before last all of the Portland area was very smoky :(16:40
NobodyCam* 20 minutes untill meeting16:40
*** stendulker has joined #openstack-ironic16:40
NobodyCamjlvillal: not at all. :) thou we have been getting rain for the last several days. which I guess is a good thing16:41
*** lifeless has joined #openstack-ironic16:41
jlvillalNobodyCam: Very good thing!16:41
*** romcheg has joined #openstack-ironic16:41
*** getvasanth has quit IRC16:42
* TheJulia tries to make pizza appear faster16:42
NobodyCama watched pizza oven never finishes16:43
NobodyCam:-p16:43
NobodyCamjlvillal: is https://bugs.launchpad.net/nova/+bug/1405131 still a isssue with all the ML2 work under way?16:44
openstackLaunchpad bug 1405131 in OpenStack Compute (nova) "Ports cannot be mapped to networks" [Low,In progress] - Assigned to Mark Goddard (mgoddard)16:44
*** meghal has joined #openstack-ironic16:44
jrollNobodyCam: it's going to continue to be an issue, that bug is basically "can't say network X goes to nic Y"16:44
NobodyCamack ok16:45
jlvillalNobodyCam: mrda and I put it on the agenda because we want to know what to do about it.  Get feedback from the team.16:45
jlvillalNobodyCam: We weren't sure exactly what to do.16:45
* jlvillal goes to get some coffee before the meeting16:46
NobodyCamlooks like there was talk about tagging the actual ports16:46
NobodyCamya16:46
*** davideagnello has joined #openstack-ironic16:47
*** harshs has joined #openstack-ironic16:48
*** davideagnello has quit IRC16:49
openstackgerritNisha Agarwal proposed openstack/ironic: ilo drivers powers off the node in ACTIVE state  https://review.openstack.org/21878516:49
*** zz_natorious is now known as natorious16:49
*** romainh has left #openstack-ironic16:50
*** davideagnello has joined #openstack-ironic16:50
openstackgerritRuby Loo proposed openstack/ironic: Refactor map_color()  https://review.openstack.org/21894816:51
*** natorious is now known as zz_natorious16:51
*** romcheg has quit IRC16:53
*** zz_natorious is now known as natorious16:53
*** romcheg has joined #openstack-ironic16:54
*** dims has quit IRC16:55
*** harshs has quit IRC16:56
lucasagomesNobodyCam, you running the meeting today?16:56
lucasagomesyou're16:56
*** aarefiev22 has joined #openstack-ironic16:57
*** cdearborn has joined #openstack-ironic16:57
NobodyCamsi16:59
lucasagomes(-:16:59
*** degorenko has quit IRC16:59
*** vishwanathj has quit IRC17:00
*** puranamr has joined #openstack-ironic17:01
*** mitchjameson has left #openstack-ironic17:01
rlooironic meeting is starting...17:02
*** saripurigopi has quit IRC17:02
*** romcheg has quit IRC17:05
*** derekh has quit IRC17:05
*** praneshp has joined #openstack-ironic17:05
*** trown|lunch is now known as trown17:06
*** romcheg has joined #openstack-ironic17:06
*** lazy_prince has quit IRC17:08
*** jjohnson2_ has joined #openstack-ironic17:14
*** achanda has joined #openstack-ironic17:18
openstackgerritJosh Gachnang proposed openstack/ironic: Clean up cleaning error handling on heartbeats  https://review.openstack.org/21376917:23
*** mjturek1 has quit IRC17:23
*** harshs has joined #openstack-ironic17:23
*** e0ne has joined #openstack-ironic17:26
*** dims has joined #openstack-ironic17:29
*** dims_ has joined #openstack-ironic17:30
*** dims has quit IRC17:30
*** mjturek1 has joined #openstack-ironic17:30
*** meghal has quit IRC17:32
*** meghal has joined #openstack-ironic17:34
*** r3mu5 has joined #openstack-ironic17:34
*** baoli has joined #openstack-ironic17:35
*** e0ne has quit IRC17:36
*** r3mu5 has left #openstack-ironic17:37
*** baoli has quit IRC17:37
*** baoli has joined #openstack-ironic17:38
*** e0ne has joined #openstack-ironic17:39
openstackgerritJosh Gachnang proposed openstack/ironic: Cache agent clean steps on node  https://review.openstack.org/21853217:40
*** stendulker has quit IRC17:42
*** ig0r__ has joined #openstack-ironic17:43
krotscheckThis might be helpful to core trying to keep track of things. It's a gerrit link that says "Give me all things in ironic/ironic-webclient not by me that have passed gate and that I haven't voted on yet": https://review.openstack.org/#/q/status:open+(project:openstack/ironic+OR+project:openstack/ironic-webclient)+-owner:self+-(label:Code-Review%253D1%252Cuser%253Dself+OR+label:Code-Review%253D2%252Cuser%253Dself)+label:Verified%253D1,n,z17:44
openstackgerritJosh Gachnang proposed openstack/ironic: Cache agent clean steps on node  https://review.openstack.org/21853217:45
*** ig0r_ has quit IRC17:46
krotscheckI recommend all cores create a link with something like that for all things they're core on.17:46
*** mjturek1 has quit IRC17:47
*** stendulker has joined #openstack-ironic17:49
*** Sukhdev has quit IRC17:51
*** romcheg has quit IRC17:52
*** mjturek1 has joined #openstack-ironic17:53
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/21696617:59
*** stendulker has quit IRC18:00
NobodyCamthank you all ... :)18:00
*** alexpilotti has quit IRC18:01
lucasagomesfolks I'm going to call it a day18:01
lucasagomeshave a great night everyone!18:01
rloogood night lucasagomes18:01
trowngood night lucasagomes18:01
thiagopnight lucasagomes18:01
JayFnn18:01
lucasagomessee y'all!18:01
NobodyCamhave a great night lucasagomes thank you for staying up with us18:01
gabriel-bezerrahave a good one, lucasagomes18:01
*** lucasagomes is now known as lucas-dinner18:01
rloogabriel-bezerra: wrt your question about adding a link18:02
jrollkrotscheck: we have things like that, but it takes more than 8 hours a day to review everything in that list :(18:02
rloogabriel-bezerra: if I look at https://wiki.openstack.org/wiki/Ironic/Drivers18:02
jrollkrotscheck: http://perm.ly/ironic-review-dashboard18:02
jrollcould use some updating.18:02
rloogabriel-bezerra: there's already a table with Cisco UCS there. maybe you can add yours there too.18:02
rloogabriel-bezerra: i think we need to clean up this wiki but for now anyway, will that suffice?18:02
gabriel-bezerrarloo: I see that. Looks better than in the bottom of the page :)18:03
NobodyCambrb18:03
rloogabriel-bezerra: the bottom of that page doesn't make any sense. not sure who/why someone added it.18:03
jrollheh, you're right rloo18:04
jroll+1 for that out of tree table18:04
rloogabriel-bezerra: is the oneview driver in ironic now?18:04
gabriel-bezerrarloo: should I add a third column with the comments regarding it being kilo specific and that the liberty version is in ironic repository?18:05
rloogabriel-bezerra: sure if you want.18:05
gabriel-bezerrarloo: the spec is approved, I'm not sure about the code18:05
rloogabriel-bezerra: whenever the oneview code is merged, you'll want to update http://docs.openstack.org/developer/ironic/deploy/drivers.html18:05
rloogabriel-bezerra: when you do, you can add a link there to the out-of-tree kilo too. I think. (Can't recall what folks are documenting there.)18:06
gabriel-bezerrarloo: great.18:06
*** davideagnello has quit IRC18:06
gabriel-bezerrarloo: thank you.18:06
rloogabriel-bezerra: yw. We welcome all documentation :-)18:07
krotscheckjroll: Then don't be surprised if contributors get frustrated when our patches bitrot-ad-nauseum.18:09
krotscheckOr die of 1000 papercuts.18:09
jrollkrotscheck: I'm not surprised, I'm just trying to help make it better next time18:09
jrollkrotscheck: our team built features a year ago that still aren't landed in various openstack projects for various reasons, I feel your pain, seriously18:10
jrolland I want to help18:10
jrollI've been at the point of quitting openstack before for the same reasons18:11
*** afaranha has quit IRC18:11
jrollyou have to figure out the system and work it, and every project is different18:11
JayFusually Ironic manages to avoid these kinds of situations via being small18:12
JayFbut clearly that's not working as well as it used to :)18:12
* jroll needs to step away for a while18:13
*** mitchjameson has joined #openstack-ironic18:15
*** pelix has quit IRC18:17
krotscheckjroll: I get it. But frankly, I have way less tolerance for frustration than others. The scope of work I'm trying to land in openstack is so large that if one project doesn't move forward, I just switch to another one. And if none of them are moving, I have a couple halo projects that aren't under the Red Tape Halo at all that I can contribute to.18:18
openstackgerritSinval Vieira Mendes Neto proposed openstack/ironic: OneView Driver for Ironic  https://review.openstack.org/19182218:18
krotscheckAnd, well, I don't have time to pester people and make noise, because of that.18:18
krotscheckCase and point, i have 5 patches on infra right now that are more than a month old, and only have positive votes.18:19
krotscheckIn total, I've got 40 patches out, 14 that are broken/contested/being argued about.18:19
thiagopkrotscheck: take a look on this -> https://review.openstack.org/#/c/214766/18:23
thiagop7 "+2" and nobody approves18:23
thiagop#openstackfeelings18:23
*** davideagnello has joined #openstack-ironic18:25
*** alexpilotti has joined #openstack-ironic18:27
krotscheckthiagop: Why?18:27
krotscheckNo, I mean seriously: Why? WIth that much support, why's the keystone team scared?18:28
krotscheckIf someone has a legitimate concern, then there should be a -1 or -2 on it.18:28
openstackgerritMerged openstack/ironic: Do not overwrite the iPXE boot script on every deployment  https://review.openstack.org/21829018:28
thiagopkrotscheck: makes no sense to me either18:28
openstackgerritMerged openstack/ironic-python-agent: Load Hardware Managers at runtime  https://review.openstack.org/21846318:29
*** dtantsur|afk is now known as dtantsur18:29
* dtantsur is shortly back18:29
NobodyCamhttp://openstackreactions.enovance.com18:29
NobodyCamhey hey dtantsur :)18:29
dtantsurI guess I missed the meeting, didn't it? :)18:29
thiagophahaha I'm on that right now NobodyCam18:29
NobodyCamdtantsur: yep it was last hour18:31
NobodyCam:-p18:31
NobodyCamwe assigned everything to you18:32
dtantsurhehe, reading now18:32
NobodyCam:)18:32
dtantsuralso folks, today I got my passport with visa back :)18:32
NobodyCamw00t :)18:32
thiagopdtantsur: how is Prague?18:32
dtantsurawesome as usual18:34
*** mjturek1 has quit IRC18:36
*** Nisha has quit IRC18:37
*** Nisha has joined #openstack-ironic18:38
NobodyCamtrown: you see this: http://logs.openstack.org/79/218879/3/check/gate-diskimage-builder-pep8/8ebd8dc/console.html#_2015-08-31_18_11_50_53318:41
trownTheJulia: do we really need to support sysv?18:41
trownNobodyCam: ya, I am working on it :)18:41
*** mgoddard has joined #openstack-ironic18:41
NobodyCam:)18:41
trownTheJulia: without it, I can just use svc-map and it will work automagically for upstart and systemd18:42
trownTheJulia: there is no support sysv in svc-map though18:42
*** mjturek1 has joined #openstack-ironic18:43
NobodyCamtrown: I expect that the best answer for that question can be found in the TripleO channel18:45
NobodyCamI'm sure they added it for some reason..18:45
trownNobodyCam: except that it is specific to ironic-agent element, which is not used at all in tripleo...yet18:46
*** meghal has quit IRC18:47
*** meghal has joined #openstack-ironic18:47
NobodyCamJoshNang: JayF jroll do you recall if the systemV support was for CoreOS?18:48
jrollcoreos is systemd18:48
*** ig0r__ has quit IRC18:48
NobodyCamTy18:49
jrollyeah np18:49
dtantsurfolks, I need more eyes on https://review.openstack.org/#/c/216604/ please. this needs to land in liberty18:50
jrolldtantsur: +1 for now, I agree with sambetts18:51
*** mjturek2 has joined #openstack-ironic18:52
dtantsurdtantsur, feel free to -1 if you want it fixed, +1 doesn't do a big service ;)18:52
dtantsurI just wanted more eyes18:52
trownNobodyCam: TheJulia, I think I actually can leave sysv support, it just will not use svc-map18:53
*** mjturek1 has quit IRC18:53
jrollNobodyCam: dtantsur do we have a launchpad for ironic-lib?18:55
NobodyCam:)18:55
TheJuliatrown: I honeslty don't remember why sysv support is included, but upstart + systemd should do it18:56
dtantsurjroll, I doubt it18:56
jrollyeah18:56
jrollfiguring out this release thing18:56
dtantsuralso we don't have gates for ironic-lib yet... I'm not to blame, patches are awaiting reviews :)18:57
*** hemna has joined #openstack-ironic18:57
jrollyeah, it's fine18:58
jrolldtantsur: I think we'll want gates before the next release18:58
jrollor before we drop it in ironic, anyway18:58
jrollper the meeting we're going to get a release out so we can put it in g-r before g-r freeze18:58
openstackgerritJulia Kreger proposed openstack/bifrost: Update iPXE agent template  https://review.openstack.org/21884318:59
dtantsurjroll, ack, makes sense18:59
openstackgerritJulia Kreger proposed openstack/bifrost: Correct log file storage location  https://review.openstack.org/21884118:59
dtantsurok folks, going to get some rest, see you tomorrow19:03
*** dtantsur is now known as dtantsur|afk19:03
NobodyCambrb19:03
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/21890619:04
openstackgerritMerged openstack/ironic-specs: CIMC PXE Drivers for Ironic  https://review.openstack.org/21759819:05
dtantsur|afksambetts, ^^19:05
*** Nisha_away has joined #openstack-ironic19:10
*** Nisha has quit IRC19:10
openstackgerritRuby Loo proposed openstack/ironic: Add release notes for 4.0.0  https://review.openstack.org/21684319:10
*** mgoddard has left #openstack-ironic19:14
*** mjturek1 has joined #openstack-ironic19:14
*** mjturek2 has quit IRC19:15
openstackgerritHarsh Shah proposed openstack/python-ironicclient: Add keyring support  https://review.openstack.org/21813219:18
trownTheJulia: I think this is ultimately a better solution, and allows debian to package IPA without any extra patches to the element: https://review.openstack.org/#/c/218879/419:23
*** baoli has quit IRC19:24
*** baoli has joined #openstack-ironic19:24
*** tideline_ has quit IRC19:28
jrollNobodyCam: BadCub could one of you add me here? https://launchpad.net/~ironic-drivers/+members19:31
jrolljust locked myself out of ironic-lib launchpad because I'm not in there :)19:31
jrollalternatively set the Driver here to ironic-drivers instead of me https://launchpad.net/ironic-lib19:31
NobodyCamjroll: not able to add you to the list... but changed the driver on LP.19:33
jrollNobodyCam: weird, but thanks19:33
NobodyCamdeva listed as the admin19:34
NobodyCamso I think he has add/edit/delete rights19:34
NobodyCam:-p19:34
jrollyeah19:35
jrolloh well, good enough for now :P19:35
NobodyCam:)19:35
BadCubit appears that I cannot add folks to ^^ either19:35
*** davideag_ has joined #openstack-ironic19:35
*** davideagnello has quit IRC19:37
jrollno worries, we'll sort it out later :)19:37
jrollFYI https://review.openstack.org/#/c/218985 should be good to go19:38
jrollironic-lib release ^19:38
jrolldtantsur|afk: so you know, ironic-lib has its own launchpad, because managed releases requires them unless you want to release in tandem. letting you know for bug reporting and such https://launchpad.net/ironic-lib19:38
openstackgerritClaudiu Belu proposed openstack/ironic: WIP: Removes unused posix-ipc requirement  https://review.openstack.org/21900319:48
*** Sukhdev has joined #openstack-ironic19:49
*** alexpilotti has quit IRC19:50
*** davideagnello has joined #openstack-ironic19:52
*** aarefiev22 has quit IRC19:53
*** davideag_ has quit IRC19:53
jrollone more quick update on ironic-lib: we will still be impacted by requirements freeze, because upper-constraints and also because the release team is stricter about releasing libraries during freeze20:00
jrollg-r change here https://review.openstack.org/#/c/219011/20:00
jrollI'll send a reply to rloo's email about this stuff20:00
rloojroll: so unless we want an exception, I would prefer to punt on using ironic-lib in ironic code, until after 4.1.0.20:01
rloojroll: thx for sending email about it20:01
*** david-lyle has quit IRC20:01
jrollrloo: I think we should start working on it, though maybe not land the patches?20:01
jrollif it doesn't seem to be buggy, why not use it20:01
rloojroll: if i understand, by the end of this week, we need to pin the version of ironic-lib, right?20:02
jrollrloo: also, 4.0.1 is going to become 4.1.0, and 4.1.0 -> 4.2.0. apparently release team is treating any "update from global requirements" patch as something that should be a minor version bump when released20:02
jrollrloo: it's done for us, but yes, it will be pinned thursday or so20:03
rloojroll: if we pin it and use that particular version, it means that if we need changes in that code, we won't be able to do it until after 4.2.020:03
rloojroll: unless there is a bug i guess?20:03
rloojroll: anyway, it seems to me that it may be restrictive wrt doing development with that library since it is something new for us.20:03
jrollrloo: right, so we can start working on the patches before 4.2, if we don't need to update ironic-lib to land them, I don't see why we couldn't land them20:04
jrollthough probably smart to just wait20:04
jrolljust in case20:04
rloojroll: so if we do something wrong wrt using that library, i'd rather do the wrongness after 4.2.20:04
jrollyeah, agree20:04
NobodyCamTheJulia: when you have a free minute could you take a look at : https://review.openstack.org/#/c/21737420:04
jrollrloo: I just don't want to tell anyone not to start writing code :)20:04
rloojroll: cuz if we work on the patches and they land (to have ironic use ironic-lib), and there's a bug, then what?20:04
rloojroll: we'd need to update ironic-lib cuz the code is in there now, but that means a new version.20:05
jrollrloo: yep, you're right. we should probably wait to land anything using it.20:05
jrollthe joy of freezing code20:05
rloojroll: at this point, i'd rather we spend our time landing features etc.20:05
*** mitchjameson has quit IRC20:05
*** david-lyle has joined #openstack-ironic20:06
* jroll wonders if frozen code still bitrots :P20:06
jrollrloo: yep. but if whoever was working on this refactor wants to keep working on it, I won't stop them, I just won't land the code :)20:06
rloojroll: unless we can do the pin'ing of ironic-lib at 4.2 time. since only ironic code uses it, i don't see why not. but it might just be more work for infra or whoever folks.20:06
jrollrloo: right, it's work for infra and more importantly, packagers20:07
rloojroll: oh, sure, i wouldn't prevent someone from submitting patch revisions but i guess we should block them from merging.20:07
jrollyep, +120:07
rloojroll: thanks for communicating that to folks :)20:07
jrolltotally, no problem :)20:07
*** aarefiev22 has joined #openstack-ironic20:10
*** dims has joined #openstack-ironic20:11
*** dims has quit IRC20:12
*** puranamr has quit IRC20:14
*** dims_ has quit IRC20:14
*** puranamr has joined #openstack-ironic20:15
*** dprince has quit IRC20:16
*** alexpilotti has joined #openstack-ironic20:16
*** meghal has quit IRC20:20
*** romcheg has joined #openstack-ironic20:28
*** aarefiev22 has quit IRC20:32
NobodyCamwhy do mondays have SO many calls :-p20:36
jrollcan we get some eyes on https://review.openstack.org/#/c/216843/ ?20:36
jrollthanks for the update btw rloo :)20:36
rloojroll: yw.20:36
*** Nisha_away has quit IRC20:37
*** achanda has quit IRC20:38
*** alexpilotti has quit IRC20:40
*** achanda has joined #openstack-ironic20:41
*** achanda has quit IRC20:41
*** achanda has joined #openstack-ironic20:41
*** dprince has joined #openstack-ironic20:42
NobodyCamtrown: wow the gate really dosen't like you today :(20:45
*** puranamr has quit IRC20:46
*** puranamr has joined #openstack-ironic20:46
trownNobodyCam: :( ...checking20:46
jrollNobodyCam: do you have time to update https://review.openstack.org/#/c/209251/9/doc/source/deploy/install-guide.rst today?20:49
jrollhoping to do the next release this week20:50
*** baoli has quit IRC20:51
rlooNobodyCam: wrt 209251, let me know if you want me to do it since I think I had some comments.20:52
jrollI'm making some as well now20:53
rloojroll: oh, so you're offering to update it too? :D20:53
jrolluh oh, what have I done20:53
jrollI don't mind updating it, but I prefer to push the +A button20:54
jroll:)20:54
rloojroll: yeah, I know. the problem with us modifying, is that we shouldn't +2 +A.20:54
jrollyep20:54
jrollstepping away for a while20:55
*** dims has joined #openstack-ironic20:57
openstackgerritgreghaynes proposed openstack/bifrost: Allow explicitly setting nginx/controller IP  https://review.openstack.org/21902621:01
NobodyCamlooking now21:02
*** linggao has quit IRC21:03
*** jjohnson2_ has quit IRC21:05
rlooTheJulia: I'm updating some of our docn. The bifrost-core consists of you and Stephanie. but ironic cores are also cores for bifrost too?21:08
*** hemna has quit IRC21:10
TheJuliarloo: that is correct21:12
rloothx TheJulia.21:12
NobodyCamrloo: looking at lines 30 & 31, is there a why for us to say something like clients older then <version / date/ something> will need to be updated???21:12
* rloo wonders how we got so many projects under ironic...21:12
*** meghal has joined #openstack-ironic21:12
rlooNobodyCam: sec, let me look21:13
rlooNobodyCam: what patch are we talking about?21:14
NobodyCamlol...21684321:14
NobodyCam:)21:14
rlooNobodyCam: oh, wrong patch. sec.21:14
NobodyCamsorry I was not clear there :p21:15
openstackgerritJulia Kreger proposed openstack/bifrost: Update iPXE agent template  https://review.openstack.org/21884321:15
rlooNobodyCam: in the index.rst file21:15
*** alexpilotti has joined #openstack-ironic21:15
openstackgerritJulia Kreger proposed openstack/bifrost: Correct log file storage location  https://review.openstack.org/21884121:15
NobodyCamrloo: yep21:15
rlooNobodyCam: hmm. You mean the older clients that didn't support the header?21:15
NobodyCamrloo: the orginal is: Clients must be updated to handle the new workflow when they30 begin sending the X-OpenStack-Ironic-API-Version header with a value >= 1.11.3121:16
*** puranamr has quit IRC21:16
rlooNobodyCam: right. Why does the client need to be updated?21:16
NobodyCamhow (as a ironic user) will I know if I need to upgrade21:16
rlooNobodyCam: Oh, I mean, the requests from the client.21:16
*** puranamr has joined #openstack-ironic21:16
rlooNobodyCam: if it says 'Client requests must be updated..' is that clearer?21:17
*** romcheg1 has joined #openstack-ironic21:17
*** dprince has quit IRC21:18
*** romcheg has quit IRC21:18
NobodyCamyea that clears it up... other wise I see a bunch of folks updating the cli21:18
rlooNobodyCam: good point.21:18
rlooNobodyCam: anything else? Should I wait?21:18
NobodyCamlet me post that and another coment real quick21:19
rlooNobodyCam: sure. no worries.21:19
*** trown is now known as trown|outttypeww21:20
openstackgerritgreghaynes proposed openstack/bifrost: Allow explicitly setting nginx/controller IP  https://review.openstack.org/21902621:20
NobodyCamrloo: posted21:20
NobodyCamrloo: Thank you :)21:21
greghaynesTheJulia: ah! I hit https://review.openstack.org/#/c/218843 and was wondering if I had just gone mad21:21
*** thrash is now known as thrash|g0ne21:25
*** cdearborn has quit IRC21:25
rlooNobodyCam: I just reread that sentence (line 30, 31). I think Devananda didn't mean our python-ironicclient, he meant any client that uses ironic.21:26
TheJuliagreghaynes: yeah, some reworking of pxe stuffs landed on ?friday? and broke us :(21:27
NobodyCamrloo: ya,21:28
rlooNobodyCam: I'm not sure how to reword this.21:29
TheJuliagreghaynes: I just commneted in the rev you posted earlier about hard coding the webserver IP address21:29
rlooNobodyCam: does eg 'Applications' seem better than 'Clients'?21:29
greghaynesTheJulia: yep, saw that21:29
TheJuliabasically saying that specific edit had to go away21:29
greghaynesTheJulia: not sure what you mean by defaults - Is there more than the group inventory default?21:30
* TheJulia may not be making much sense atm, apparently food food hates me21:30
TheJuliagreghaynes: so there are role level defaults that get imported21:30
TheJuliarolename/defaults/mainyml21:30
greghaynesgotcha21:30
TheJuliathey are overridden by a higher level, but if they are all set to what you want, then you don't need to override, and I mean by that, some sort of automagic has done the setting for you21:31
greghaynesYep, probably good to set at role level for the roles to be reconsumable individually though21:31
NobodyCamrloo: Custom clients wishing to take advantage of this new work flow must be updated to begin sending the X-OpenStack-Ironic-API-Version header with a value >= 1.11  .... maybe??21:32
*** alexpilotti has quit IRC21:33
jrollrloo: NobodyCam "API clients wishing..."21:33
NobodyCamoh ^^^^ better :)21:33
TheJuliagreghaynes: yup, the whole goal we have is making the smaller chunks re-usable... with some sanity and minimal hairloss21:33
rloojroll, NobodyCam: what is an API client? Don't we mean that if you use that version, you have to change your workflow to get the node from enroll -> available?21:34
*** e0ne has quit IRC21:34
jrollwell, maybe I should read it21:35
rlooI think the problem is that 'Client' has different meanings.21:35
jrollyeah21:36
jrollso I actually think current wording is fine, personally21:36
mrdaMorning Ironic21:36
jrollrloo: ^21:36
jrollmrda: morning :)21:36
rloojroll: yeah, me too. cuz i can't think of anything better right now. the problem is if others interpret it as our client package21:36
NobodyCamgood morning mrda21:36
*** hemna has joined #openstack-ironic21:36
rloomorning mrda21:36
mrdao/21:37
jrollrloo: API clients would also fit fine, and maybe disambiguate a bit21:37
thiagopmorning mrda21:37
rloojroll: ok, i'll change to API client :-)21:37
jrollNobodyCam: ^ good with you?21:37
jrollthanks rloo :)21:37
NobodyCamjroll: rloo: ya I think that is good... :)21:38
NobodyCamand Thank you21:38
NobodyCam:)21:38
NobodyCamTheJulia: have you seen the error on 218833: TASK: [bifrost-deploy-nodes-dynamic | Error if deploy_image is not present, and instance_info is not defined] ***21:40
TheJuliagreghaynes: SpamapS: I know you two are busy and have proposed a few things to bifrost.  Would you it help you guys if we can back after you guys and proposed doc updates, or do you guys have the time and minimal context switches that you can do them?21:42
greghaynesTheJulia: ECONTEXT21:42
greghaynesTheJulia: Which change?21:42
TheJuliagreghaynes: several changes :)21:42
greghaynesI can poke at docs, I havent revisited some stuff purely because im trying to get them fully tested before doing finishing touches21:43
TheJuliaokay, thats better context for us too then21:43
greghayneshopefully will be setup to test vlan and other patches pretty quickly...21:43
jrollslight tangent: one day I'm going to convince people to submit docs updates WITH the code changes :)21:43
greghaynesslow booting servers are slow, etc21:43
NobodyCammy fear is we could lose track of what still needs docs and what does not21:43
* NobodyCam assumes TheJulia is talking about /my doc comment21:44
greghaynesNobodyCam: TheJulia oh, that one sounds like the static dhcp assignment one. I think that one might be getting abandoned21:44
TheJuliaNobodyCam: a couple different commits could use doc changes, I'm just noticing the trend going over stuff while I wait for hopefully the gate to pass21:44
greghaynesIt seems to work, but its sort of a layering violation I think were going to work around differently21:45
*** jcoufal has quit IRC21:45
TheJuliagreghaynes: but... I really like that change :(21:45
greghaynesheh21:45
greghaynesTheJulia: So, I did test that one21:45
greghaynesTheJulia: and so far its the only totaly success weve had :)21:45
NobodyCambrb21:46
TheJuliaNobodyCam: 218833... error if... thats a red herring :)21:47
*** Sukhdev has quit IRC21:49
openstackgerritRuby Loo proposed openstack/ironic: Add release notes for 4.0.0  https://review.openstack.org/21684321:50
*** alexpilotti has joined #openstack-ironic21:51
*** Sukhdev has joined #openstack-ironic21:51
jrollwoo, thanks ruby21:52
rloojroll: glad you liked it :D21:52
TheJuliagreghaynes: fwiw, your nework info module changes, I think I went into testing it last week and got hit by something else.  If https://review.openstack.org/#/c/218879/ lands, then the non voting job should work and will test that for us :)21:53
greghaynesah, ok21:54
* greghaynes will review21:54
rlooNobodyCam: let us know if you have time to work on https://review.openstack.org/#/c/209251/9/doc/source/deploy/install-guide.rst21:54
rlooNobodyCam: today or tomorrow. I can't do it today (too late now), but I can work on it tomorrow.21:55
*** alexpilotti has quit IRC21:55
jrolldoes anybody see any release blockers in this list?21:56
jrollhttps://bugs.launchpad.net/python-ironicclient21:56
*** garthb_ has joined #openstack-ironic21:56
*** garthb_ has quit IRC21:57
NobodyCamrloo: sorry I haven't had a chance to update that. I will try today, but if I haven't done it by the the time you look please feell free to21:59
rlooNobodyCam: ok, thx.22:00
NobodyCamjroll: rloo : +2 from me on release notes. just waiting on the gate jobs for +a22:00
jrollNobodyCam: they'll block your +A if you want to add it now :P22:01
NobodyCam??? We can wait until they are ready22:02
*** meghal has quit IRC22:02
*** meghal has joined #openstack-ironic22:02
*** alexpilotti has joined #openstack-ironic22:03
*** meghal has quit IRC22:03
TheJuliawin 622:03
TheJuliadoh22:03
*** meghal has joined #openstack-ironic22:03
rlooNobodyCam, jroll: yeah, I sometimes do +A cuz I don't want to wait/forget to do it afterwards. if Jenkins doesn't like it, it won't get merged anyway.22:03
jroll^ exactly22:04
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279622:06
*** thiagop has quit IRC22:06
rloojroll: I took a quick look at the client bugs. I don't see anything 'important' that needs to be addressed before we release.22:06
*** meghal has quit IRC22:06
rloojroll: although we should probably look at patches too?22:06
jrollrloo: I agree after a quick skim, going to look once more22:06
jrollyeah, getting there :)22:06
*** meghal has joined #openstack-ironic22:06
rloojroll: I looked at client patches. i don't think we need anything there. might want to check tomorrow morning with others. gotta go now. ciao.22:10
openstackgerritJulia Kreger proposed openstack/bifrost: Update iPXE agent template and boot.ipxe ownership  https://review.openstack.org/21884322:10
TheJuliaNobodyCam: ^^^ it should fix bifrost's gate in one revision. :(22:11
jrollrloo: have a good night, thank you! :)22:14
*** garthb has quit IRC22:17
*** garthb has joined #openstack-ironic22:17
*** garthb_ has joined #openstack-ironic22:17
*** garthb_ has quit IRC22:17
*** garthb has quit IRC22:18
*** garthb_ has joined #openstack-ironic22:18
*** garthb has joined #openstack-ironic22:19
*** meghal has quit IRC22:22
*** meghal has joined #openstack-ironic22:22
*** meghal has quit IRC22:27
*** meghal1 has joined #openstack-ironic22:27
*** dguerri` is now known as dguerri22:27
openstackgerritDan Prince proposed openstack/ironic: Add retry options to iBoot power driver  https://review.openstack.org/21905122:29
*** dguerri is now known as dguerri`22:31
*** jaypipes has quit IRC22:32
*** romcheg1 has quit IRC22:32
*** alexpilotti has quit IRC22:33
*** mjturek1 has quit IRC22:35
*** meghal1 has quit IRC22:36
*** meghal has joined #openstack-ironic22:36
NobodyCamhave a good night rloo :)22:37
*** hemna has quit IRC22:38
NobodyCamTheJulia: oh was the file created as another user :-p22:39
TheJuliayup :(22:41
NobodyCamLGTM, +2, lets see if Haomeng has time to take a look at it, or any one else have time for a quick review: https://review.openstack.org/21884322:43
*** Haomeng|2 has joined #openstack-ironic22:43
* jroll looks22:43
NobodyCamawesome Thank you jroll :)22:43
jroll+A22:44
NobodyCamw00t Ty jroll :)22:45
*** Haomeng has quit IRC22:46
* TheJulia dances22:46
*** meghal has quit IRC22:51
openstackgerritMerged openstack/bifrost: Update iPXE agent template and boot.ipxe ownership  https://review.openstack.org/21884322:57
*** cdearborn has joined #openstack-ironic22:57
NobodyCamTheJulia: wow 218879 is having a hard time getting thru the check jobs22:58
NobodyCam:(22:58
*** alexpilotti has joined #openstack-ironic23:00
TheJuliaohh... I wonder if the dib job is failing silently23:02
TheJuliahmmm23:04
*** lucas-dinner has quit IRC23:05
SpamapSTheJulia: I think we can do docs once we settle on an approach. We're kind of shotgunning.23:06
TheJulia10-423:07
TheJuliaNobodyCam: the one log I dug through has nothing to do with the elemnt we want merged. :(23:09
TheJuliaNobodyCam: it is using ironic-deploy, I wonder if lucas was ever able to get them to move away from it23:18
*** meghal has joined #openstack-ironic23:23
*** sambetts has quit IRC23:28
*** sambetts has joined #openstack-ironic23:28
NobodyCamTheJulia: ack :)23:30
*** alexpilotti has quit IRC23:34
NobodyCamok folks going to head out23:39
mrdaHave fun NobodyCam!23:41
*** cdearborn has quit IRC23:44
*** shoutm has joined #openstack-ironic23:46
*** smoriya has joined #openstack-ironic23:49
*** Sukhdev has quit IRC23:51
*** achanda_ has joined #openstack-ironic23:58
*** naohirot has joined #openstack-ironic23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!