Friday, 2015-06-12

*** Sukhdev has quit IRC00:03
*** Sukhdev has joined #openstack-ironic00:03
*** Sukhdev has quit IRC00:08
*** mtanino has quit IRC00:30
*** naohirot has joined #openstack-ironic00:33
*** ijw has joined #openstack-ironic00:36
*** r-daneel_ has joined #openstack-ironic00:45
*** r-daneel has quit IRC00:45
*** saripurigopi has joined #openstack-ironic00:48
*** smoriya has joined #openstack-ironic00:49
*** saripurigopi has quit IRC00:55
*** saripurigopi has joined #openstack-ironic01:02
*** mitchjameson has quit IRC01:08
*** r-daneel_ has quit IRC01:46
*** achanda has joined #openstack-ironic01:46
*** mitchjameson has joined #openstack-ironic01:53
*** yonglihe has quit IRC01:59
*** yonglihe has joined #openstack-ironic01:59
*** achanda has quit IRC02:02
*** Marga_ has quit IRC02:21
*** saripurigopi has quit IRC02:24
*** rloo has quit IRC02:27
*** ramineni has joined #openstack-ironic02:38
*** Marga_ has joined #openstack-ironic02:39
*** Marga_ has quit IRC02:43
*** Marga_ has joined #openstack-ironic02:43
*** Marga_ has quit IRC02:45
*** Marga_ has joined #openstack-ironic02:45
*** Marga_ has quit IRC02:46
*** Marga_ has joined #openstack-ironic02:50
*** Marga_ has quit IRC02:52
*** yonglihe has quit IRC02:59
openstackgerritYongli He proposed openstack/bifrost: local install with -K if needed  https://review.openstack.org/18784103:08
*** yonglihe has joined #openstack-ironic03:11
*** spandhe has quit IRC03:15
openstackgerritTan Lin proposed openstack/ironic: Lower capability value  https://review.openstack.org/19090003:30
openstackgerritMerged openstack/ironic: remove unneeded sqlalchemy-migrate requirement  https://review.openstack.org/19069603:35
*** dguerri` is now known as dguerri03:44
*** dguerri is now known as dguerri`03:54
*** saripurigopi has joined #openstack-ironic03:55
*** mitchjameson has quit IRC03:58
*** yog_ has joined #openstack-ironic04:01
*** Marga_ has joined #openstack-ironic04:09
*** Marga_ has quit IRC04:09
*** Marga_ has joined #openstack-ironic04:10
*** Sukhdev has joined #openstack-ironic04:19
*** yuikotakada has joined #openstack-ironic04:22
yuikotakadaTGIF, Ironic04:22
openstackgerritAnusha Ramineni proposed openstack/ironic-specs: Add Zapping support to iLO drivers  https://review.openstack.org/14540404:23
*** Marga_ has quit IRC04:25
openstackgerritGopi Krishna S proposed openstack/ironic: Address follow-up comments on ucs drivers  https://review.openstack.org/19091304:29
*** rameshg87 has joined #openstack-ironic04:34
rameshg87good morning ironic04:34
yuikotakadarameshg87: good morning04:34
rameshg87yuikotakada: o/04:35
saripurigopigood morning rameshg87,yuikotakada04:41
yuikotakadasaripurigopi: g'morning04:41
rameshg87saripurigopi: o/04:44
*** zz_natorious is now known as natorious04:52
openstackgerritMerged openstack/ironic: Add additional logging around cleaning  https://review.openstack.org/19072804:53
*** chlong is now known as chlong_mtg04:59
openstackgerritTan Lin proposed openstack/ironic: Set the capability value to lowercase  https://review.openstack.org/19090004:59
*** natorious is now known as zz_natorious05:00
*** bkero has quit IRC05:10
*** bkero has joined #openstack-ironic05:10
*** smoriya has quit IRC05:28
*** gridinv_ has quit IRC05:31
*** mitchjameson has joined #openstack-ironic05:43
*** zz_naotok has quit IRC05:54
*** yuikotakada has quit IRC05:56
*** vishwana_ has quit IRC05:58
*** vishwanathj has joined #openstack-ironic06:00
chlong_mtgping06:04
chlong_mtgCan I ask an ironic question here?06:04
*** yuikotakada has joined #openstack-ironic06:04
*** radek_ has joined #openstack-ironic06:05
chlong_mtgironic question: I'm having problems registering newly created VM's in ironic cli for use with my overcloud to scale out. Any ideas? I'm following the instructions here: https://www.rdoproject.org/TripleO-CLI#Post-Deployment06:06
chlong_mtgI'm always receiving this error: InvalidParameterValue: SSH connection cannot be established: Failed to establish SSH connection to host 192.168.122.106:06
*** chlong_mtg is now known as chlong06:06
*** amotoki has joined #openstack-ironic06:16
*** Sukhdev has quit IRC06:19
*** Haomeng has joined #openstack-ironic06:27
openstackgerritTan Lin proposed openstack/ironic: Remove unnecessary validation in PXE  https://review.openstack.org/19095206:38
*** lazy_prince has joined #openstack-ironic06:41
*** Guest____ has joined #openstack-ironic06:45
*** amotoki has quit IRC06:47
lazy_princeo/06:48
*** amotoki has joined #openstack-ironic06:50
*** Nisha has joined #openstack-ironic06:50
*** Guest____ is now known as jcoufal06:51
*** jcoufal is now known as Guest7641006:51
*** Guest76410 has quit IRC06:53
*** jcoufal_ has joined #openstack-ironic06:53
*** mitchjameson has quit IRC06:54
*** dtantsur|afk is now known as dtantsur06:54
dtantsurmorning Ironic06:54
dtantsurTGIF!06:54
yuikotakadalazy_prince: o/06:55
yuikotakadadtantsur: TGIF!06:55
dtantsur:)06:55
lazy_prince:)06:55
*** romainh has joined #openstack-ironic06:56
rameshg87chlong: may be you should check with dtantsur about rdo thing. he might be able to point out whom to check with.06:57
rameshg87dtantsur: lazy_prince: morning o/06:57
chlongrameshg87, OK06:57
dtantsurrameshg87, o/06:57
chlongdtantsur, ping06:58
dtantsurchlong, hi!06:58
chlongironic question: I'm having problems registering newly created VM's in ironic cli for use with my overcloud to scale out. Any ideas? I'm following the instructions here: https://www.rdoproject.org/TripleO-CLI#Post-Deployment06:58
chlongdtantsur, who should I be speaking to about this ?06:58
dtantsurchlong, let's continue on #rdo, I probably can help you06:59
*** derekh has joined #openstack-ironic06:59
*** ijw has quit IRC07:03
yuikotakadadtantsur: IMO, we Ironic-inspector should set feature freeze date same as other projects in order to recognize schedule. Currently there is just rough release date, right? WDYT?07:04
openstackgerritRamakrishnan G proposed openstack/ironic: IPA: Do a soft power off at the end of deployment  https://review.openstack.org/18566707:04
dtantsuryuikotakada, for 2.1.0? depending on what you mean by "other projects" :) everyone is doing their own thing now, e.g. Ironic is likely to have shorted FF07:05
yuikotakadadtantsur, yeah, for v2.1.007:05
yuikotakada"other projects" means like nova or ironic and so on07:05
dtantsuryuikotakada, we can just follow Ironic, as it's the closest "other project" for us :) nova might have a different schedule07:06
*** naohirot has quit IRC07:06
yuikotakadaoh07:06
yuikotakadaare we following to ironic? That's right, I forgot it :p07:07
dtantsurI guess we do. Let's see what we agree on for Ironic actually (if we actually agree on something) :D07:07
yuikotakada:D07:09
*** athomas has joined #openstack-ironic07:11
*** Haomeng|2 has joined #openstack-ironic07:14
*** Haomeng has quit IRC07:16
openstackgerritRamakrishnan G proposed openstack/ironic: Add boot interface in Ironic  https://review.openstack.org/16651207:21
*** rameshg87 is now known as rameshg87-lunch07:22
*** ifarkas has joined #openstack-ironic07:28
*** Nisha has quit IRC07:51
*** dtantsur is now known as dtantsur|bbl07:51
*** dlpartain has joined #openstack-ironic07:52
openstackgerritMerged stackforge/proliantutils: Add iSCSI boot support in RIS  https://review.openstack.org/18716207:56
*** ijw has joined #openstack-ironic08:04
*** Nisha has joined #openstack-ironic08:05
openstackgerritRamakrishnan G proposed openstack/ironic: IPA: Do a soft power off at the end of deployment  https://review.openstack.org/18566708:07
*** ijw_ has joined #openstack-ironic08:08
*** dlpartain has left #openstack-ironic08:10
*** ijw has quit IRC08:10
*** rameshg87-lunch is now known as rameshg8708:11
*** ijw_ has quit IRC08:13
*** erelwell has joined #openstack-ironic08:13
*** chlong is now known as chlong_afk08:16
*** lucasagomes has joined #openstack-ironic08:16
*** yuikotakada has quit IRC08:17
openstackgerritMerged openstack/ironic: Remove unnecessary validation in PXE  https://review.openstack.org/19095208:18
*** subscope has joined #openstack-ironic08:20
*** jistr has joined #openstack-ironic08:20
*** yuikotakada has joined #openstack-ironic08:21
*** stendulker has joined #openstack-ironic08:27
*** ndipanov has joined #openstack-ironic08:31
*** dmitchell has joined #openstack-ironic08:35
*** e0ne has joined #openstack-ironic08:38
*** MattMan has quit IRC08:41
openstackgerritGopi Krishna S proposed openstack/ironic: Address follow-up comments on ucs drivers  https://review.openstack.org/19091308:48
*** NobodyCam has quit IRC08:49
*** NobodyCam has joined #openstack-ironic08:50
*** lucasagomes has quit IRC08:52
*** ifarkas has quit IRC08:52
*** ifarkas has joined #openstack-ironic08:53
*** dmitchell__ has joined #openstack-ironic08:54
*** lucasagomes has joined #openstack-ironic08:56
*** e0ne has quit IRC08:57
*** dmitchell has quit IRC08:58
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Wake-On-Lan Power interface  https://review.openstack.org/17907809:02
*** chlong_afk has quit IRC09:03
*** ndipanov has quit IRC09:04
*** dguerri` is now known as dguerri09:05
*** ijw has joined #openstack-ironic09:08
*** pelix has joined #openstack-ironic09:09
*** ijw has quit IRC09:13
stendulkerHi09:14
stendulkerFor dhclient request, dnsmasq dhcp is responding with the IP, but that IP is not getting set to the interface.09:16
stendulkerNode perpetually requests for the IP and dnsmasq pertually send the IP address to it. Without the node setting the received IP.09:17
stendulkerAny idea what could be the issue here?09:17
stendulkerI face this issue with CoreOS and openSUSE linux09:18
openstackgerritRamakrishnan G proposed openstack/ironic: Refactor method to add or update capability string  https://review.openstack.org/16245109:18
stendulkeruser images of CoreOS and openSUSE linux09:18
openstackgerritGopi Krishna S proposed openstack/ironic: Address follow-up comments on ucs drivers  https://review.openstack.org/19091309:20
*** Marga_ has joined #openstack-ironic09:34
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Capabilities should accept values as dictionary  https://review.openstack.org/18293409:37
*** ifarkas has quit IRC09:38
*** ifarkas has joined #openstack-ironic09:39
openstackgerritGopi Krishna S proposed openstack/ironic: Address follow-up comments on ucs drivers  https://review.openstack.org/19091309:43
*** ndipanov has joined #openstack-ironic09:44
*** mgoddard1 has joined #openstack-ironic09:46
*** mgoddard has quit IRC09:47
*** ndipanov has quit IRC09:49
*** ndipanov has joined #openstack-ironic09:49
*** Nisha has quit IRC09:50
*** romainh has quit IRC09:50
*** mgoddard1 has quit IRC09:50
rameshg87lucasagomes: hi09:51
lucasagomesrameshg87, hello there!09:51
rameshg87lucasagomes: a quick question09:51
*** toabctl has quit IRC09:52
rameshg87lucasagomes: I think it's better if we make the raid stuff not to wait for zapping09:52
rameshg87lucasagomes: because we have seen that raid can be used in cleaning as well09:52
*** toabctl has joined #openstack-ironic09:52
rameshg87lucasagomes: since zapping/cleaning is just a difference of priority values, can we get the raid stuff merged before zapping ?09:53
rameshg87lucasagomes: raid generic stuff has now 2x+2s and inband raid had 4 +2s (in previous patch set)09:53
rameshg87wdyt ?09:53
lucasagomesrameshg87, /me thinks09:57
lazy_princestendulker: did you check the dhclient log files..?09:57
lucasagomesrameshg87, so how one triggers raid? Via cleaning? Or passthru?09:58
rameshg87lucasagomes: through zapping or cleaning09:58
rameshg87lucasagomes: the create_configuration will be decorated and the priority value will be configurable09:59
rameshg87lucasagomes: if it's non-zero it can be done during cleaning09:59
rameshg87lucasagomes: so that means it will work with the existing framework itself09:59
lucasagomesrameshg87, ack yeah sounds good10:01
lucasagomesif that was the decision to have on zapping and/or cleaning10:01
lucasagomesit seems fine10:01
rameshg87yeah10:01
* rameshg87 goes and removes gerrit dependency of raid spec10:01
rameshg87lucasagomes: have mentioned it in spec10:02
rameshg87https://review.openstack.org/#/c/173214/11/specs/liberty/ironic-generic-raid-interface.rst L17-2410:02
* lucasagomes reviews it10:03
*** e0ne has joined #openstack-ironic10:04
*** e0ne is now known as e0ne_10:04
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add generic RAID configuration spec for liberty  https://review.openstack.org/17321410:05
stendulkerlazy_prince: what is path of dhclient logs10:05
*** e0ne_ is now known as e0ne10:06
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add inband RAID configuration spec for liberty  https://review.openstack.org/17321810:06
stendulkerlazy_prince: Did not nee any thing under /var/log10:06
*** ijw has joined #openstack-ironic10:06
lazy_princeit depends on distro but it should be in /var/log/dhcpcd10:06
lazy_princeyou could also try running dhclient in verbose mode to see if it reports any error..10:07
*** romainh has joined #openstack-ironic10:10
*** alexpilotti has joined #openstack-ironic10:10
stendulkerlazy_prince: Running on opensuse. Did not find dhclient binary on the node. Probably n/w service is issuing  the dhclient request10:11
*** ijw has quit IRC10:12
lazy_princestendulker:  can you see if you have dhcpcd or something similar..?10:13
lazy_princeon SLES, I could see there is dhcpcd...10:13
lucasagomesrameshg87, question... so it was made possible to run on cleaning so we could have software raid?10:15
lucasagomesor that spec is specific for hardware raid?10:15
stendulkerlazy_prince: dhcpd binary also do not exist here10:15
rameshg87lucasagomes: JayF first pointed out that it can be run on cleaning with software raid10:16
lazy_princedhcpd or dhcpcd..?10:16
lucasagomesrameshg87, yeah... it indeed seems possible10:16
rameshg87lucasagomes: but then we generally came to an agreement that it can left out as operator's choice to make it available as cleaning or zapping10:16
lucasagomesanyway... just a question, it's out of the scope of the spec anyway10:16
lucasagomesrameshg87, seems sane10:16
lucasagomesyeah10:16
lazy_princestendulker: dhcpd or dhcpcd..?10:17
rameshg87lucasagomes: if someone wants to make sure the node is set everytime with same raid configuration before it's available10:17
rameshg87they could make it as part of cleaning10:17
rameshg87in hp hardware at least, the tenant can modify raid configuration with inband utilities10:17
lazy_princestendulker: also check if the log files exists in /var/lib..10:18
lucasagomesrameshg87, I see10:18
lucasagomesrameshg87, +a10:19
lucasagomesthanks happy hacking10:19
stendulkerlazy_prince: no dhcpd or dhcpcd10:19
rameshg87\o/10:20
rameshg87thanks :)10:20
stendulkerno releant logs in /var/lib as well10:20
stendulkerlazy_prince: all i could see on server side the mac address beig given IP but no ack from the node for that IP10:20
lazy_princestendulker: aha.. so then which utility is requesting IP address..10:20
*** mgoddard has joined #openstack-ironic10:21
stendulkerlazy_prince: 03:05:26.652789 IP 0.0.0.0.bootpc > 255.255.255.255.bootps: BOOTP/DHCP, Request from 9c:b6:54:af:93:3c (oui Unknown), length 31210:21
stendulker03:05:26.653016 IP 10.10.1.197.bootps > 10.10.1.190.bootpc: BOOTP/DHCP, Reply, length 32610:21
rameshg87lucasagomes: and there is an inband one too. please have a look at it when you get some time10:21
saripurigopiI'm installing devstack on ubuntu14.1 , it is failing with 2015-07-23 09:39:56.624 | ++ die 251 'PXE boot file /usr/lib/syslinux/pxelinux.0 not found.'10:22
rameshg87lucasagomes: I guess you are last spec core to not +2 it yet :D10:22
*** athomas has quit IRC10:23
lucasagomesrameshg87, ack10:25
lazy_princestendulker: here..10:25
lazy_princefor me the tcpdump shows as follows:10:25
lazy_prince06:27:43.576668 IP 0.0.0.0.bootpc > 255.255.255.255.bootps: BOOTP/DHCP, Request from 00:50:56:98:1b:d2 (oui Unknown), length 54810:25
lazy_prince06:27:43.577277 IP 192.168.0.51.bootps > 255.255.255.255.bootpc: BOOTP/DHCP, Reply, length 34410:25
lazy_princein your case the bootpc 2nd line has a ip address.. however in my case it 255.255.255.25510:26
stendulkerlazy_prince: in your case it looks like  broadcasting10:26
*** romainh has quit IRC10:27
lazy_princethats how it should be as the client still does not have an ip address10:27
lazy_princeonce IP address is received, then it should only be renewal10:28
stendulkerstendulker: yes, but for ubuntu deploy ramdisk it does get IP through DHCP , though I have not captured  tcpdump of tht10:29
*** athomas has joined #openstack-ironic10:29
stendulkerlazy_prince: Your thing makes sense. But its dnsmasq thing, tht should take care of the protocol..10:29
lazy_princestendulker: these lines that i have posted are for ubuntu deploy images.. :)10:30
stendulkerlazy_prince: ok :)10:30
openstackgerritYuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials  https://review.openstack.org/17078210:30
stendulkerlazy_prince: this is suse whole disk image10:30
stendulkerlazy_pricne: being deployed on to the BM10:30
lazy_princestendulker: are you sure, your system has to get 10.10.1.190 IP address..? it could also be that some other is renewing the lease.. just a thought..10:31
openstackgerritMerged openstack/ironic-specs: Add generic RAID configuration spec for liberty  https://review.openstack.org/17321410:31
*** yuikotakada has quit IRC10:32
lazy_princethe other thing to check is if the image has driver for your NIC and that someone else is not using the same IP address for something else..10:32
stendulkerlazy_prince: That part I had confirmed tht no other dhcp is responding to this mac10:32
rameshg87ifarkas: ^^^^^10:33
stendulkerlazy_prince: I will try to deploy again and chk what tcpdump shows for ubuntu deploy ramdisk10:33
*** subscope has quit IRC10:33
*** yuanying has quit IRC10:34
*** rameshg87 is now known as rameshg87-away10:34
ifarkasrameshg87-away, thanks, will review it10:41
*** subscope has joined #openstack-ironic10:50
devanandahttp://www.uefi.org/sites/default/files/resources/UEFI_Plugfest_May_2015_HTTP_Boot_Redfish_Samer_El-Haj_ver1.2.pdf10:50
devanandaalso, good morning10:50
stendulkerlazy_prince: For ubuntu deploy ramdisk also I get similar kind of entries.10:51
stendulkerlazy_prince: 03:56:16.455182 IP 0.0.0.0.bootpc > 255.255.255.255.bootps: BOOTP/DHCP, Request from 9c:b6:54:af:93:3c (oui Unknown), length 30010:51
stendulkerlazy_prince: 03:56:16.456255 IP 10.10.1.197.bootps > 10.10.1.194.bootpc: BOOTP/DHCP, Reply, length 32610:51
*** ramineni has quit IRC10:52
stendulkerlazy_prince: But ubuntu does accept 10.10.1.194 as its IP and moves forward with deploy10:52
* lazy_prince out of ideas now..10:53
stendulkerlazy_prince: thank you for your help :)10:54
lazy_princestendulker: I was not able to help.. but anyways, i will accept your ty.. :P10:55
*** rameshg87-away is now known as rameshg8710:56
rameshg87ifarkas: it just got merged (again) :)10:56
*** dtantsur|bbl is now known as dtantsur10:57
ifarkasrameshg87, hah, I misread it11:00
ifarkasrameshg87, but that is excellent news!!!11:00
rameshg87:)11:01
*** stendulker has quit IRC11:02
*** ijw has joined #openstack-ironic11:06
*** romainh has joined #openstack-ironic11:09
*** ijw has quit IRC11:12
yuriyzHi Ironic11:22
*** yog_ has quit IRC11:22
*** saripurigopi has quit IRC11:22
yuriyzlucasagomes please look at https://review.openstack.org/#/c/174785/14/ironic/drivers/modules/deploy_utils.py , validation for capabilities in lowercase11:23
lucasagomesyuriyz, good afternoon11:23
lucasagomesyuriyz, I will do in a bit, I gotta go out collect something in the POST and grab some lunch11:23
lucasagomeswill review it right after I get back11:24
*** lucasagomes is now known as lucas-hungry11:24
*** ndipanov has quit IRC11:33
dtantsurmorning, devananda, yuriyz, and everyone I missed during my language lessons :)11:33
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/18990011:34
dtantsurany specs-core up for reviewing the ENROLL spec https://review.openstack.org/#/c/179151 please?11:34
openstackgerritMerged openstack/bifrost: Correct CSV processing in inventory  https://review.openstack.org/19078111:35
*** saripurigopi has joined #openstack-ironic11:37
*** e0ne is now known as e0ne_11:38
openstackgerritGopi Krishna S proposed openstack/ironic: Address follow-up comments on ucs drivers  https://review.openstack.org/19091311:39
*** krtaylor has joined #openstack-ironic11:42
*** saripurigopi has quit IRC11:42
openstackgerritMerged openstack/bifrost: Change pip install method and add six to requirements  https://review.openstack.org/19076811:44
openstackgerritMerged stackforge/ironic-discoverd: Use keystonemiddleware auth credentials  https://review.openstack.org/17078211:48
*** ndipanov has joined #openstack-ironic11:48
*** e0ne_ has quit IRC11:49
rameshg87dtantsur: hi, do you want someone else also to take a look at https://review.openstack.org/#/c/189241/11:57
rameshg872 x +2s already :)11:58
openstackgerritRamakrishnan G proposed openstack/ironic: Add driver interface for RAID configuration  https://review.openstack.org/15523011:58
openstackgerritRamakrishnan G proposed openstack/ironic: Refactor method to add or update capability string  https://review.openstack.org/16245111:58
dtantsurrameshg87, well, I don't known IPA really well...11:58
*** zhenguo has quit IRC11:58
rameshg87dtantsur: okay. will wait for someone's to have a look at it. np.11:59
sambettsmorning dtantsur12:02
dtantsursambetts, morning!12:02
sambettsUGT morning ironic :)12:02
openstackgerritJohn Trowbridge proposed stackforge/ironic-discoverd: Rename edeploy plugin  https://review.openstack.org/18994412:02
openstackgerritJohn Trowbridge proposed stackforge/ironic-discoverd: Fix edeploy plugin puts too much data in Ironic extra column  https://review.openstack.org/18942712:02
sambetts(it totally afternoon for me right now)12:02
rameshg87sambetts: o/12:03
sambettso/ rameshg8712:03
rameshg87sambetts: that's what ugt was for right ? :)12:03
sambettsrameshg87: haha yup12:03
rameshg87it's morning whenever we begin work :)12:03
dtantsur++ once I had morning at 5pm12:04
dtantsur(after the summit, maybe you remember_12:04
*** tiagogomes_ has joined #openstack-ironic12:05
TheJuliagood morning everyone12:06
dtantsurTheJulia, morning12:06
sambettsMorning TheJulia12:06
*** ijw has joined #openstack-ironic12:06
*** dprince has joined #openstack-ironic12:06
rameshg87dtantsur: :)12:07
* rameshg87 goes home12:07
*** rameshg87 has quit IRC12:08
*** ndipanov has quit IRC12:09
tiagogomes_Hi, I am getting an "No valid host was found. There are not enough hosts available" error when trying to provision with Nova. My baremetal flavor is http://paste.openstack.org/show/285134/ , and the hypervisor details are this http://paste.openstack.org/show/285135/ . Anyone has any clue about this?12:10
dtantsurtiagogomes_, hey, here's a guide I once wrote on this error: https://repos.fedorapeople.org/repos/openstack-m/docs/master/troubleshooting/troubleshooting-overcloud.html#no-valid-host-found-error12:11
*** ijw has quit IRC12:12
openstackgerritSam Betts proposed stackforge/ironic-discoverd: Provide hook into process when a node isn't found  https://review.openstack.org/19012412:14
sambettsdtantsur:12:14
sambettsdtantsur: ^ I moved the find node info stuff out like we discussed, which got rid of the complexity issue :)12:15
dtantsurcool!12:15
tiagogomes_thanks for the link dtantsur, I'll have a look12:15
tiagogomes_dtantsur, "Make sure that enough nodes are in available state according to ironic node-list". I can't see available in any column. Should it be on the "Provisioning State" column?12:22
dtantsurtiagogomes_, yep. If you have older ironic and/or client, it may look like None (or NOSTATE?)12:23
*** subscope has quit IRC12:23
tiagogomes_dtantsur, I am using Kilo version. It displays None12:23
*** ndipanov has joined #openstack-ironic12:23
dtantsurtiagogomes_, which client version?12:23
tiagogomes_dtantsur, 0.4.112:24
dtantsurtiagogomes_, ok, so it's fine, this guide is for RDO, where we have 0.6 (at least) already12:24
dtantsurNone is fine anyway12:24
*** trown|outttypeww is now known as trown12:30
lazy_princejroll: ironic neutron plugin refactored as a ML2 extension driver -> https://github.com/kumarom/ironic-neutron-plugin.git12:31
v92nice12:34
openstackgerritMerged openstack/ironic: Add vendor-passthru to attach and boot an ISO  https://review.openstack.org/18831712:35
tiagogomes_mmm, everything looks fine but I still get the "No valid host was found" error12:35
dtantsurhmmm12:36
*** jjohnson2 has joined #openstack-ironic12:37
dtantsurtiagogomes_, could you paste your `ironic node-show UUID`?12:37
*** subscope has joined #openstack-ironic12:38
tiagogomes_dtantsur, http://paste.openstack.org/show/285138/12:38
*** MattMan has joined #openstack-ironic12:38
tiagogomes_dtantsur, I have instance_info set, because I tried Ironic without Nova before. Could be because of that?12:39
dtantsurtiagogomes_, no, but I see your node properties != your flavor. If you use exact filters (we in RDO don't IIRC), that may be the problem12:40
dtantsurtiagogomes_, what if you make your flavor = your nodes (ram and disk-wise)?12:40
tiagogomes_dtantsur that's what I had before, and it didn't work12:41
dtantsurthe same error?12:41
tiagogomes_yep12:41
dtantsurweird. anyway before trying further, I suggest bringing them in sync12:42
dtantsurtiagogomes_, any exceptions in Nova or Ironic conductor logs?12:42
*** yog_ has joined #openstack-ironic12:43
tiagogomes_dtantsur, http://paste.openstack.org/show/285139/12:45
devanandadtantsur: http://www.gnu.org/software/libtool/manual/html_node/Libtool-versioning.html12:45
tiagogomes_Ironic conductor doesn't show any error12:46
devanandafwiw, just having coffee with mordred and discussing api versioning and whether vendor_passthru changes should bump it (answer: no)12:46
dtantsurdevananda, one of the reasons I was aleays avoiding libtool. As they say "Note that this can cause problems because libtool’s compatibility requirements are actually stricter than is necessary."12:47
dtantsurdevananda, I don't disagree, I just point it as source of inconsistency in the whole approach12:47
*** mordred has joined #openstack-ironic12:47
devanandadtantsur: ironic never claimed to express a contract around, or support in any way, what is in /vendor_passthru/12:48
*** trown is now known as trown|brb12:48
devanandathough the semantic of how one discovers that -- that's our contract12:48
devanandaI was reviewing lucas-hungry's change to add File support to vendor responses12:48
devanandadidn't know WSME could do that :)12:49
dtantsurdevananda, I still don't understand while exposing a list of strings via API (= my feature detection proposal) is worse than making hard work on every API-touching patch feature gating it's changes (= Nova approach)12:49
dtantsurdevananda, wow, haven't see that patch12:49
devananda18971612:49
dtantsuroh amazing! I wonder what output format is, but it's anyway amazing12:50
devanandawtf? I dont have stable branch rights on ironic now12:51
dtantsurO__O12:51
* devananda pokes infra team12:55
devanandadtantsur: http://www.gnu.org/software/libtool/manual/libtool.html#Updating-version-info12:59
devanandathis is a different way to express the min/max/current that our API expresses today13:00
*** e0ne has joined #openstack-ironic13:01
*** krtaylor has quit IRC13:02
devanandadtantsur: so, what I think you want is for the client to express "does the REST API support version X?" whereas our implementatoin, today, only allows the client to express "plesae give me version X"13:03
*** lucas-hungry is now known as lucasagomes13:03
devanandadtantsur: and I agree that a client who expects version X does not care about the existence of new interfaces -- so there is no benefit to that client if our REST API hides them13:04
lucasagomesdevananda, morning tiagogomes_13:04
lucasagomesdevananda, yeah, we need that for the iPXE dynamic config :-)13:05
lucasagomesso we are going to generate the conf files and iPXE will download it directly from our api13:05
lucasagomesslick13:05
*** trown|brb is now known as trown13:05
*** ijw has joined #openstack-ironic13:06
BadCubmorning folks13:07
*** [1]cdearborn has joined #openstack-ironic13:07
lucasagomesdtantsur, the output format is the raw content of the file13:08
lucasagomesif you do a wget <api url>/vendor_passthru/method it will download a file13:08
lucasagomesBadCub, good ugt morning13:09
BadCubmornin lucasagomes and happy friday!13:09
devanandadtantsur: we need to increment the api version number when adding an interface (rest api endpoint, etc) but we don't need to hide it when the requested version was less than that13:10
devanandadtantsur: so that a client who depends on that interface is ble t odetect that older servers do not support it13:10
openstackgerritVladyslav Drok proposed openstack/ironic: Remove tools/flakes.py  https://review.openstack.org/19106113:10
lucasagomesBadCub, TGIF :-) happy friday there too13:10
dtantsurdevananda, I still prefer string-based detection (I was even thinking about using spec names as feature names), but sounds good as well13:11
dtantsurdevananda, what about other API changes? e.g. we feature-gated new provision verbs, which I think was a mistake. wdyt?13:11
dtantsurBadCub, morning, happy Friday :)13:11
*** ijw has quit IRC13:11
devanandadtantsur: string based? spec names as api version? huh?13:11
dtantsurdevananda, it's still about potential API like $ ironic feature-supported <feature-name>13:12
dtantsurbut well, I'm ok with versions, unless we go insane while hiding new stuff from users :)13:12
devanandadtantsur: omg no13:12
*** achanda has joined #openstack-ironic13:13
dtantsurwell, we did hide from users thing like new management verbs. I think lucasagomes tries to introduce microversion for new argument to existing API. I don't think it should happen any more, wdyt, devananda?13:13
* lucasagomes reads13:14
lucasagomesoh, yeah cause that's a new feature in the api right?13:14
lucasagomesbe able to fetch a subset of attributes on the resources13:14
lucasagomes(for ref, the patch is this one https://review.openstack.org/#/c/183491/)13:15
mordredthe thing is - I think that's talking about different things13:15
dtantsurI meant to say, bumping version is good, but not hiding this feaure, right?13:15
mordredironic feature-supported is a thing implemented by the client program and client library13:15
devanandadtantsur: so for S/NOSTATE/ACTIVE/, we had to have a backwards-capable wrapper13:15
mordredit can answer that question13:15
lucasagomesdtantsur, I iwll read the scrollback, I dunno what it meas by hiding13:15
mordreddtantsur: I, btw, agree with you that hiding the feature is not useful, and I think I've convinced devananda of that too :)13:16
dtantsurlucasagomes, this: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L58-L6913:16
dtantsurlucasagomes, or this: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L79-L8713:16
dtantsurmordred, good to hear :)13:16
lucasagomesdtantsur, oh right13:17
mordreddtantsur: I think the question the client interaction is asking for is "I am programmed against API version 4, can you (the server) support that for me please"13:17
lucasagomesI thought it was done because someone might want to check like schema13:17
mordredto which the server can respond either "yes" or "no"13:17
devanandamordred: hiding new endpoints - yes, I agree, not useful13:17
* lucasagomes thinks we should have schemas in our api13:17
mordredsupporting API version 4 does not say anything about theoretical existence of things that are not part of vesrion 413:17
devanandaand in order to support version 4, sometimes the server needs to wrap / translate / transform some data so that it looks the same as it did in version 413:18
mordredif the client, on the other hand requests version 4 and then uses version 5 - it _is_ a bug in the client program13:18
mordredbut I do not believe it's useful for the server to enforce that bug13:18
dtantsurdevananda, only when change is breaking IMO13:18
mordredit _might_ be useful for the client library to detect and warn the user about such a behavior13:18
mordredsince it means the program may be accidentally incompatible with a different cloud13:19
mordredbut now we're getting in to some esoteric problems13:19
dtantsurdevananda, I don't agree https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L58-L69 is sane, unless we account for insane clients (which I've seen tbh, but they're hard to account for)13:19
[1]cdearbornMorning Ironic!13:20
TheJuliagood morning13:20
dtantsur[1]cdearborn, morning!13:20
devanandadtantsur: looking at list of features we've bumped api version for so far --13:21
devananda# v1.2: Renamed NOSTATE ("None") to AVAILABLE ("available")13:21
devananda# v1.3: Add node.driver_internal_info13:21
devananda# v1.4: Add MANAGEABLE state13:21
devananda# v1.5: Add logical node names13:21
devananda# v1.6: Add INSPECT* states13:21
devanandatwo of those added a new resource: driver_internal_info and "name"13:21
devanandatwo added new verbs -- which affects both the allowable values in a POST request, and the set of possible returned values for provision-state13:22
devanandaentirely new semantics in the client became possible -- but the old semantics were still valid, too13:22
devanandaonly 1.2 was incompatible in the strictest sense13:22
dtantsurright13:22
BadCubmorning and happy friday dtantsur devananda mordred13:23
*** krtaylor has joined #openstack-ironic13:26
NobodyCamGood morning Ironicers and.....13:26
NobodyCamTGIF!13:26
dtantsurNobodyCam, hey, TGIF!13:26
[1]cdearbornmorning NobodyCam, +25!13:26
NobodyCammorning dtantsur and [1]cdearborn :)13:27
mordredmorning BadCub !13:27
NobodyCamhow goes this Friday13:27
NobodyCammorning mordred :)13:27
devanandadtantsur: so, in summary, I dont think we need to hide compatible changes any longer, but we should be bumping the current version on any api-affecting change13:27
mordredmorning NobodyCam !13:27
NobodyCammorning devananda13:27
NobodyCam:)13:27
[1]cdearbornI officially have a lab connection to my desk, so I don't have to hike to the gym any more to do git review.  Life Is Good!13:28
dtantsurdevananda, ++ agreed13:28
dtantsurdevananda, can/should we move this discussion to the ML for visibility once we have at least rough consensus now?13:28
devanandadtantsur: and if we add a small incompatible change that we CAN wrap, then we dont need to bump the minimum supported13:28
devanandafor example - changing the default state for newly created nodes to ENROLL or MANAGEABLE13:29
devanandathat behavior change MUST be guarded with a new version13:29
*** achanda_ has joined #openstack-ironic13:29
dtantsurdefinitely (it's in the spec IIRC)13:29
devanandabut we CAN support the old behavior still, if a client requests version < 1.7 (for instance)13:29
*** achanda has quit IRC13:31
dtantsurdevananda, speaking of which, any chances you have a look at the ENROLL spec (179151)? It's a hard change, I think it's fair to drag you into reviewing it :)13:32
devanandadtantsur: not yet. will look in a minute13:32
dtantsurthnx!13:32
*** bradjones has quit IRC13:35
*** bradjones has joined #openstack-ironic13:38
*** bradjones has quit IRC13:38
*** bradjones has joined #openstack-ironic13:38
*** derekh is now known as derekh_afk13:39
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Refactor node's and driver's vendor passthru to a common place  https://review.openstack.org/19012113:50
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Allow vendor methods to serve static files  https://review.openstack.org/18971613:50
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Change return value of [driver_]vendor_passthru to dict  https://review.openstack.org/19012013:50
* lucasagomes fixes the py34 problem13:50
*** krtaylor has quit IRC13:51
*** dmitchell has joined #openstack-ironic13:52
NobodyCammorning lucasagomes13:52
lucasagomesbtw I installed py34 now, but when I run the tests a get loads of "object is not iterable"13:52
lucasagomesI know the cause and that it's a mock bug13:52
lucasagomesbut it kinda sucks :-(13:52
lucasagomesNobodyCam, morning13:52
devanandadtantsur: 179151 +2'd -- I'm good with approving it, but also happy to give others a couple more days13:52
devanandalucasagomes: see my response o nthe patch fixing py34 on fedora?13:53
lucasagomesdevananda, we have a patch for that? /me looks13:53
lucasagomesI haven't13:53
* lucasagomes finds it13:53
lucasagomeshttps://review.openstack.org/#/c/190835/13:53
devanandayes13:54
*** dmitchell__ has quit IRC13:54
lucasagomesdevananda, right, what version pf py34 you have there? I think it may affect other OSs later13:54
lucasagomescause it's a bug in mock13:54
lucasagomesfedora just happens to have a newer version13:54
lucasagomesAFAUI, I will read more but I might be wrong13:54
lucasagomesdevananda, but that's a super valid point, I would like to have some fedora gating13:55
NobodyCamlucasagomes: when you have a few minute could I by chance draw on your RH experence for a biFrost review13:56
NobodyCamhttps://review.openstack.org/#/c/18529413:56
lucasagomesNobodyCam, ack13:56
NobodyCamawesome TY13:56
TheJuliaty lucasagomes13:57
devanandalucasagomes: ++ on fedora jobs, but I think they'll need to run as third-party CI :-/13:57
lucasagomesdevananda, http://packages.ubuntu.com/trusty/python3 seems that's 3.4.0 that we run on gate13:57
devanandayup13:57
lucasagomesThe following fails on python 3.4.2, 3.4.3 and 3.5.0a2 (downloaded from python.org and compiled on Ubuntu 14.04).13:57
devanandaoooh13:57
lucasagomesonce it's updated to 3.4.2+ it's going to fail on ubuntu too13:58
lucasagomesdevananda, http://bugs.python.org/issue23661.13:58
lucasagomesthat's the version of the mock in python, it's not OS related13:58
sambettslucasagomes: I get the 3.4.2 failures on my local machine :(13:58
devanandamordred: ^13:58
lucasagomessambetts, right13:58
devanandalucasagomes: can you start a discussion about that in -infra ?13:59
NobodyCammorning sambetts13:59
lucasagomesdevananda, ack, I will comment on that patch first and jump on -infra13:59
devanandaty13:59
* devananda jumps on a meeting13:59
jrollI wish we could get python cores to land that mock patch, though I doubt they'd backport it all the way back to 3.4.214:01
jrollg'morning everyone14:01
dtantsurjroll, morning14:05
dtantsurdevananda, ty, yeah let's make people aware of this spec :)14:05
lazy_princejroll: morning14:06
lazy_princejroll: ironic neutron plugin refactored as a ML2 extension driver -> https://github.com/kumarom/ironic-neutron-plugin.git14:06
*** ijw has joined #openstack-ironic14:06
NobodyCammorning jroll lazy_prince :)14:06
jrolllazy_prince: yep, saw that, nice :)14:06
lazy_princeNobodyCam: its actually night here..14:07
jrollit's always morning in ironiclandia!14:07
jrolllazy_prince: http://www.total-knowledge.com/~ilya/mips/ugt.html :)14:08
lazy_prince:)14:08
*** achanda_ has quit IRC14:08
*** rbudden has joined #openstack-ironic14:10
NobodyCam:) jroll ++++ on UGT link!!! hehehehe14:10
jroll:P14:10
*** ijw has quit IRC14:12
*** yog_ has quit IRC14:13
jrollany opposition to +A on https://review.openstack.org/#/c/173218/614:13
jroll4x +2 on the previous patchset, this one had no changes just a rebase14:13
* jroll does it14:13
jrolloh I guess I was wrong, I must have been thinking of a different spec... oops.14:14
jroll¯\_(ツ)_/¯14:15
dtantsurlol14:16
dtantsurwell, this spec does not look too complicated to me14:16
*** bnemec has joined #openstack-ironic14:16
*** dmitchell__ has joined #openstack-ironic14:17
*** dmitchell has quit IRC14:20
*** [1]cdearborn has quit IRC14:20
openstackgerritMerged openstack/ironic-specs: Add inband RAID configuration spec for liberty  https://review.openstack.org/17321814:21
sambettsdtantsur: If I change NodeNotFoundHook into just a function instead of a class, don't we lose the enforcement that the abc abstract class thing gives us?14:22
dtantsursambetts, it does not gives us much actually14:22
dtantsurI doubt it enforces function signatures for example..14:23
sambettsdtantsur: I just realised I forgot the @abc.abstractmethod, <- I'm pretty sure that does14:23
jrollit doesn't, nova broke us once by adding an argument to a driver method14:23
dtantsursambetts, that's the only thing it checks IIRC14:23
jrollthough idk if those are abstract, ignore me14:23
dtantsursambetts, I meant, IIRC it checks for overloading, but not for signatures14:24
*** cdearborn has joined #openstack-ironic14:25
sambettsdtantsur: Just reading the docs it seems that it just ensures that the function is overloaded by subclasses, nothing more14:26
dtantsursambetts, right. And if we have one function, we're already sure we have a function :)14:26
mjturek1hey all, maurosr and I had noticed that there's a check in devstack that sets the rootfs type to ramfs if the node RAM is set to <1024. We were wondering if this might be a sanity check worth bringing into ironic proper. We're thinking that, during a deploy, if the flavor's RAM is <1024, ramfs is automatically used as the rootfstype rather than what's in ironic.conf.14:27
openstackgerritJulia Kreger proposed openstack/bifrost: Add os_ironic_facts module  https://review.openstack.org/19109314:27
maurosrthis lines https://github.com/openstack-dev/devstack/blob/master/lib/ironic#L353-L356 specifically14:29
*** krtaylor has joined #openstack-ironic14:32
*** ijw has joined #openstack-ironic14:34
TheJuliamjturek1: I think that would be something that would be more deployment specific detail.  Additionally we have many users focusing on use of the Ironic Python Agent which realistically requires a very different footprint to deploy.14:35
mjturek1TheJulia: ahh generally a larger footprint for IPA?14:36
TheJuliamjturek1: like 2.5-3GB minimum14:37
mjturek1gotcha14:37
jrollo.o14:38
mjturek1TheJulia: maurosr and I are willing to implement the check, think it'd be worth at least showing the community? Of course it's understandable if it's ultimately deemed not worthwhile14:39
*** zz_jgrimm is now known as jgrimm14:39
jrollTheJulia: that seems... high.14:39
jrollis that coreos ramdisk or dib14:39
TheJuliajroll: coreos14:39
jrollhrm14:39
dtantsurI was assuming only dib ramdisk has this problem...14:39
jrollI guess maybe with pulling images into the ramdisk14:39
jrollit's totally fine with 1gb with cirros... so that plus the size of your image I guess14:40
*** romainh has left #openstack-ironic14:41
*** jgrimm is now known as zz_jgrimm14:41
TheJuliajroll: I seem to remember one of my cohorts ran into issues trying to boot the coreos IPA image in <1.5GB of ram and just didn't get anywhere until we took it to 2GB.14:42
jrollTheJulia: odd, the gate runs on 1gb :)14:43
TheJuliaHmm..14:43
dtantsurjroll, (maybe stupid question follows) can't we somehow stream an image to the hard driver (aka curl | dd), so that we don't have to keep it in memory?14:43
jrolldtantsur: yeah. totally. got some time to work on it? :)14:44
TheJuliajroll: very interesting.....14:44
dtantsurjroll, likely to have this cycle14:44
dtantsurno right now though14:44
jrolldtantsur: comstud started working on making image writing better/pluggable before he stopped working on this stuff, never ressurected it14:44
TheJuliadtantsur: I think it gets written through qemu-img out to disk14:44
TheJuliabut if it supports stdin, then that could work14:44
jrollit was more about vhd support, but may help here https://review.openstack.org/#/c/128701/14:45
jrollmaybe not, dunno14:45
dtantsurif nobody takes this task, I'll get to it this cycle. lowering memory requirements is a must..14:45
jrollhow many 2gb servers do people deploy these days? :)14:45
jrolland/or huge images14:45
jrollI guess that might be the bigger issue14:46
dtantsurwell, people tend to deploy on weird things :)14:46
jrollheh, yeah14:46
jrollour smallest flavor is 32gb so it hasn't been a priority for us :P14:46
dtantsurwell, for servers it's valid :)14:49
dtantsurjroll, though if people try to deploy Windows image with a lot of stuff installed....14:49
*** e0ne is now known as e0ne_14:49
dtantsur... and do it on a server with only 16 GiB....14:49
jrolldtantsur: yep. agree14:49
jlvillalGood morning Ironic14:50
TheJuliadtantsur: I wouldn't mind seeing a smaller footprint for testing bifrost since we currently only provision 1 VM :(14:50
TheJuliagood morning14:50
dtantsurjlvillal, morning, happy Friday14:51
jlvillaldtantsur, Thanks :)  I hope you enjoy your weekend too!14:51
dtantsurit's gonna be pretty hot here :) maybe some swimming, will see14:52
jlvillaldtantsur, Warm weather sounds good to me.  It will take a lot more warm weather here before I start complaining.14:52
jlvillaldtantsur, I will be in Moscow in July.  My wife is warning me that it will probably be hot.14:53
*** yog_ has joined #openstack-ironic14:53
*** erelwell is now known as bethelwell14:53
jlvillalTheJulia, Good morning to you too :)14:53
dtantsurjlvillal, it can be either deadly hot, or deadly cold with rain. no one can guess :)14:53
jlvillaldtantsur, You are not making me excited to go :P14:54
dtantsurlol sorry :)14:54
*** cdearborn has quit IRC14:54
dtantsurjlvillal, May is the best time to go nearly anywhere, other seasons might suck14:54
*** e0ne_ is now known as e0ne14:55
NobodyCammorning jlvillal14:56
jlvillalNobodyCam, Thanks and you too.14:57
NobodyCam:)14:57
*** jcoufal_ has quit IRC15:07
sambettsdtantsur: If I move the Error raise into the helper function in process and don't pass in the failures list, then we won't be able to include the failures that occur in the preprocessing hook loop in the Error thats raised15:09
openstackgerritChristopher Dearborn proposed openstack/ironic: Fix failing unit tests under py34  https://review.openstack.org/19083515:10
*** alex_xu is now known as alexus15:11
dtantsursambetts, right. nevermind then, we can always refactor later15:11
*** achanda has joined #openstack-ironic15:11
sambettsdtantsur: Yeah, cool, I'll push a new patch set with your comments addressed :)15:13
openstackgerritJulia Kreger proposed openstack/bifrost: Changes to ironic-install module for NUC testing in Vagrant  https://review.openstack.org/19110915:15
openstackgerritJulia Kreger proposed openstack/bifrost: Vagrantfile and vagrant.yml for testing NUC  https://review.openstack.org/19111015:15
*** yog_ has quit IRC15:16
*** e0ne is now known as e0ne_15:18
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Update devstack plugin for the recent authentication change  https://review.openstack.org/19111115:19
*** cdearborn has joined #openstack-ironic15:19
TheJuliadevananda: ^^^ two reviews posted from your AMT/vagrant stuffs15:20
*** zhenguo has joined #openstack-ironic15:20
*** dprince has quit IRC15:22
lazy_princenight all..15:24
* lazy_prince goes home..15:24
*** lazy_prince has quit IRC15:25
*** achanda has quit IRC15:25
*** e0ne_ is now known as e0ne15:27
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Change release model to independent releases  https://review.openstack.org/18517115:32
*** hemna has joined #openstack-ironic15:32
jroll^^ folks I've finally updated the release model spec with feedback from the review and the ML, pls to review :)15:33
jrolldevananda: ^ especially15:33
*** yog_ has joined #openstack-ironic15:33
*** Marga_ has quit IRC15:37
*** rameshg87 has joined #openstack-ironic15:39
openstackgerritSam Betts proposed stackforge/ironic-discoverd: Provide hook into process when a node isn't found  https://review.openstack.org/19012415:40
*** absubram has joined #openstack-ironic15:41
rameshg87jroll: hi15:41
rameshg87jroll: can you please +A the change https://review.openstack.org/#/c/189241/ if it seems okay15:41
jrollhey rameshg87, what's up :)15:41
jrolldtantsur: any reason you didn't +A that?15:42
jrollrameshg87: will do unless dmitry has a reason not to15:42
*** absubram has quit IRC15:42
rameshg87jroll: yeah sure ..15:42
jrollbbiaf15:43
*** absubram has joined #openstack-ironic15:43
*** tiagogomes_ has left #openstack-ironic15:46
*** hemna has quit IRC15:48
dtantsursee y'all next week, g'night!15:49
*** dtantsur is now known as dtantsur|afk15:49
NobodyCamhave a good weekend dtantsur|afk15:49
*** hemna has joined #openstack-ironic15:49
*** achanda has joined #openstack-ironic15:50
lucasagomesrameshg87, jroll it lgtm, should I wait to +a that?15:50
dtantsur|afkjroll, not +A as I don't know IPA well enough :)15:51
rameshg87lucasagomes: looks like I can't test wol driver for a while. I took my old cpu to nearby hardware shop, I expected a SMPS failure, but it seems something wrong with the board, it's not powering on. :(15:58
lucasagomesrameshg87, hmm15:59
lucasagomesrameshg87, powering on via WOL? or powering on in general?15:59
rameshg87lucasagomes: powering on in general.15:59
rameshg87lucasagomes: not even with power button :(15:59
lucasagomesoh damn16:01
*** subscope has quit IRC16:04
*** wuhg has joined #openstack-ironic16:05
jrolldtantsur|afk: I see16:10
jrolllucasagomes: yeah go ahead16:10
lucasagomesjroll, morning ugt :-)16:10
lucasagomesdone16:10
jrollheh morning :)16:11
jrollthanks!16:11
rameshg87\o/16:11
*** bnemec has quit IRC16:16
openstackgerritLaura Moore proposed openstack/ironic-specs: Update of the Ironic ML2 Integration spec  https://review.openstack.org/18852816:18
rameshg87clif_h: hi16:20
jrollrameshg87: he's on vacation, what's up16:20
rameshg87jroll: oh okay16:21
rameshg87jroll: wanted to discuss something regarding https://review.openstack.org/#/c/161832/16:21
jrollrameshg87: I can try :)16:22
rameshg87jroll: had some comments on it and thought it was better to go through16:22
rameshg87jroll: yeah sure16:22
rameshg87jroll: first - https://review.openstack.org/#/c/161832/15/ironic/dhcp/neutron.py16:22
*** dguerri is now known as dguerri`16:22
rameshg87jroll: is there any real-use case of separate caching vs cleaning network ?16:22
jrollrameshg87: so I don't think there's a real use case for different caching vs *provisioning* network16:23
jrollrameshg87: but I think there's a use case for different cleaning network16:23
jrollas you don't know the state of the machine when you bring it up there16:23
jrolland we don't have a concept of a provisioning network yet16:23
*** e0ne has quit IRC16:24
rameshg87jroll: okay, so later on it can be merged into provisioning network ?16:24
jrollrameshg87: I think so yeah16:24
jrollI don't have a strong opinion on that16:25
rameshg87jroll: my question because it adds more methods to our dhcp api and once it is added, it's hard to recall back16:25
*** viktors is now known as viktors|afk16:26
zhenguohi all, I create a new project ironic-dashboard as the UI component for Ironic.16:26
jrollrameshg87: indeed. not sure if there's a better way to do that16:26
*** Marga_ has joined #openstack-ironic16:27
rameshg87okay. may be will keep that aside and see if provisioning network concept takes shape by that time16:27
zhenguoplease help to review https://review.openstack.org/#/c/191142/ and https://review.openstack.org/#/c/191131/16:27
zhenguothe init repo is here: https://github.com/niuzhenguo/ironic-dashboard16:28
rameshg87jroll: and https://review.openstack.org/#/c/161832/15/ironic/drivers/modules/agent.py L64416:28
*** dprince has joined #openstack-ironic16:28
rameshg87jroll: I meant if we do something in validate(), the API waits till we complete it16:29
jrollrameshg87: I don't get your point16:30
jrollzhenguo: have you seen krotscheck 's work on this? https://github.com/krotscheck/ironic-webclient16:30
rameshg87jroll: and hence we could fail the API call itself, instead of doing the error checks in vendorpassthru16:30
jrollzhenguo: or is this for horizon?16:30
zhenguojroll, yes it's a plugin for horizon16:30
rameshg87jroll: for example we just return in vendor passthru if node.provision_state != states.AVAILABLE16:31
rameshg87jroll: the vendor passthru just silently exists16:31
jrollzhenguo: ok16:31
zhenguojroll, like other project's UI project like manila-ui, tuscar-ui,16:31
rameshg87jroll: we could have done it in validate() and thrown an error back to the caller of the API16:31
jrollrameshg87: what line are you looking at?'16:31
jrolloh sorry 64416:31
rameshg87jroll: https://review.openstack.org/#/c/161832/15/ironic/drivers/modules/agent.py L64416:31
jrollrameshg87: yeah, looking16:32
openstackgerritMario Villaplana proposed openstack/ironic-specs: Adds spec for creating a key manager with Barbican to Ironic  https://review.openstack.org/18860916:33
rameshg87jroll: a lot of such checks down moved to validate() will help in better experience with the API16:33
jrollrameshg87: I think I agree, it should at least return an error16:33
rameshg87jroll: okay will comment on the review16:34
jrollrameshg87: I will too16:34
rameshg87jroll: https://review.openstack.org/#/c/161832/15/ironic/drivers/modules/agent.py L70716:34
rameshg87jroll: wondering if creation of neutron ports by ironic in available state will lead to race between nova and ironic if node gets selected at the same time16:35
rameshg87jroll: we just lock the ironic node in available state and do stuff, but nova doesn't know about it16:35
*** zhenguo has quit IRC16:35
rameshg87do stuff = caching16:36
jrollrameshg87: urgh. yeah.16:36
jrollI guess we didn't think about that16:36
jrollrameshg87: I think the confusion there is because we don't use neutron for dhcp16:37
rameshg87yeah, that solves most of the problems :)16:37
jrollheh16:37
rameshg87but yeah16:37
rameshg87if neutron port creation failure in nova just leads to nova picking up another node, then it's fine16:37
rameshg87I mean at least okay in my opinion, because (node in caching) = (not available)16:38
jrollrameshg87: yeah, but let's avoid reschedules if possible :)16:39
*** zz_natorious is now known as natorious16:39
jrollif it does it 3 times in a row the boot will fail16:39
rameshg87yeah16:39
jrollwe'll have to think about this one... I'll leave a note on the review16:39
jrollthanks for catching that :)16:39
rameshg87okay16:39
rameshg87:)16:39
rameshg87jroll: and it sort of seems like we are misusing vendor interface16:40
jrollrameshg87: well, the bash ramdisk can't support this :)16:41
rameshg87jroll: agent is not a vendor and our own stuff :)16:41
jrollrameshg87: I tend to think vendor passthru name is a bad name16:42
jrollrameshg87: it's really more for things specific to a single driver (rather, a single implementation of an interface)16:42
rameshg87yeah it is16:43
jrollthough I guess I'd like this to be a real endpoint one day16:43
rameshg87jroll: +116:44
rameshg87jroll: yeah I will think about nova thing further if I can find anything about it16:45
rameshg87just saw clif_h change on nova side and getting full picture of it16:45
jrollrameshg87: I'll think about it as well16:46
jrollrameshg87: have you looked at arsenal also?16:46
*** Sukhdev has joined #openstack-ironic16:46
rameshg87jroll: what's that ?16:46
rameshg87I just looked at the review link that he posted in comment16:46
jrollrameshg87: https://github.com/rackerlabs/arsenal16:47
jrollbasically a service that discovers images and ironic nodes, and decides how to cache what and on where16:47
jrollalso has docs https://arsenal.readthedocs.org/en/latest/16:47
rameshg87oh great ..16:47
*** trown is now known as trown|lunch16:48
rameshg87jroll: you guys are miles ahead of ironic everytime :)16:48
jrollrameshg87: ha :)16:48
rameshg87jroll: will go through it16:48
openstackgerritChristopher Dearborn proposed openstack/ironic: Fix failing unit tests under py34  https://review.openstack.org/19083516:48
rameshg87and nice name too16:49
jrollrameshg87: turns out you move quickly when profit is a motivator :P16:49
* rameshg87 is a fan of arsenal club in epl16:49
rameshg87jroll: :)16:49
jrollheh16:49
*** cdearborn has quit IRC16:50
lucasagomesrameshg87, w00t! WOL upstream16:50
rameshg87\o/16:50
lucasagomesI will drink a pint for that tonight16:50
rameshg87lucasagomes: cheers16:50
jroll\o/16:50
lucasagomesno more keep merging it downstream here16:50
lucasagomes(it wasn't that painful, it's pretty much self contained. But anyway, glad I don't have to do it anymore)16:51
rameshg87lucasagomes: and you can let those people know who were waiting for this to get upstream :)16:51
lucasagomesyup16:51
* lucasagomes prepares FYI emaisl16:52
lucasagomesemails*16:52
*** athomas has quit IRC16:52
* rameshg87 is sad couldn't still try wol out16:52
* rameshg87 will get hardware for it from somewhere16:52
* rameshg87 thinks it's time to go to bed 16:53
*** bethelwell has quit IRC16:53
rameshg87goodnight ugt jroll lucasagomes ironic16:53
jrollsee ya rameshg87 :)16:53
rameshg87see you next weekend16:53
jrollhave a good weekend16:53
lucasagomesrameshg87, good ugt night! Enjoy the weekend16:53
* lucasagomes will go very soon too16:53
rameshg87happy weekend to both of you too ..16:53
lucasagomesthanks16:53
*** rameshg87 has quit IRC16:53
jrollhave a good one lucasagomes \o16:53
openstackgerritMerged openstack/ironic-python-agent: Add power_off command in standby extension  https://review.openstack.org/18924116:54
lucasagomesjroll, cheers!16:54
* jroll dives into work16:54
lucasagomesyou too enjoy it16:54
openstackgerritMerged openstack/ironic: Wake-On-Lan Power interface  https://review.openstack.org/17907816:54
lucasagomesI mean the weekend, but work as well :-)16:54
jrollheh yeah :P16:54
jlvillallucasagomes: Ciao.  Enjoy your weekend.  I hope you have sun in Ireland :)16:55
lucasagomesjlvillal, thanks you too! Have a good night enjoy the weekend16:56
jlvillal:)16:56
lucasagomesjlvillal, yeah, it's cloudy at the moment. But on the weekend I think the sun will show up!16:56
* lucasagomes *fingers crossed*16:56
jlvillallucasagomes: I hope so :)  it is supposed to be hot here.  31-32 I think.16:56
lucasagomes31C?16:56
lucasagomesIt's 17C :-(16:57
jlvillal17C here at the moment.  But weekend warmer, I hope.16:57
lucasagomesactually anything near 20C is already pretty "hot" here. The thermometers maxes out at 25C here, at least it saves mercury16:57
*** Marga_ has quit IRC16:57
jlvillalLOL!16:57
*** Marga_ has joined #openstack-ironic16:57
jlvillallucasagomes: I lived in Huntingdon, England for three years.  So I think somewhat similar.16:58
lucasagomesjlvillal, I see, well in 4 years here. The max I've seem was 27C16:58
jlvillal:(16:58
lucasagomesit's ok... coming from brazil I'm glad it's cold16:58
jlvillallucasagomes: You adapt.  Now when it hits 20C you think it is so hot ;)16:59
lucasagomesI had enough sun from 3 lives already16:59
lucasagomesfor*16:59
lucasagomesjlvillal, yeah16:59
lucasagomes20C is like wow16:59
lucasagomesanyhoo, I see you all later17:00
lucasagomeshave a good night17:00
*** lucasagomes is now known as lucas-beer17:00
jlvillallucasagomes: Thanks.  You too.17:00
sambettsnight all :)17:00
jlvillalsambetts: Ciao.17:00
lucas-beersambetts, night17:00
openstackgerritChristopher Dearborn proposed openstack/ironic: Fix failing unit tests under py34  https://review.openstack.org/19083517:15
NobodyCamnight sambetts17:15
NobodyCamhave a good weekend17:15
*** Sukhdev has quit IRC17:27
*** Sukhdev has joined #openstack-ironic17:33
*** trown|lunch is now known as trown17:38
*** amotoki has quit IRC17:41
*** spandhe has joined #openstack-ironic17:42
*** achanda has quit IRC17:46
*** jistr has quit IRC17:48
*** pelix has quit IRC17:52
*** Sukhdev has quit IRC17:56
*** achanda has joined #openstack-ironic17:59
*** e0ne has joined #openstack-ironic18:00
*** mitchjameson has joined #openstack-ironic18:06
*** e0ne has quit IRC18:13
*** achanda has quit IRC18:17
*** cdearborn has joined #openstack-ironic18:18
*** Sukhdev has joined #openstack-ironic18:19
*** e0ne has joined #openstack-ironic18:19
*** dprince has quit IRC18:20
*** yog_ has quit IRC18:22
*** e0ne is now known as e0ne_18:25
openstackgerritImre Farkas proposed openstack/ironic: DRAC: fix set/get boot device for 11g  https://review.openstack.org/19117718:27
*** valkyrja has quit IRC18:27
*** e0ne_ has quit IRC18:30
*** dmitchell has joined #openstack-ironic18:32
*** Marga_ has quit IRC18:32
*** bethelwell has joined #openstack-ironic18:33
*** yog_ has joined #openstack-ironic18:33
*** dmitchell__ has quit IRC18:33
jlvillalDoes anyone know much about ironic/nova/compute/manager.py?  Is it owned by Ironic team?  Is it a copy of some Nova code?18:35
jlvillallifeless: ^^^18:35
NobodyCamjlvillal: in our repo...18:36
jlvillalNobodyCam: Yes18:36
NobodyCamthat the clustered compute manager18:36
jlvillalNobodyCam: Do you know if it is code unique to Ironic?  Or something that gets copied in occasionally from Nova.18:37
jlvillalI'm thinking unique to Ironic, but not sure.18:37
*** Marga_ has joined #openstack-ironic18:38
*** hemna_ has joined #openstack-ironic18:39
NobodyCamas last update was 12/9/2014 I expect its not copied..18:39
jlvillalNobodyCam: Thank you!18:39
*** hemna has quit IRC18:40
*** bethelwell has quit IRC18:41
NobodyCamjlvillal: I was just looking to see if we had any doc I could point you to18:42
jlvillalNobodyCam: Thanks.  I'm not quite sure how it is used.  I got detoured because I was going to enable flake8 checking for that directory.18:43
jlvillalNobodyCam: I only had to fix a few things in that file.18:43
NobodyCamthe etherpad is still around: https://etherpad.openstack.org/p/ironic-nova-friction18:44
openstackgerritJohn L. Villalovos proposed openstack/ironic: Enable flake8 checking of ironic/nova/*  https://review.openstack.org/19118318:45
jlvillalNobodyCam: ^^^18:46
jlvillalNobodyCam: I will look at the etherpad.  Thanks.18:47
openstackgerritJohn L. Villalovos proposed openstack/ironic: Enable flake8 checking of ironic/nova/*  https://review.openstack.org/19118318:48
*** mgoddard1 has joined #openstack-ironic18:48
*** valkyrja has joined #openstack-ironic18:49
*** mgoddard has quit IRC18:49
*** mgoddard1 has quit IRC18:53
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP TEST: See what happens if remove ironic/nova  https://review.openstack.org/19119119:03
jlvillalNobodyCam: ^^^ Trying something crazy ;)19:03
* NobodyCam clicks19:03
*** hemna__ has joined #openstack-ironic19:03
NobodyCamjlvillal: why?19:04
*** wuhg has quit IRC19:04
jlvillalNobodyCam: Just curious if it is actually being used.  I'm not sure.  So I thought I would do a test.19:04
jlvillalNobodyCam: py27 tests worked without it...19:05
*** hemna_ has quit IRC19:05
NobodyCamhttp://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-service-to-use-the-bare-metal-service19:05
jlvillalNobodyCam: Oh, thanks for that.19:06
NobodyCamyes it used19:06
NobodyCammainly by folks with > 1 conductor in the same compute regon19:06
jlvillalNobodyCam: Okay, I think I see.19:07
jlvillalNobodyCam: Nova calls into it.  But Ironic doesn't call it, so that is why the unit tests passed.  Thanks.19:09
jlvillalNobodyCam: So much to learn!19:09
NobodyCamyes that is a out of tree nova driver19:09
NobodyCam:-p19:09
*** dontalton has joined #openstack-ironic19:09
*** e0ne has joined #openstack-ironic19:09
*** e0ne is now known as e0ne_19:18
jrolljlvillal: pls don't remove that :(19:21
jrolljlvillal: it's an out of tree thing... it's meant so you can run more than one nova-compute host19:21
jrollhowever it gets racy as all hell19:21
jrollbut it mostly works (we run it)19:21
jrollthat should be nuked from orbit like this cycle or else, once we figure out the better compute story19:22
NobodyCamjroll: +++19:22
*** e0ne_ is now known as e0ne19:22
*** cdearborn has quit IRC19:22
NobodyCambut I do tend to look at it as an out of tree compute driver :-p ... that may just be me19:23
*** cdearborn has joined #openstack-ironic19:24
*** yog_ has quit IRC19:28
*** absubram has quit IRC19:37
*** mgoddard has joined #openstack-ironic19:52
*** dontalton has quit IRC19:52
jlvillaljroll, I won't remove it! :)  I was just doing a test because I didn't know what it was for...19:52
jrollgotcha20:00
*** v92 has quit IRC20:07
*** dmellado has quit IRC20:07
*** aignatov has quit IRC20:07
*** cascone has quit IRC20:07
*** Guest19563 is now known as dan20:07
*** v92 has joined #openstack-ironic20:07
*** dmellado has joined #openstack-ironic20:07
*** hemna__ is now known as hemna20:09
*** cascone has joined #openstack-ironic20:10
*** EmilienM is now known as EmilienM|afk20:10
*** aignatov has joined #openstack-ironic20:11
*** ifarkas has quit IRC20:12
lifelessjlvillal: its unique20:14
jlvillallifeless: Thank you.20:14
lifelessjlvillal: one could argue that it should be in Nova20:15
lifelessjlvillal: but that discussion was more cost than benefit back in the split-out-day, so...20:15
jlvillallifeless: My feeling is that probably the Ironic team likes to have it in Ironic.  As they can more easily make changes if needed.  Though it doesn't seem to change much.20:16
jlvillalBut I've been known to be wrong :)20:16
lifeless:)20:17
jrollwell, the thing is20:17
jrollnova won't accept it :P20:17
jrollit's completely broken by their standards, even though it mostly works20:17
*** Sukhdev has quit IRC20:19
*** krtaylor has quit IRC20:19
*** e0ne has quit IRC20:19
*** natorious is now known as zz_natorious20:19
jrollis anyone here going to the nova midcycle? devananda maybe?20:20
* jlvillal started to say yes and then realized it was not the Ironic midcycle20:25
lifelessI am, if I get my a into gear and book stuff20:27
*** greghaynes has quit IRC20:27
jrollheh20:27
*** alexpilotti has quit IRC20:35
*** alexpilotti has joined #openstack-ironic20:37
jlvillaljroll: NobodyCam: Slightly worrying that the Jenkins job passed for the patch removing the ironic/nova/ directory.  https://review.openstack.org/19119120:39
jrolljlvillal: right, it's untested20:40
jrollor not used in dsvm stuff, I should say20:40
openstackgerritJulia Kreger proposed openstack/bifrost: Support for AMT  https://review.openstack.org/19110920:40
jlvillaljroll: Okay.  I was thinking it would be tested in one of the tempest jobs.  Thanks.20:40
jrolljlvillal: nope, our testing story is kind of horrible20:40
jrolljlvillal: this is another fun example https://review.openstack.org/#/c/165430/20:41
jlvillaljroll: I'll make a bug about that.  Just to trake it.20:41
jlvillals/trake/track/20:41
jlvillalMaybe can figure out how to test it in the future...20:41
jlvillaljroll: Uh.  That's not good either.20:42
jrollyah.20:42
*** zz_natorious is now known as natorious20:47
*** natorious is now known as zz_natorious20:48
*** zz_natorious is now known as natorious20:49
TheJuliaAnyone fired up the ucs driver recently?20:54
BadCubnope20:55
BadCublol20:55
TheJuliaLooks like the driver is borked20:56
TheJuliafiling a bug20:56
*** trown is now known as trown|outttypeww21:03
*** davideagnello has quit IRC21:08
*** derekh_afk has quit IRC21:08
*** davideagnello has joined #openstack-ironic21:08
*** krtaylor has joined #openstack-ironic21:08
TheJuliahmm... doesn't actually appear to be the driver...21:09
*** vishwanathj has quit IRC21:13
*** vishwanathj has joined #openstack-ironic21:13
openstackgerritJulia Kreger proposed openstack/bifrost: Enable agent_ilo driver  https://review.openstack.org/19121821:14
openstackgerritJulia Kreger proposed openstack/bifrost: Adding agent_ucs driver to the driver list  https://review.openstack.org/19121921:14
*** dguerri` is now known as dguerri21:17
spandhelucas-beer: hey! yt?21:20
*** cdearborn has quit IRC21:21
*** Sukhdev has joined #openstack-ironic21:25
*** zz_jgrimm is now known as jgrimm21:33
*** jjohnson2 has quit IRC21:34
-openstackstatus- NOTICE: Gerrit will be offline for project renames between 22:00 and 22:30 UTC21:43
*** ChanServ changes topic to "Gerrit will be offline for project renames between 22:00 and 22:30 UTC"21:43
*** ndipanov has quit IRC21:52
*** radek_ has quit IRC21:52
-openstackstatus- NOTICE: Gerrit is offline for project renames. ETA 20:3022:04
*** ChanServ changes topic to "Gerrit is offline for project renames. ETA 20:30"22:04
*** Marga_ has quit IRC22:07
*** Marga_ has joined #openstack-ironic22:07
-openstackstatus- NOTICE: Gerrit is offline for project renames. ETA 22:4022:08
jlvillalHmmm.  Gerrit is down.  Sunshine outside.  3pm.... :)22:08
*** ChanServ changes topic to "Gerrit is offline for project renames. ETA 22:40"22:08
TheJulialol22:11
*** Marga_ has quit IRC22:12
*** Marga_ has joined #openstack-ironic22:12
*** gdmallar has joined #openstack-ironic22:30
TheJuliaOn a plus side, it is back up and the inspector repo has been renamed22:38
cineramaoh yay22:41
*** ChanServ changes topic to "Gerrit is offline for project renames. ETA 20:30"22:42
-openstackstatus- NOTICE: Gerrit is back online. Zuul reconfiguration for renamed projects is still in progress, ETA 23:30.22:42
*** ijw has quit IRC22:45
*** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"22:50
*** dguerri is now known as dguerri`22:54
*** natorious is now known as zz_natorious23:05
*** lucas-beer has quit IRC23:06
*** rbudden has quit IRC23:15
*** hemna has quit IRC23:17
*** lucas-beer has joined #openstack-ironic23:17
*** rbudden has joined #openstack-ironic23:18
*** openstackgerrit has quit IRC23:22
*** lucas-beer has quit IRC23:22
*** openstackgerrit has joined #openstack-ironic23:22
*** davideag_ has joined #openstack-ironic23:23
*** Sukhdev has quit IRC23:25
*** davideag_ has quit IRC23:25
*** davideagnello has quit IRC23:27
jlvillalNobodyCam: I added you as a reviewer to: https://review.openstack.org/#/c/191248/   Updating the text a little for the Ironic meeting.23:31
*** Marga_ has quit IRC23:37
*** Marga_ has joined #openstack-ironic23:37
*** Sukhdev has joined #openstack-ironic23:38
NobodyCamhehehe23:38
NobodyCamjlvillal: +1'd23:39
NobodyCamgood catch23:39
jlvillalNobodyCam: Thanks :)23:39
NobodyCam:)23:41
*** ijw has joined #openstack-ironic23:41
*** Marga_ has quit IRC23:47
*** zhenguo has joined #openstack-ironic23:48
*** mgoddard has quit IRC23:49
*** dan is now known as Guest7440923:49
openstackgerritJeremy Stanley proposed openstack/ironic-inspector: Update .gitreview file for project rename  https://review.openstack.org/19127123:50
*** dan_ has joined #openstack-ironic23:50
*** dan_ is now known as Guest6854923:50
*** bkero has quit IRC23:57

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