Friday, 2015-04-10

*** achanda has quit IRC00:07
*** ijw has joined #openstack-ironic00:09
*** Sukhdev has joined #openstack-ironic00:10
*** achanda has joined #openstack-ironic00:11
*** achanda has quit IRC00:16
TheJuliaNobodyCam: soft power off would make the universe a happier place00:21
jrollsoft power off doesn't work without a connection to the OS :(00:22
* BadCub is opposed to a happy universe lol00:22
TheJuliaCats and cats will behave, the weather will be perfect year round, etc.00:22
TheJuliaerr, cats and dogs00:23
BadCubwe live in Palm Springs, the weather is for the most part perfect all year round lol00:23
TheJuliajroll: Well, there is a signal, but the OS has to be responsive to it... and it would totally depend on the driver00:24
jrollTheJulia: right, but some people have their machines configured to block any OS <-> BMC communication00:24
TheJulia:(00:26
jrollmeh, better than everything getting owned00:27
jroll:P00:27
BadCubI think it time to go do things.00:27
BadCubHave a good evening folks :)00:28
*** BadCub is now known as BadCub_Hiding00:28
*** jamielennox|away is now known as jamielennox00:28
jrollNobodyCam: still around?00:37
*** alex_xu has quit IRC00:42
*** kbs1 has joined #openstack-ironic00:43
*** kbs has quit IRC00:43
*** bradjones has quit IRC00:45
*** alex_xu has joined #openstack-ironic00:45
*** bradjones has joined #openstack-ironic00:50
*** jerryz has joined #openstack-ironic00:51
*** Marga_ has quit IRC01:05
*** Marga_ has joined #openstack-ironic01:05
*** ijw_ has joined #openstack-ironic01:05
*** ijw has quit IRC01:08
openstackgerritMerged openstack/ironic: Agent driver fails without Ironic-managed TFTP  https://review.openstack.org/17098401:16
*** rfchapman has quit IRC01:17
*** mtanino has quit IRC01:18
*** Marga__ has joined #openstack-ironic01:22
*** Marga_ has quit IRC01:22
*** igordcard_ has quit IRC01:24
*** Marga__ has quit IRC01:24
*** Marga_ has joined #openstack-ironic01:24
*** chenglch has joined #openstack-ironic01:55
*** ijw_ has quit IRC02:01
*** korekhov_ has joined #openstack-ironic02:04
*** Sukhdev has quit IRC02:05
*** korekhov has quit IRC02:07
*** korekhov_ has quit IRC02:08
*** harlowja is now known as harlowja_away02:26
*** meghal has quit IRC02:27
*** jcoufal has quit IRC02:28
*** rloo_afk has quit IRC02:35
*** achanda has joined #openstack-ironic02:40
*** dttocs has quit IRC02:40
*** ramineni has joined #openstack-ironic02:40
*** achanda has quit IRC02:45
NobodyCamjroll: no I'm not02:48
BadCub_Hidinglol02:48
*** achanda has joined #openstack-ironic02:50
BadCub_HidingI pinged up https://review.openstack.org/#/c/166386/ in the -qa room to get reviews02:50
NobodyCam:)02:50
*** achanda has quit IRC03:20
*** Marga_ has quit IRC03:31
*** bradjones has quit IRC03:40
*** bradjones has joined #openstack-ironic03:43
*** bradjones has quit IRC03:43
*** bradjones has joined #openstack-ironic03:43
*** Marga_ has joined #openstack-ironic03:45
*** Marga_ has quit IRC03:47
*** Marga_ has joined #openstack-ironic03:47
*** korekhov has joined #openstack-ironic04:04
*** meghal has joined #openstack-ironic04:05
*** dtantsur|afk has quit IRC04:09
*** korekhov has quit IRC04:09
*** dtantsur has joined #openstack-ironic04:10
*** meghal1 has joined #openstack-ironic04:13
*** meghal has quit IRC04:16
*** korekhov has joined #openstack-ironic04:18
*** korekhov has quit IRC04:23
*** achanda has joined #openstack-ironic04:36
openstackgerritNisha Agarwal proposed openstack/ironic: Update ilo drivers documentation for inspection  https://review.openstack.org/17006504:59
*** bradjones has quit IRC05:12
*** pradipta has joined #openstack-ironic05:15
*** bradjones has joined #openstack-ironic05:21
*** bradjones has quit IRC05:26
*** bradjones has joined #openstack-ironic05:30
*** korekhov has joined #openstack-ironic05:38
*** Sukhdev has joined #openstack-ironic05:38
*** ramineni1 has joined #openstack-ironic05:47
*** ramineni has quit IRC05:49
*** achanda has quit IRC05:51
*** achanda has joined #openstack-ironic05:52
*** jcoufal has joined #openstack-ironic05:57
*** korekhov has quit IRC05:59
openstackgerritMichael Davies proposed openstack/python-ironicclient: Cache negotiated api microversion for this server  https://review.openstack.org/17194605:59
*** korekhov has joined #openstack-ironic06:00
*** Haomeng has joined #openstack-ironic06:01
*** Haomeng|2 has quit IRC06:04
*** korekhov has quit IRC06:04
*** bradjones has quit IRC06:13
*** bradjones has joined #openstack-ironic06:19
*** korekhov has joined #openstack-ironic06:30
openstackgerritShivanand Tendulker proposed openstack/ironic: Fix for automated boot iso issue with IPA ramdisk  https://review.openstack.org/17231306:32
*** Sukhdev has quit IRC06:35
*** ramineni1 has quit IRC06:37
*** ramineni has joined #openstack-ironic06:41
*** jamielennox is now known as jamielennox|away07:20
dtantsurMorning Ironic!07:21
*** korekhov has quit IRC07:23
*** achanda has quit IRC07:24
*** andreykurilin_ has joined #openstack-ironic07:25
*** achanda has joined #openstack-ironic07:25
*** achanda has quit IRC07:30
*** mrda is now known as mrda-weekend07:31
Haomengdtantsur: good morning:)07:31
dtantsuro/07:31
Haomengdtantsur: I found an issue - https://bugs.launchpad.net/python-ironicclient/+bug/144243607:31
openstackLaunchpad bug 1442436 in python-ironicclient "ironic *-show commands return empty properties when the input uuid is "" or space char" [Low,In progress] - Assigned to Haomeng,Wang (whaom)07:31
dtantsuroh interesting07:32
* dtantsur is trying07:32
Haomengdtantsur: both ironic client and api server does not validate uuid, for such empty string, will show empty properties07:32
Haomengdtantsur: so we have two solution, fix on client side or server side, how do you think? any idea?07:33
dtantsurHaomeng, this should be fixed on client side, because server receives a valid request07:33
dtantsuridentical to chassis-list07:34
Haomengdtantsur: yes, i thinkso,client should valide the input first07:34
dtantsurand returns a valid response. nothing to be fixed there07:34
HaomengHaomeng: to avoid to run into backend07:34
Haomengdtantsur: ok, i will propose the client fix, thank you:)07:34
dtantsurnp07:34
*** kevinbenton has quit IRC07:35
*** kevinbenton has joined #openstack-ironic07:41
jcoufaldtantsur: hey, so what do I have to do in order to use discover-ramdisk.kernel and *.ramdisk from glance for discovery?07:41
dtantsurjcoufal, download them into /tftpboot? :)07:41
dtantsurdiscoverd (unlike Ironic) does not touch TFTP configuration at all07:42
jcoufaldtantsur: so that means not using them from glance ;)07:43
dtantsuryep07:43
jcoufalso I cannot use them from glance07:44
dtantsuryep :)07:44
dtantsurnot sure what you want to, but currently you can07:44
dtantsur* can't07:44
jcoufalI see07:44
jcoufalI missunderstood your mail07:44
jcoufalso what I wanted is to upload the images to glance and tell discoverd to find it there :)07:45
jcoufalbut apparently it doesn't work07:45
dtantsurthe problem is: discoverd does not find images or use them. at all. it fully relies on a pre-configured TFTP environment07:45
dtantsursorry for not being clear enough, but that's Fri morning :)07:45
jcoufaldtantsur: no problem, that's fine07:46
jcoufalso basically I need to create /tftpboot directory and rename the images to match discovery.*07:47
jcoufalto make it work07:47
dtantsurjcoufal, it's pretty surprising you don't have it all already07:51
dtantsurhow do you create your env?07:51
jcoufaldtantsur: via instack-virt-setup07:54
jcoufaldtantsur: I don't know why it was not there07:55
*** lucasagomes has joined #openstack-ironic07:55
dtantsurjcoufal, did you follow https://repos.fedorapeople.org/repos/openstack-m/instack-undercloud/internal-html/build-images.html ?07:55
dtantsurthe last command should have done it07:55
jcoufalyup07:55
jcoufaloh the last command not07:55
dtantsur(despite being described as "load images in glance")07:55
jcoufalI tried to avoid it07:55
dtantsur:)07:55
dtantsurjcoufal, then yeah, you have to copy files manually. not a big deal actually07:56
jcoufalbut there is no mkdir, so I didn't do it :)07:56
*** romcheg has quit IRC07:59
*** romcheg has joined #openstack-ironic07:59
*** romcheg has quit IRC08:04
dtantsurlucasagomes, morning! may I ask you for discoverd reviews again? I'd like to enter kind-of-RC state for git master, and I have 4 outstanding reviews now: https://review.openstack.org/#/q/project:stackforge/ironic-discoverd,n,z08:05
dtantsur(also see my answers on the documentation patch)08:06
lucasagomesdtantsur, hi, sure :-( but I will take a look soonish08:09
* lucasagomes got dragged into some atomic + ironic thing08:09
dtantsuroh, sorry to hear08:09
dtantsur:D08:09
dtantsurlucasagomes, take your time then08:09
*** derekh has joined #openstack-ironic08:10
*** jistr has joined #openstack-ironic08:11
*** romcheg has joined #openstack-ironic08:15
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Do not save auth token on TFTP server in PXE driver  https://review.openstack.org/15981908:18
dtantsurfolks, do you think we should target https://bugs.launchpad.net/ironic/+bug/1441691 to rc?08:28
openstackLaunchpad bug 1441691 in Ironic "iscsi_ilo driver : deploy fails with ipa ramdisk if user image do not have boot iso property in UEFI boot mode" [Medium,In progress] - Assigned to Shivanand Tendulker (shivanand-tendulker)08:28
dtantsurlucasagomes, ^^^08:28
dtantsurfix is https://review.openstack.org/#/c/172313/08:28
lucasagomes+108:29
*** meghal1 has quit IRC08:32
*** edcragg has joined #openstack-ironic08:34
*** athomas has joined #openstack-ironic08:35
*** ndipanov has joined #openstack-ironic08:44
*** a1exhughe5 has joined #openstack-ironic08:44
*** rameshg87 has joined #openstack-ironic08:53
rameshg87good afternoon ironic08:53
dtantsurg'afternoon08:54
rameshg87dtantsur: o/08:55
*** romcheg has quit IRC09:02
*** achanda has joined #openstack-ironic09:05
*** romcheg has joined #openstack-ironic09:09
*** achanda has quit IRC09:12
*** igordcard_ has joined #openstack-ironic09:13
pshigeRameshg87: good afternoon09:14
rameshg87good afternoon pshige09:16
*** pelix has joined #openstack-ironic09:24
*** igordcard_ has quit IRC09:33
*** athomas has quit IRC09:35
rameshg87dtantsur: how do you get fedora 21 cloud image ?09:36
rameshg87dtantsur: from dib i am still getting f20 cloud image only09:36
rameshg87build from disk-image-create in dib09:36
dtantsurI personally don't get F21 image :) I've been using RHEL for some time now..09:36
dtantsurbut there was some env argument iirc09:37
dtantsurrameshg87, maybe use DIB_LOCAL_IMAGE to point to a locally downloaded copy09:37
rameshg87dtantsur: oh okay, download f21 and point it. let me give a try09:39
*** athomas has joined #openstack-ironic09:42
*** andreykurilin_ has quit IRC09:42
*** igordcard_ has joined #openstack-ironic09:44
*** saripurigopi has joined #openstack-ironic09:44
*** bradjones has quit IRC09:57
*** bradjones has joined #openstack-ironic10:02
*** bradjones has quit IRC10:02
*** bradjones has joined #openstack-ironic10:02
*** dtantsur is now known as dtantsur|brb10:02
lucasagomesderekh, ^ you know if we can get F21 from DIB?10:02
derekhlucasagomes:             export DIB_RELEASE=2110:04
derekhI think that should do it10:04
lucasagomesderekh, ta much!10:04
lucasagomesrameshg87, ^10:04
rameshg87lucasagomes: derekh: oh great .. thanks10:05
*** pradipta has quit IRC10:05
* rameshg87 tries10:05
lucasagomesrameshg87, lemme know if it works :-) (is interested as well)10:05
*** achanda has joined #openstack-ironic10:09
*** chenglch has quit IRC10:10
rameshg87lucasagomes: works, started fetching f21 image anyway :)10:10
lucasagomes:-) thanks10:10
rameshg87derekh: why isn't f21 still the default ?10:11
derekhrameshg87: we havn't switched CI yet, the first step us this https://review.openstack.org/#/c/169778/ but has had the reviews yet10:12
derekh*hasn't had ...10:12
rameshg87oh okay10:13
openstackgerritAnusha Ramineni proposed stackforge/proliantutils: Missing argument in set_http_boot_url  https://review.openstack.org/17236610:14
*** achanda has quit IRC10:14
openstackgerritMerged openstack/ironic: Nit fixes for boot_mode being overwritten  https://review.openstack.org/17172210:20
*** thrash|g0ne is now known as thrash10:21
openstackgerritVladyslav Drok proposed openstack/ironic: Update session headers during initialization of AgentClient  https://review.openstack.org/17215410:32
* rameshg87 goes home on travel for a long weekend 10:34
rameshg87see you folks on tuesday and happy weekend everyone ...10:34
*** rameshg87 has quit IRC10:34
*** stendulker has joined #openstack-ironic10:35
*** romcheg has quit IRC10:48
openstackgerritsonu proposed openstack/python-ironicclient: Added unit tests for command-line shell  https://review.openstack.org/17237110:49
*** yuanying has quit IRC11:02
*** ukalifon1 has joined #openstack-ironic11:05
*** ramineni has quit IRC11:05
stendulkerdtantsur: Good Morning :)11:06
stendulkerdtantsur: Can you please have a look at this review when you get time. https://review.openstack.org/#/c/168844/ iLO driver documentation for UEFI secure boot. Have addressed your comments on same.11:08
*** viktors|afk is now known as viktors11:11
openstackgerritSHIGEMATSU Mitsuhiro proposed openstack/ironic: Add Ceilometer to Ironic's Conceptual Architecture  https://review.openstack.org/17238011:18
*** lucasagomes is now known as lucas-hungry11:26
*** romcheg has joined #openstack-ironic11:26
*** stendulker has quit IRC11:37
openstackgerritSHIGEMATSU Mitsuhiro proposed openstack/ironic: Add Ceilometer to Ironic's Conceptual Architecture  https://review.openstack.org/17238411:43
*** saripurigopi has quit IRC11:45
*** trown|outttypeww is now known as trown11:49
*** dtantsur|brb is now known as dtantsur11:49
*** rsalevsky has quit IRC11:59
*** rsalevsky has joined #openstack-ironic12:04
*** zhenguo has quit IRC12:07
*** lucas-hungry is now known as lucasagomes12:16
jrollmornin' everybody :)12:16
lucasagomesjroll, yo morning12:18
jrollhey lucas :)12:19
*** dprince has joined #openstack-ironic12:19
*** krtaylor has quit IRC12:23
*** bauzas is now known as bauwser12:24
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Check hooks availability on start-up  https://review.openstack.org/17239412:25
dtantsurjroll, morning!12:26
jrollhiya dtantsur :)12:33
*** jcoufal has quit IRC12:34
*** jcoufal has joined #openstack-ironic12:38
pshigejroll: morning!12:39
jrollhey pshige :)12:40
*** Marga_ has quit IRC12:41
openstackgerritMerged stackforge/ironic-discoverd: Pre-release documentation update  https://review.openstack.org/17121112:44
*** krtaylor has joined #openstack-ironic12:52
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add support for 'baremetal show' command  https://review.openstack.org/17173913:05
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Introduce openstackclient plugin  https://review.openstack.org/17167213:05
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal create  https://review.openstack.org/17217113:05
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add support for microversion  https://review.openstack.org/17217013:05
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implement the rest of the args for baremetal list  https://review.openstack.org/17169913:05
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implement baremetal delete command  https://review.openstack.org/17175113:05
*** kalpase has joined #openstack-ironic13:06
*** kalpase has left #openstack-ironic13:06
*** Marga_ has joined #openstack-ironic13:13
*** kkoski has joined #openstack-ironic13:14
*** alexpilotti has joined #openstack-ironic13:15
*** alexpilotti has quit IRC13:25
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: README: More updates for Troubleshooting section  https://review.openstack.org/17207713:31
dtantsurlucasagomes, ^^^ should fix your concerns on the readme patch13:31
lucasagomesdtantsur, awesome, /me looks13:31
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal create  https://review.openstack.org/17217113:34
*** dmellado has quit IRC13:42
*** dmellado has joined #openstack-ironic13:44
*** gilliard is now known as gillllliard13:47
*** jistr has quit IRC13:49
*** jistr has joined #openstack-ironic13:56
NobodyCamgood morning ironic14:11
NobodyCamand ofc TGIF!14:12
jrollmorning NobodyCam, tgif indeed :)14:13
NobodyCam:)14:13
dtantsurNobodyCam, morning, really TGIF :)14:14
*** mtanino has joined #openstack-ironic14:14
NobodyCammorning dtantsur :)14:18
NobodyCamHow are our outstanding DOC patches looking14:18
jrollI must admit I never got to them14:19
NobodyCam:-p14:19
jrollbut they don't need to merge before RC is cut14:19
jrolldoc site is published from master14:19
dtantsurwe have one patch we might want target to rc: https://review.openstack.org/17231314:21
NobodyCamand 17133414:22
TheJuliarandom off the wall question of the day, would there ever be a case that someone would want to update instance info after a node is deployed?14:33
NobodyCamTheJulia: we actually have a bug about this that14:34
NobodyCamso I expect so14:34
TheJuliabut not presently done so?14:35
NobodyCamTheJulia: https://bugs.launchpad.net/ironic/+bug/130127914:35
openstackLaunchpad bug 1301279 in OpenStack Compute (nova) "Changing node's properties in Ironic after node is deployed will count as available resources in Nova" [Medium,Confirmed] - Assigned to Nha Pham (phqnha)14:35
NobodyCamso dont do it :-p14:35
NobodyCamhehheeh14:36
TheJuliawow14:36
NobodyCam:(14:36
TheJuliaso yeah... I suppport changing those in the new os_ironic :(14:38
TheJuliaansible module14:38
*** maurosr has quit IRC14:42
*** maurosr has joined #openstack-ironic14:43
*** Marga_ has quit IRC14:43
*** ukalifon1 has quit IRC14:43
*** viktors has quit IRC14:45
*** jgrimm has quit IRC14:48
*** Marga_ has joined #openstack-ironic14:50
*** mariojv has joined #openstack-ironic14:50
*** mjturek1 has quit IRC14:55
*** mjturek1 has joined #openstack-ironic14:55
*** maurosr has quit IRC14:58
*** maurosr has joined #openstack-ironic15:03
*** BadCub_Hiding is now known as BadCub15:04
BadCubmorning folks15:04
NobodyCammornign BadCub15:04
NobodyCambrb15:06
jlvillalGood morning: BadCub NobodyCam TheJulia dtantsur lucasagomes and the rest of Ironic15:06
dtantsurmorning TheJulia, BadCub, jlvillal!15:07
lucasagomesBadCub, NobodyCam jlvillal good morning!15:07
*** maurosr has quit IRC15:08
*** rwsu has quit IRC15:09
BadCubg'morning lucasagomes , NobodyCam , TheJulia , dtantsur , jlvillal Happy Friday!15:11
NobodyCammornign lucasagomes15:11
NobodyCammornign jlvillal15:12
BadCubhey jlvillal how are you coming along on https://review.openstack.org/#/c/171334/ & https://review.openstack.org/#/c/172168/1 ?15:16
jlvillalBadCub: One was abandoned and the other one I will be working on this morning.15:18
BadCubIs anyone able to put eyes on some doc patches? We have about 4 on the pad that need +2/+A on em.15:18
BadCubjlvillal: 172168 is abandoned then?15:19
* dtantsur tries to find the pad15:19
BadCub#link https://etherpad.openstack.org/p/IronicReviewDay15:19
jlvillalBadCub: Yeah, I merged the work into 17133415:19
BadCubdtantsur: ^15:19
dtantsurack, will have a look15:19
BadCubjlvillal: okay, coolness. I will take that off the pad then15:19
jrollBadCub: jfyi, rc1 can be cut before doc patches land15:19
jrolldocs are published from master15:20
BadCubjroll: yes, I am aware. They are just kinda sitting there nagging lol15:20
jrollk :)15:20
BadCubTrying to get Tempest to review the the one patch we need/want from them, but no luck so far :(15:21
*** krtaylor has quit IRC15:27
*** krtaylor has joined #openstack-ironic15:28
*** stendulker has joined #openstack-ironic15:29
*** maurosr has joined #openstack-ironic15:32
*** zz_jgrimm has joined #openstack-ironic15:32
*** Marga_ has quit IRC15:34
*** Marga_ has joined #openstack-ironic15:35
*** maurosr has quit IRC15:39
*** alexpilotti has joined #openstack-ironic15:41
stendulkerdevananda: Hi15:41
*** zz_jgrimm has quit IRC15:41
stendulkerdevananda: This is regarding bug https://bugs.launchpad.net/ironic/+bug/144169115:41
openstackLaunchpad bug 1441691 in Ironic "iscsi_ilo driver : deploy fails with ipa ramdisk if user image do not have boot iso property in UEFI boot mode" [Medium,In progress] - Assigned to Shivanand Tendulker (shivanand-tendulker)15:41
*** maurosr has joined #openstack-ironic15:42
stendulkerdevananda: This defect fix is important for iLO drivers. The fix has been reviewed by 3 core reviewers and they are satified with the fix.15:42
stendulkerdevananda: Can you please take this fix for rc1.15:43
jrollBadCub: ^15:43
stendulkerdevananda: Thank you15:43
* BadCub looks15:44
-openstackstatus- NOTICE: gerrit has been restarted to address a hung event stream. change events between 15:00 and 15:43 utc which were lost will need to be rechecked or have approval workflow votes reapplied for zuul to act on them15:44
*** a1exhughe5 has quit IRC15:45
NobodyCamstendulker: devananda is on PTO until monday15:45
stendulkerNobodyCam: Hi, Good Morning :)15:45
stendulkerNobodyCam: Needed help in pushing defect fix for https://bugs.launchpad.net/ironic/+bug/1441691 into rc115:46
openstackLaunchpad bug 1441691 in Ironic "iscsi_ilo driver : deploy fails with ipa ramdisk if user image do not have boot iso property in UEFI boot mode" [Medium,In progress] - Assigned to Shivanand Tendulker (shivanand-tendulker)15:46
*** maurosr has quit IRC15:46
*** romcheg has quit IRC15:47
*** Marga_ has quit IRC15:48
*** Marga_ has joined #openstack-ironic15:49
*** rsalevsky has left #openstack-ironic15:49
BadCubstendulker: looking at ^ now15:49
stendulkerBadCub: Thank you :)15:50
BadCubstendulker: ^ is a go for RC115:51
*** alexpilotti has quit IRC15:51
*** rwsu has joined #openstack-ironic15:51
*** alexpilotti has joined #openstack-ironic15:51
*** hemna has joined #openstack-ironic15:52
stendulkerBadCub: Thank you for your support :)15:52
*** maurosr has joined #openstack-ironic15:54
NobodyCamstendulker: just a question.. why the change on hte menu entry from  "install" to "boot_partition" ??15:54
*** enikanorov has quit IRC15:55
*** bnemec is now known as beekneemech15:55
stendulkerNobodyCam: this is done to keep it in sync with the pxe menu entries. It was changed to 'boot_partition' in kilo15:56
stendulkerNobodyCam: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/elilo_efi_pxe_config.template#L1015:56
stendulkerNobodyCam: This config file gets into action for the final boot of user image15:57
NobodyCamstendulker: ack.. can you add that to the commit message?15:58
stendulkerNobodyCam: Sure. Will do it right away.15:58
NobodyCam:) TY15:59
stendulkerNobodyCam: Thank you for the quick review :)16:00
NobodyCam:)16:00
*** maurosr has quit IRC16:00
openstackgerritMerged stackforge/ironic-discoverd: README: More updates for Troubleshooting section  https://review.openstack.org/17207716:01
*** dttocs has joined #openstack-ironic16:06
*** jistr has quit IRC16:06
openstackgerritShivanand Tendulker proposed openstack/ironic: Fix for automated boot iso issue with IPA ramdisk  https://review.openstack.org/17231316:07
*** jcoufal has quit IRC16:08
*** romcheg has joined #openstack-ironic16:11
stendulkerNobodyCam: Have done it. Pls take a look when you get time.16:11
NobodyCamstendulker: :) Ty Just waiting to see test results16:12
NobodyCam:-p16:12
stendulkerNobodyCam: Sure. Have re-ran the tests and flake tests after rebasing it.16:13
NobodyCam:)16:13
dtantsurg'night folks, enjoy your weekend!16:14
*** dtantsur is now known as dtantsur|afk16:15
*** zz_jgrimm has joined #openstack-ironic16:18
NobodyCamhave a good weekend dtantsur|afk16:19
*** maurosr has joined #openstack-ironic16:20
*** zz_jgrimm has quit IRC16:23
*** dttocs has quit IRC16:25
*** maurosr has quit IRC16:25
*** ijw has joined #openstack-ironic16:27
*** lsmola_ has quit IRC16:28
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Client should fall back to the lower versions if necessary  https://review.openstack.org/17133416:29
NobodyCamjlvillal: you should add a Co-Authored-By: <you> to the commit message on that16:30
NobodyCam:-p16:30
jlvillalNobodyCam: Okay :)  Thanks16:30
NobodyCam:)16:31
*** maurosr has joined #openstack-ironic16:31
*** zz_jgrimm has joined #openstack-ironic16:31
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Client should fall back to the lower versions if necessary  https://review.openstack.org/17133416:31
* BadCub moves ^ back to the NEEDS APPROVAL list16:35
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Client should fall back to the lower versions if necessary  https://review.openstack.org/17133416:36
*** athomas has quit IRC16:36
*** maurosr has quit IRC16:37
*** dprince has quit IRC16:37
NobodyCamjlvillal: is there a test for the case rloo found. which was user requests a valid version (ie. 1.4) but then asks for some thing that is not supported like setting the name on the node, which was added in 1.5?16:38
*** zz_jgrimm has quit IRC16:39
NobodyCamsee her comment on rev 10 of http.py line 81 ish16:39
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add support for 'baremetal show' command  https://review.openstack.org/17173916:44
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal create  https://review.openstack.org/17217116:44
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add support for microversion  https://review.openstack.org/17217016:44
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create  https://review.openstack.org/17246116:44
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implement baremetal delete command  https://review.openstack.org/17175116:44
*** jerryz has quit IRC16:44
*** derekh has quit IRC16:47
*** trown is now known as trown|lunch16:48
*** bradjones has quit IRC16:49
*** maurosr has joined #openstack-ironic16:50
NobodyCamBadCub: stendulker : 172313 +a's16:50
NobodyCam+a'd even16:50
*** bradjones has joined #openstack-ironic16:51
*** bradjones has quit IRC16:51
*** bradjones has joined #openstack-ironic16:51
stendulkerNobodycam, lucasgomes: Thank you :)16:52
BadCubNobodyCam: lucasagomes thnx for re-reviewing ^16:52
NobodyCam:)16:52
lucasagomes171827 has a typo, but I think it's grand16:53
BadCubeep. I missed the typo lol16:54
*** zz_jgrimm has joined #openstack-ironic16:55
stendulkerlucasagomes: Thank you :) and sorry for typo.16:56
lucasagomesnp16:57
lucasagomes+a'ed16:57
*** tiagogomes_ has quit IRC16:57
NobodyCamlucasagomes: w00 h00...16:58
BadCubawesome! ty lucasagomes :)16:58
lucasagomesnp :D16:58
NobodyCamhave any plans for this weekend lucasagomes ?16:58
*** maurosr has quit IRC17:00
openstackgerritNisha Agarwal proposed openstack/ironic: Update ilo drivers documentation for inspection  https://review.openstack.org/17006517:00
*** zz_jgrimm has quit IRC17:00
*** dprince has joined #openstack-ironic17:01
NobodyCammornig Nisha17:01
*** maurosr has joined #openstack-ironic17:04
BadCubhiya Nisha17:05
NobodyCambrb17:07
*** mjturek1 has quit IRC17:07
*** maurosr has quit IRC17:10
*** zz_jgrimm has joined #openstack-ironic17:16
*** achanda has joined #openstack-ironic17:16
jlvillalNobodyCam: Yes there is a test case for that.17:18
jlvillalNobodyCam: https://review.openstack.org/#/c/171334/16/ironicclient/tests/unit/test_http.py  L12117:18
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal set for power state  https://review.openstack.org/17251717:18
NobodyCamahh I was just blind... /me blames the lack of koffi17:19
*** ijw has quit IRC17:19
*** achanda has quit IRC17:20
*** achanda has joined #openstack-ironic17:21
NobodyCamlucasagomes: i'm just looking over your comemnt on 171334 (about falling-forward)17:21
*** zz_jgrimm has quit IRC17:21
*** Marga_ has quit IRC17:21
lucasagomesNobodyCam, right17:21
lucasagomesjust a note17:21
lucasagomesthat's what we were discussing with mrda and rloo the other day17:22
*** Marga_ has joined #openstack-ironic17:22
JayFdevananda: BadCub: et.al: would you like me to send an email to the list that specs are open + detail the changes to the template + note that anyone wanting to push a K spec for L should move it out of kilo-archive/ into liberty/17:22
*** bandicot has joined #openstack-ironic17:22
NobodyCamfrom what I recall if a user requests a version then the client shouldn't attempt to over rite it (or nogotiate) a different version17:22
NobodyCamJayF: thta would be super kewl17:23
NobodyCamlucasagomes: lines 10-11 of the commit message seem to say that too17:23
NobodyCamIf the user did not request a specific version, the client should negotiate17:24
*** maurosr has joined #openstack-ironic17:24
*** pelix has quit IRC17:25
NobodyCamso the act of requesting a version causes the error (behavior) your seeing with your local testing17:25
*** bandicot has quit IRC17:26
NobodyCamor am I missunderstanding something ... /me notes the lack of coffee comment above ^^^17:26
NobodyCamlol17:26
lucasagomeswell I specified the version to simulate a old version of the client17:27
*** bandicot has joined #openstack-ironic17:27
BadCubJayF: ++ on ^17:27
lucasagomesbut if u have an old version which will pass 1.0 by default17:27
lucasagomesand u try to set the node name with that17:27
lucasagomesit will return 40617:27
*** zz_jgrimm has joined #openstack-ironic17:28
*** maurosr has quit IRC17:29
jlvillallucasagomes: Old version of the client?17:30
lucasagomesyeah17:30
jlvillallucasagomes: But how can we fix that?  Wouldn't that require updating the client?17:30
lucasagomesI think returning 406 for old versions is wrong in Ironic17:31
JayFBadCub: NobodyCam: RFR -> https://etherpad.openstack.org/p/IronicSpecsOpenLibertyEmail17:31
*** zz_jgrimm is now known as jgrimm17:32
jlvillallucasagomes: Okay.  So you are suggesting a change to Ironic server.17:32
lucasagomesbecause that's not backward compat, before if u try to update a field that doesn't exist you would get a 40017:32
*** ijw has joined #openstack-ironic17:32
lucasagomesnow you get 40617:32
lucasagomesyeah17:32
lucasagomesI was talking about it with mrda and rloo17:32
BadCubJayF: Awesome! TY! :)17:32
*** meghal has joined #openstack-ironic17:33
jlvillallucasagomes: Works for me.  If the client doesn't send a version field.  But I don't understand all the possible impacts though.17:33
lucasagomesyeah :-)17:33
lucasagomesanyhoo, it's friday and pretty late here17:33
lucasagomesso I will just call it a day17:33
jlvillallucasagomes: Go have a great weekend :)17:33
BadCubg'night and have an awesome weekend lucasagomes :)17:33
NobodyCamhave a great weekend lucasagomes17:34
lucasagomesjlvillal, BadCub NobodyCam you guys too! Enjoy the weekend17:34
lucasagomessee y'all on monday17:34
*** lucasagomes has quit IRC17:34
* BadCub goes to fix some late breakfast17:34
openstackgerritShivanand Tendulker proposed openstack/ironic: iLO driver documentation for UEFI secure boot  https://review.openstack.org/16884417:34
NobodyCamJayF: looks good to me... I did one minor edit!17:35
*** dttocs has joined #openstack-ironic17:36
JayFNobodyCam: BadCub: Sending this without an explicit +1 from deva seems scary. Do you two count in his stead? :P17:36
*** ijw has quit IRC17:36
*** jgrimm has quit IRC17:36
NobodyCamwe can wait to monday.. for deva to return. tho I'm okay ethier way17:37
JayFI'm going to send it then17:37
BadCubJayF: I spoke with devananda before he left and got he go-ahead to open specs for L. But am good either way as well17:37
BadCubs/he/the17:38
JayFNobodyCam: I just sent it without your edits because I didn't repaste before hitting send. whoops :/17:38
JayFlol17:38
BadCublol17:38
NobodyCamlol it was super minor17:38
JayFyeah I saw :)17:38
*** dttocs has quit IRC17:40
*** maurosr has joined #openstack-ironic17:41
*** korekhov has joined #openstack-ironic17:43
*** harlowja_away is now known as harlowja17:44
*** trown|lunch is now known as trown17:44
*** maurosr has quit IRC17:46
openstackgerritMerged openstack/ironic: Fix for automated boot iso issue with IPA ramdisk  https://review.openstack.org/17231317:46
openstackgerritMerged openstack/ironic: Update installation service overview  https://review.openstack.org/16634617:47
openstackgerritMerged openstack/ironic: Add notes about upgrading juno->kilo to docs  https://review.openstack.org/17182717:47
NobodyCamw00 hoo :)17:48
*** igordcard_ has quit IRC17:48
*** photocyte has joined #openstack-ironic17:49
BadCubsweet17:50
*** zz_jgrimm has joined #openstack-ironic17:50
*** edcragg has quit IRC17:50
NobodyCambrb17:52
*** zz_jgrimm has quit IRC17:54
*** maurosr has joined #openstack-ironic17:57
*** ijw has joined #openstack-ironic18:10
*** maurosr has quit IRC18:11
*** meghal has quit IRC18:12
*** romcheg has quit IRC18:14
*** romcheg has joined #openstack-ironic18:15
*** meghal has joined #openstack-ironic18:15
*** mgoddard has quit IRC18:18
*** bauwser is now known as bauzas18:25
mgagneI would like to explore the ability to associate more neutron ports than physical ports to a node. As anyone tried it?18:30
NobodyCammgagne: not that I know of.18:31
JayF Hm.18:31
JayFmorgabra: ^ Have we tried to do that? Or is it direct 2:2 mapping?18:31
mgagneit's the first time I'm really digging into ironic code18:31
jrollJayF: we haven't bothered with it, because we happen to have two nics and two networks18:31
JayFjroll: that's what I thought18:32
NobodyCammgagne: welcome to the fold :) hehe18:32
morgabranova chooses a port from the ports ironic exposes to it18:32
jrollmgagne: that's something I'm going to hack on in liberty if nobody else does18:32
mgagneso I'm wondering if the 1:1 association between neutron port and phy port is hardcoded or not. or if info is just "info" not some sort of primary key18:32
morgabraand iirc will fail if you try to add more ports than exist ironic ports18:32
mgagnejroll: I'm doing it right now =)18:32
jrollmgagne: it's an assumption nova networks api makes18:32
morgabraalso, 'port' is so overloaded it's so hard to talk about it :P18:32
mgagnemorgabra: yep, I'm removing the protection atm but wondering about the ironic port state update18:33
*** zz_jgrimm has joined #openstack-ironic18:33
*** mgoddard has joined #openstack-ironic18:33
mgagnewe are using neutron18:33
jrollmgagne: right, same goes for neutron, though, by nova networks api I mean the pluggable interface that neutron and nova-networks uses18:33
mgagnehmm, I just find out that neutron port id is "only" stored in the extra field, nothing in Ironic *requires* neutron18:34
morgabraI don't think ironic keeps any mapping right? I don't see why it wouldn't work18:34
jrollmgagne: ironic's dhcp stuff requires neutron18:34
mgagne-> /extra/vif_port_id18:34
mgagneall I found18:34
morgabrait's a little wierd because it uses the mac from the ironic port in the neutron port18:34
jrollthe only mapping ironic keeps is from nova18:34
morgabraso if you have more than 1...18:34
mgagnejroll: you mean network_info right?18:34
mgagnemorgabra: will check that dhcp stuff, we are not planning on using neutron dhcp18:35
morgabrayeah, we don't use it either18:35
jrollmgagne: I'm not sure :)18:35
jrollmgagne: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L382-38618:35
mgagnejroll: one more thing to check then :P18:36
stendulkerNobodyCam: Has this merge got stuck? https://review.openstack.org/#/c/169162/18:37
*** pas-ha has quit IRC18:37
jrollmgagne: I guess I'm not sure where I would have meant network_info18:37
morgabrayeah, I would think you could get away with just letting neutron generate the mac if you run out of ironic macs to assign18:37
mgagnejroll: well, I thought it was it since I'm in this part of the code in nova =)18:37
stendulkerNobodyCam: One test has failed for it.18:38
*** ijw_ has joined #openstack-ironic18:38
*** zz_jgrimm has quit IRC18:38
mgagnemorgabra: AFAIK, I don't care about mac since I'm planning on using tagged vlans18:38
jrollmgagne: hey, you want to do what we do/want to do :D18:38
jrollwe being rackspace18:38
mgagnejroll: =)18:38
*** photocyte has quit IRC18:38
jrollwe currently have two networks that are tagged vlans18:38
morgabramgagne: in fact, it's sort of annoying that it works the way it does for our use case18:39
mgagnejroll: https://github.com/rackerlabs/ironic-neutron-plugin/blob/master/scripts/interfaces/interfaces.template#L18-L2218:39
*** hemna has quit IRC18:39
morgabrabecause if an instance ever fails to clean up or something you get stuck with a neutron port holding the mac for your ironic node18:39
mgagnejroll: are you guys using vlan translation?18:39
NobodyCamstendulker: looks like its still in post gate testing18:39
mgagnejroll: to not expose the actual vlan id?18:39
jrollmgagne: lmao, nice find18:39
mgagnemorgabra: true, will add it to my check list18:39
morgabramgagne: oh, I should delete that, it was leftover from the before-time18:40
stendulkerNobodyCam: ok. My doc review would get into merge conflict with that it. Hence monitoring that :)18:40
*** maurosr has joined #openstack-ironic18:41
*** ijw has quit IRC18:41
*** zz_jgrimm has joined #openstack-ironic18:42
* BadCub grumbles at CWT18:42
*** achanda has quit IRC18:43
jrollBadCub: YES18:43
jrollhate that.18:43
*** Sukhdev has joined #openstack-ironic18:43
BadCubjroll: I am stuck on the "spinning wheel of processing" doom18:43
mgagnewhat I find unfortunate, is the inability (in Nova) to allow a driver to declare and inject his own network template without overriding injected_network_template config18:44
* BadCub bangs face on desk and closes CWT browser window18:45
jrollmgagne: well, you can only have one driver per compute, so why not override that config?18:45
mgagnehttps://review.openstack.org/#/c/171847/ but than InstanceMetadata class doesn't allow you to pass a custom template file18:45
jrollah, I see18:46
mgagnejroll: true but why have scheduler_use_baremetal_filters if you can only have one set of filters =)18:46
*** maurosr has quit IRC18:47
mgagnejroll: but you are right, I can do without it18:47
jrollyeah, I think that's just a convenience thing18:47
mgagneyep18:47
*** zz_jgrimm has quit IRC18:51
*** Marga_ has quit IRC18:51
*** achanda has joined #openstack-ironic18:52
*** Marga_ has joined #openstack-ironic18:52
*** degorenko has quit IRC18:54
*** Sukhdev has quit IRC18:55
*** Sukhdev has joined #openstack-ironic18:59
*** enikanorov_ has joined #openstack-ironic18:59
*** mariojv has quit IRC19:04
*** Marga_ has quit IRC19:05
*** zz_jgrimm has joined #openstack-ironic19:06
*** Marga_ has joined #openstack-ironic19:06
*** Marga_ has quit IRC19:06
*** Marga_ has joined #openstack-ironic19:07
*** maurosr has joined #openstack-ironic19:07
*** Marga_ has quit IRC19:08
*** ndipanov has quit IRC19:09
*** Marga_ has joined #openstack-ironic19:09
*** bandicot has quit IRC19:10
*** Marga_ has quit IRC19:11
*** hemna has joined #openstack-ironic19:18
*** photocyte has joined #openstack-ironic19:19
*** mariojv has joined #openstack-ironic19:22
*** mariojv has quit IRC19:34
NobodyCamwoo hoo travel booked for vancouver :)19:41
BadCubfinally!! Only took an hour to get it done lol19:42
JayFI was booked two weeks ago, you slackers :P19:42
JayFbtw, j* is going to be at the summit19:42
NobodyCamgota luv CWT ... why I know not ..but19:42
JayFbut nobody else from our end :(19:42
* BadCub procrastinates on anything CWT related19:42
JayFNobodyCam: oh man, they just changed the policy here from "we like if you use cwt" to "use this or else"19:42
NobodyCamno mrda19:42
JayFNobodyCam: and it gave me a sad19:43
JoshNangBadCub: just a heads up, i'm debugging some issues with cleaning in our preprod right now. seems like nodes get deadlocked, but i'm only seeing it if you throw a bunch of nodes into cleaning at the same time19:43
JayFNobodyCam: mrda doesn't work for Rackspace, he works for ǝɔɐdsʞɔɐᴚ19:43
NobodyCamlol19:43
BadCubJayF: ouch. That sucks :( Us HPers feel the pain of CWT19:43
* JoshNang misses booking travel with hipmunk19:43
BadCubJoshNang: okay. Thanks for the heads up. You think we will end up with a bug on that?19:44
JoshNangBadCub: not sure. i also have some downstream code (to migrate from what we've been running for a year to what is upstream) that i need to rule out19:44
BadCubJoshNang: okay cool :-)19:45
*** bradjones has quit IRC19:55
*** Haomeng has quit IRC20:01
*** bradjones has joined #openstack-ironic20:01
*** Haomeng has joined #openstack-ironic20:02
NobodyCamwow HP Global Security Services has a lot to say about travel to Canada20:02
JayFNobodyCam: I had to watch a video telling me how not to get abducted by 'terrorists'20:08
NobodyCamlol20:09
BadCuboh my... Somehow I think getting abducted by terrorists in Canada is pretty unlikely20:09
NobodyCamCanadian 'terrorists' eehh20:10
NobodyCamhttps://www.youtube.com/watch?v=1BFPt001PYU20:11
*** achanda has quit IRC20:12
* TheJulia wonders if she wants to click the link20:12
NobodyCamCanadian 'terrorists' eehh20:13
* BadCub coaxes TheJulia to click the link... Just click it......20:13
*** stendulker has quit IRC20:14
BadCubbrb20:14
*** mariojv_ has joined #openstack-ironic20:15
pshige:-)20:16
*** dprince has quit IRC20:17
*** Marga_ has joined #openstack-ironic20:20
*** Marga_ has quit IRC20:21
*** korekhov has quit IRC20:21
*** Marga_ has joined #openstack-ironic20:21
*** korekhov has joined #openstack-ironic20:22
NobodyCammorning pshige20:23
BadCubhiya pshige :)20:25
*** Marga_ has quit IRC20:30
NobodyCamjlvillal: just curios are you by chance looking to the comment lucas made on the client microversion patch. or are mrda / rloo looking at that? did follow as closly as i should have this morning :-p20:30
*** Marga_ has joined #openstack-ironic20:30
pshigeNobodyCam, BadCub: morning20:31
jlvillaljlvillal: I am not looking at that comment as he said the server needs to be changed, not the client.20:31
NobodyCamjlvillal: ya20:31
jlvillalBadCub: ^^^  I'm talking to myself20:31
NobodyCamjlvillal: thats not a good sign20:32
jlvillal:)20:32
NobodyCambut it is friday lol20:32
NobodyCam:-p20:32
BadCubjlvillal: I noticed... It is s sign that you need to take off for the weekend lol20:32
jlvillalBadCub: In about four hours I hope :)20:32
jlvillalBadCub: I'm working on making some more test cases.20:33
BadCubbout the same here :)20:33
pshigealready Saturday morning in Japan :-)20:33
BadCubawesome!20:33
BadCubpshige: what you doing online on a Saturday?20:33
NobodyCampshige: oh rub it in... :(20:33
NobodyCamlol20:33
pshigelol20:37
NobodyCampshige: I hear the saturday morning cartoons are quite good over there!20:38
*** korekhov_ has joined #openstack-ironic20:41
*** korekho__ has joined #openstack-ironic20:43
*** korekhov_ has quit IRC20:43
*** korekhov has quit IRC20:45
pshigeFrom 0730 to 1100 JST20:46
NobodyCam:-p :)20:46
pshige0546 JST now20:46
NobodyCamwow thats early.. I'm usaly up around 0600 pst but its at least 1/2 hour before I attempt to speak to anyone lol20:48
BadCublol20:48
* BadCub does not speak until first cup of coffee is consumed20:48
pshigeI am now drinking first cup of coffee :-)20:53
NobodyCam:)20:53
*** Sukhdev has quit IRC20:59
*** achanda has joined #openstack-ironic20:59
*** trown is now known as trown|outttypeww21:08
*** kkoski has quit IRC21:10
*** Marga_ has quit IRC21:12
*** zz_jgrimm is now known as jgrimm21:16
*** thrash is now known as thrash|wknd21:17
*** jgrimm is now known as zz_jgrimm21:17
*** alexpilotti has quit IRC21:27
*** photocyte has quit IRC21:30
*** Marga_ has joined #openstack-ironic21:31
*** Marga_ has quit IRC21:34
*** Marga_ has joined #openstack-ironic21:34
*** Marga_ has quit IRC21:35
*** Marga_ has joined #openstack-ironic21:36
*** korekho__ has quit IRC21:38
*** korekhov has joined #openstack-ironic21:38
*** korekhov has quit IRC21:41
*** korekhov has joined #openstack-ironic21:41
*** ijw_ has quit IRC21:42
*** pas-ha has joined #openstack-ironic21:42
*** ijw has joined #openstack-ironic21:42
*** ijw has quit IRC21:43
*** bradjones has quit IRC21:56
*** harlowja is now known as harlowja_away21:58
* NobodyCam steps away for a bit22:02
*** mgoddard has quit IRC22:04
*** bradjones has joined #openstack-ironic22:04
*** andreykurilin__ has joined #openstack-ironic22:08
*** romcheg has quit IRC22:13
*** sambetts has quit IRC22:19
*** sambetts has joined #openstack-ironic22:21
*** harlowja_away is now known as harlowja22:21
openstackgerritJosh Gachnang proposed openstack/ironic: Fix stuck locks for async clean steps  https://review.openstack.org/17258222:24
JoshNangBadCub: ^ the issue i was talking about. pretty minor fix22:24
JoshNangi tested 20 nodes at the same time, all of them finished with that. before i was getting a handful at best.22:24
BadCubJoshNang: That is AWESOME news22:26
*** andreykurilin__ has quit IRC22:28
*** andreykurilin__ has joined #openstack-ironic22:28
JoshNang:) i'm just glad we caught it before rc122:30
BadCubgreat catch! I added ^ to the pad and tagged it for rc122:31
JoshNangthanks!22:31
BadCubnp. Thank you for catching it and putting a fix up so fast :)22:32
NobodyCamJoshNang: should I be sad that it was only a one line cehck to tests and yet the doc string removes four raises :(22:32
NobodyCamcehck = change22:33
NobodyCam:-p22:33
jrollJoshNang: wherever the @messaging.expected_exceptions or whatever come in, should those be removed?22:33
JoshNangNobodyCam: heh yeah, the rpc tests are pretty much 'can you call this function? yes! good enough'22:33
JoshNangjroll: hmmm22:34
jrollunless those just weren't on this method?22:34
JoshNangthese ones, yeah? https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L83122:35
jrollyeah22:35
JoshNanglemme look where those are caught/handled22:36
*** hemna has quit IRC22:36
* NobodyCam goes an eats a peep22:38
* BadCub dives into chocolate egg22:41
JoshNangjroll: so i think leaving the expected exceptions is alright. we don't want to cleanfail or raise an exception for a node because the conductor is out of workers. and, because it is heartbeat triggered, it'll retry22:43
jrollJoshNang: I guess I don't know what those actually do :)22:44
jrollbut yeah, seems alright22:45
JoshNangjroll: hehe22:45
JoshNangdocs: http://docs.openstack.org/developer/oslo.messaging/server.html22:45
jrollcool22:46
jroll+222:47
JoshNang\o/22:47
*** bradjones has quit IRC22:50
JayFRequest for image: Pixie Boots polishing up it's drum, with caption "Ironic: Now with cleaning"22:54
BadCubJayF: would have to draw a maid's outfit on Pixie Boots for that to work lol22:55
*** Marga_ has quit IRC22:56
*** EmilienM is now known as EmilienM|afk22:56
JayFso I just said to JoshNang22:56
jrolllol22:56
JayFwe need a pixie boots image corresponding to every state22:56
JoshNangheh22:56
JayFfor zapping I see a marvin the martian style disintegrator22:56
BadCublol22:56
BadCubor a cattle prod22:56
jrollheh22:56
*** bradjones has joined #openstack-ironic22:57
NobodyCamoh we should redo the cat status codes to be PixieBoots based22:57
JayFBadCub: ಠ_ಠ22:58
BadCubsomehow I think boor Pixie Boots can have some very "interesting" looks22:59
BadCubs/boor/poor22:59
*** mariojv_ has quit IRC22:59
BadCubmy fingers say it's Friday and my head says it's cocktail time (almost)23:00
jrollI would love to see a NobodyCam / BadCub version of pixie boots23:00
NobodyCamieek23:00
jrollit's past cocktail time but I have to drive to the airport later :/23:00
BadCubThat could be...... Dangerous lol23:00
jrollheh23:01
*** Marga_ has joined #openstack-ironic23:01
NobodyCamof at a minimum NSFW23:01
BadCubcensored by most network TV stations even23:01
NobodyCamof = or23:01
NobodyCamlol23:01
jrollyeah, I'd get a kick out of it23:01
JayFNobodyCam: make it light red23:02
JayFNobodyCam: I'm 100% serious, do it, and don't tell Julia I told you to23:02
jrollLOL23:02
jrollyeah, she'll never see this23:02
NobodyCamlol23:02
* BadCub tries to wipe these bad things off the IRC board lol23:03
*** pas-ha has quit IRC23:04
BadCubbrb23:06
* BadCub sends PixieBoots off to the spa for a makeover lol23:12
NobodyCamjroll: JoshNang: BadCub: I would like to give others time to look at that patch before landing but do we expect folks to look at things over the weekend?23:16
JoshNangNobodyCam: that's reasonable23:16
JoshNangthere's definitely people on before our monday morning due to timezones23:16
NobodyCamJoshNang: I'll +2 with a comment to why I not +a'ing "right away"23:17
JoshNangNobodyCam: kk. thanks!23:17
BadCubNobodyCam: sounds good. I doubt/hope folks aren't looking at patches over the wekeend though23:17
JoshNangme too!23:17
* BadCub will not be doing such things23:18
NobodyCamJoshNang: there ya go23:19
NobodyCamif I pop in on sunday I'll +a that guy23:20
JoshNangsweet!23:20
* BadCub smacks NobodyCam 's hands. You be diggin in the back yard Sunday LOL We have more archaeological finds to discover23:21
*** andreykurilin__ has quit IRC23:21
NobodyCamlol23:22
*** davideagnello has quit IRC23:26
* TheJulia immagines BadCub sitting back with a nice hat watching NobodyCam dig23:29
BadCubTheJulia: don't forget the cocktail! LOL23:30
BadCuboh, and fan with mister attachment hehehe23:30
TheJuliaI didn't want to mention that, but yes, that was there as well23:30
TheJulianot somebody else with palm fronds?23:31
BadCubthat sounds good too..... *shy grin*23:31
*** achanda has quit IRC23:36
*** achanda has joined #openstack-ironic23:42
*** jerryz has joined #openstack-ironic23:43
*** meghal has quit IRC23:45
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Client should fall back to the lower versions if necessary  https://review.openstack.org/17133423:50
*** ijw has joined #openstack-ironic23:52
*** ijw_ has joined #openstack-ironic23:55
*** ijw has quit IRC23:58

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