Monday, 2015-06-22

*** s-moriya has joined #openstack-ironic00:09
*** naohirot has joined #openstack-ironic00:21
*** bradjones has quit IRC00:22
*** bradjones has joined #openstack-ironic00:23
*** bradjones has quit IRC00:23
*** bradjones has joined #openstack-ironic00:23
*** Mark_Melb has joined #openstack-ironic00:35
*** Mark_Melb has quit IRC00:35
*** amotoki has joined #openstack-ironic00:50
*** jgrimm_ is now known as zz_jgrimm_01:27
*** Sukhdev has joined #openstack-ironic01:52
*** linkedinyou has quit IRC02:06
*** yog_ has joined #openstack-ironic02:56
openstackgerritYuiko Takada proposed openstack/ironic-inspector: Load authenticate token from HTTP header  https://review.openstack.org/17423103:05
*** ramineni has joined #openstack-ironic03:09
*** achanda has joined #openstack-ironic03:10
*** yuanying has quit IRC03:16
*** alexpilotti has joined #openstack-ironic03:17
openstackgerritYuiko Takada proposed openstack/python-ironic-inspector-client: Change to Capital letters  https://review.openstack.org/19392403:21
*** Marga_ has joined #openstack-ironic03:30
*** Marga_ has quit IRC03:30
*** Marga_ has joined #openstack-ironic03:31
*** achanda has quit IRC03:33
*** zz_natorious is now known as natorious03:35
*** natorious is now known as zz_natorious03:37
*** Sukhdev has quit IRC03:52
*** yuanying has joined #openstack-ironic04:07
*** yuanying has quit IRC04:07
*** yuanying has joined #openstack-ironic04:08
*** yuanying_ has joined #openstack-ironic04:24
*** yuikotakada has joined #openstack-ironic04:25
*** yuanying_ has quit IRC04:26
*** yuanying has quit IRC04:28
*** pradipta has joined #openstack-ironic04:35
*** spandhe has joined #openstack-ironic04:39
*** Marga_ has quit IRC04:41
*** yuanying has joined #openstack-ironic04:46
*** ijw_ has joined #openstack-ironic04:46
*** yuanying has quit IRC04:48
*** Marga_ has joined #openstack-ironic04:49
*** yuanying has joined #openstack-ironic05:02
*** yuanying has quit IRC05:05
*** radek has joined #openstack-ironic05:07
openstackgerritMichael Davies proposed openstack/python-ironicclient: Cache negotiated api microversion for server  https://review.openstack.org/17367405:10
*** Haomeng|2 has joined #openstack-ironic05:11
*** yuanying has joined #openstack-ironic05:12
*** Haomeng has quit IRC05:15
*** ijw_ has quit IRC05:15
*** radek__ has joined #openstack-ironic05:18
*** radek has quit IRC05:21
*** yuanying has quit IRC05:22
*** ijw_ has joined #openstack-ironic05:23
*** yuanying has joined #openstack-ironic05:23
*** yuanying has quit IRC05:25
*** yuanying has joined #openstack-ironic05:33
*** yuanying has quit IRC05:36
*** saripurigopi has joined #openstack-ironic05:36
saripurigopigood morning Ironic...05:36
*** yuanying has joined #openstack-ironic05:39
*** yuanying has quit IRC05:41
yuikotakadasaripurigopi, hi, good morning05:44
*** alexpilotti has quit IRC05:50
*** spandhe has quit IRC05:56
*** spandhe has joined #openstack-ironic05:56
*** coolsvapz is now known as coolsvap06:02
saripurigopimorning yuikotakada :-)06:02
*** spandhe has quit IRC06:13
*** yuanying has joined #openstack-ironic06:14
*** dasm has joined #openstack-ironic06:16
*** Marga_ has quit IRC06:17
*** yuanying has quit IRC06:19
*** Marga_ has joined #openstack-ironic06:23
*** saripurigopi has quit IRC06:27
*** saripurigopi has joined #openstack-ironic06:27
*** pradipta has quit IRC06:34
*** yuanying has joined #openstack-ironic06:36
mrdahi saripurigopi and yuikotakada06:38
saripurigopihi mrda06:39
saripurigopiwhere can I download the presentation ppts of Vancouver summit? I can see the videos @ https://www.openstack.org/summit/vancouver-2015/summit-videos/presentation, but looking for ppts alone.06:40
mrdasaripurigopi: at previous summits the power points were linked off the summit-videos page06:42
*** pradipta has joined #openstack-ironic06:46
saripurigopimrda : okay, don't see any links now.06:46
openstackgerritAnusha Ramineni proposed stackforge/proliantutils: Add RIS support for firmware update  https://review.openstack.org/19395206:46
mrdasaripurigopi: no, neither do I.  Sorry.06:46
yuikotakadamrda, hi :)06:48
*** stendulker has joined #openstack-ironic06:50
mrdahi yuikotakada06:51
saripurigopiHi stendulker06:51
stendulkersaripurigopi: Hi06:51
saripurigopistendulker: if you've "How to make Ironic bare-metal provisioning more secure and reliable?" presentation ppt, could it share it?06:52
stendulkersaripurigopi: The youtube video of same is there.06:53
saripurigopistendulker, just want the ppt alone.06:53
stendulkersaripurigopi: They too were hosted at someplace. Will check and let you know the link06:54
saripurigopistendulker: sure, thank you.06:55
stendulkersaripurigopi: Check it here http://files.meetup.com/3498272/2.%20How%20to%20make%20ironic%20baremetal%20provisioning%20more%20secure%20and%20rel.pdf06:57
saripurigopistendulker: got it, thanks.06:57
*** yuikotakada has quit IRC06:57
*** yuikotakada has joined #openstack-ironic07:06
dtantsurMorning Ironic!07:08
*** bethelwell has joined #openstack-ironic07:09
yuikotakadadtantsur, g'morning :)07:11
dtantsuro/07:11
dtantsurgood job putting up https://review.openstack.org/#/c/19394107:11
mrdahi dtantsur07:12
dtantsurmrda, hey!07:12
yuikotakadadtantsur,Thanks, running .sh on gate seems possible ;)07:13
*** bethelwell has quit IRC07:13
dtantsuryuikotakada, yeah, lgtm, just one small comments: it's not a functional test, it's integration test. so please no "-functest" postfix :)07:14
yuikotakadadtantsur, ok, I will fix.07:14
dtantsur(the confusion is because Ironic is running functional tests the same way)07:14
*** athomas has joined #openstack-ironic07:15
*** Guest87092 has quit IRC07:23
*** d0ugal has joined #openstack-ironic07:24
*** d0ugal is now known as Guest1487507:24
*** foexle has joined #openstack-ironic07:24
*** Guest14875 is now known as d0ugal07:25
*** d0ugal has quit IRC07:25
*** d0ugal has joined #openstack-ironic07:25
dtantsuryuikotakada, responded to your session proposal letter07:26
dtantsurwhich reminds me, that trown and I should also write one...07:26
yuikotakadadtantsur, thanks a lot! I'm reading now.07:28
*** dguerri` is now known as dguerri07:31
*** Marga_ has quit IRC07:34
*** dguerri is now known as dguerri`07:35
*** chlong has quit IRC07:39
*** ndipanov has joined #openstack-ironic07:39
*** bradjones has quit IRC07:43
*** bradjones has joined #openstack-ironic07:45
*** bradjones has quit IRC07:45
*** bradjones has joined #openstack-ironic07:45
*** saripurigopi_ has joined #openstack-ironic07:46
*** saripurigopi has quit IRC07:47
openstackgerritNaohiro Tamura proposed openstack/ironic: Add IPA support for iscsi_irmc driver  https://review.openstack.org/18180707:55
*** bradjones has quit IRC08:02
*** Marga_ has joined #openstack-ironic08:03
*** lucasagomes has joined #openstack-ironic08:03
*** e0ne has joined #openstack-ironic08:05
*** yuikotakada has quit IRC08:05
*** Marga_ has quit IRC08:06
*** yuanying has quit IRC08:07
*** Marga_ has joined #openstack-ironic08:07
*** Nisha has joined #openstack-ironic08:08
*** e0ne has quit IRC08:11
*** bradjones has joined #openstack-ironic08:14
*** bradjones has quit IRC08:14
*** bradjones has joined #openstack-ironic08:14
*** Marga_ has quit IRC08:17
*** dguerri` is now known as dguerri08:20
*** Marga_ has joined #openstack-ironic08:21
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/19402308:27
*** romcheg has joined #openstack-ironic08:32
*** oomichi has joined #openstack-ironic08:32
oomichidtanstur: hi08:33
oomichidtantsur: is it fine to operate the test script for inspector on the gate now?08:34
*** yuriyz has quit IRC08:34
*** Nisha has quit IRC08:34
oomichidtantsur: I heard inspector needed to run Tempest on the gate, but that is not necessary now?08:35
*** bradjones has quit IRC08:38
*** yuriyz has joined #openstack-ironic08:38
*** pelix has joined #openstack-ironic08:39
*** bradjones has joined #openstack-ironic08:40
*** bradjones has quit IRC08:40
*** bradjones has joined #openstack-ironic08:40
*** rameshg87 has joined #openstack-ironic08:41
*** jcoufal has joined #openstack-ironic08:41
dtantsuroomichi, well, until stuff with tempest plugins settles, we'd like to have at least some gate testing08:43
dtantsuroomichi, btw, do you by change know answer to another my question on that patch?08:44
*** derekh has joined #openstack-ironic08:44
oomichidtantsur: ok, thanks for your explanation. maybe yuiko's gate patch would be really nice as the first step :)08:45
oomichidtantsur: which question?08:45
dtantsuroomichi, how does it happen that gate is testing the current patch (in gerrit), not git master?08:46
dtantsurit looks like we're fetching the latest git..08:46
openstackgerritNaohiro Tamura proposed openstack/ironic: Add unit test for ilo_deploy _configure_vmedia_boot()  https://review.openstack.org/19404708:47
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Cache Keystone client instance  https://review.openstack.org/18668508:47
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/19237308:47
*** davideagnello has joined #openstack-ironic08:50
oomichidtantsur: ah, "the latest git" means project-config repo?08:51
dtantsuroomichi, no, the project.08:51
dtantsuroomichi, e.g. ironic-inspector08:51
dtantsur(I'm moving it to #infra)08:51
oomichidtantsur: ironic-inspector?08:52
*** davideagnello has quit IRC08:55
sambettsMorning ironi08:55
oomichidtantsur: ok08:55
sambettsironic o/08:55
*** marzif has joined #openstack-ironic08:56
*** jcoufal has quit IRC08:57
*** oomichi has quit IRC09:01
*** linkedinyou has joined #openstack-ironic09:02
*** e0ne has joined #openstack-ironic09:05
*** MattMan has joined #openstack-ironic09:11
*** romainh has joined #openstack-ironic09:13
*** dguerri is now known as dguerri`09:20
*** dguerri` is now known as dguerri09:20
dtantsursambetts, morning!09:21
* dtantsur brb09:21
*** dtantsur is now known as dtantsur|brb09:21
*** e0ne is now known as e0ne_09:22
*** Nisha has joined #openstack-ironic09:23
Nishadtantsur|brb,09:23
Nishahi09:23
*** e0ne_ is now known as e0ne09:27
*** romcheg has quit IRC09:29
*** romcheg has joined #openstack-ironic09:34
*** ifarkas has joined #openstack-ironic09:43
*** naohirot has quit IRC09:56
rameshg87jroll: when you get a chance, please have a look at https://review.openstack.org/#/c/180014/09:58
*** amotoki has quit IRC10:10
openstackgerritMerged openstack/ironic-python-agent: IPA:'shred' utility to use configured iterations  https://review.openstack.org/19175110:11
*** romcheg has quit IRC10:12
*** jamielennox is now known as jamielennox|away10:25
*** romcheg has joined #openstack-ironic10:26
*** bethelwell has joined #openstack-ironic10:32
*** yuikotakada has joined #openstack-ironic10:34
*** Nisha has quit IRC10:34
*** rsFF has quit IRC10:36
*** davideagnello has joined #openstack-ironic10:39
*** rsFF has joined #openstack-ironic10:42
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Do not preserve partitions after ephemeral  https://review.openstack.org/18268810:42
*** davideagnello has quit IRC10:43
*** d0ugal has quit IRC10:43
*** ramineni has quit IRC10:43
*** gabriel-bezerra has quit IRC10:43
*** clif_h has quit IRC10:43
*** dtantsur|brb has quit IRC10:43
*** persia has quit IRC10:43
*** eglute_s has quit IRC10:43
*** Guest11697 has quit IRC10:43
*** adam_g has quit IRC10:43
*** zz_natorious has quit IRC10:43
*** zigo has quit IRC10:43
*** jroll has quit IRC10:43
*** comstud has quit IRC10:43
*** persia has joined #openstack-ironic10:43
*** persia has quit IRC10:43
*** persia has joined #openstack-ironic10:43
*** comstud has joined #openstack-ironic10:43
*** clif_h has joined #openstack-ironic10:43
*** zz_natorious has joined #openstack-ironic10:43
*** dtantsur has joined #openstack-ironic10:43
*** d0ugal has joined #openstack-ironic10:43
*** ramineni has joined #openstack-ironic10:43
*** zz_natorious is now known as natorious10:43
*** jroll has joined #openstack-ironic10:43
*** adam_g has joined #openstack-ironic10:43
*** d0ugal is now known as Guest2901110:43
*** mgagne has joined #openstack-ironic10:43
*** romainh has quit IRC10:43
*** mgagne is now known as Guest8120210:43
*** adam_g has quit IRC10:43
*** adam_g has joined #openstack-ironic10:43
*** romainh has joined #openstack-ironic10:43
*** zigo has joined #openstack-ironic10:44
*** Guest29011 is now known as d0ugal10:44
*** d0ugal has quit IRC10:44
*** d0ugal has joined #openstack-ironic10:44
*** e0ne is now known as e0ne_10:44
*** eglute has joined #openstack-ironic10:44
yuikotakadadtantsur, thank you very much about https://review.openstack.org/#/c/193941/ ! Sorry for not having been there, I was in the train :(10:47
*** gabriel-bezerra has joined #openstack-ironic10:48
*** s-moriya has quit IRC10:50
dtantsuryuikotakada, no prob :)10:52
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Do not preserve partitions after ephemeral  https://review.openstack.org/18268810:53
*** e0ne_ is now known as e0ne10:53
*** subscope has joined #openstack-ironic10:56
*** ramineni has quit IRC10:56
*** romainh has quit IRC10:59
*** linkedinyou has quit IRC11:04
rameshg87lucasagomes: hi11:05
lucasagomesrameshg87, hi there11:05
TheJuliagood morning11:05
rameshg87lucasagomes: any quick review or trying out of https://review.openstack.org/#/c/166513/ is much appreciated :)11:05
rameshg87lucasagomes: refactored pxe into boot interface11:05
rameshg87lucasagomes: thinking to refactor ipxe out of PXEBootInterface in the next commit11:06
lucasagomesrameshg87, oh sure! I'm just finishing something up and will take a look11:06
rameshg87thanks11:06
lucasagomesrameshg87, btw, one thing. The pxe_ipmitool driver will still offer ipxe?11:06
rameshg87lucasagomes: yes11:06
lucasagomescause that does now, I wonder if we should really refactor it into ipxe_...11:06
rameshg87PXEBootInterface has both pxe and ipxe now11:06
lucasagomesrameshg87, ah gotcha!11:06
lucasagomesperfect11:06
lucasagomesso it's backward compat11:07
rameshg87lucasagomes: yeah11:07
rameshg87lucasagomes: also I am thinking of delaying this patch for l211:07
rameshg87lucasagomes: anyway it will get delayed11:07
rameshg87but just want to make sure I don't want to get this in l1 with just 4 days to go11:07
*** dguerri is now known as dguerri`11:07
lucasagomes++11:08
lucasagomesI will review it soonish11:08
rameshg87because there are whole lot of combinations (uefi, localboot, wholediskimage, etc) and all drivers are impacted11:08
*** dguerri` is now known as dguerri11:08
rameshg87everything is not in gate :(11:08
*** stendulker has quit IRC11:10
lucasagomesx.x11:11
lucasagomesyeah, too many features11:11
openstackgerritJulia Kreger proposed openstack/bifrost: Add dynamic node unprovision role  https://review.openstack.org/19368811:14
openstackgerritJulia Kreger proposed openstack/bifrost: Remove un-necessary checks from the dynamic test  https://review.openstack.org/19368711:14
dtantsurmorning (ugt) TheJulia, lucasagomes, rameshg8711:15
yuikotakadadtantsur: Is it OK updating ironic.yaml instead of creating ironic-inspector.yaml newly ?11:15
openstackgerritJulia Kreger proposed openstack/bifrost: Correct ipv4 networking config issue in dynamic path  https://review.openstack.org/19368911:15
rameshg87yuikotakada: TheJulia: dtantsur: o/11:15
openstackgerritJulia Kreger proposed openstack/bifrost: Add delete role for dynamic inventory testing  https://review.openstack.org/19369011:15
dtantsuryuikotakada, I believe it is. anyone it's something AJaeger requested11:16
dtantsur* anyway11:16
rameshg87TheJulia: are you planning to add support for iscsi_ilo driver in bifrost ?11:16
*** daemontool_ has joined #openstack-ironic11:16
lucasagomesdtantsur, good ugt morning11:16
yuikotakadarameshg87, TheJulia, lucasagomes: o/11:16
rameshg87TheJulia: currently it has dependency on swift, but there is a spec proposed for adding support for a webserver on conductor to host images - https://review.openstack.org/#/c/193478/111:18
rameshg87TheJulia: good if you had a look at that spec11:18
*** marzif has quit IRC11:18
yuikotakadadtantsur, I read his comment now. I don't understand why. Do you?11:18
dtantsuryuikotakada, probably he wants to group everything related to Ironic in one file. We're part of Ironic team after all :)11:19
dtantsuryuikotakada, you can always ping him on #openstack-infra, if you want more details..11:19
lucasagomesyuikotakada, morning11:20
rameshg87dtantsur: yuikotakada: nisha was trying to ping one of you, had a question on inspector11:20
rameshg87dtantsur: yuikotakada: Nisha was trying to use latest ironic with latest ironic-inspector11:20
rameshg87seems like https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/inspector.py#L91-L96 is causing driverloaderror11:20
rameshg87the file https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/__init__.py is not empty and doesn't have __version_info__11:21
*** Marga_ has quit IRC11:21
dtantsurrameshg87, ugh, that's bad11:21
yuikotakadadtantsur, hm, ok. Well, can I add gate job also?11:21
dtantsurrameshg87, but I have an easy solution: please review https://review.openstack.org/#/c/193176 :)11:21
rameshg87dtantsur: oh okay :)11:22
dtantsuryuikotakada, there's no point in having non-voting gate job. we'll create it, once we make a check job voting11:22
yuikotakadarameshg87, Today? I talked with him maybe on the last Friday.11:22
rameshg87yuikotakada: yeah, just a few hours back11:22
rameshg87dtantsur: will take a look after I go home and relogin11:23
* rameshg87 goes home 11:23
dtantsurrameshg87, thnx!11:23
*** rameshg87 has quit IRC11:23
dtantsurlucasagomes, can I also ask you to review https://review.openstack.org/#/c/193176 ? It's blocking some of the inspector work..11:23
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/19398511:24
lucasagomesdtantsur, will do11:24
yuikotakadadtantsur, you mean, I should make check job voting, right?11:26
*** romainh has joined #openstack-ironic11:27
dtantsuryuikotakada, not right now please :) what exactly do you want to achieve now?11:27
openstackgerritJulia Kreger proposed openstack/bifrost: Add delete role for dynamic inventory testing  https://review.openstack.org/19369011:27
*** jlvillal has quit IRC11:28
yuikotakadadtantsur, I'd like to run both check and gate job safely. What should I do? Adding gate job with non-voting makes no sense, right?11:28
dtantsuryuikotakada, first of all you wait :) we need to finally get our non-voting check job, and watch it for some time.11:29
openstackgerritJulia Kreger proposed openstack/bifrost: Individual playbook for creating a test VM  https://review.openstack.org/19369111:29
openstackgerritJulia Kreger proposed openstack/bifrost: Update the dynamic test playbook  https://review.openstack.org/19369211:29
dtantsuryuikotakada, then we propose the same non-voting job to Ironic11:29
dtantsurwe also create a similar job on a client, once we merge https://review.openstack.org/#/c/193150/11:29
yuikotakadadtantsur, ok, in the separate patch?11:29
dtantsuryep, definitely11:30
dtantsurfinally, after we have all the jobs running for some time (say, a couple of weeks), we start making them voting11:30
dtantsurthat's a slow thing, yeah :)11:30
yuikotakadadtantsur, ok. I'm looking forward that devstack-test.sh runs on the gate. We will not need to run that by myself anymote :)11:32
dtantsuryeah, such a relief :) even non-voting it would help A LOT11:32
*** saripurigopi_ has quit IRC11:35
*** daemontool_ has quit IRC11:38
*** marzif has joined #openstack-ironic11:39
yuikotakadadtantsur, BTW, we need to talk about future tasks. We have to run integration tests in Tempest or in Ironic-tree with Tempest-lib some time, right? will you abandon https://review.openstack.org/#/c/192196/ and wait for (1)adam_g's patch set will be merged or (2) Tempest-lib's something ?11:40
*** Marga_ has joined #openstack-ironic11:40
dtantsuryuikotakada, I've raised  https://review.openstack.org/#/c/192196/ mostly to facilitate discussion on microversions. Don't mind it for now. We should go for Tempest once Adam's patch lands.11:40
*** Marga_ has quit IRC11:41
dtantsurbut with our new check jobs we're no longer in hurry11:41
*** foexle has quit IRC11:41
*** Marga_ has joined #openstack-ironic11:41
dtantsuryuikotakada, I would say, once we have our simple check jobs, lets forget about tempest for some time. Let it settle.11:41
*** marzif_ has joined #openstack-ironic11:42
yuikotakadadtantsur, yep. but his patch set is left as it is for about 2 months :(11:43
yuikotakadadtantsur, OK, for a while.11:44
dtantsuryuikotakada, that's why I suggest to not bother with Tempest for a while.. We can't speed it up and we need to finally make 2.0.0 release :)11:44
*** lucasagomes is now known as lucas-hungry11:45
dtantsuryuikotakada, what we did forget is translation jobs. Could you please revive your patch?11:48
yuikotakadadtantsur, Ah! I will, soon11:49
dtantsurthanks11:49
openstackgerritMerged openstack/ironic-inspector: Remove ironic running check from inspector startup  https://review.openstack.org/19219111:53
*** dprince has joined #openstack-ironic12:00
openstackgerritVictor Sergeyev proposed openstack/ironic: Switch from MySQL-python to PyMySQL  https://review.openstack.org/18437812:02
openstackgerritAndreas Jaeger proposed openstack/ironic-inspector: Setup Translations  https://review.openstack.org/19410512:07
*** e0ne is now known as e0ne_12:18
openstackgerritMerged openstack/ironic-inspector: Setup Translations  https://review.openstack.org/19410512:19
*** e0ne_ is now known as e0ne12:19
*** amotoki has joined #openstack-ironic12:19
*** romainh has left #openstack-ironic12:25
*** chlong has joined #openstack-ironic12:25
*** chlong has quit IRC12:28
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add support for inspection using ironic-inspector  https://review.openstack.org/19411612:28
* dtantsur started moving some code around :)12:29
yuikotakadaI know, of course. Reading :)12:29
*** Marga_ has quit IRC12:36
*** alexpilotti has joined #openstack-ironic12:40
*** viktors has joined #openstack-ironic12:41
dtantsurtrown, good job sending a email, let's see what people think12:43
trowndtantsur: thanks12:44
openstackgerritJulia Kreger proposed openstack/bifrost: Addition of dynamic enrollment and deployment playbooks  https://review.openstack.org/19373712:50
openstackgerritJulia Kreger proposed openstack/bifrost: Add dynamic inventory description to README  https://review.openstack.org/19370912:50
openstackgerritVictor Sergeyev proposed openstack/ironic: Switch from MySQL-python to PyMySQL  https://review.openstack.org/18437812:51
*** bradjones has quit IRC12:52
openstackgerritJulia Kreger proposed openstack/bifrost: Change Bifrost over to using dynamic inventory  https://review.openstack.org/19388512:54
*** bradjones has joined #openstack-ironic12:54
*** bradjones has quit IRC12:54
*** bradjones has joined #openstack-ironic12:54
openstackgerritMerged openstack/ironic: Rename ilo_power._attach_boot_iso to improve readability  https://review.openstack.org/19304212:56
*** e0ne is now known as e0ne_12:58
*** e0ne_ is now known as e0ne12:59
*** lucas-hungry is now known as lucasagomes13:03
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Clean nodes stuck in DEPLOYING state when ir-cond restarts  https://review.openstack.org/19413213:03
lucasagomesif you guys have some time, mind taking a peek at ^? It's small13:03
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Clean nodes stuck in DEPLOYING state when ir-cond restarts  https://review.openstack.org/19413213:09
*** bethelwell has quit IRC13:11
*** coolsvap is now known as coolsvap|away13:13
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add support for inspection using ironic-inspector  https://review.openstack.org/19411613:21
dtantsurhuh, is gerrit down?13:24
*** dasm is now known as dasm|afk13:26
sambettsdtantsur: git review etc is working for me13:26
sambettsdtantsur: but the review page isn't loaded13:27
sambettsloading*13:27
dtantsurgertty does not work as well..13:28
dtantsur(does not update, I mean)13:28
sambettsdtantsur: I guess the gerrit API/web frontend is down :(13:30
dtantsurgot up for me13:30
sambettsdtantsur: Yup its back for me too13:30
*** zz_jgrimm_ is now known as jgrimm13:35
*** jroll has quit IRC13:36
*** jroll has joined #openstack-ironic13:36
*** thiagop has quit IRC13:39
*** Marga_ has joined #openstack-ironic13:41
*** absubram has quit IRC13:41
sambettsdtantsur: woah?! they've added a new field to gerrit13:42
dtantsurwhere? Oo13:42
sambettsif you just go to the home page there are 4 fields for each patch now, one called Rollcall-vote13:43
dtantsursambetts, I suspect that's because you've just submitted a patch to a repo that has it13:43
dtantsursambetts, e.g. governance?13:43
sambettsdtantsur: ooh maybe, but then its showing up in the my reviews page for all of them :/13:44
dtantsursambetts, yeah, the same happens with gertty13:45
sambettsdtantsur: wow, I have never seen that before13:47
*** zigo has quit IRC13:47
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Clean nodes stuck in DEPLOYING state when ir-cond restarts  https://review.openstack.org/19413213:51
*** r-daneel has joined #openstack-ironic13:52
*** saripurigopi has joined #openstack-ironic13:55
*** e0ne is now known as e0ne_13:57
*** e0ne_ is now known as e0ne13:57
*** zigo has joined #openstack-ironic14:05
openstackgerritYuiko Takada proposed openstack/python-ironic-inspector-client: Change to Capital letters  https://review.openstack.org/19392414:09
*** beekneemech is now known as bnemec14:13
jrollmorning all :)14:25
dtantsurmorning jroll14:26
jrollheya dtantsur!14:26
*** marzif_ has quit IRC14:39
*** Nisha has joined #openstack-ironic14:39
*** rameshg87 has joined #openstack-ironic14:39
*** marzif_ has joined #openstack-ironic14:39
sambettsMorning jroll o/14:40
jrollmorning sambetts :)14:40
TheJuliarameshg87: I think it would be possible with the pass-through capability that comes with dynamic inventory, it would really be a matter of enabling the driver... once it supports standalone mode :)14:42
rameshg87TheJulia: okay, just want to make sure ilo driver has everything to work on standlone mode14:44
rameshg87TheJulia: last time we missed it when pxe driver was changed14:44
TheJuliarameshg87: exactly.  With regards to the spec, I think it would help explain the mechanisms that leverage the webserver.  Seems a little weird for auto iso creation as well since a user should just be able to have a pre-built image14:46
rameshg87yaeah14:47
rameshg87okay.14:47
TheJuliarameshg87: I think, if I was doing a large deployment, I might already have the webserver capabilitys built out, but without knowing the use mechanism I wouldn't know if it would help me or not14:48
TheJulia(guess it would be a good idea for me to go read the virtual media code)14:48
*** jlvillal has joined #openstack-ironic14:49
rameshg87TheJulia: yeah, basically the webserver will be used in the same way for current ipxe14:50
*** absubram has joined #openstack-ironic14:50
rameshg87TheJulia: to host intermediate files, and one addition from that is to host the generated boot iso files to boot the instance. let me know if you need some help from that part in virtual media driver.14:50
* rameshg87 should go and checkout bifrost as well14:50
rameshg87anyways we have time until Nishas code is merged :)14:51
*** jlvillal has quit IRC14:51
*** jlvillal has joined #openstack-ironic14:52
TheJuliaOkay, that helps context wise, its what I thoguht, but I do really need to skim through the driver.  Maybe later today :)14:52
*** saripurigopi has quit IRC14:54
rameshg87TheJulia: ack14:54
*** marzif has quit IRC14:55
*** pelix has quit IRC14:56
*** jcoufal has joined #openstack-ironic15:01
*** wendar_ is now known as wendar15:01
openstackgerritMerged openstack/bifrost: Change job logging such that successful jobs save logs  https://review.openstack.org/19311415:03
NobodyCammorning from the person who is not here15:04
openstackgerritMerged openstack/ironic: Add disk layout check on re-provisioning  https://review.openstack.org/18001415:04
jrollmorning NobodyCam :P15:04
openstackgerritYuiko Takada proposed openstack/python-ironic-inspector-client: Change to Capital letters  https://review.openstack.org/19392415:05
dtantsurNobodyCam, morning!15:05
*** mtanino has joined #openstack-ironic15:05
sambettsMorning NobodyCam o/15:05
*** davideagnello has joined #openstack-ironic15:07
*** mjturek1 has joined #openstack-ironic15:07
TheJuliagood morning NobodyCam15:07
TheJuliaNobodyCam: you have to say hello, it is in the rules ;)15:07
NobodyCammorning dtantsur,TheJulia sambetts :)15:11
NobodyCamhehehehe15:11
BadCubmorning folks.15:12
openstackgerritRamakrishnan G proposed openstack/ironic: Refactor method to add or update capability string  https://review.openstack.org/16245115:13
sambettsMorning BadCub15:13
BadCubhiya sambetts15:13
*** rameshg87 has quit IRC15:13
lucasagomesTheJulia, sambetts jroll NobodyCam BadCub morning15:17
jrolllucasagomes: morning! I was just going to come looking for you15:17
dtantsurBadCub, o/15:18
* TheJulia wonders what this channel's morning greeting stats are....15:18
lucasagomeslol \o/15:18
BadCubhiya lucasagomes dtantsur :)15:18
*** rloo has joined #openstack-ironic15:18
*** yuikotakada has quit IRC15:18
jrolllucasagomes: I'm updating https://review.openstack.org/#/c/187829/3/specs/liberty/network-provider.rst and wondering if maybe the driver should just be per node15:18
devanandag'morning, all15:19
lucasagomesjroll, right, it sounds "easier" tho ports having it is possible and more fine grained15:19
BadCubmorning devananda15:19
trowngood morning devananda15:19
lucasagomesI dunno if it's realistic tho15:19
lucasagomesdevananda, morning15:19
trownthanks for your response on ahc-tools15:19
jrolllucasagomes: well thing is there will be ports and then port groups for aggregated connections15:19
jrolllucasagomes: and I'd like the driver api to be per-node or else it might get complex calling it, idk15:20
rloohello & mornin' devananda, jroll, BadCub, lucasagomes, trown15:20
jrollmorning BadCub trown TheJulia and rloo :)15:20
jrolloh and of course deva ^^15:20
BadCubhiya rloo jroll :)15:21
lucasagomesjroll, oh... right, yeah... specially if all ports in the same group needs to have the same driver (does it right?)15:21
trowngood UGT morning jroll BadCub lucasagomes TheJulia rloo15:21
lucasagomesrloo, trown good ugt morning15:21
BadCubhiya trown :)15:21
jrolllucasagomes: yeah, I think it would, I'm just going to do this to keep it simple15:21
jrollthis being the per node thing15:21
lucasagomesjroll, fair enuff15:21
trownTheJulia: are morning greeting stats kept somewhere?15:22
trownI think we would certainly win15:22
jrolllol15:22
jrolltrown: maybe if you dig hard enough in http://activity.openstack.org/dash/browser/15:23
TheJuliatrown: I don't know... it would be amusing to study them15:23
jrollthe data is there, perhaps you can add it to the dashboard :P15:23
trownhaha15:24
* rloo cringes at the thought of it15:24
trownjroll: that is a cool dashboard, I had not seen that15:24
jrollyeah, I learned about it in vancouver15:24
*** keekz_ is now known as keekz15:24
*** pelix has joined #openstack-ironic15:29
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Update network provider spec  https://review.openstack.org/18782915:30
jrollwheeeeeeeeee15:30
openstackgerritMerged openstack/python-ironic-inspector-client: Change to Capital letters  https://review.openstack.org/19392415:32
dtantsurfolks, our bug dashboard got some love finally: http://ironic-bugs.divius.net/15:35
dtantsurand g'morning rloo, devananda!15:36
*** Guest81202 is now known as mgagne15:36
*** mgagne has joined #openstack-ironic15:36
rloohiya dtantsur!15:36
dtantsurdevananda, everyone wants their code under Ironic umbrella, can I suggest moving the bug dashboard in? :D15:36
jrolllol15:36
devanandahaha :)15:36
devanandadtantsur: also - where is the code for your dashboard now?15:37
dtantsurdevananda, https://github.com/divius/ironic-bug-dashboard but please don't look at it, it's BAD15:37
* dtantsur is ashamed actually15:37
* devananda looks15:38
dtantsuroh no15:38
* dtantsur runs away15:38
devanandaaaaaah my eyes! (j/k)15:38
rloodtantsur, if you had asked us to review it, no one would have looked :)15:38
trownhaha15:38
dtantsurc'mon folks, everyone must write shitty code at least once a year15:39
trowndtantsur: my laughing was at rloo's comment15:40
dtantsurnot to forget this wonderful feeling of not giving a **** about quality, tests, proper design and other boring stuff15:40
dtantsur:D15:40
*** Sukhdev has joined #openstack-ironic15:40
*** marzif_ has quit IRC15:41
*** marzif_ has joined #openstack-ironic15:41
jrolldtantsur: wait, we don't do that every day?15:42
lucasagomesdtantsur, hah, I hope it doesn't become a ironic-bug(gy)-dashboard15:42
jroll:P15:43
dtantsurLOL15:44
dtantsurlucasagomes, the only time it got broken was due to nice situation with unicode in Py215:44
dtantsurafter we got non-ASCII in the bug title or someone's name...15:44
lucasagomesdtantsur, nice, more stable than the glibc then15:44
jrolldevananda: "if ironic provided a blob store for each node..." interesting, we were just talking about this internally the other day for firmware-y things15:46
devananda:)15:46
jrollwe've also talked about doing benchmarking during cleaning in the past15:47
jrollvia the agent15:47
lucasagomesjroll, yeah, before a node is scheduled as well. So you can use the information for scheduling... e.g storage nodes disks >= X iops15:49
jrolllucasagomes: yeah, that's clearly the next step once you have the data15:50
*** romcheg has quit IRC15:50
*** yog_ has quit IRC15:51
dtantsurjroll, so yeah, that's what we're aiming for15:52
jrolldtantsur: cool. my thoughts around it were more of "all the hardware is still here and still works" :)15:53
jrollbut I welcome our highly targeted scheduling overlords15:53
*** cinerama` is now known as cinerama15:53
lucasagomes+1 both15:53
dtantsuryep, I like both things, I want to make them possible this cycle15:53
lucasagomesactually benchmarking it after to see if the hardware still conformant15:53
jrollyeah15:54
lucasagomesconformant*15:54
jrollthe root disks we currently use degrade to become slower before they die completely15:54
jrollfor example15:54
lucasagomesright, yeah... would be good to have a task for it and even being able to detect such things15:55
trownjroll: ya so you could have a threshold where you knew they were close to dying and could possibly prevent them from dying while a tenant had the machine15:55
lucasagomesand somehow inform the operator15:55
jrolltrown: indeed :)15:56
lucasagomes(hey, a disk on this machine died *set* maintenance)15:56
jrollyep +215:56
* jroll tries to remember to reply to that thread later15:56
jrollfriendly reminder ironic/neutron meeting in 3 minutes for anyone interested15:56
jroll#openstack-meeting-415:57
devanandajroll: i'll lurk, but again have another meeting conflict15:57
sambettsthanks for the reminder jroll :)15:57
jrolldevananda: yeah no worries. at this point going to start needing ironic people reviewing specs15:58
*** lazy_prince has joined #openstack-ironic16:00
* lucasagomes *joins*16:01
lucasagomesdevananda, if you have a min later on https://review.openstack.org/#/c/194132/ (this helps mitigate that problem with the node stuck on DEPLOYING)16:03
*** romcheg has joined #openstack-ironic16:03
*** jcoufal has quit IRC16:09
*** e0ne has quit IRC16:11
*** pradipta has quit IRC16:12
*** e0ne has joined #openstack-ironic16:13
*** viktors has left #openstack-ironic16:15
rloohey devananda, wrt 'Bare metal service' vs 'Bare Metal service'. did you see https://review.openstack.org/#/c/191900/?16:17
rloodevananda: makes me so sad16:17
dtantsurrloo, I don't like the whole s/Ironic/ironic/ thing16:18
*** david-ly_ is now known as david-lyle16:18
dtantsurit's a name16:18
rloodtantsur: I don't even understand their description of when to Capitalize or not, and I don't want to get into a discussion with them about that16:19
lucasagomesI don't like it either (as per my comment) but I'm trusting the docs folks to do it16:19
rloodtantsur: but the 'Bare metal' vs 'Bare Metal'. Now I think that is grammatically incorrect.16:19
trownhuh, I thought all the services were capitalized...TIL16:19
*** pleia2_ is now known as pleia216:20
rlooit makes me sad cuz I would hope that someone could say to me 'Bare metal service' is grammatically correct vs 'well, it sez here to use this...'16:20
dtantsurI wonder how people are supposed to read texts with "ironic inspector" not capitalized...16:20
lucasagomesbtw https://wiki.openstack.org/wiki/Documentation/Conventions#Service_and_project_names16:22
lucasagomesit says "Bare metal"16:22
SpamapSTheJulia: hi. I have to run my car off to a service appointment in a bit, but when I get back, I want to discuss whether you thihnk we should pursuse the mellanox insanity, or aggressively replace our mellanox with sane intel nics.16:22
SpamapSgreghaynes: ^16:22
jrollhey y'all... https://review.openstack.org/#/c/187829 has general buyoff from ironic/neutron subteam peoples. would love some reviews this week.16:22
dtantsurlucasagomes, I wonder if the author of this "text" wants his/her name lower case as well16:22
rloolucasagomes: yes, it does say that. but c'mon folks, think about it.16:22
greghaynesSpamapS: Hey16:23
trownrloo: I agree with you...Bare metal looks weird16:23
dtantsurrloo, I'm close to going and -1ing all the patches based on "my English teacher disagree with you"16:23
lucasagomesrloo, dtantsur right... Well the thing is we are kinda delegating it to the docs people no?16:23
rloolucasagomes: had a quick discussion with devananda last thurs and he agreed that it should be 'Bare Metal'. I think we need to update the governance thingy first though.16:23
lucasagomesright16:23
dtantsurlucasagomes, our common sense? I do not.16:23
lucasagomeswell, right, but that's the whole point engineers vs doc writers16:24
dtantsurlucasagomes, no, it's language grammar16:24
rloolucasagomes, dtantsur: ok, so I think the thing-to-do is find out from devananda how the governance thing gets updated. if there's a patch for that, we can -2 this patch until that patch goes through.16:24
SpamapSgreghaynes: interested in your thoughts on the NIC thing too16:24
lucasagomesrloo, right16:24
dtantsurlucasagomes, names are capitalized in most langauges I'm aware of16:24
greghaynesSpamapS: Do we know how the nic's are setup in the box? one big worry is will replacing the NIC's remove the bit of info we use to correlate HP cloud info with the info we get off the box16:25
mariojv-hi all16:25
SpamapSgreghaynes: we could of course just abandon ipxe and do pxe the way we did before.16:25
rloo'we' may have made the mistake when we submitted whatever that got into http://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml16:25
lucasagomesrloo, dtantsur another thing that makes me a bit ew about that patch is the OpenStack Networking service16:25
lucasagomessince we have 2 ?!16:25
SpamapSgreghaynes: the boxes have serial numbers. We don't use MACs to coordinate with the NOC16:25
lucasagomesnova network and neutron16:25
dtantsurrloo, lucasagomes, the only reason I didn't -1ed all the stuff is because I have enough rage-against-guidelines talks already :D16:25
greghaynesSpamapS: Yes, but we cant tie a serial number with hardware we talk to without some other bit of info16:25
mariojv-quick question about general best practices for a new openstack contributor. is it usually worth it to continue work on a spec that has been -2ed? i would significantly revise the spec16:25
jrolllucasagomes: we're 100% incompatible with nova-network right?16:26
SpamapSgreghaynes: errr, sure we can. The serial # is visible in ILO16:26
jrollmariojv-: depends on the spec? :)16:26
rloolucasagomes: my guess is that when networks were in nova, it was part of Nova Compute?16:26
dtantsurjroll, I believe so16:26
lucasagomesjroll, yes, that's why I wanted the docs to say Neutorn instead of OpenStack Networking16:26
lucasagomesor at least put a (Neutron)16:26
lucasagomesin front of that16:26
mariojv-jroll: ok - just wanted to know if that was something that really shouldn't be done. thanks16:26
SpamapSgreghaynes: that serial # is what has a rack# and thus can be serviced.16:26
jrollmariojv-: generally was it -2'd for "we don't ever want this" or?16:26
rloolucasagomes: and that when referring to neutron, it is 'OpenStack Networking'16:26
SpamapSgreghaynes: maybe I'm missing what you are concerned about?16:26
jrollmariojv-: I could probably tell you if you link16:26
mariojv-jroll: for not seeing benefit16:26
mariojv-sure16:26
lucasagomesrloo, the thing is the network bits still in nova16:27
mariojv-https://review.openstack.org/#/c/18860916:27
lucasagomesand it's not deprecated16:27
mariojv-jroll: ^16:27
greghaynesSpamapS: Yes, then we need to tie ilo with actual hardware which would mean figuring out that tool to hit the ilo from linux which may or may not be easier than just correlating MAC addresses16:27
rloolucasagomes: in the documentation, when OpenStack Networking is used, is it referring to neutron, nova network, or both?16:27
lucasagomesrloo, for us always Neutron16:27
rloolucasagomes: so in our case, it is correct to use OpenStack Networking.16:28
lucasagomesrloo, but since now they only write OpenStack Networking it sounds a bit confusing... I wonder if someone will confuse it16:28
jrollmariojv-: sounds like you should probably: 1) chat with devananda; 2) keep working on it; 3) IMO and sounds like deva's, drop the CONF storage stuff. baby steps. :)16:28
lucasagomesrloo, oh sorry i think I misread ur question16:28
mariojv-alright, thanks16:28
rloolucasagomes: i'm not quite sure what they use when something can be configured with either nova networking or neutron, but we don't have that problem16:28
lucasagomesrloo, so I don't know, outside Ironci if OpenStack Networking is used I don't know whether it's neutron or nova network16:28
lucasagomesperhaps it's only neutron... /me will dig16:29
rloolucasagomes: I *think* it should == neutron. cuz anything in nova is part of 'OpenStack Compute'.16:29
lucasagomesyeah16:29
TheJuliaSpamapS: yes yes, ping me when you get back16:30
jrollrloo: lucasagomes: I think nova-network is considered part of "openstack networking"16:30
lucasagomesjroll, *sigh*16:31
rloojroll: me so sad. again.16:31
SpamapSgreghaynes: they're not going to replace the NICs without giving us the new MAC's16:31
lucasagomesso that docs can be very confusing... I did suggest calling it "OpenStack Networking (neutron)" on that docs on a previous review I will comment again16:32
SpamapSgreghaynes: so I think your concerns are valid, and we should be sure that we get the new MAC<->SERIAL mapping.16:32
rloolucasagomes: did you ask them how they distinguish neutron from nova networking?16:32
SpamapSgreghaynes: unless we change our minds and just plod forward with the mellanoxes. ;)16:32
lucasagomesrloo, yup https://review.openstack.org/#/c/191900/4/doc/source/deploy/install-guide.rst L37416:33
lucasagomesbut no answer16:33
jrollrloo: lucasagomes: we should point out to them, in case it's not clear, that we only support neutron and not nova-net16:33
lucasagomesI will add the comment again on the new patch set16:33
greghaynesSpamapS: yea, fair enough16:33
rloothx lucasagomes16:34
*** dguerri is now known as dguerri`16:42
rloodevananda: when you get the chance, would you please give your ack on this: https://review.openstack.org/#/c/194230/16:45
*** rwsu has joined #openstack-ironic16:45
rloolucasagomes, dtantsur: ^^ change to 'Bare Metal' :)16:45
lucasagomesrloo, hah w00t16:46
trownrloo, nice one16:46
*** teju has joined #openstack-ironic16:46
* lucasagomes add his name as reviwer, grabs pop corn16:47
*** derekh has quit IRC16:55
*** pelix has quit IRC16:57
*** lazy_prince has quit IRC16:58
*** cinerama has quit IRC16:58
*** cinerama has joined #openstack-ironic17:00
*** teju has left #openstack-ironic17:00
lucasagomesfolks I will call it a day17:01
lucasagomeshave a good night everyone!17:01
*** lucasagomes is now known as lucas-dinner17:02
dtantsuryeah, g'night from me as well :)17:02
*** dtantsur is now known as dtantsur|afk17:03
*** e0ne has quit IRC17:05
*** rwsu has quit IRC17:19
*** mitchjameson has joined #openstack-ironic17:19
*** trown is now known as trown|lunch17:22
*** jjohnson2 has joined #openstack-ironic17:25
*** harlowja has joined #openstack-ironic17:28
devanandarloo: I hvae replied17:30
devanandalucas-dinner: enjoy the show ;)17:30
rloodevananda: thx. but I'm thinking b*, no way am I updating non-Ironic docs :)17:31
devanandahehehe17:31
*** ifarkas has quit IRC17:31
rloodevananda: otherwise, great reply!17:31
devanandarloo: i'm sure we can find someone else to do it17:31
rloodevananda: I'm quite positive we can. I think some people like to do such changes...17:31
harlowjadtantsur|afk u want to be in futurist-core, now that u have some knowledge of it?17:32
*** ukalifon1 has joined #openstack-ironic17:33
*** spandhe has joined #openstack-ironic17:34
*** rwsu has joined #openstack-ironic17:36
*** marzif_ has quit IRC17:39
*** Sukhdev has quit IRC17:40
*** Sukhdev has joined #openstack-ironic17:43
*** ijw_ has quit IRC17:46
*** achanda has joined #openstack-ironic17:54
*** amotoki has quit IRC18:00
sambettsNight all o/18:02
TheJuliaGoodnight!18:03
*** ukalifon1 has quit IRC18:04
*** romainh has joined #openstack-ironic18:05
*** natorious is now known as zz_natorious18:08
harlowjarloo i prefer it called Bare-Metal, like in http://www.lemans1970-was-sonst.de/pics/baremetal_002.jpg18:08
harlowjalol18:08
rlooharlowja: submit your own patch then!18:09
harlowja:(18:09
rlooharlowja: We google'd it, and it seems like 'bare metal' is the correct usage. Or more frequent usage anyway.18:09
*** ukalifon has joined #openstack-ironic18:09
harlowjaor all caps? like in http://buybaremetal.com//wp-content/uploads/2013/12/Bare-Metal-Standard-Midwest-truck.jpg18:10
rlooharlowja: really, I think folks want Bear-metal :D18:10
harlowjalol18:10
*** e0ne has joined #openstack-ironic18:12
*** romcheg has quit IRC18:12
openstackgerritMario Villaplana proposed openstack/ironic-specs: Adds spec for creating a key manager with Barbican to Ironic  https://review.openstack.org/18860918:16
SpamapSTheJulia: back at my desk now18:17
TheJuliawoot, so I noticed something in the packet capture you sent me18:21
TheJuliait says it is gPXE in one of the informational fields that we can match18:21
TheJuliaSpamapS: https://review.openstack.org/#/c/193385/6/playbooks/roles/ironic-install/templates/dnsmasq.conf.j218:22
*** e0ne is now known as e0ne_18:23
*** e0ne_ is now known as e0ne18:24
*** Sukhdev has quit IRC18:24
*** trown|lunch is now known as trown18:26
SpamapSTheJulia: ah! Ok, I'll try that patch next18:27
* TheJulia crosses her fingers18:27
openstackgerritStephanie Miller proposed openstack/bifrost: Refactor ironic-install playbook to support CentOS & RHEL  https://review.openstack.org/19374218:28
*** romcheg has joined #openstack-ironic18:29
openstackgerritStephanie Miller proposed openstack/bifrost: Refactor ironic-install playbook to support CentOS & RHEL  https://review.openstack.org/19374218:32
openstackgerritMerged openstack/ironic: Finish switch to inspector and inspector-client  https://review.openstack.org/19317618:40
*** ukalifon has quit IRC18:40
openstackgerritMario Villaplana proposed openstack/ironic-specs: Adds spec for creating a key manager with Barbican to Ironic  https://review.openstack.org/18860918:49
*** ijw_ has joined #openstack-ironic19:01
*** e0ne has quit IRC19:04
*** e0ne has joined #openstack-ironic19:05
*** ijw__ has joined #openstack-ironic19:11
*** ijw_ has quit IRC19:13
*** Nisha has quit IRC19:14
*** rwsu has quit IRC19:15
*** romcheg has quit IRC19:18
*** romcheg has joined #openstack-ironic19:22
*** romcheg has quit IRC19:22
*** romcheg has joined #openstack-ironic19:22
*** Nisha has joined #openstack-ironic19:25
*** jgrimm is now known as zz_jgrimm19:26
TheJuliaSpamapS: any luck?19:35
*** e0ne is now known as e0ne_19:35
*** e0ne_ is now known as e0ne19:35
*** ijw_ has joined #openstack-ironic19:37
*** achanda has quit IRC19:37
*** rwsu has joined #openstack-ironic19:38
*** marzif_ has joined #openstack-ironic19:39
*** rwsu has quit IRC19:40
*** rwsu has joined #openstack-ironic19:40
*** ijw__ has quit IRC19:40
*** lucas-dinner has quit IRC19:43
*** mgarza has joined #openstack-ironic19:47
SpamapSTheJulia: sorry, been pulled in a few different directions and haven't been able to insert myself into the evils of ilo yet.19:51
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: Add spec for user-settable Node affinity  https://review.openstack.org/19326319:52
TheJuliano worries19:52
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/19432019:53
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/19432119:54
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/19398619:54
*** lucas-dinner has joined #openstack-ironic19:56
*** lauramoore has joined #openstack-ironic20:00
*** marzif_ has quit IRC20:07
*** rwsu has quit IRC20:10
*** romcheg1 has joined #openstack-ironic20:12
*** romcheg has quit IRC20:13
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/19432020:13
*** Marga_ has quit IRC20:15
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/19434720:23
*** Sukhdev has joined #openstack-ironic20:27
*** Sukhdev has quit IRC20:33
*** Sukhdev has joined #openstack-ironic20:36
*** dprince has quit IRC20:38
*** Marga_ has joined #openstack-ironic20:43
*** Marga_ has quit IRC20:47
*** Sukhdev_ has joined #openstack-ironic20:49
*** Sukhdev__ has joined #openstack-ironic20:52
*** Sukhdev has quit IRC20:52
*** Sukhdev_ has quit IRC20:56
*** zz_natorious is now known as natorious21:01
*** trown is now known as trown|outttypeww21:02
*** e0ne has quit IRC21:03
*** jjohnson2 has quit IRC21:04
*** e0ne has joined #openstack-ironic21:06
jrollSukhdev__: remind me, is bind_requested in the binding profile or in the top level or the port-create request?21:06
Sukhdev__jroll: this will be part of the binding profile21:07
jrollah ok, thanks21:08
Sukhdev__jroll: we will use host-id, which is at top level, to set it to "-Irnonic-" so that ML2 drivers can filter on ironic ports21:08
jrollSukhdev__: but we're also using bind_requested yes?21:09
Sukhdev__jroll: yes21:09
*** e0ne has quit IRC21:09
*** rwsu has joined #openstack-ironic21:10
jrollSukhdev__: thanks21:10
jrollSukhdev__: looks like host_id is in the binding profile as well21:13
jrollSukhdev__: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L72721:13
Sukhdev__jroll: no, it is part port binding - which contains binding profile21:13
jrollSukhdev__: oops, you're right, my bad21:14
jrollso that code needs to change as well eh21:14
* jroll does it21:14
Sukhdev__jroll: give me a sec  - in a meeting - will ping you in a bit21:14
jrollSukhdev__: ok, this isn't a huge problem except that I don't see this called out in a spec anywhere21:15
openstackgerritJohn L. Villalovos proposed openstack/ironic: API: Get a subset of fields from Nodes  https://review.openstack.org/18349121:17
*** marzif_ has joined #openstack-ironic21:20
*** absubram has quit IRC21:24
*** romainh has quit IRC21:37
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/19434721:37
*** Sukhdev__ has quit IRC21:40
*** Sukhdev has joined #openstack-ironic21:41
*** zigo has quit IRC21:43
Sukhdevjroll; I am back -21:44
jrollhi Sukhdev21:46
Sukhdevjroll: So, what we are saying is that we will use everything as-is (as called out in spec) + additionally, we will use host-id as well21:46
jrollSukhdev: ok, so we'll need to define what we want in host_id, and change the nova code to put it there21:46
Sukhdevjroll: I am assuming when lauramoore updates the spec she will add it there21:46
Sukhdevjroll: correct -21:46
jrollSukhdev: ok, so I want to change the nova code right now. what goes in host_id? :)21:47
Sukhdevjroll: my suggestion is that we use "-ironinc-"21:47
jrollSukhdev: seems like it might be valuable to have the ironic node id or something there no?21:47
Sukhdevjroll: note the "-" in front and end of ironic21:47
jrollSukhdev: I'd suggest "ironic-{node_id}"21:48
Sukhdevjroll: you can put anything you want - be sure to have leading "-" and trailing "-" - this ensures we will always be unique21:48
*** zigo has joined #openstack-ironic21:49
Sukhdevjroll: that will be fine, except, it will be very long - but, thats OK -21:49
jrollSukhdev: I think filtering ports by ironic node id will be super useful21:50
Sukhdevjroll: however, be sure to add leading - and trailing -21:50
jrollmorganfainberg: ^ wdyt21:50
jrollSukhdev: how does a - make it unique?21:50
Sukhdevjroll: since host-id is used to specify the name of a host - what if somebody (accidentally) uses the same name?21:51
*** romainh has joined #openstack-ironic21:51
jrollSukhdev: accidentally use "ironic-5c80dd47-05bd-43d2-8e8c-0f6795bbe4e1" for a virtual machine compute host?21:51
jrollSukhdev: they could also use "-ironic-5c80dd47-05bd-43d2-8e8c-0f6795bbe4e1-"21:51
Sukhdevjroll: but. leading and trailing "-" is prohibited by RFC, this will make us unique21:52
*** kevinbenton has quit IRC21:52
jrollSukhdev: there's no RFC for nova-compute host names :)21:52
Sukhdevjroll: you are probably OK - with that - when I was proposing "Ironic" then it is more likely21:53
jrollyeah21:53
Sukhdevjroll: actually there is - let me look it up21:53
jrollSukhdev: so on the ironic side we'll need to fetch a list of ports sometimes - having node id in host_id field will be useful21:54
Sukhdevjroll: RFC 112321:54
*** kevinbenton has joined #openstack-ironic21:54
*** TheJulia has quit IRC21:55
*** TheJulia has joined #openstack-ironic21:55
Sukhdevjroll: I think ironic+node-id is unique enough, but, to avoid paranoia we could throw in dashes, if it becomes an issue. I personally do not care really21:55
jrollSukhdev: nova-compute host name doesn't have to be RFC compliant: https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L4821:55
jrollit's just a string21:55
*** mtanino has quit IRC21:57
Sukhdevjroll: I think there is a bit of confusion - nova can allow it, but, the real hosts can not have that name - hence, an added level of protection21:57
jrollSukhdev: sure, but that isn't a guarantee. anyway :)21:58
Sukhdevjroll: try setting a host name on a real machine with leading and trailing dashes - should be reject21:58
Sukhdevrejected21:58
jrollright... but nova doesn't necessarily tie those two together21:58
jrollcompute host name can be different than computer hostname21:58
jrollit's a moot point, anyway, I think this will be fine21:58
Sukhdevjroll: I think will concede to it for now - if the need be, we can alway use it21:59
Sukhdevjroll: lets go with ironic+node-id - this is good22:00
jrollyep, will do22:00
Sukhdevjroll: one crazy thought about it - filtering may not be very efficient22:02
jrollSukhdev: yeah, we'll need to think about it22:02
Sukhdevjroll: filtering will work, but, may be bit inefficient - if there are thousands of nodes/ports22:03
*** mgarza has quit IRC22:04
jrollSukhdev: indeed22:05
*** lucas-dinner has quit IRC22:08
*** kklimonda has quit IRC22:09
mrdaMorning Ironic22:10
jrollSukhdev: https://review.openstack.org/#/c/186855 and https://review.openstack.org/#/c/194413/ is what I see this looking like22:11
jrolllauramoore: ^^22:11
Sukhdevjroll: let me review - give me a sec22:11
jrollyeah, no rush, just a first shot22:13
*** zigo has quit IRC22:15
*** marzif__ has joined #openstack-ironic22:16
*** zigo has joined #openstack-ironic22:17
*** marzif_ has quit IRC22:19
*** marzif__ has quit IRC22:21
Sukhdevjroll: I had a similar idea in mind - https://review.openstack.org/#/c/179255/6/nova/network/neutronv2/api.py (from my old WIP patch) after line 635 - simply add port_req_body['port']['host-id] = bind_host_id22:21
*** romcheg1 has quit IRC22:22
jrollSukhdev: yeah, seems similar22:22
Sukhdevjroll: correct22:23
Sukhdevjroll: So, are you proceeding with a spec for nova or without it?22:23
jrollSukhdev: talked with nova folks, sounds like as long as we have good commit messages, pointers to ironic spec, etc, we don't need one22:24
Sukhdevjroll: oh cool - that is great22:24
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/19442122:35
krotscheckI have been told to do this: https://review.openstack.org/19442422:37
*** radek__ has quit IRC22:44
*** thrash is now known as thrash|g0ne22:57
*** romainh has left #openstack-ironic22:59
*** rwsu has quit IRC23:00
*** egon_ is now known as egon23:10
morganfainbergjroll: looking at the scrollback now23:12
morganfainbergsorry on the delay, was doing lunch/errands23:12
jrollmorganfainberg: ?23:12
jrollkrotscheck: woot23:12
*** EntropyWorks has joined #openstack-ironic23:12
jrollmrda: morning, I totally missed you there :)23:13
morganfainbergjroll: you pinged me earlier23:13
jrollmorganfainberg: if I did I didn't mean to :)23:13
morganfainberg<jroll>morganfainberg: ^ wdyt23:13
jrolloooops23:13
morganfainbergjroll: i was going to ask what that meant to me :P23:13
jrollyeah that should have been morgabra23:13
morganfainbergahahah23:13
*** ndipanov has quit IRC23:13
jrollunless you have opinions on networks :)23:13
morganfainbergall good23:13
morganfainbergnope23:13
jrollheh23:14
jrollmy bad23:14
* morganfainberg hides back under the rock23:14
mrdahey jroll23:16
EntropyWorksI'm wondering how far can I test Ironic using devstack as the starting point. would I be able to send IPMI commands or call VirtualBox VBoxManager to simulate provisioning a bare metal server?23:16
jrollEntropyWorks: yeah, the *_ssh drivers allow you to sub in vbox/virsh/etc23:16
jrollEntropyWorks: default devstack uses virsh23:16
EntropyWorksinteresting. I'll have to take a look at that driver then. thanks.23:17
jrollyep, np23:19
jrollyou can also set it up with ipmi and real hardware, but idk if that's all automated23:19
EntropyWorksI'd love to show someone Ironic and the steps to provision some VM's in place of real hardware. Populating Ironic to know that 3 servers are available. Have Virtualbox boot maybe an empty VM which then actually is seen by Ironic and added to its inventory.23:24
openstackgerritMichael Davies proposed openstack/python-ironicclient: Cache negotiated api microversion for server  https://review.openstack.org/17367423:25
jrollEntropyWorks: yep, vbox can totally do that for you. set IRONIC_SSH_VIRT_TYPE=vbox in localrc, and IRONIC_VM_SSH_ADDRESS=your.virtualbox.host.ip and it should just work (tm)23:27
*** yuanying has joined #openstack-ironic23:29
*** mitchjameson has quit IRC23:38
*** mitchjameson has joined #openstack-ironic23:38
*** Nisha_away has joined #openstack-ironic23:40
*** Sukhdev has quit IRC23:41
*** Nisha_away has quit IRC23:42
*** ijw__ has joined #openstack-ironic23:43
*** Nisha has quit IRC23:43
*** Nisha has joined #openstack-ironic23:44
*** athomas has quit IRC23:44
devanandaEntropyWorks: see our quickstart guide yet?23:45
*** yuikotakada has joined #openstack-ironic23:46
*** ijw_ has quit IRC23:47
*** ijw_ has joined #openstack-ironic23:53
*** ijw__ has quit IRC23:56

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