Tuesday, 2016-05-24

*** jhesketh_ is now known as jhesketh00:02
*** alexpilotti has quit IRC00:02
*** rama_y has quit IRC00:05
*** mtanino has joined #openstack-ironic00:08
*** moshele has joined #openstack-ironic00:09
*** clif_h has joined #openstack-ironic00:19
*** mrda has joined #openstack-ironic00:19
*** devananda has joined #openstack-ironic00:19
*** ppiela has joined #openstack-ironic00:19
*** d0ugal has joined #openstack-ironic00:19
*** PixieBoots has joined #openstack-ironic00:19
*** vdrok has joined #openstack-ironic00:20
*** patrickeast has joined #openstack-ironic00:20
*** agordeev has joined #openstack-ironic00:20
*** ildikov has joined #openstack-ironic00:21
*** BadCub has joined #openstack-ironic00:21
openstackgerritMerged openstack/ironic: Joined 'tags' column while getting node  https://review.openstack.org/25306500:23
*** rbudden has quit IRC00:29
openstackgerritVincent S. Cojot proposed openstack/ironic: Make the vbox driver work on headless VirtualBox machines  https://review.openstack.org/31127800:32
*** baoli has quit IRC00:32
*** baoli has joined #openstack-ironic00:33
*** Sukhdev has quit IRC00:33
*** baoli has quit IRC00:33
*** baoli has joined #openstack-ironic00:34
*** hoangcx has joined #openstack-ironic00:42
*** YiboCai has joined #openstack-ironic00:43
*** moshele has quit IRC00:44
*** vishwanathj has quit IRC00:50
*** sergek has quit IRC00:53
*** Sukhdev has joined #openstack-ironic00:54
*** mbound has joined #openstack-ironic00:55
*** sergek has joined #openstack-ironic00:55
*** mbound has quit IRC00:59
*** mtanino has quit IRC01:05
openstackgerritVincent S. Cojot proposed openstack/ironic: Make the vbox driver work on headless VirtualBox machines  https://review.openstack.org/31127801:29
*** mtanino has joined #openstack-ironic01:30
*** YiboCai has quit IRC01:30
*** YiboCai has joined #openstack-ironic01:31
*** praneshp has quit IRC01:32
*** mtanino has quit IRC01:35
ElCoyote__jroll: I dropped the unittests because I couldn't get them to work.01:38
*** Sukhdev has quit IRC01:40
*** jaybeale has quit IRC01:44
*** jaybeale has joined #openstack-ironic01:45
*** jaybeale has quit IRC01:50
*** Nisha_away has joined #openstack-ironic01:58
*** adu has joined #openstack-ironic02:01
*** Nisha_away has quit IRC02:02
openstackgerritJim Rollenhagen proposed openstack/ironic: Make the vbox driver work on headless VirtualBox machines  https://review.openstack.org/31127802:13
jrollElCoyote__: the tests are important so I fixed them up ^^02:13
* jroll nn02:13
*** penick has quit IRC02:15
*** sacharya has joined #openstack-ironic02:19
*** hoonetorg has quit IRC02:20
*** hoonetorg has joined #openstack-ironic02:20
*** praneshp has joined #openstack-ironic02:28
*** adu has quit IRC02:44
*** ChrisAusten has joined #openstack-ironic02:49
*** adu has joined #openstack-ironic02:54
*** Sukhdev has joined #openstack-ironic03:01
*** baoli has quit IRC03:05
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/31988003:11
guestIs it possible to attach floating IP and get metadata from the metadata server at the same time? If i attach router(for floating IP) to the network then i can not recevie metadata. If i don't attach gateway then i can not attach floating IPs as well but i am able to get metadata. Is it the problem with my setup or is the expected behaviour?03:17
*** Egyptian[Home] has quit IRC03:18
*** adu has quit IRC03:29
*** sdake_ has quit IRC03:29
*** Nisha_away has joined #openstack-ironic03:36
openstackgerritTan Lin proposed openstack/ironic-specs: Tooling for recovering nodes  https://review.openstack.org/31981203:39
*** baoli has joined #openstack-ironic03:44
*** david-lyle_ has joined #openstack-ironic03:55
*** david-lyle has quit IRC03:57
*** vmud213 has joined #openstack-ironic04:00
*** vmud213 has quit IRC04:06
*** links has joined #openstack-ironic04:07
*** vmud213 has joined #openstack-ironic04:07
*** ijw has quit IRC04:17
*** sdake has joined #openstack-ironic04:17
*** sdake has quit IRC04:18
*** sivaramakrishna has joined #openstack-ironic04:26
*** absubram has joined #openstack-ironic04:31
*** absubram_ has joined #openstack-ironic04:32
*** absubram has quit IRC04:36
*** absubram_ is now known as absubram04:36
openstackgerritJohn L. Villalovos proposed openstack/ironic: Patch to test Grenade.  https://review.openstack.org/31110904:37
*** appu103_ has joined #openstack-ironic04:44
*** rama_y has joined #openstack-ironic05:02
*** sabeen1 has quit IRC05:06
openstackgerritJohn L. Villalovos proposed openstack/ironic: Patch to test Grenade.  https://review.openstack.org/31110905:07
*** causten_ has joined #openstack-ironic05:11
*** ChubYann has quit IRC05:11
*** appu103_ is now known as aparnav05:13
*** links has quit IRC05:14
*** ChrisAusten has quit IRC05:15
*** jvgrant has quit IRC05:15
*** adreznec has quit IRC05:15
*** jvgrant has joined #openstack-ironic05:15
*** dansmith has quit IRC05:16
*** baoli has quit IRC05:16
*** links has joined #openstack-ironic05:18
*** dansmith has joined #openstack-ironic05:18
*** dansmith is now known as Guest7113205:18
*** adreznec has joined #openstack-ironic05:19
*** sacharya_ has joined #openstack-ironic05:21
*** sacharya has quit IRC05:23
*** links has quit IRC05:24
*** sacharya has joined #openstack-ironic05:26
*** sacharya_ has quit IRC05:28
*** Sukhdev_ has joined #openstack-ironic05:45
*** Sukhdev has quit IRC05:46
*** amotoki has joined #openstack-ironic05:56
*** Fdaisuke has quit IRC06:00
*** Fdaisuke has joined #openstack-ironic06:03
*** ijw has joined #openstack-ironic06:03
*** links has joined #openstack-ironic06:04
*** mjura has joined #openstack-ironic06:07
*** rcernin has joined #openstack-ironic06:08
*** sdake has joined #openstack-ironic06:11
*** mjura has quit IRC06:13
*** mjura has joined #openstack-ironic06:13
*** izaakk has quit IRC06:14
*** izaakk has joined #openstack-ironic06:14
*** evgenyl has quit IRC06:15
*** morgabra has quit IRC06:15
*** jiangfei has quit IRC06:15
*** serverascode has quit IRC06:15
*** marios has quit IRC06:15
*** mgkwill has quit IRC06:15
*** zhenguo_ has quit IRC06:16
*** ChrisMead has quit IRC06:16
*** jiangfei has joined #openstack-ironic06:16
*** betherly has quit IRC06:16
*** NobodyCam has quit IRC06:16
*** serverascode has joined #openstack-ironic06:17
*** marios has joined #openstack-ironic06:17
*** ChrisMead has joined #openstack-ironic06:17
*** mgkwill has joined #openstack-ironic06:18
*** lekha-away has quit IRC06:18
*** pas-ha has quit IRC06:18
*** evgenyl has joined #openstack-ironic06:18
*** zhenguo_ has joined #openstack-ironic06:18
*** NobodyCam has joined #openstack-ironic06:19
*** betherly has joined #openstack-ironic06:19
*** pas-ha has joined #openstack-ironic06:19
*** lekha-away has joined #openstack-ironic06:20
*** akrish9 has quit IRC06:20
*** lintan has quit IRC06:21
*** cppforlife_ has quit IRC06:21
*** lintan has joined #openstack-ironic06:21
*** serverascode has quit IRC06:22
*** cppforlife_ has joined #openstack-ironic06:23
*** morgabra has joined #openstack-ironic06:24
*** _vs has joined #openstack-ironic06:25
*** _vsaienko has joined #openstack-ironic06:25
*** sdake has quit IRC06:25
*** moshele has joined #openstack-ironic06:26
*** anush has joined #openstack-ironic06:26
*** harlowja has quit IRC06:28
*** romcheg has quit IRC06:29
*** romcheg has joined #openstack-ironic06:31
*** serverascode has joined #openstack-ironic06:31
*** _vs has quit IRC06:32
*** _vsaienko has quit IRC06:32
*** sdake has joined #openstack-ironic06:37
*** jcoufal has joined #openstack-ironic06:44
*** fragatin_ has joined #openstack-ironic06:45
*** Sukhdev_ has quit IRC06:46
*** fragatina has quit IRC06:47
*** fragatin_ has quit IRC06:50
*** sdake has quit IRC06:53
*** fragatina has joined #openstack-ironic06:57
*** rama_y has quit IRC06:57
*** jcoufal has quit IRC06:59
*** fragatina has quit IRC07:02
*** tesseract has joined #openstack-ironic07:04
*** ijw has quit IRC07:05
*** fragatina has joined #openstack-ironic07:07
*** fragatina has quit IRC07:08
*** tesseract has quit IRC07:08
*** fragatina has joined #openstack-ironic07:09
*** jtomasek has quit IRC07:10
*** jcoufal has joined #openstack-ironic07:11
*** athomas has joined #openstack-ironic07:12
*** parlos has joined #openstack-ironic07:12
*** sacharya has quit IRC07:13
*** chopmann has joined #openstack-ironic07:19
*** jcoufal has quit IRC07:19
*** rwsu has joined #openstack-ironic07:19
*** rwsu has quit IRC07:22
parlosGood Morning Ironic!07:26
*** jcoufal has joined #openstack-ironic07:32
*** Nisha_away has quit IRC07:43
*** dtantsur|afk is now known as dtantsur07:52
dtantsurMorning Ironic07:53
*** Fdaisuke has quit IRC07:57
_milan_morning dtantsur07:58
_milan_morning parlos07:59
*** Fdaisuke has joined #openstack-ironic07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** fragatina has quit IRC08:01
*** kromanenko__ has joined #openstack-ironic08:02
*** kromanenko_ has joined #openstack-ironic08:02
dtantsurhey _milan_08:03
*** mbound has joined #openstack-ironic08:04
_milan_dtantsur, so you've split the boot management spec? (https://review.openstack.org/#/c/320003/1/specs/approved/inspector-network-separation.rst)08:04
*** mbound has quit IRC08:05
*** mbound has joined #openstack-ironic08:06
dtantsur_milan_, I did08:06
_milan_dtantsur, cool08:06
* _milan_ reads08:06
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
*** lucasagomes_ is now known as lucasagomes08:12
lucasagomesJayF, hey, will do08:12
lucasagomesmorning all08:12
lucasagomesJayF, oh thanks for the rebase08:13
*** sacharya has joined #openstack-ironic08:13
_milan_morning lucasagomes08:14
_milan_dtantsur, I'm wondering, doesn't this change introduce REST API impact between inspector and ironic?08:15
*** dmk0202 has joined #openstack-ironic08:16
dtantsurmorning lucasagomes08:16
dtantsur_milan_, which one?08:16
_milan_dtantsur, between inspector and ironic08:16
_milan_dtantsur, nothing to be signaled through there?08:16
dtantsur_milan_, which change are you referring to right now? :)08:17
_milan_dtantsur, sorry :)08:17
* _milan_ pastes08:18
_milan_https://review.openstack.org/#/c/320003/1/specs/approved/inspector-network-separation.rst08:18
*** sacharya has quit IRC08:18
* _milan_ is wondering which party should call the network.add_provisioning_network08:20
*** e0ne has joined #openstack-ironic08:20
_milan_Ironic or Inspector?08:20
_milan_dtantsur, ^08:20
dtantsur_milan_, inspector can't call it, it's not a public API08:21
_milan_dtantsur, ok, so if one starts introspection thru  inspector API /<node_id>/introspect, if the network separation is in place, that won't work?08:23
dtantsur_milan_, correct08:23
* _milan_ wonders whether to state that in this spec08:24
_milan_as well as should we address that?08:24
*** odyssey4me_ is now known as odyssey4me08:26
_milan_dtantsur, ^08:26
dtantsur_milan_, we can't address that from within inspector, we don't have an access to internal API's08:27
*** praneshp has quit IRC08:28
_milan_dtantsur, shouldn't that API get exposed then? would that make sense?08:28
dtantsur_milan_, not sure to be honest. this is going to take substantial time with unclear benefit to end users08:30
*** e0ne has quit IRC08:31
*** e0ne has joined #openstack-ironic08:32
_milan_dtantsur, you know, I can imagine the bugs saying: "I can't do openstack baremetal introspection start my_node when network separation is in place" but I've just got the false impression that this call to attach the port to a network would be handled by inspector... But that api isn't public.08:34
* _milan_ will suggest the spec says explicitly that call is going to be performed by ironic08:34
_milan_and that inspector introspect API will not work anyway08:35
dtantsurthe inspector API is kind of low-level, yeah08:36
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Wait for at least one suitable disk to appear on start up  https://review.openstack.org/32029508:37
dtantsurJayF, ^^^08:37
*** ig0r_ has joined #openstack-ironic08:40
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add a configurable sleep before IPA starts working  https://review.openstack.org/31874508:40
*** derekh has joined #openstack-ironic08:44
aarefievmorning08:45
_milan_aarefiev, morning!08:47
aarefiev_milan_, dtantsur: inspector introspect API already has a problem because it's not touching ironic08:47
_milan_aarefiev, which is that?08:47
dtantsuraarefiev, right08:47
aarefievone more, one less :)08:47
dtantsuralso morning aarefiev :)08:47
* _milan_ not that familiar with those08:48
aarefiev_milan_: it doesn't change node state e.g08:48
_milan_aarefiev, I see. So true08:49
aarefievI think we have something in docs mentioning this08:50
_milan_anyways, I think it's fair to state that the spec won't help the inspector situation so the users (and me ;) aren't under false impression it might08:50
*** mgould|afk is now known as mgould08:51
mgouldmorning Ironic!08:51
_milan_morning mgould!08:52
dtantsurhey mgould08:53
*** athomas has quit IRC08:53
*** _milan_ has quit IRC08:57
mgouldmorning dtantsur milan08:57
mgouldalso morning aarefiev08:57
aarefievmorning mgould08:58
*** daemontool_ has joined #openstack-ironic09:00
*** electrofelix has joined #openstack-ironic09:00
*** athomas has joined #openstack-ironic09:03
vsaienkoMorning mgould, aarefiev, _milan_, dtantsur, lucasagomes and all Ironic'ers!09:19
dtantsurvsaienko, o/09:20
*** sdake has joined #openstack-ironic09:20
aarefievmorning vsaienko09:21
mgouldmorning vsaienko09:23
sergekmorning all o/09:23
*** zigo_ is now known as zigo09:28
lucasagomesvsaienko, hi there, good morning09:30
*** jtomasek_ has joined #openstack-ironic09:33
*** sivaramakrishna has quit IRC09:34
mgouldmorning sergek lucasagomes09:35
*** _milan_ has joined #openstack-ironic09:36
*** _milan_ is now known as milan09:39
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/31988009:43
lucasagomesmgould, morning09:44
openstackgerritIshant Tyagi proposed openstack/ironic-specs: Support for virtual macs - HP Oneview  https://review.openstack.org/32033109:44
*** sambetts|afk is now known as sambetts09:55
sambettsMorning all09:56
mgouldmorning sambetts09:58
*** sdake has quit IRC10:01
*** hoangcx_ has joined #openstack-ironic10:02
dtantsurmorning sambetts10:03
*** hoangcx has quit IRC10:03
sambettsHey mgould dtantsur10:03
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844010:10
openstackgerritAline Bousquet proposed openstack/ironic-lib: Add support for metrics  https://review.openstack.org/30152610:12
*** e0ne has quit IRC10:13
*** e0ne has joined #openstack-ironic10:14
*** sacharya has joined #openstack-ironic10:15
vdrokmorning everyone10:16
*** sacharya has quit IRC10:19
mgouldmorning vdrok10:22
vdrokgood morning mgould10:22
sambettso/ vdrok10:23
vdrokhi sambetts10:23
*** early has quit IRC10:23
*** early has joined #openstack-ironic10:23
*** ptx0 has quit IRC10:29
*** thrash|g0ne is now known as thrash10:29
*** Egyptian[Home] has joined #openstack-ironic10:35
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands  https://review.openstack.org/17251710:36
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands  https://review.openstack.org/21672710:36
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create  https://review.openstack.org/17246110:36
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec  https://review.openstack.org/28416010:36
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add provision state commands  https://review.openstack.org/20611910:36
openstackgerritShivanand Tendulker proposed openstack/ironic: Baremetal provisioning in UEFI secure boot mode for iLO drivers  https://review.openstack.org/23192710:36
*** e0ne_ has joined #openstack-ironic10:41
*** hoangcx_ has quit IRC10:41
*** e0ne has quit IRC10:42
*** moshele has quit IRC10:48
*** moshele has joined #openstack-ironic10:53
*** hparekh has quit IRC10:59
*** lazy_prince has quit IRC10:59
openstackgerritShivanand Tendulker proposed openstack/ironic-python-agent: Use ironic-lib to create configdrive  https://review.openstack.org/29646610:59
*** lazy_prince has joined #openstack-ironic11:00
*** Guest67116 has quit IRC11:00
*** sivaramakrishna has joined #openstack-ironic11:00
*** sivaramakrishna is now known as Guest9619311:01
*** dprince has joined #openstack-ironic11:02
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Returns CPU flags in the CPU inventory  https://review.openstack.org/30829211:10
openstackgerritVladyslav Drok proposed openstack/ironic: Gracefully degrade start_iscsi_target for Mitaka ramdisk  https://review.openstack.org/31918311:11
*** hparekh has joined #openstack-ironic11:13
*** kiran- has joined #openstack-ironic11:16
*** sdake has joined #openstack-ironic11:17
*** lucasagomes is now known as lucas-hungry11:18
vmud213morning all11:20
vdrokmorning vmud21311:21
vmud213morning vdrok11:21
*** Guest96193 has quit IRC11:24
sambettshi vmud21311:27
vmud213Hi sambetts11:27
vmud213morning11:28
jrollmorning everyone11:28
sambettsHey jroll!11:29
jrollsambetts: hey, morning :)11:29
jrollsambetts: random question, if we had grenade working today, do you think the multitenant network stuff is doable by end of june?11:30
vmud213morning jroll11:30
sambettsjroll: the existing patches?11:30
jrollsambetts: ya11:30
jrollI'm thinking it is, but then again we thought it was landable in mitaka :)11:30
jrollif we get on a phone and pile on it, I think we can do it though11:31
sambettsjroll: yeah, I think so, I'm really concered about merging the portgroups API :/11:31
jrollsambetts: because we should be modeling that in neutron instead?11:32
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [WIP] [POC] Ansible deploy driver alt impl  https://review.openstack.org/31004211:32
sambettsjroll: well I'm concered that if neutron decides to model them that we'll then have an API we'll need to support/deprecate out because neutron is now handling it11:33
sambettsand we can't technically ever remove it once its there11:33
jrollsambetts: sure, but do we need to model the physical bits on our end anyway?11:33
jrolle.g. we still have port11:34
sambettsjroll: right but I think thats fine, modeling a bond in Ironic is a little weird to me, because it implys that the switch is preconfigured with the bond11:34
sambettsand that its a static thing11:34
*** ig0r_ has quit IRC11:34
jrollsambetts: idk, I'm thinking it's fine, it models the fact that we expect it to be configured into a bond11:35
sambettsmodeling a port, makes sense because its a physical thing, which is what Ironic models, but adding bond we're modeling an intangiabl logical thing11:35
jrollif the backend changes to create bonds in neutron instead of just submitting all the physical data, the api doesn't need to change11:36
mgoulddoes neutron currently model bonds?11:36
jrollI don't think so, not sure11:37
sambettsmgould: not right now, but there was a rumour they might be11:37
jroll:|11:37
jrolllet's not stop progress based on rumors11:37
sambettsthats why I've asked sukdev to do some digging11:37
jrolllast time we did that it was for vlan aware vms11:37
*** kiran- is now known as kirankv11:37
jrollin kilo? liberty?11:37
jrollstill not done11:37
* mgould doesn't have a good sense of neutron's scope11:38
sambettsjroll: the difference is that then the bond is an operator decision, if its in neutron its a user decision11:38
jrollsambetts: sure, and at least in my deployment I want it to be an op decision11:38
jroll"these should always have a bond"11:39
sambetts:/ that breaks VM parity though right?11:39
dtantsurmorning jroll, vmud213, vdrok11:39
jrollI tend to think most deployments would be the same, if you have two ToRs in the rack, connected to the same upstream, why would that ever be two distinct links?11:39
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add boot information into the inventory  https://review.openstack.org/30829211:40
sambettsjroll: 2 different access VLANs?11:40
sambettsjroll: so you can use the full bandwidth of 1 link for a particular network connection11:41
jrollwhy would you do that instead of putting both on a trunk11:41
jrollmmm11:41
jrollI mean, the alternative is that ironic doesn't do bonding until neutron decides to11:41
jrollwhich I think is pretty sad11:41
sambettsunless we push to get it into neutron11:42
* jroll wonders if both models are okay, if you want users to control it you just don't create port groups11:42
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [WIP] [POC] Callback from ramdisk support for Ansible driver  https://review.openstack.org/31081611:42
jrollI don't expect our team to have the time or the knowledge to land a major feature in neutron like that11:42
jrollif we're being honest11:42
dtantsurjroll, maybe a stupid question, but still: do we *require* portgroups to land before the remaining network separation work?11:43
sambettsNo11:43
dtantsurjroll, aka can we land only ports in newton, then bonding in ocata, if we lack time?11:43
mgouldif we implement bonds, and then neutron do, would we remove our code and call theirs?11:44
milansambetts, morning!11:44
sambettso/ milan11:44
milanjroll, morning!11:44
jrolldtantsur: we don't require it, but it's a major part of the set of "what we want to accomplish with this"11:44
jrollhi milan11:44
jrollmgould: it's unclear to me11:44
dtantsurjroll, I'm thinking baby steps here.. it's better to land something rather than nothing at all, right?11:45
dtantsurI guess a lot of people will be happy even with basic support for network separation11:45
dtantsurmgould, removing API is not something we can easily do..11:45
jrollI feel like there's a path of "if operator wants to force bonding, use portgroups. if they want to leave it to users, skip portgroups and allow users to do it in neutron (if that's a thing)"11:45
milansambetts, https://review.openstack.org/#/c/308379/ have you got a minute?11:46
dtantsurI just don't want us to get blocked by the ongoing portgroup discussion.. it's a big chunk of API after all11:46
jrollit's a simple API though, just basic crud11:46
jrollbut I do see your point11:46
sambettsI think we're shooting into the dark right now, we need to find out if thats even on the cards from neutron's side, like I said it was a rumour that they wanted to support it for virt11:47
sambettsI'm just throwing the spanners to make sure we don't do something stupid11:47
jrollhttps://review.openstack.org/#/q/project:openstack/neutron-specs+bonding11:48
jrollhttps://review.openstack.org/#/q/project:openstack/neutron-specs+bonds11:48
*** sdake has quit IRC11:49
mgoulddtantsur: so if we add our own bonding API, we have to support it forever, even if we switch to using neutron for bonding11:51
sambettsright11:51
sambettsthats what worries me11:51
jrollsambetts: http://lists.openstack.org/pipermail/openstack-dev/2016-May/095766.html11:51
dtantsurmgould, kind of yes11:52
jrollwhat worries me is users needing to create bonds on their own, forgetting to do so, and we have a switch go down and they lose their entire network :)11:52
jrollor having NO CLUE AT ALL how to create bonds in neutron11:52
* jroll brb11:53
sambetts++11:53
*** aparnav has quit IRC11:54
* mgould likes dtantsur's idea: land "ports only" in newton, add bonding in ocata (by which time neutron may support them...)11:54
mgouldbut yeah, if neutron have no plan to ever support bonding, the question is kinda moot11:54
sambettsexactly11:54
mat128morning ironicers!11:55
sambettsliterally playing devils advocate11:55
mat128interesting topic this morning :D11:55
mgouldmorning mat12811:55
*** aparnav has joined #openstack-ironic11:55
mgouldmorning aparnav11:56
mat128I think I'm with jroll on this, I want flavors which are advertised as bond, and others which could be "flat" but I don't want the end user to choose, for the reasons mentioned11:56
mat128o/11:56
*** ig0r_ has joined #openstack-ironic11:57
openstackgerritAbhishek Kekane proposed openstack/ironic: Remove unused is_valid_cidr method  https://review.openstack.org/32039011:57
mgouldmat128, jroll: do users actually care about bonding, or is it just a means to the end of "more network throughput"?11:58
jrollmgould: HA for switching gear is the main reason11:58
mat128in our case they care, but if we let it up to them to set it up, it might not be setup at all and then they depend on it11:58
jrollusers don't care about "bonding", they care about "network fails less often"11:58
mat128s/it/a single switch/11:58
mgouldjroll: gotcha, thanks11:58
jrollmgould: and us as operators care about "we can take down a switch without data plane impact"11:59
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [PoC] Make streaming_write to be proper Ansible module  https://review.openstack.org/31860911:59
jrolle.g. code upgrades, crashes, etc11:59
mgouldso you'd advertise the flavor as "HA networking" rather than "oodles of network packets per second"11:59
jrollyes12:00
mgouldcool, thanks12:00
vdrokmorning dtantsur milan and jroll12:00
vdrokand morning mat128 !12:00
*** aparnav has quit IRC12:00
milanmorning vdrok and mat128 :)12:00
*** jjohnson2 has joined #openstack-ironic12:00
mat128hello vdrok12:00
dtantsurmorning mat12812:00
jrollheya vdrok12:01
mat128mgould: that's the main reason everything is redundant (power, network, cooling, etc.)12:01
* mgould nods12:01
*** jcoufal has quit IRC12:02
xavierrgood morning  Ironic12:05
xavierrmorning vdrok :)12:05
vdrokmorning xavierr12:05
dtantsurmorning xavierr12:05
mgouldmorning xavierr12:05
xavierrmorning mgould12:05
xavierrmorning dtantsur12:05
mat128morning milan xavierr dtantsur ;)12:06
milanxavierr, o/ :)12:06
xavierrmorning mat128 milan \o12:07
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [PoC] Add callback for logging into Ironic log  https://review.openstack.org/31861012:08
xavierrI ran 7.5 kilometres (4.6 miles) this morning12:08
* xavierr is feeling awesome12:08
vmud213morning dtantsur12:08
vmud213sambetts,dtantsur:  I am not sure why i am getting merge conflict for https://review.openstack.org/#/c/30350312:09
*** lucas-hungry is now known as lucasagomes12:10
dtantsurvmud213, did you try rebasing your change?12:10
vmud213patch related to initialize_hardware .. is reverted12:10
vmud213yes..i tried.But no conflicts as the changes are reverted12:11
dtantsurvmud213, then upload the rebased version. gerrit is sometimes overly paranoid with conflict detection12:11
mgouldxavierr: congrats!12:11
vmud213dtantsur: will do that. thanks12:11
xavierrmgould :D12:12
openstackgerritAline Bousquet proposed openstack/ironic-lib: Add support for metrics  https://review.openstack.org/30152612:12
*** trown|outtypewww is now known as trown12:13
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands  https://review.openstack.org/17251712:14
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands  https://review.openstack.org/21672712:14
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create  https://review.openstack.org/17246112:14
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec  https://review.openstack.org/28416012:14
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add provision state commands  https://review.openstack.org/20611912:14
parlosBye bye12:14
openstackgerritvinay kumar muddu proposed openstack/ironic-python-agent: Wait for all interfaces to be up before node lookup  https://review.openstack.org/30350312:15
*** jcoufal has joined #openstack-ironic12:16
* milan rereads http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/network-provider.html because of https://review.openstack.org/#/c/320003/ and I'm wondering: shouldn't the interface of the network_provider be exposed? Such that e.g Inspector could plumb the node to provisioning network?12:17
*** parlos has quit IRC12:19
*** ayoung has quit IRC12:19
openstackgerritDmitry Tantsur proposed openstack/ironic-specs: Tenant network separation for in-band inspection  https://review.openstack.org/32000312:20
*** rbudden has joined #openstack-ironic12:21
TheJuliaGood morning12:23
vdrokgood morning TheJulia12:25
*** divya has joined #openstack-ironic12:25
lucasagomesTheJulia, morning12:25
divyaHi Ironicers12:26
sambettsmilan: the problem I have with inspector + networking is that at the point we're inspecting in the same way we might not have a mac address we may also not have the switch information, because we're meant to get that from inspection12:26
vdrokhi divya12:27
dtantsurmorning TheJulia, divya12:27
xavierrhi lucasagomes12:27
lucasagomesxavierr, hello there :-)12:27
xavierrmorning TheJulia divya12:27
xavierrlucasagomes o/12:28
divyamorning TheJulia, vdrok, xavierr12:28
milansambetts, yeah, for the POST@continue endpoint, however, for POST@introspection/node_id we do12:28
divyaHi Sambetts12:28
sambettsmilan: ?12:29
sambettsdivya: Hi12:29
milansambetts, more and more I think we should make POST@continue into some POST@registry to just receive an uuid and IPMI creds12:29
dtantsurmy concern is different, I don't think ironic-inspector should be aware of so many things that Ironic does, like boot management or network separation12:30
milansambetts, when the administrator triggers the introspection through the introspect/node_id?12:30
dtantsurwe have an "inspect" interface in ironic which is a front-end for inspector (modulo unfinished driver composition reform)12:30
milanTheJulia, divya morning!12:31
sambettsright, but in either case, we might not have enough information to program the swtich to perform any network separation12:31
divyaGuys, i have a basic question. if i am trying to boot a physical bare metal server what would be the PXE server/TFTP IP in openstack controller node?12:32
milansambetts, what I'm thinking about is the inspector calling the network flip should the request to introspect come through the POST@introspection/<node_id> by doing POST@ironic/<node_id>/networ_manager/whatnot if the Inspector config says so12:33
milanbut dtantsur has a point12:33
*** ptx0 has joined #openstack-ironic12:33
* milan just wondering cause we're putting down the specs atm12:33
mgouldmorning TheJulia12:33
dtantsurmilan, we don't expose API just for the sake of inspector.. every API we expose *will* be used by someone (and probably in a bad way)12:34
dtantsurmilan, so before exposing the network interface to users we better have a good understanding around how it will be used12:34
sambettsI think inspector should not be networking aware12:34
milandtantsur, it's not about because of Inspector rather than because of design considerations but fair enough12:35
* milan just asking12:35
*** rloo has joined #openstack-ironic12:36
divyamgould, i have a basic question. if i am trying to boot a physical bare metal server what would be the PXE server/TFTP IP in openstack controller node?12:36
milansambetts, I'd say needn't be rather than shouldn't12:36
milansambetts, *ideally12:37
sambettsmilan: tbh I'd rather inspector wasn't boot process aware, and that wasn't all handled in ironic by the driver12:37
mgoulddivya: for deployment or introspection?12:37
sambettsmilan: hence the manage_boot spec12:37
milansambetts, ideally ;) and I agree12:38
*** dprince has quit IRC12:38
milanmy other point is once we declare the support to separate tenant networks we render that inspector API broken12:39
milanshould we deprecate it?12:39
sambettsmilan: its not broken, it'd still work in a standalone or flat environment12:39
mgoulddivya: how are you deploying Ironic?12:40
milansambetts, but that means inspector should be networking-aware to handle, no?12:40
sambettsmilan: nope12:40
dtantsurdivya, I don't quite get the "what" question. Do you mean which software to use? Or how does Ironic manage it? or..?12:40
milansambetts, ?12:40
dtantsurmilan, we work perfectly well with flat network already. we will work if something (ironic or a user or a script) will set network for us12:41
sambettsmilan: because inspector continues to see it as a flat network, ironic will configure the node to be in that network12:41
divyamgould, i mean the [pxe] tftp_server = <IP> in /etc/ironic/ironic.conf12:42
divya?12:42
milandtantsur, sambetts thx I see12:42
sambettsmilan: but thats if it has enough information to configure the switch, which on a first inspection it probably won;t12:42
dtantsurdivya, that's an IP for your TFTP server; probably the same machine as Ironic conductor, but can be different as well.12:42
dtantsurdivya, we expect a TFTP server to be preconfigured somehow12:43
NobodyCamgood morning Ironicers12:43
milandtantsur, sambetts should we then have a doc/spec that describes this as expected behaviour  for the Inspector w/r network separation?12:43
dtantsursambetts, yeah, kind of chicken-and-egg12:43
dtantsurmorning NobodyCam12:43
dtantsurmilan, what would you put there?12:43
NobodyCammorning dtantsur :)12:43
divyadtantsur, mgould, Thanks. in case the TFP IP is my management ip address of my openstack server/TFTP server, my bare metal doesn't have TOR connection then how it will reach TFTP server IP?12:45
sambettsdtantsur: yup, in my environment, we have a guide for adding a new machine into Ironic and as part of that process we describe manually configuring the switch to put the node into the inspection network12:45
milandtantsur,  maybe docs: "...Network Separation (in Notes probably):  dear user and milan ;) note please that the endpoint POST@introspection/node_id won't manage network flip on your behalf. Rather, we assume you set up all this before calling our endpoint." - ish12:45
dtantsurdivya, your bare metal must have connection to the conductor/TFTP for now. we're working on improving this situation in the future, but now we only support flat networks12:45
sambettsmilan: ++12:46
dtantsursambetts, it's hard to avoid, unless something can discovery network connectivity out-of-band12:46
dtantsur(which I doubt)12:46
divyadtantsur, bare metal is connected to l2 switch and l2 switch is connected to TFTP server/openstack server12:47
divya?12:47
sambettsdtantsur: exactly, (magic hardware aside) we rely on in-band LLDP to get that info12:47
milansambetts, so you have sort of a default config on the Switch that plumbs nodes to an introspection network?12:47
dtantsursambetts, oh, do you already have LLDP discovery implemented downstream?12:47
dtantsurdivya, kind of this, yes. remember that you'll also need DHCP access to Neutron12:48
sambettsdtantsur: yeah, and we've left comments on the ones implementing it upstream but no response yet12:48
*** catintheroof has joined #openstack-ironic12:49
*** vmud213 has quit IRC12:49
dtantsursambetts, maybe ping the authors directly via their email, then create a competing patch? it's been a while since the last updated..12:49
divyadtantsur, DHCP access to Neutron means? can you please elaborate?12:50
*** mgoddard__ has quit IRC12:50
dtantsurdivya, DHCP for your nodes is served by the Neutron service, which is probably also on what you call "openstack server". I'm not aware how exactly it works, sorry12:50
*** mgoddard__ has joined #openstack-ironic12:51
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [WIP] [POC] Callback from ramdisk support for Ansible driver  https://review.openstack.org/31081612:51
sambettsdtantsur: yeah, I had plans to put up a competing patch, but havn't had the time with the other specs I've been pushing, if your good with it then I'll bump the priority my end12:51
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [PoC] Make streaming_write to be proper Ansible module  https://review.openstack.org/31860912:51
dtantsursambetts, I'd love to make some progress re LLDP discovery in Newton12:52
sambettsdtantsur: Cool, I'll ensure that I get some time to chase that12:52
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [PoC] Add callback for logging into Ironic log  https://review.openstack.org/31861012:52
dtantsursambetts, great, thanks!12:53
NobodyCammorning sambetts, and ugh just read jrolls email on bonding :p12:54
*** ipukha has quit IRC12:54
*** ipukha has joined #openstack-ironic12:54
sambettsNobodyCam: yeah, I was playing devils advocate this morning :P12:54
*** baoli has joined #openstack-ironic12:55
divyadtantsur, i am also not sure how it works. When i try to boot i am getting "BOOT failed. PXE Network" i suspect bare metal is not reaching TFTP server.12:55
NobodyCam:)12:55
*** baoli_ has joined #openstack-ironic12:56
mgouldmorning NobodyCam12:56
NobodyCammorning mgould :)12:56
xavierrhi NobodyCam, good morning12:57
NobodyCammorning xavierr :)12:57
*** david-lyle has joined #openstack-ironic12:57
*** rloo has quit IRC12:57
*** rloo has joined #openstack-ironic12:58
milanNobodyCam, o/12:58
NobodyCamhe hey mornig milan :)12:59
milan:)12:59
*** baoli has quit IRC13:00
mgoulddivya: if your baremetal node can't see the TFTP server, then yes, that'll happen13:00
*** jistr is now known as jistr|mtg13:01
*** lucasagomes is now known as lucas-afk13:01
*** thiagop has joined #openstack-ironic13:01
*** david-lyle_ has quit IRC13:01
divyamgould, do u have any clue to reach TFTP server?13:01
thiagopGood morning, Ironicers!13:03
NobodyCammorning thiagop13:03
mgoulddivya, what's the output of `pgrep -a tftp` on your openstack server?13:03
thiagopmgould: heard that you like mountains :D13:04
thiagopNobodyCam: hiya!13:04
mgouldalso, how did you deploy this cluster? TripleO? devstack? Something else?13:04
mgouldthiagop: yep :-) You too?13:04
divya2346 /usr/sbin/in.tftpd --listen --user tftp --address [::]:69 --secure /var/lib/tftpboot13:04
thiagopmgould: yup13:04
jrollwho wants to kill some code? :) https://review.openstack.org/#/c/316820/ https://review.openstack.org/#/c/316824/13:04
dtantsurme ME!13:05
jrolldoeeeet13:05
thiagopmgould: But only have the opportunity to trek up to 2.000mts13:05
* jroll steps away13:05
*** mjturek1 has left #openstack-ironic13:05
*** nico-afk is now known as nicodemos13:05
thiagopmgould: There is not that much high mountains in Brazil, less even where I live13:06
nicodemosmorning, ironic13:06
xavierrmorning nicodemos13:06
divyamgould, the output is 2346 /usr/sbin/in.tftpd --listen --user tftp --address [::]:69 --secure /var/lib/tftpboot13:06
mgouldmorning nicodemos13:06
NobodyCammorning nicodemos13:06
mgouldthiagop: the highest mountain in the UK is 1345m, but it does start near sea level :-)13:07
mgoulddivya: OK, thanks13:07
*** sdake has joined #openstack-ironic13:09
thiagopmgould: but you're on europe. A couple of hours flight from the alps?!13:09
mgouldthiagop: yep :-) A bit too far for a weekend, but I've been there on vacation a couple of times13:10
jlvillalvsaienko, vdrok: Hopefully you noticed that I made some changes to the Grenade related patches.13:10
thiagopmgould: my dream!13:10
vdrokmorning jlvillal, yep13:10
jlvillalvsaienko, vdrok: I made all the Ironic ones be independent of each other. So that we could get them merged independently.13:10
milandtantsur, sambetts I've wrapped up  our discussion: https://bugs.launchpad.net/ironic-inspector/+bug/158519613:10
openstackLaunchpad bug 1585196 in Ironic Inspector "[RFE] add network separation caveats to Inspector documentation " [Undecided,New]13:10
*** nicodemos has quit IRC13:10
mgoulddivya: I don't think that helps, unfortunately: looks like it's only listening on IPv613:11
dtantsurmilan, that's definitely on an RFE :)13:11
dtantsur* not13:11
thiagopmgould: maybe we can schedule something before/after summit near barcelona, han?! :D13:11
vsaienkojlvillal good morning13:11
jlvillalvsaienko, vdrok: I think we got 3-4 merged yesterday. I have us only needing 8 more patches to be merged. And 3 of those are the tempest timeout ones.13:11
milandtantsur, Oops13:11
jlvillalvdrok, vsaienko Good morning :)13:11
thiagopmorning jlvillal13:11
mgouldthiagop: https://www.facebook.com/miles.gould/media_set?set=a.10150155172582234.288638.521097233&type=3 https://www.dropbox.com/sc/gzrvwdu9tqdjrj6/AABrv7VoMU_CJZMjp0VvlGyTa13:11
jlvillalthiagop, :)13:11
divyamgould, can u please suggest a way/doc to debug further?13:12
milandtantsur, fixed ;)13:12
sambettsdtantsur, milan: I wish we had a better way to track things like that13:12
openstackgerritVladyslav Drok proposed openstack/ironic: Gracefully degrade start_iscsi_target for Mitaka ramdisk  https://review.openstack.org/31918313:12
mgouldthiagop: sorry, first link was wrong. try https://www.dropbox.com/sc/lgxrv0opj1kyend/AACQV-MAuE2pKlX7cREaj80ua13:12
milansambetts, better a bug than nothing ;)13:12
mgoulddivya: how are you deploying ironic? I can be more help if I know which set of deployment code to look at13:13
mgouldthiagop: post-summit climbing trip sounds like an *excellent* idea13:13
thiagopmgould: holy cow! *_*13:13
mgouldI think milan and ifarkas might also be interested13:13
milanat least I feel i've done some work besides having had a good conversation :)13:13
*** sdake has quit IRC13:13
sambettsmilan: heh :-P yeah13:14
milanmgould, thiagop sure thing :)13:14
milan*unless ropes are necessary13:14
jlvillalsambetts, I'll let you and vdrok figure out which function the retry should be done in https://review.openstack.org/319183 :)13:14
* milan total noob in that13:14
mgouldmilan: I'm sure we can climb unroped if you'd prefer :-)13:14
jlvillalsambetts, vdrok: I admit I like the current patch. It looks cleaner. But that doesn't mean it is the most correct though.13:15
*** nicodemos has joined #openstack-ironic13:15
milanmgould :D OK13:15
milannot much safer, right ;)13:15
jlvillaljroll, Do you have an opinion on https://review.openstack.org/319183 ? Since you started it :)13:15
divyamgould, i just installed devstack and trying to test bare metal deployment.13:15
xavierrnice pics mgould :)13:15
mgoulddivya: cool, thanks13:15
milanjlvillal, morning!13:15
jlvillalmilan, :)13:16
mgouldxavierr: thanks!13:16
ifarkasmgould, I am totally in! :-)13:16
mgould\o/13:17
NobodyCammorning jlvillal13:17
mgoulddivya: trying to work out where the TFTP server is set up in devstack13:18
jlvillalNobodyCam, Morning! :)13:18
mgouldfor ironic-inspector, we use dnsmasq's built-in tftp server, but AIUI for deployment we use a separate one13:18
mgouldfor deployment we use neutron-dhcp-agent to assign IPs to nodes13:19
milanmgould, where to subscribe for the trip? ;)13:19
mgouldmilan: I'll make sure you're kept in the loop :-)13:20
milanmgould, thanks! :)13:20
*** cdearborn has joined #openstack-ironic13:20
thiagop#pixiesay trekkers, trekkers everywhere!13:20
PixieBootsʕ•͡ᴥ•ʔ: trekkers, trekkers everywhere!13:20
sambettsdtantsur: I was just reviewing your cpu flags patch and it was all looking good until I refreshed and now that change id has changed into the boot info patch :/13:21
sambettsdtantsur: was that intentional?13:21
jrolljlvillal: define opinion?13:21
sambettsheh13:21
sambettsjlvillal, jroll: The real question is Should the driver gracefully degrade, or should the client?13:22
dtantsursambetts, no, I think I've screwed it. lemme fix13:22
jrollsambetts: well, turns out it's an async command, so the agent will take it and fail later13:22
*** _vs has joined #openstack-ironic13:22
*** _vsaienko has joined #openstack-ironic13:22
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add boot information into the inventory  https://review.openstack.org/32044813:22
jrollsambetts: I *think*, judging by the working code that's how it seems to me13:23
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Returns CPU flags in the CPU inventory  https://review.openstack.org/30829213:23
dtantsursambetts, done ^^^13:23
sambettsdtantsur: thanks :D13:23
*** fellypefca has joined #openstack-ironic13:24
sambettsjroll: we're a little screwed, without some sync way for detecting failure then right? Although if its an async command how did we end up with the logs in ir-cond, sent through on the next heartbeat?13:24
jrollsambetts: yeah, next heartbeat it will check the result13:25
jrollbut, double-checking13:25
mgoulddivya: looks like we use xinetd to serve TFTP for deployment13:25
mgouldtry `pgrep -a xinetd`?13:25
sambettsjroll: might need to move all that code around then anyway13:25
*** _vsaienko has quit IRC13:25
*** _vs has quit IRC13:25
jlvillalmgould, Not xinetd. xinetd runs the real tftdp13:25
mgouldoh, OK13:25
jlvillaltftpd13:25
mgouldshould devstack configure that correctly, or do you have to do anything special?13:26
*** causten__ has joined #openstack-ironic13:26
jlvillalmgould, I'm assuming that devstack configures whatever it is using correctly or we would break in the gate. But that is only applicable to the way we run it in the gate.13:27
openstackgerritDmitry Tantsur proposed openstack/ironic-specs: The driver composition reform  https://review.openstack.org/18837013:27
*** ayoung has joined #openstack-ironic13:27
jrollsambetts: I lied, the result with invalid args is syncronous https://gist.github.com/jimrollenhagen/87b90a2deff21fdaff85226451c9bb7d13:28
jrollso yeah, would much prefer the client itself handle it13:28
jrollit does return a 200 though13:28
ElCoyote_jroll: I noticed you had submitted a new PatchSet.. thank you so much for coming to my rescue. When I added the first unit tests, I got python exceptions on both (as reported by Jenkins).. so I dropped them at a later patchset..13:28
ElCoyote_jroll: Is there somewhere where I can learn about the API for the unit-tests?13:29
jrollElCoyote_: right, the unit tests were incorrect13:29
jrollumm13:29
zhenguo_jroll, jlvillal: mind to have a look at this https://bugs.launchpad.net/nova/+bug/1582680? a very obvious ironic related nova bug13:29
openstackLaunchpad bug 1582680 in OpenStack Compute (nova) "Ironic: wrong check for ready to deploy" [Undecided,In progress] - Assigned to Zhenguo Niu (niu-zglinux)13:29
mgoulddivya: I need to go to a meeting now, but it looks like http://docs.openstack.org/developer/ironic/deploy/install-guide.html#pxe-setup covers TFTP configuration13:29
mgouldgood luck!13:29
*** lindycoder has joined #openstack-ironic13:30
*** causten_ has quit IRC13:30
*** causten__ has quit IRC13:30
jrollElCoyote_: we primarily use https://testtools.readthedocs.io/en/latest/ and https://docs.python.org/3/library/unittest.mock.html13:31
ElCoyote_jroll: Thanks.. I think this is what I was looking for.13:32
ElCoyote_jroll: Thank you so much.. I wouldn't have been able to write the unittests at this time (I'm new at the OpenStack code)13:32
jrollElCoyote_: happy to help13:33
jlvillaljroll, I'm probably being dumb. On your comment in https://review.openstack.org/#/c/319183/ . You mention you would prefer it being done in the client (and -1 it), but to me it seems like it is.13:33
jrollzhenguo_: +1'd13:33
jlvillalBeing done in the client that it.13:33
jlvillalBeing done in the client that is.13:33
jrolljlvillal: urgh, it changed since last night13:33
sambettsheh13:34
*** ildikov has quit IRC13:34
*** ildikov has joined #openstack-ironic13:34
jrolljlvillal: +113:34
jlvillaljroll, thanks13:35
jrollnp, thanks for pointing that out13:35
*** jjohnson2 has quit IRC13:35
* jroll fetches clearly needed coffee13:35
*** parlos has joined #openstack-ironic13:35
jlvillalvdrok, vsaienko jroll rloo NobodyCam TheJulia devananda  As an FYI. I will be in training for the rest of this week13:36
jlvillalAnd anyone else who cares :)13:36
jrolljlvillal: sorry to hear that :P13:36
* jlvillal will learn about golang :)13:36
*** jjohnson2 has joined #openstack-ironic13:37
sambettsnot you too XD13:37
jlvillalheh13:37
jlvillalIt was a free class. So I said, sure sign me up :)13:37
*** _vs has joined #openstack-ironic13:37
dtantsurmorning jlvillal! oh, so you're joining their sect? :D13:37
jlvillaldtantsur, Heh. I'll see...13:38
jlvillaldtantsur, lucas-afk As an FYI. I will be in training for the rest of this week13:42
* jlvillal forgot to include you in the list of people the first time :(13:42
*** _vs has quit IRC13:42
openstackgerritMerged openstack/ironic: Remove deprecated driver_periodic_task  https://review.openstack.org/31682013:43
*** ametts has joined #openstack-ironic13:46
*** chopmann has quit IRC13:46
*** _vsaienko has joined #openstack-ironic13:46
*** m3m0 has quit IRC13:47
*** _vsaienko has quit IRC13:47
*** mgoddard__ has quit IRC13:49
*** m3m0 has joined #openstack-ironic13:49
*** chopmann has joined #openstack-ironic13:49
*** chopmann_ has joined #openstack-ironic13:51
*** mgoddard_ has joined #openstack-ironic13:53
*** _vsaienko has joined #openstack-ironic13:53
*** _vs has joined #openstack-ironic13:53
*** chopmann has quit IRC13:54
*** rloo has quit IRC13:59
*** sabeen has joined #openstack-ironic14:03
*** jistr|mtg is now known as jistr14:04
*** divya has quit IRC14:07
*** _vs has quit IRC14:10
*** Guest71132 is now known as dansmith14:11
*** _vs has joined #openstack-ironic14:11
*** dansmith is now known as Guest2162114:12
*** lucas-afk is now known as lucasagomes14:12
lucasagomesjlvillal, cool, enjoy it!14:12
*** absubram has quit IRC14:13
*** penick has joined #openstack-ironic14:13
*** Guest21621 is now known as dansmith14:14
*** ChrisAusten has joined #openstack-ironic14:14
*** [1]cdearborn has joined #openstack-ironic14:14
*** rloo has joined #openstack-ironic14:18
*** penick has quit IRC14:21
*** sacharya has joined #openstack-ironic14:24
*** sacharya has quit IRC14:24
*** cdearborn has quit IRC14:24
parlosGood day Ironic14:25
sambettsHi parlos14:25
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Tests for testing chssis-create command.  https://review.openstack.org/32048314:26
parlosGot a question wrt, the deployment/boot process of a BM..14:27
jrolllucasagomes: one more deprecation removal https://review.openstack.org/#/c/316824/14:27
* lucasagomes looks14:28
*** logan- has quit IRC14:29
parlosAs a BM boots, the boot request is caught by the DHCP server that redirects to a tftp image as to boot. This image in turn takes appropriate actions, download data and boot.14:30
*** logan- has joined #openstack-ironic14:30
parlosIf the BM has already been deployed, is the tftp image still deployed, and the image figures out(by talking to someone) that is should just boot from disk?14:31
sambettsparlos: once the BM is deployed, a different tftp image that chain loads to the local disk will be served to the BM, unless you have local boot configured in which case Ironic will actually set the boot order on the BM to point at the local disk14:34
*** _vs has quit IRC14:34
*** _vsaienko has quit IRC14:34
*** _vsaienko has joined #openstack-ironic14:34
*** _vs has joined #openstack-ironic14:34
*** e0ne has joined #openstack-ironic14:34
*** e0ne_ has quit IRC14:35
parlossambetts: Ok, this means that ironic has control over a dhcp server..?14:35
parlosIs that server 'embedded' into ironic?14:35
sambettsparlos: No, Ironic sets dhcp options on the neutron port assigned to that server, and neutron configures the DHCP server that it has setup for that network14:36
parlossambetts; so we could send other options to the neutron port...14:38
parlossambetts: the default boot image (used for deployment) is that a standard boot image, or is it constructed/assembled per BM?14:39
*** marios_ has joined #openstack-ironic14:40
*** zzzeek has quit IRC14:40
sambettsparlos: this depends on the driver you are running, the standard agent/iscsi deploy interfaces in ironic expect a prebuilt image stored in glance, and then the uuid of tht image set on each node you want to use it14:41
*** ChrisAusten has quit IRC14:41
*** david-lyle has quit IRC14:42
*** marios has quit IRC14:42
*** YiboCai has quit IRC14:42
*** ricardoas has quit IRC14:42
*** [1]cdearborn has quit IRC14:42
*** mgoddard_ has quit IRC14:42
*** athomas has quit IRC14:42
*** konetzed has quit IRC14:42
parlossambetts: its one of the ir-deploy-pxe-ssh images (nova image-list)14:43
sambettsparlos: in devstack yes, if you do an ironic node-show <uuid>, if you look in the driver_info you'll see fields were that image is configured for that node to use14:44
*** _vs has quit IRC14:44
sambettsparlos: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L388 and https://github.com/openstack/ironic/blob/master/ironic/dhcp/neutron.py are good places in the code to look at if you want to understand the pxe deployment flow a little better14:44
parlossambetts: thanks :)14:44
*** fragatina has joined #openstack-ironic14:45
*** _vsaienko has quit IRC14:45
*** fragatina has quit IRC14:45
*** fragatina has joined #openstack-ironic14:46
*** rcernin has quit IRC14:47
*** _vsaienko has joined #openstack-ironic14:48
*** _vs has joined #openstack-ironic14:49
*** sdake has joined #openstack-ironic14:49
*** rloo has quit IRC14:53
*** links has quit IRC14:57
*** _vs has quit IRC14:58
*** trandles has joined #openstack-ironic14:58
*** _vsaienko has quit IRC14:59
*** rcernin has joined #openstack-ironic14:59
*** rloo has joined #openstack-ironic15:00
*** marios_ has quit IRC15:01
openstackgerritVasyl Saienko proposed openstack/ironic: Update resources subnet CIDR  https://review.openstack.org/31708215:01
*** marios_ has joined #openstack-ironic15:01
*** _vsaienko has joined #openstack-ironic15:02
parlossambetts: I've tried to understand the flow better, so is the https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L464 called/passed every time a BM boots, or only when the node is deployed as subsequent boots will be caught by the neutron dhcp options pointing to the simplified (boot local) image?15:02
*** absubram has joined #openstack-ironic15:02
*** _vs has joined #openstack-ironic15:03
*** zzzeek has joined #openstack-ironic15:03
openstackgerritVincent S. Cojot proposed openstack/ironic: Make the vbox driver work on headless VirtualBox machines  https://review.openstack.org/31127815:03
*** jistr is now known as jistr|mtg15:04
*** rama_y has joined #openstack-ironic15:04
*** mgoddard has joined #openstack-ironic15:04
lucasagomesparlos, it's called once when the image is being deployed15:05
sambettsparlos: so that function will be called once, when the node moves from deploying state to active state, and it configures the environement including the dhcp and tftp to boot the tenant image from that point forward15:05
openstackgerritMerged openstack/ironic: Remove "periodic_interval" config option  https://review.openstack.org/31682415:05
jrollElCoyote_: so does setting boot device while power is on work for non-headless vbox?15:05
ElCoyote_No15:05
*** e0ne has quit IRC15:06
jrollElCoyote_: so we probably want to remove the 'if headless' there then, right?15:06
ElCoyote_It fails in all cases but no-one had noticed cause I guess most people are using tripleo with KVM15:06
ElCoyote_jroll: I didn't want to step on other people's toes.15:06
jroll(on line 771)15:06
jrollElCoyote_: it's new code, not sure what you'd be stepping on :)15:07
ElCoyote_Ok, let me change that.15:07
*** ChrisAusten has joined #openstack-ironic15:07
jrollthanks15:07
openstackgerritVincent S. Cojot proposed openstack/ironic: Make the vbox driver work on headless VirtualBox machines  https://review.openstack.org/31127815:08
parloslucasgomes sambetts: ok... so, if there would be a new state, this would then have to talk to neutron again, and tell it to serve up another image if that device boots again.15:08
lucasagomesparlos, sorry I may be out of context, when you say "image" you mean the initrd & kernel image or the [i]PXE ROM image?15:09
* lucasagomes reads scrollback15:10
parloslucasgomes; image would be netboot image (that downloads to the local disk)15:10
*** joprovost has joined #openstack-ironic15:10
parlossorry for being ambiguous.15:11
jrollElCoyote_: lgtm, +1'd15:11
sambettsparlos: we don't serve the tenant image via tftp15:11
ElCoyote_jroll: Thank you for all your help.15:11
jrollElCoyote_: you're welcome, thanks for the fix and the perseverence :)15:11
parlossambetts, that I guessed, but the image that the device boots from after/during the PXE boot is served via tftp..15:12
lucasagomesparlos, no problem... so the user/tenant image is already written to the disk. What happens is, the kernel and ramdisk from that image (if no local boot is used) will be fetched from TFTP (or HTTP)15:12
lucasagomesbecause in this case PXE is the bootloader, and the local disk have no bootloader installed15:12
lucasagomesso PXE remotely fetches the kernel/initrd and load it in memory15:13
lucasagomesonce that's loaded it assumes the control and boot into the local disk15:13
lucasagomesthe instructions to do it is written in the PXE configuration file15:13
*** _vsaienko has quit IRC15:13
*** mgould is now known as mgould|brb15:13
lucasagomeswhich says what is the UUID of the root filesystem in the disk (see root=UUID=<uuid>)15:13
*** joprovost has quit IRC15:14
lucasagomesparlos, here's the template https://github.com/openstack/ironic/blob/master/ironic/tests/unit/drivers/pxe_config.template#L1115:15
lucasagomesroot={{ ROOT }}15:15
*** jrist has quit IRC15:15
*** _vs has quit IRC15:15
lucasagomes{{ ROOT }} will be replaced with the UUID of the filesystem of the user image15:15
lucasagomesand that's instruct the ramdisk to mount that fs and continue the boot from there15:15
parloslucasgomes; Hrm, as I'm old I usually revert back to lilo (dont scream).. So your saying that the entire kernel/initrd is coming across the net? I thought that once deployed, if the device rebooted, the netboot loader just kinda read a config file that said boot from /dev/hda1 (yeah I'm old)15:16
lucasagomes(note that this is a big assumption of Ironic about the image, it wouldn't work for images that relies on root filesystem not being a partition such as ostree, btrfs subvolumes, lvm logical volumes and so on)15:17
lucasagomesparlos, heh no worries, lilo is great! So yes, your assumptions are correct, in a default deployment the kernel and initrd will come across the net15:18
lucasagomesparlos, that's configurable tho, if you want to boot directly from the local disk it's possible by using the local boot feature in Ironic15:18
*** e0ne has joined #openstack-ironic15:18
*** joprovost has joined #openstack-ironic15:18
parloslucasgomes, so would this not mean that the BM will always use the kernel that the ironic has?15:19
parlosor is it possible that the kernel/initrd are replaced as the BM boots?15:19
lucasagomesparlos, that means that yes, any kernel update within the image will be ignored if it's not booting from local15:19
*** ishant has joined #openstack-ironic15:20
lucasagomesparlos, we don't have a mechanism for that in place15:20
parloshrm, so the BM image must be linux compatible...  (not so bad, but some people will not like it...)15:20
lucasagomescan be done manually, but it's not striaght forward15:20
lucasagomesparlos, for different OSes, I would recommend using "whole disk image" in Ironic15:21
sambetts^ what lucas said15:21
parlosJust trying to figure out how ex. fuel/MAAS/etc.. changes so that the device boots locally.. sure requires a boot loader on the disc.15:21
lucasagomeswhich is a image that contains everything, partition table and bootloader in place15:21
*** _vsaienko has joined #openstack-ironic15:22
*** _vs has joined #openstack-ironic15:22
parlosAh; that was my starting point.15:22
parlosimage that is self contained.15:22
lucasagomesright15:22
*** Junhongl_ has joined #openstack-ironic15:22
lucasagomesthat's supported as well15:22
lucasagomesand if you're deploying something like, windows images, that would be the way forward15:23
lucasagomesironic does not care what the image is in the whole disk image, it will just copy it byte-by-byte onto the device15:23
lucasagomes(same for partition images unless they use local boot)15:23
*** mjura has quit IRC15:26
parloslucasgomes; so now if one of these whole image BMs happen to reboot, and the BM bios setting happen to start with PXE, what is pushed over? anything or simply nothing, hoping (or by config) that next boot device is local disk?15:27
*** Junhongl_ has left #openstack-ironic15:27
sambettsparlos: we remove all pxe configuration for local boot images15:28
lucasagomesparlos, the DHCP will asnwer it but the net boot will fail because there won't be any PXE configuration file15:28
lucasagomesand the device should fall back to the next boot device15:28
lucasagomesif booting from disk is not disabled in that example of yours, eventually it should boot from it15:29
parlosok, thanks for clearing that out.15:29
lucasagomesotherwise it just won't boot anything15:29
openstackgerritVladyslav Drok proposed openstack/ironic: Add support for API microversions in Tempest tests  https://review.openstack.org/26035815:29
dtantsurvdrok, nice ^^^!15:30
* dtantsur waits for the gate to pass15:30
*** parlos is now known as parlos|brb15:31
vdrokdtantsur: heh, yep, seems to pass locally15:31
dtantsurfantastic!15:31
dtantsurwe are in desperate need of this patch in inspector, it's blocking our tempest work15:31
jrollwoohoo15:31
openstackgerritRuby Loo proposed openstack/ironic-specs: follows cycle-with-intermediary release model  https://review.openstack.org/31818815:32
jrolldtantsur: inspector uses ironic tempest tests?15:32
*** _vs has quit IRC15:33
*** _vsaienko has quit IRC15:33
*** ishant has quit IRC15:33
dtantsurjroll, no yet. but we want to reuse bits from the ironic tempest plugin15:33
jrollah15:34
jrolldtantsur: that alone might be worth the effort of factoring it to a different repo15:34
*** ishant has joined #openstack-ironic15:34
dtantsurjroll, we anyway have to install ironic, so it's not pressing :)15:34
*** ChrisAusten has quit IRC15:35
*** links has joined #openstack-ironic15:36
dtantsurjroll, did we come to any conclusion about these numerous nova-compatible console specs?15:36
dtantsurI don't want us to look like we're just avoiding reviewing them (though I personally do)15:36
dtantsurlucasagomes, ^^^15:37
*** dmk0202 has quit IRC15:38
*** dmk0202 has joined #openstack-ironic15:38
*** _vsaienko has joined #openstack-ironic15:38
*** _vs has joined #openstack-ironic15:38
lucasagomesdtantsur, ++15:39
*** jistr|mtg is now known as jistr15:40
*** [1]cdearborn has joined #openstack-ironic15:40
*** athomas has joined #openstack-ironic15:40
*** konetzed has joined #openstack-ironic15:40
*** Junhongl_ has joined #openstack-ironic15:40
lucasagomesyuikotakadamori, is https://review.openstack.org/#/c/319505/ a new version of https://review.openstack.org/#/c/306755/ ?15:41
* lucasagomes thinks he's stating the obvious here, but I just want a confirmation15:42
lucasagomesjroll, ^ you may want to update ur blog post with the new link as well15:42
*** _vsaienko has quit IRC15:42
*** _vs has quit IRC15:42
openstackgerritRuby Loo proposed openstack/ironic-lib: Remove deprecated disk util configs  https://review.openstack.org/31883415:42
JayFmaybe better to abandon the out of date merge req with a link to the new one?15:42
JayFsince the link in the ml archive won't be fixed15:43
*** fragatina has quit IRC15:43
*** Junhongl_ has quit IRC15:44
*** Junhongl_ has joined #openstack-ironic15:44
*** david-lyle has joined #openstack-ironic15:45
*** YiboCai has joined #openstack-ironic15:45
*** ricardoas has joined #openstack-ironic15:45
sambettswelll ... thats just lovely isn't it ... /me working on some IPA tests, and finds he can't import netutils into test_hardware because of circlur imports :'(15:46
*** jrist has joined #openstack-ironic15:46
JayFyou can use mock.patch.object in that case, right?15:47
JayFand mock up the version of it imported into the module in use?15:47
*** Junhongl_ has quit IRC15:47
*** jrist has quit IRC15:47
sambettsactually its not into test_hardware its into hardware.py ... arse... thats even worse...15:48
*** jrist has joined #openstack-ironic15:48
*** Junhongl has joined #openstack-ironic15:48
* sambetts blames lucasagomes 15:48
* lucasagomes reads15:48
*** marios_ has quit IRC15:48
lucasagomeshah sorry15:49
sambettslucasagomes: can you explain why you are importing IPA.cmd.agent into ipa.netutils?15:49
* lucasagomes dunno if he has something to do with it but...15:49
*** penick has joined #openstack-ironic15:49
*** fragatina has joined #openstack-ironic15:49
*** marios has joined #openstack-ironic15:49
sambettsyou've left a FIXME on it15:49
lucasagomessambetts, have a link handy to the code?15:49
*** marios is now known as Guest9795415:49
sambettslucasagomes: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/netutils.py#L2515:50
*** Guest97954 is now known as marios15:50
*** Junhongl has quit IRC15:51
lucasagomessambetts, we can try removing it, it is a FIXME because something went wrong with the imports15:53
lucasagomesI can't remember the exactly situation15:53
* lucasagomes checks the history15:53
* sambetts may be able to work around it tbh, I think my test scope was a little large 15:53
sambettsactually no, I can't /me keeps forgoting it wasn't in the test fail15:54
sambettsfile*15:54
lucasagomessambetts, so I think the problem is that tests are relying on configuration options that are created as part of the cmd.agent module15:55
*** morgabra has quit IRC15:55
*** morgabra has joined #openstack-ironic15:55
lucasagomesso, if you don't import it there the configs are never registered15:55
lucasagomesand tests fails15:55
lucasagomestherefore the FIXME heh15:55
*** kromanenko_ has quit IRC15:56
sambettsshouldn't that import be in the test file though not in the actual code?15:56
*** kromanenko__ has quit IRC15:56
*** MattMan has quit IRC15:56
sambettsI'll trying and work it out, I need to fix it for this patch I'm working on15:56
sambettstry and work *15:57
*** MattMan has joined #openstack-ironic15:57
*** jrist has quit IRC15:57
*** david-lyle has quit IRC15:57
*** YiboCai has quit IRC15:57
*** ricardoas has quit IRC15:57
lucasagomessambetts, so the netutils also uses that configs15:57
lucasagomese.g lldp_timeout is created at cmd.agent.py but used in netutils15:58
*** mgoddard_ has joined #openstack-ironic15:58
lucasagomessambetts, honestly, I would just remove all the confg options from cmd.agent and put it in another place that gets imported15:58
lucasagomesimported by other modules*15:58
sambettsyeah, might be the best idea I guess15:59
*** parlos|brb is now known as parlos15:59
*** wajdi has joined #openstack-ironic15:59
*** _vs has joined #openstack-ironic16:00
*** _vsaienko has joined #openstack-ironic16:00
*** jaybeale has joined #openstack-ironic16:01
*** mgoddard has quit IRC16:01
*** _vs has quit IRC16:02
*** fragatina has quit IRC16:02
*** _vsaienko has quit IRC16:02
jrolldtantsur: for consoles, no we didn't, we need to16:04
dtantsuryeah..16:04
*** jcoufal has quit IRC16:04
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Get root device hints from the node object  https://review.openstack.org/31853816:05
lucasagomesjroll, maybe we should bring it to the ML ? I can do it if you think it's worth16:05
jrolldtantsur: I like the separate service myself16:05
jrolllucasagomes: sure!16:05
*** _vsaienko has joined #openstack-ironic16:05
dtantsurlucasagomes, it was already on the ML16:05
*** _vs has joined #openstack-ironic16:06
lucasagomesdtantsur, on the newton recap ?16:06
dtantsurI don't think we have a good track of deciding anything there tbh16:06
lucasagomesright I mean a specific ML thread about it16:06
*** jrist has joined #openstack-ironic16:06
*** david-lyle has joined #openstack-ironic16:06
*** YiboCai has joined #openstack-ironic16:06
*** ricardoas has joined #openstack-ironic16:06
dtantsurlucasagomes, yuikotakadamori had one16:06
lucasagomesdtantsur, oh, I missed that then. Will take a look16:06
dtantsurbut I'm afraid we'll have to actively decide something, ML threads don't usually get a lot of answers16:07
lucasagomesyeah it's a shame :-(16:07
lucasagomesI saw rloo's ML about making decisions16:07
rloohuh? (good morning everyone)16:08
* lucasagomes should reply to it too16:08
lucasagomesrloo, hi there16:08
jrollyeah, we just had a thread about that16:08
*** mgould|brb is now known as mgould16:09
dtantsurmorning rloo16:11
openstackgerritVasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy  https://review.openstack.org/26915716:13
openstackgerritVasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624416:13
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643216:13
openstackgerritVasyl Saienko proposed openstack/ironic: Add 'neutron' network driver  https://review.openstack.org/31739316:13
openstackgerritVasyl Saienko proposed openstack/ironic: Add network_interface API  https://review.openstack.org/31739216:13
openstackgerritVasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326216:13
openstackgerritVasyl Saienko proposed openstack/ironic: Add network_drivers  https://review.openstack.org/28585216:13
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack  https://review.openstack.org/29352016:13
openstackgerritVasyl Saienko proposed openstack/ironic: Create common neutron module  https://review.openstack.org/31739016:13
openstackgerritVasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849616:13
openstackgerritVasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port  https://review.openstack.org/25636516:13
openstackgerritVasyl Saienko proposed openstack/ironic: Refactor ironic enroll-node code  https://review.openstack.org/25636416:13
openstackgerritVasyl Saienko proposed openstack/ironic: Add network_interface node field DB, OBJ  https://review.openstack.org/31739116:13
openstackgerritVasyl Saienko proposed openstack/ironic: Add configure_provision_network function  https://review.openstack.org/25636716:13
openstackgerritVasyl Saienko proposed openstack/ironic: Update Ironic VM network connection  https://review.openstack.org/25636616:13
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to specify node arch  https://review.openstack.org/31738916:13
openstackgerritVasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation  https://review.openstack.org/25859616:13
sambettswoah?! hehe16:13
parlosthe cirros images that comes with devstack.. if I change the boot_device to disk, and reboot it (from virsh), why does it look like it still does some network activities?16:13
vsaienkoI'm sorry for the spam, reviews are welcome :)16:13
*** moshele has quit IRC16:13
*** e0ne has quit IRC16:14
*** _vsaienko has quit IRC16:14
*** _vs has quit IRC16:14
thiagopvsaienko: even on "DO NOT REVIEW"? lol16:15
lucasagomesdtantsur, jroll replied to the thread16:15
*** jtomasek_ has quit IRC16:16
*** chopmann_ has quit IRC16:16
jrolllucasagomes: thanks16:16
*** sacharya has joined #openstack-ironic16:18
*** _vs has joined #openstack-ironic16:18
*** _vsaienko has joined #openstack-ironic16:18
* sambetts starts crying as he realises how many circlar imports there are in IPA 16:21
*** derekh has quit IRC16:22
*** cdearborn has joined #openstack-ironic16:24
*** dmk0202 has quit IRC16:24
*** ig0r_ has quit IRC16:24
*** _vs has quit IRC16:25
rlooit's OK sambetts, really, it is bad but you can do it!16:25
*** _vs has joined #openstack-ironic16:25
TheJuliasambetts: Coffee.... or chocolate...... or maybe beer will make it all better :)16:25
sambettsTheJulia: heh :-P its getting to that time over here16:26
parlosGood evening, and have a nice day.16:27
*** catintheroof has quit IRC16:28
*** parlos has quit IRC16:29
*** alex_xu has quit IRC16:35
*** sdake_ has joined #openstack-ironic16:35
*** [1]cdearborn has quit IRC16:37
*** sdake has quit IRC16:38
NobodyCamoh morning rloo and jroll16:38
rloomorning NobodyCam16:38
NobodyCam:)16:38
*** alex_xu has joined #openstack-ironic16:38
jrollohai NobodyCam :)16:39
* milan gone16:39
NobodyCam:)16:40
NobodyCamnight milan16:40
milannight NobodyCam :)16:40
NobodyCam:)16:40
openstackgerritRuby Loo proposed openstack/ironic: minor changes to security documentation  https://review.openstack.org/32056616:42
*** _vs has quit IRC16:43
*** _vsaienko has quit IRC16:43
vdrokmorning rloo and NobodyCam16:44
vdrokand good night everyone :)16:44
NobodyCammorning vdrok how are you doing today16:45
*** milan has quit IRC16:45
vdrokNobodyCam: doing good :) what about you?16:46
rloohi vdrok16:46
NobodyCamI could use more time off but other wise doing good16:46
thiagopgood night vdrok16:46
thiagopNobodyCam: how was the rv trip?16:47
NobodyCammorning thiagop :) it was awesome :)16:47
*** daemontool_ has quit IRC16:49
*** harlowja has joined #openstack-ironic16:51
*** ifarkas has quit IRC16:53
*** mbound has quit IRC16:54
lucasagomesvdrok, night16:54
*** fellypefca has quit IRC16:55
*** e0ne has joined #openstack-ironic16:59
*** ChrisAusten has joined #openstack-ironic17:00
lucasagomesI will call it a day as well17:01
lucasagomeshave a great evening all, talk to you tomorrow :-(17:02
lucasagomes:-)*17:02
*** lucasagomes is now known as lucas-afk17:02
openstackgerritVincent S. Cojot proposed openstack/ironic: Make the vbox driver work on headless VirtualBox machines  https://review.openstack.org/31127817:04
*** joprovost1 has joined #openstack-ironic17:04
*** joprovost has quit IRC17:04
*** joprovost1 is now known as joprovost17:04
*** trown is now known as trown|lunch17:08
openstackgerritSam Betts proposed openstack/ironic-python-agent: Support LLDP data as part of interfaces in invertory  https://review.openstack.org/32058417:10
sambettsdtantsur: ^ first attempt to rework our downstream solution into a upstream one17:11
* sambetts -> home 17:11
sambettsnight all17:11
NobodyCamhave a good night lucas-afk17:11
NobodyCamnight sambetts17:11
*** sambetts is now known as sambetts|afk17:11
* devananda sees LLDP and looks17:11
devanandag'night sambetts|afk !17:11
sambetts|afk:D cya devananda17:11
NobodyCamgood morning devananda17:12
* sambetts|afk just spotted spelling mistake in commit message... will fix 17:12
*** e0ne has quit IRC17:12
openstackgerritSam Betts proposed openstack/ironic-python-agent: Support LLDP data as part of interfaces in inventory  https://review.openstack.org/32058417:13
JayFsambetts|afk: ^ that is disabled by default, I hope?17:14
JayFsambetts|afk: otherwise would have a material impact on the total deploy + cleaning time17:14
JayFsambetts|afk: because waiting for LLDP could take 30-60s or longer depending on switch configuration :/17:14
devanandaalso, are we guaranteed to get any LLDP packets this late inthe process?17:15
*** moshele has joined #openstack-ironic17:17
devanandaJayF: "or longer" yea ... juniper's maximum value for lldp advertisement interval is 9 hours17:18
dtantsursambetts|afk, awesome, g'night17:18
dtantsuroh, yeah.. I remember our downstream ramdisk to wait something like a minute17:19
dtantsuralso morning devananda17:19
JayFWe currently use the lldp code in part of the clean process in downstream17:19
JayFto confirm port mappings17:19
JayFand it can take 30s+ in a good situation, and wait damn near forever if something goes wrong17:20
dtantsurthese constants "wait until something happens" piss me of for real17:20
* dtantsur imagines what JayF will say when he sees https://review.openstack.org/31874517:20
dtantsuroh, you did already17:20
JayFyep17:20
JayFdefaulted to 017:20
JayFI totally get that some hardware has to wait like that :)17:21
dtantsurthat's terrible, but yeah17:21
JayFand if you care about deployment speed, you don't buy that hardware17:21
dtantsuralso good catch about per node configuration17:21
JayFif it had not defaulted to 0, I would have -1'd it :)17:21
dtantsurI would have -1'ed it myself :D17:22
dtantsurand thanks for reviews17:22
*** ayoung has quit IRC17:22
*** ayoung has joined #openstack-ironic17:23
*** athomas has quit IRC17:25
*** ChubYann has joined #openstack-ironic17:26
*** ricardoas has quit IRC17:27
*** jralph has quit IRC17:29
*** e0ne has joined #openstack-ironic17:30
*** athomas has joined #openstack-ironic17:31
dtantsursee you tomorrow17:32
*** dtantsur is now known as dtantsur|afk17:32
*** ayoung has quit IRC17:35
*** ayoung has joined #openstack-ironic17:36
*** ricardoas has joined #openstack-ironic17:36
*** baoli_ has quit IRC17:40
NobodyCamnight dtantsur|afk17:41
*** ricardoas has quit IRC17:42
*** praneshp has joined #openstack-ironic17:47
*** trown|lunch is now known as trown17:50
*** mgould is now known as mgould|afk17:53
*** Sukhdev has joined #openstack-ironic17:54
*** mbound has joined #openstack-ironic17:54
*** jralph has joined #openstack-ironic17:58
*** jralph has quit IRC17:58
*** jralph has joined #openstack-ironic17:59
*** mbound has quit IRC17:59
*** jralph has quit IRC18:00
*** jralph has joined #openstack-ironic18:01
openstackgerritVincent S. Cojot proposed openstack/ironic: Make the vbox driver work on headless VirtualBox machines  https://review.openstack.org/31127818:05
*** mgoddard_ has quit IRC18:06
ElCoyote_jroll: I rolled back my latest change: I only want to poweroff the VM forcibly if the end-user has requested 'headless' mode. It broke stuff in Jenkins and I am not even sure the end-user would want that on a desktop.18:06
*** athomas has quit IRC18:07
jrollElCoyote_: okay, fair enough18:07
*** mbound has joined #openstack-ironic18:08
ElCoyote_so basically PatchSet 28 is PatchSet 25.18:08
*** joprovost1 has joined #openstack-ironic18:09
*** joprovost has quit IRC18:09
*** joprovost1 is now known as joprovost18:09
ElCoyote_what's the relationship between drivers/modules/virtualbox.py and drivers/modules/ssh.py (the vbox part)..? Aren't these two different drivers?18:09
jrollthey are - virtualbox.py uses the virtualbox web api thing instead of vboxmanage18:11
*** sdake_ is now known as sdake18:11
ElCoyote_jroll: ok, so it appears Bin Lu patched virtualbox.py to poweroff the VM before setting boot device too.. and I patched ssh.py (the vbox part)18:11
jrollElCoyote_: heh, nice18:12
*** jralph_ has joined #openstack-ironic18:12
*** david-lyle_ has joined #openstack-ironic18:12
ElCoyote_his patch looks cleaner and more involved than mine (it saves the current boot device and does other magic tricks).18:12
ElCoyote_...but we both fixed Vbox.. :)18:13
*** jaybeale has quit IRC18:13
ElCoyote_I'm gonna change the first line of my commit to reflect that..18:13
openstackgerritVincent S. Cojot proposed openstack/ironic: Make the ssh driver work on headless VirtualBox machines  https://review.openstack.org/31127818:15
*** jrist has quit IRC18:15
ElCoyote_jroll: if you're ok with that patchset (includes the unittests you wrote.. Thanks for that).. Would you mind +1'ing ?18:15
*** jrist has joined #openstack-ironic18:16
*** jralph has quit IRC18:16
*** david-lyle has quit IRC18:16
jrollElCoyote_: sure, lemme look18:16
*** electrofelix has quit IRC18:16
jrolldone18:17
*** ijw has joined #openstack-ironic18:17
*** ijw has quit IRC18:17
*** ijw has joined #openstack-ironic18:17
*** swatson has quit IRC18:18
*** fragatina has joined #openstack-ironic18:22
*** ig0r_ has joined #openstack-ironic18:23
*** rcernin has quit IRC18:29
*** rbrud has joined #openstack-ironic18:31
*** mbound has quit IRC18:35
*** praneshp has quit IRC18:38
*** praneshp has joined #openstack-ironic18:39
*** clenimar has joined #openstack-ironic18:39
*** jvgrant has quit IRC18:45
*** links has quit IRC18:48
*** e0ne has quit IRC18:53
*** jvgrant has joined #openstack-ironic18:54
NobodyCamis that 31127818:55
NobodyCamI have it open to review18:55
*** causten_ has joined #openstack-ironic19:00
*** ChrisAusten has quit IRC19:03
*** jaybeale has joined #openstack-ironic19:12
openstackgerritAndre keedy proposed openstack/ironic-inspector: Replace node data via uuid/data/unprocessed API  https://review.openstack.org/30837919:17
openstackgerritXavier proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers  https://review.openstack.org/28619219:18
*** ishant has quit IRC19:20
*** sdake has quit IRC19:21
*** sdake has joined #openstack-ironic19:21
*** _vs has joined #openstack-ironic19:21
*** _vsaienko has joined #openstack-ironic19:21
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643219:23
*** baoli has joined #openstack-ironic19:27
*** _vsaienko has quit IRC19:27
*** _vs has quit IRC19:29
*** _vsaienko has joined #openstack-ironic19:29
*** _vs has joined #openstack-ironic19:29
*** sdake has quit IRC19:30
*** sdake has joined #openstack-ironic19:31
openstackgerritMerged openstack/ironic-lib: Clarify which projects are meant to use the ironic-lib  https://review.openstack.org/31925119:32
*** e0ne has joined #openstack-ironic19:38
*** baoli has quit IRC19:39
*** ig0r_ has quit IRC19:39
openstackgerritXavier proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers  https://review.openstack.org/28619219:41
*** dmk0202 has joined #openstack-ironic19:54
*** _vs has quit IRC19:55
*** _vsaienko has quit IRC19:55
*** rbrud has quit IRC20:01
*** sdake_ has joined #openstack-ironic20:02
*** nicodemos is now known as nico-afk20:05
*** sdake has quit IRC20:05
*** baoli has joined #openstack-ironic20:07
*** sabeen has quit IRC20:09
*** _vs has joined #openstack-ironic20:12
*** _vsaienko has joined #openstack-ironic20:12
*** lhcheng has joined #openstack-ironic20:13
*** Sukhdev has quit IRC20:13
openstackgerritXavier proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers  https://review.openstack.org/28619220:14
*** _vs has quit IRC20:15
*** _vsaienko has quit IRC20:15
openstackgerritThiago Paiva Brito proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers  https://review.openstack.org/28619220:17
*** rbudden has quit IRC20:18
*** rbudden has joined #openstack-ironic20:18
JayFthiagop: did you ever file that bug you say you found in the ironicclient yesterday?20:18
thiagopJayF: I was trying to reproduce it first, but got overwhelmed by internal affairs20:18
*** sabeen has joined #openstack-ironic20:19
*** ayoung has quit IRC20:21
*** _vsaienko has joined #openstack-ironic20:23
*** _vs has joined #openstack-ironic20:23
*** afaranha has quit IRC20:25
*** baoli has quit IRC20:29
*** baoli has joined #openstack-ironic20:29
*** baoli has quit IRC20:34
*** baoli has joined #openstack-ironic20:35
*** rbrud has joined #openstack-ironic20:38
*** Egyptian[Home] has quit IRC20:38
*** Egyptian[Home]1 has joined #openstack-ironic20:38
openstackgerritThiago Paiva Brito proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers  https://review.openstack.org/28619220:44
*** e0ne has quit IRC20:49
*** jaybeale_ has joined #openstack-ironic20:49
*** _vsaienko has quit IRC20:50
*** _vs has quit IRC20:50
*** e0ne has joined #openstack-ironic20:50
*** jaybeale has quit IRC20:51
*** mbound has joined #openstack-ironic20:55
*** wajdi_ has joined #openstack-ironic20:56
*** fragatina has quit IRC20:57
*** jaybeale has joined #openstack-ironic20:58
*** jaybeale_ has quit IRC20:58
*** wajdi has quit IRC20:59
*** mbound has quit IRC21:00
*** wajdi has joined #openstack-ironic21:00
*** rcernin has joined #openstack-ironic21:03
*** wajdi_ has quit IRC21:03
*** Sukhdev has joined #openstack-ironic21:04
*** ayoung has joined #openstack-ironic21:05
*** wajdi has quit IRC21:05
*** Egyptian[Home]1 has quit IRC21:06
*** jjohnson2 has quit IRC21:07
*** jralph_ is now known as jralph21:09
*** jralph is now known as jralph_21:10
*** [1]cdearborn has joined #openstack-ironic21:10
*** trown is now known as trown|outtypewww21:12
*** jralph_ has quit IRC21:12
*** jralph has joined #openstack-ironic21:12
*** _vs has joined #openstack-ironic21:12
*** _vsaienko has joined #openstack-ironic21:12
*** fragatina has joined #openstack-ironic21:20
*** vishwanathj has joined #openstack-ironic21:20
rloohey JayF, wrt https://review.openstack.org/#/c/301526/. I'm not questioning the ability to specify the prefix or delimiter.21:25
*** cdearborn has quit IRC21:25
rlooJayF: I am questioning why the library has to generate the prefix from possibly a list of strings.21:25
JayFah, okay21:25
JayFgot it21:25
JayFI didn't understand the comment21:25
*** baoli has quit IRC21:25
JayFalthough I am ... interested in how we can factor that without having a CONF.host in the ironic-lib21:25
rlooJayF: the code has 'prefix_parts', 'name_parts', and honestly, I find it ugly to read.21:26
*** _vs has quit IRC21:26
*** baoli has joined #openstack-ironic21:26
rlooJayF: huh? Why can't you pass the host when initializing/getting the logger?21:26
JayFwell that makes a whole hell of a lot of sense21:26
rlooJayF: and what does that have to do with the prefix thing?21:26
JayFrloo: just the other thing about that patch that was unsettled mentally for me21:27
*** _vsaienko has quit IRC21:27
rlooJayF: Oh, sorry. Maybe I could have worded it differently. I'm a bit annoyed that I didn't question all this the first time I looked at the patch.21:27
JayFit's fine, honestly I think my understanding of it is muddied by having run similar code in a downstream patch :/21:28
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: OneView client with managers  https://review.openstack.org/28680321:28
JayFI thought at one point it'd be more helpful to have run some version of the code before reviewing it21:28
JayFnow I think the bias it provokes probably overrides the operational experience it gives21:28
rlooJayF: that's a problem of being "too close" to the code. That's why we have reviewers :)21:29
rlooJayF: anyway, it is always a classic complaint in school. Why didn't I get full marks if my program works. (Cuz it ain't up to snuff son. or whatever.)21:30
jlvillalkrtaylor, Any chance you could run the QA meeting tomorrow?21:30
*** e0ne has quit IRC21:30
jlvillalkrtaylor, I am in class all week21:30
*** dmk0202 has quit IRC21:32
*** _vsaienko has joined #openstack-ironic21:33
*** _vs has joined #openstack-ironic21:33
openstackgerritGabriel Bezerra proposed openstack/python-oneviewclient: OneView client with managers  https://review.openstack.org/28680321:34
*** sdake has joined #openstack-ironic21:34
*** ametts has quit IRC21:35
*** dmk0202 has joined #openstack-ironic21:37
*** Egyptian[Home] has joined #openstack-ironic21:37
*** sdake_ has quit IRC21:38
*** lindycoder has quit IRC21:41
*** rcernin has quit IRC21:43
*** stevebak` is now known as stevebaker21:46
*** penick_ has joined #openstack-ironic21:46
*** penick has quit IRC21:46
*** penick_ is now known as penick21:46
mrdaMorning Ironic21:50
*** Sukhdev has quit IRC21:50
NobodyCammorning mrda21:50
mrdao/21:50
*** Sukhdev has joined #openstack-ironic21:53
*** _vsaienko has quit IRC21:54
*** _vs has quit IRC21:55
*** baoli has quit IRC21:59
*** baoli has joined #openstack-ironic21:59
*** _vsaienko has joined #openstack-ironic22:01
*** _vs has joined #openstack-ironic22:01
*** penick has joined #openstack-ironic22:01
*** penick has quit IRC22:02
xavierrmorning mrda :)22:06
*** xavierr is now known as xavierr_afk22:06
*** penick has joined #openstack-ironic22:07
*** moshele has quit IRC22:07
*** dmk0202 has quit IRC22:09
*** sdake has quit IRC22:09
*** _vsaienko has quit IRC22:09
*** _vs has quit IRC22:09
*** _vsaienko has joined #openstack-ironic22:14
*** _vs has joined #openstack-ironic22:14
*** rloo has quit IRC22:15
*** sdake has joined #openstack-ironic22:19
*** jaybeale has quit IRC22:19
*** thiagop has quit IRC22:20
*** [1]cdearborn has quit IRC22:20
*** _vsaienko has quit IRC22:21
*** _vs has quit IRC22:22
*** _vsaienko has joined #openstack-ironic22:24
*** _vs has joined #openstack-ironic22:24
*** Egyptian[Home] has quit IRC22:25
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/31693722:28
openstackgerritMerged openstack/ironic: Remove unused is_valid_cidr method  https://review.openstack.org/32039022:29
*** _vs has quit IRC22:30
*** _vsaienko has quit IRC22:31
*** _vsaienko has joined #openstack-ironic22:36
*** _vs has joined #openstack-ironic22:36
*** _vs has quit IRC22:39
*** _vsaienko has quit IRC22:40
*** Sukhdev has quit IRC22:43
*** _vsaienko has joined #openstack-ironic22:46
*** _vs has joined #openstack-ironic22:47
*** _vs has quit IRC22:52
*** _vsaienko has quit IRC22:52
*** _vsaienko has joined #openstack-ironic22:53
*** _vs has joined #openstack-ironic22:54
*** baoli has quit IRC22:55
*** trandles has left #openstack-ironic22:56
*** absubram has quit IRC22:58
*** _vs has quit IRC23:01
*** _vsaienko has quit IRC23:02
*** openstackgerrit has quit IRC23:03
*** vishwanathj has quit IRC23:03
*** openstackgerrit has joined #openstack-ironic23:03
*** _vsaienko has joined #openstack-ironic23:05
*** _vs has joined #openstack-ironic23:05
*** Egyptian[Home] has joined #openstack-ironic23:10
*** david-lyle_ is now known as david-lyle23:12
*** _vs has quit IRC23:16
*** _vsaienko has quit IRC23:18
*** aNupoisc has joined #openstack-ironic23:20
*** jcoufal has joined #openstack-ironic23:20
*** _vsaienko has joined #openstack-ironic23:20
*** _vs has joined #openstack-ironic23:21
*** Sukhdev has joined #openstack-ironic23:21
aNupoiscHi guys. I am trying to do bifrost on few nodes for the first time and have encountered a problem while running enroll-dynamic.yaml23:21
aNupoiscIt seems to be api_timeout error23:21
aNupoiscI checked Ironic is running23:22
aNupoischere is the error http://pastebin.com/2QAQNzic23:22
aNupoisccan anyone help me to deal with it23:22
cineramahi aNupoisc23:22
aNupoischi cinerama23:23
*** ptx0 has quit IRC23:24
*** ptx0 has joined #openstack-ironic23:26
cineramaaNupoisc, anything interesting in ironic's logs?23:27
*** _vsaienko has quit IRC23:27
aNupoisccinerama: can you tell the path for ironic logs23:28
*** _vs has quit IRC23:28
aNupoiscCINERAMA: i think ironic service not getting connected23:29
aNupoiscfrom api_timeout23:29
*** thrash is now known as thrash|g0ne23:31
cineramaaNupoisc: oh the api_timeout in your output? that's one of the parameters supplied to the os_ironic module in ansible that we use to drive ironic23:32
aNupoiscyes the one in the output23:33
aNupoisccinemar: so i mean where do you think the problem lies23:33
*** _vsaienko has joined #openstack-ironic23:33
*** _vs has joined #openstack-ironic23:34
cineramaaNupoisc: try /var/log/upstart/ironic* for logs23:36
aNupoisccinerama: i can see in ironic-api.log keystonemiddleware.auth_token - Rejecting request23:39
*** Sukhdev has quit IRC23:39
aNupoiscit is unable to establish connection to 127.0.0.1: 3535723:39
*** rbudden has quit IRC23:39
cineramaaNupoisc, okay that gives me more to work with23:39
*** _vs has quit IRC23:40
cineramaaNupoisc, did you install ironic with bifrost or on your own23:40
*** _vsaienko has quit IRC23:40
aNupoiscso i run env-setup and then run bifrost-install23:42
aNupoiscso it should set ironic right?23:42
aNupoisccinemark: but i am windering Ironic API is accessible without authentication right?23:43
*** causten_ has quit IRC23:44
cineramaaNupoisc, yes it is. did you change the setting for noauth_mode when you installed?23:44
aNupoiscnope23:45
*** _vsaienko has joined #openstack-ironic23:45
*** _vs has joined #openstack-ironic23:45
aNupoisccan i check what is noauth_mode set to?23:45
cineramaaNupoisc, so the ways you would change the settings used by the playbooks are 1) on the command line using -e or 2) in playbooks/inventory/group_vars23:46
cineramaaNupoisc, also check auth_strategy23:47
aNupoisccinerama: so this i should check inside host deployer right? or inside VM23:48
cineramaaNupoisc, on the deployer23:48
yuikotakadamorigood morning, ironic23:48
aNupoisccinerama: i did not submittted anything with -e23:48
aNupoisci will check grpoup_vars23:48
cineramaaNupoisc, also what happens when you do "ironic node-list"23:49
*** Sukhdev has joined #openstack-ironic23:49
*** _vs has quit IRC23:51
aNupoisccinemark: i don't see auth_strategy or noauth_mode in group_vars23:52
*** oomichi has joined #openstack-ironic23:52
aNupoisccinerama: In VM ironic node-list says Invalid openstack identity credentials23:53
*** _vsaienko has quit IRC23:53
aNupoiscin deployer ironic is not installed23:53
*** sdake has quit IRC23:55
*** _vsaienko has joined #openstack-ironic23:55
*** _vs has joined #openstack-ironic23:56
cineramaaNupoisc, okay so right now i'm only worried about the machine that bifrost and ironic are on23:57
aNupoisccinerama: so means my VM23:57
aNupoisccinerama: i confused you :)23:58
cineramaaNupoisc: if you're running ironic and bifrost there, then we're talking about the VM :)23:58
aNupoiscso it says Invalid openstack identity credentials23:59

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