Tuesday, 2014-12-09

*** smoriya has joined #openstack-ironic00:00
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Use oslo.utils and oslo.concurrency  https://review.openstack.org/13811600:07
* NobodyCam steps away to look for some food00:07
*** arif-ali has quit IRC00:09
*** arif-ali has joined #openstack-ironic00:12
JayFhttps://review.openstack.org/#/c/92627/13 needs a final core review to land00:13
*** romcheg1 has joined #openstack-ironic00:19
jroll^ +A00:21
*** romcheg has quit IRC00:21
NobodyCamlol /me stops looking00:21
jrollNobodyCam: actually00:21
jrollmy name is on that patch00:21
JayFNobodyCam: it's okay, jroll was the original author, you should probably look00:22
jrollso your review would be good00:22
jrollbut I don't think I wrote any of that code :P00:22
JayFNobodyCam: also it won't merge cleanly so someone has to rebase it00:22
*** romcheg has joined #openstack-ironic00:31
NobodyCamany reason you would want to add a log around line 78 of :https://review.openstack.org/#/c/92627/13/ironic_python_agent/netutils.py00:33
*** romcheg1 has quit IRC00:33
JayFSeems like a useful addition, and another patchset has to be published now anyway, so ...00:34
NobodyCamthats about all I see00:35
*** Masahiro has joined #openstack-ironic00:38
*** eghobo has joined #openstack-ironic00:42
*** Masahiro has quit IRC00:43
*** andreykurilin_ has quit IRC00:44
*** romcheg has left #openstack-ironic00:44
*** eghobo has quit IRC00:45
*** Marga_ has quit IRC00:47
*** eghobo has joined #openstack-ironic00:47
*** eghobo has quit IRC00:49
*** Marga_ has joined #openstack-ironic00:52
*** eghobo has joined #openstack-ironic00:52
*** smoriya has quit IRC00:55
*** Marga_ has quit IRC00:56
*** eghobo has quit IRC01:02
*** eghobo has joined #openstack-ironic01:03
*** eghobo has quit IRC01:04
*** smoriya has joined #openstack-ironic01:06
*** spandhe has joined #openstack-ironic01:24
*** killer_prince is now known as lazy_prince01:26
*** shakamunyi has quit IRC01:28
openstackgerritMerged openstack/ironic: Fix image_info passed to IPA for image download  https://review.openstack.org/13998801:35
*** marcoemorais has quit IRC01:40
*** marcoemorais has joined #openstack-ironic01:41
*** jerryz has joined #openstack-ironic01:42
*** alexpilotti has joined #openstack-ironic01:44
*** ryanpetrello has joined #openstack-ironic01:45
*** chenglch has joined #openstack-ironic01:48
*** ChuckC has quit IRC01:50
*** alexpilotti has quit IRC02:06
adam_ganyone happen to know if its possible to push MTU settings per neutron port via extra_dhcp_opts?02:07
*** ryanpetrello has quit IRC02:10
Haomengadam_g: not sure if veth_mtu in ovs_neutron_plugin.ini can help you02:11
jrolladam_g: I kind of doubt it, but I'd ask neutron02:12
*** ChuckC has joined #openstack-ironic02:12
cuihaozhiHaomeng: morning :)02:16
*** ChuckC has quit IRC02:16
*** ChuckC has joined #openstack-ironic02:16
adam_gHaomeng, was hoping to set it on the client side. you can set 'dhcp-option-force=26,1300' in a file and configure the dhcp agent to pass as an additional --conf-file to dnsmasq, but curious if thats possible in the same way we set up tftp/pxe stuff per-port02:17
*** lazy_prince has quit IRC02:20
*** Masahiro has joined #openstack-ironic02:26
*** marcoemorais has quit IRC02:31
*** Masahiro has quit IRC02:31
*** marcoemorais has joined #openstack-ironic02:31
*** killer_prince has joined #openstack-ironic02:33
*** killer_prince is now known as lazy_prince02:33
*** rloo has quit IRC02:40
Haomengadam_g: ok02:41
*** ramineni has joined #openstack-ironic02:43
ramineniJoshNang: hi02:44
*** marcoemorais has quit IRC02:48
*** dlaube has quit IRC03:11
*** spandhe has quit IRC03:20
*** david-lyle is now known as david-lyle_afk03:24
*** ryanpetrello has joined #openstack-ironic03:27
*** Masahiro has joined #openstack-ironic03:42
raminenijroll, JayF: hi03:44
*** Masahiro has quit IRC03:47
*** bradjones has quit IRC03:51
*** achanda has joined #openstack-ironic03:59
*** ryanpetrello has quit IRC04:02
*** dlaube has joined #openstack-ironic04:06
*** achanda has quit IRC04:17
*** achanda has joined #openstack-ironic04:17
*** rameshg87 has joined #openstack-ironic04:21
*** lazy_prince has quit IRC04:22
*** achanda has quit IRC04:22
*** lazy_prince has joined #openstack-ironic04:22
*** wanghua has quit IRC04:24
*** pcrews has quit IRC04:29
*** chenglch has quit IRC04:33
*** rushiagr_away is now known as rushiagr04:34
*** achanda has joined #openstack-ironic04:37
*** achanda has quit IRC04:40
*** Masahiro has joined #openstack-ironic04:43
*** Masahiro has quit IRC04:47
devanandaevening, all04:51
devanandareminder -- meeting is in 10 minutes04:51
* devananda wonders if anyone is around :)04:55
lintanI will join04:55
jrollramineni: hi, what's up?04:55
raminenijroll : hi , IPA supports disk_erase during zapping right?  which tool IPA uses to do disk_erase?04:57
jrollramineni: not upstream (yet) but yes04:58
jrollit uses hdparm for normal SSDs04:58
raminenijroll : is it opensource tool? I want to explore the same for our servers too :)04:58
mrda\o04:59
victor_lowtherhdparm has been opensource for at least 15 years. :)04:59
jrollramineni: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L272-27904:59
jrollramineni: ironic does not call that command upstream at the moment, though04:59
* victor_lowther remembers using it to enable IRQ batching and MWDMA on old ATA controllers for great performance boosts.05:00
jrollramineni: this is how we do our fancy LSI nytro cards https://github.com/rackerlabs/onmetal-ironic-hardware-manager/blob/master/onmetal_ironic_hardware_manager/__init__.py#L18105:00
raminenijroll : thanks , will check :)05:00
jroll:)05:00
devanandameeting time!05:00
devanandaremember, new room -- #openstack-meeting-305:00
jrollramineni: we're running it in production today, it works well05:00
jrollramineni: however, we don't erase spinning disks at the moment05:01
Haomengok05:01
raminenivictor_lowther: :P , my bad , havent used it..so no idea05:01
raminenijroll: oohok05:01
* jroll wonders what time it is for dtantsur|afk and friends05:03
sambettsjroll: Its 5am for me and 6am for dtantsur|afk05:07
jrollaha05:08
jrollthanks :)05:08
*** rushiagr is now known as rushiagr_away05:09
sambettsjroll: what time is it for you guys?05:10
jrollsambetts: 9pm in cali05:11
sambettsah, thanks05:12
sambetts:)05:13
*** pensu has joined #openstack-ironic05:14
*** harlowja is now known as harlowja_away05:16
*** dlaube has quit IRC05:17
devanandaNobodyCam: dont suppose you're around?05:24
*** dlaube has joined #openstack-ironic05:29
*** dlaube has quit IRC05:30
*** Nisha has joined #openstack-ironic05:30
*** yuanying_ has quit IRC05:37
*** Masahiro has joined #openstack-ironic05:44
*** rakesh_hs has joined #openstack-ironic05:45
*** yuanying has joined #openstack-ironic05:46
*** Masahiro has quit IRC05:48
*** faizan_ has joined #openstack-ironic06:02
jrolllintan: hmm, maybe you're right, idk06:02
jrolllike06:03
jrollthis has been tested in devstack06:03
jrollwith dnsmasq06:03
*** chenglch has joined #openstack-ironic06:03
lintanjroll: so it's wired06:04
jrollhm?06:05
lintanjroll: I didn't try devstack but it doesn't work06:05
jrollinteresting06:05
sirushtiHi all, I brought this up in the meeting but could this spec https://review.openstack.org/#/c/97150/ get some reviews please?06:06
jrolllintan: sorry, but I need to go, let's chat more later06:06
jrollg'night everyone06:07
lintanjroll: OK, lucas test on my commit and approve it, we can ask him next week06:07
jrollok, thanks06:07
lintangood night06:07
devanandaI'm also fading fast ... see everyone tomorrow!06:10
*** lazy_prince has quit IRC06:13
*** rushiagr_away is now known as rushiagr06:14
mrdanight devananda06:14
*** Marga_ has joined #openstack-ironic06:14
Haomengdevananda: night:)06:21
openstackgerritDevananda van der Veen proposed openstack/ironic-specs: fix the filename for hardware-capabilities.rst  https://review.openstack.org/14025206:22
*** killer_prince has joined #openstack-ironic06:25
*** killer_prince is now known as lazy_prince06:25
*** faizan_ has quit IRC06:39
*** rameshg87 has quit IRC06:41
*** rameshg87 has joined #openstack-ironic06:41
jiangfeihi, haomeng06:42
Haomenghi06:42
Haomengjiangfei: hi06:42
*** russellb has quit IRC06:43
*** lazy_prince has quit IRC06:44
jiangfeinode-create should valid the input driver do you mean, we use curl -x post /node -d '{"driver": "xxxxxx"}' has faultstring .but ironic node-create just Bad Request (HTTP 400), no faulutstring06:44
*** russellb has joined #openstack-ironic06:44
*** lazy_prince has joined #openstack-ironic06:45
Haomengjiangfei: yes06:45
Haomengjiangfei: it is better that we valid the driver, and tell user the input driver is invalid06:46
Haomengjiangfei: you can run with command line06:46
jiangfeiHaomeng: it's also modify the error message?06:46
Haomengjiangfei: I am prepare to commit the patch, but not sure if it is handled by cliend side06:46
Haomengjiangfei: two cases, the one is that the driver is invalid/not enabled, it is not listed in our ironic.conf enable_driver06:47
jiangfeiHaomeng: yes06:47
Haomengjiangfei: another one is that the driver is valid, it is listed in conf, but no any conductor support such driver06:47
Haomengjiangfei: so can not find the conductor06:48
jiangfeiHaomeng: nice06:48
Haomengjiangfei: maybe no any conductor is started06:48
Haomengjiangfei: but it is complex if we have multi conductor06:48
Haomengjiangfei: so current behavior is that we try to lookup all conductors to find which supports the input driver06:49
jiangfeiHaomeng: yes06:49
Haomengjiangfei: in fact, our drivers are plugable, so we can check from python exts to see if it is loaded already06:49
Haomengjiangfei: any idea here?06:50
jiangfeiHaomeng: i would like to '  check from python exts to see if it is loaded already'06:51
Haomengjiangfei: yes, maybe the conductor side code to be changed to check python exts first06:52
Haomengjiangfei: then to check for each conductors06:52
Haomengjiangfei: but have to check each conductor env's python exts06:52
Haomengjiangfei: the conductors can be distributed06:52
Haomengjiangfei: so here some complex06:53
jiangfeiHaomeng:  the conductors can has different drivers?06:53
Haomengjiangfei: yes, it can be installed on different nodes with different confs06:53
*** marcoemorais has joined #openstack-ironic06:56
*** marcoemorais1 has joined #openstack-ironic06:58
jiangfeiHaomeng: may be we should add retry times in  conductors in this case.07:00
*** marcoemorais has quit IRC07:01
Haomengjiangfei: I think we just check from conductor side to see if python exts is loaded07:01
jiangfeiHaomeng: if not return the error message , or try to check it  the other conductors07:03
*** Marga_ has quit IRC07:05
Haomengjiangfei: I am not sure if driver-list can list all conductors drivers07:05
Haomengjiangfei: if it is case, we can check with driver-list result07:05
Haomengjiangfei: have to check driver-list code logic07:06
jiangfeiHaomeng: yes07:06
Haomengjiangfei: :)07:07
Haomengjiangfei: if you'd like to fix this, go ahead please07:07
Haomengjiangfei: dont worry07:07
jiangfeiHaomeng: thanks, i will07:07
Haomengjiangfei: thanks07:07
Haomengjiangfei: if our ironic driver-list can not list all dist-conductor's drivers, that should be another bug I think:)07:11
jiangfeiHaomeng:i will test it07:13
Haomengjiangfei: we can check the code logic, that we will know if it support multi-conductors07:14
Haomengjiangfei: and we can levreage the logic to do pre-check for node-craete -d xxx07:14
*** k4n0 has joined #openstack-ironic07:15
jiangfeiHaomeng:ok07:17
Haomengtjiangfei: hank you07:17
jiangfeiHaomeng: :)07:18
Haomengjiangfei: :)07:18
*** Masahiro has joined #openstack-ironic07:33
*** Masahiro has quit IRC07:37
*** ifarkas has joined #openstack-ironic07:40
*** mrda is now known as mrda-away08:07
*** Nisha has quit IRC08:16
*** jistr has joined #openstack-ironic08:20
openstackgerritDmitry Tantsur proposed openstack/ironic-specs: Set blueprint link for drivers capabilities spec  https://review.openstack.org/14026708:26
*** dtantsur|afk is now known as dtantsur08:27
dtantsurMorning08:27
*** ndipanov_gone is now known as ndipanov08:29
dtantsurdevananda, https://review.openstack.org/140267 <-- blueprint for driver capabilities08:31
*** jcoufal has joined #openstack-ironic08:33
*** Masahiro has joined #openstack-ironic08:33
*** MattMan has quit IRC08:33
dtantsurifarkas, o/ once you have some time, could you start reviewing a long (6 for now) patch queue starting with https://review.openstack.org/#/c/140064/ ?08:35
*** MattMan has joined #openstack-ironic08:36
*** Masahiro has quit IRC08:38
ifarkasdtantsur, sure, will do ;-)08:39
dtantsurcool!08:39
* dtantsur did some refactoring yesterday08:39
dtantsurifarkas, fun thing: zuul was so surprised with my 4 patches uploaded yesterday, that I have to use 'recheck' today to make it run tests on them :)08:41
ifarkasdtantsur, that must have been a nice surprise ;-)08:42
dtantsurheh08:42
rameshg87ifarkas, hi08:43
ifarkasrameshg87, hi08:43
rameshg87ifarkas, there was lot of discussions yesterday on RAID configuration. had a chance to see them ?08:43
rameshg87ifarkas, i think we should post a full spec as all agreed this is in scope08:44
ifarkasrameshg87, I haven't yet, but I will read back the log08:45
rameshg87ifarkas, okay. i am think of addressing the comments in the spec, but wanted to discuss with you08:47
rameshg87ifarkas, let me know once you have followed up on what has happened, then we can discuss08:47
ifarkasrameshg87, ok08:48
GheRiveromorning all08:48
rameshg87morning GheRivero08:49
dtantsurGheRivero, rameshg87, o/08:52
rameshg87dtantsur, morning o/08:52
*** Haomeng|2 has joined #openstack-ironic08:52
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Enable iscsi_ilo driver to share boot images  https://review.openstack.org/13729108:53
rameshg87dtantsur, when/how is a short-spec accepted for current release ?08:53
rameshg87dtantsur, by getting some +1s from cores ?08:53
*** Haomeng has quit IRC08:54
*** jcoufal_ has joined #openstack-ironic08:55
dtantsurrameshg87, short specs are accepted to backlog or converted directly to long specs after seeing some consensus on the idea08:57
dtantsurrameshg87, in your case it makes sense to convert it to long spec and retarget to specs/kilo08:57
dtantsurbrb08:57
rameshg87dtantsur, okay08:58
*** jcoufal has quit IRC08:59
*** dlpartain has joined #openstack-ironic09:08
*** dividebin has quit IRC09:11
*** dividehex has joined #openstack-ironic09:14
*** romcheg has joined #openstack-ironic09:19
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add support for VirtualBox WebService.  https://review.openstack.org/13792609:24
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add support for VirtualBox WebService.  https://review.openstack.org/13792609:26
*** athomas has joined #openstack-ironic09:28
*** derekh has joined #openstack-ironic09:35
*** erwan_taf has joined #openstack-ironic09:37
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Enable boot images sharing for iLO driver(s)  https://review.openstack.org/13729109:41
*** andreykurilin_ has joined #openstack-ironic09:42
rameshg87dtantsur, can you please have a look at some of my reviews when you get time, some them have been waiting without feedback, posted the full specs - https://review.openstack.org/137291, https://review.openstack.org/137567, https://review.openstack.org/13792609:42
*** rameshg87 is now known as rameshg87-away09:43
*** lazy_prince is now known as killer_prince09:46
*** igordcard has joined #openstack-ironic09:47
*** pelix has joined #openstack-ironic09:48
dtantsurrameshg87-away, yeah, will try to find time today09:48
*** andreykurilin_ has quit IRC09:49
*** andreykurilin_ has joined #openstack-ironic09:49
*** killer_prince is now known as lazy_prince09:49
*** dtantsur is now known as dtantsur|brb09:49
*** dlpartain has quit IRC09:50
*** andreykurilin__ has joined #openstack-ironic10:00
*** andreykurilin_ has quit IRC10:00
*** lazy_prince is now known as killer_prince10:07
*** chenglch has quit IRC10:18
*** vdrok has joined #openstack-ironic10:19
*** jistr has quit IRC10:21
*** Masahiro has joined #openstack-ironic10:22
*** Marga_ has joined #openstack-ironic10:24
*** Masahiro has quit IRC10:27
openstackgerritMerged stackforge/ironic-discoverd: Wait for power off state before calling discovery done  https://review.openstack.org/14006410:30
*** erwan_taf has quit IRC10:30
*** erwan_taf has joined #openstack-ironic10:33
*** marcoemorais1 has quit IRC10:40
*** killer_prince is now known as lazy_prince10:41
*** wanghua has joined #openstack-ironic10:41
*** ramineni has quit IRC10:43
openstackgerritMerged stackforge/ironic-discoverd: Refactoring: consolidate standard plugins in one module  https://review.openstack.org/14013410:45
openstackgerritMerged stackforge/ironic-discoverd: Refactoring: drop features incompatible with Kilo changes  https://review.openstack.org/14013510:45
*** andreykurilin__ has quit IRC10:48
*** aignatov has quit IRC10:48
openstackgerritVladyslav Drok proposed openstack/ironic: Do not strip 'glance://' prefix from image hrefs  https://review.openstack.org/13905710:48
*** jistr has joined #openstack-ironic10:49
*** aignatov has joined #openstack-ironic10:49
sambettsmorning dtantsur10:59
rameshg87-awaydtantsur|brb, thanks11:07
* rameshg87-away is going home11:07
*** rameshg87-away has quit IRC11:08
*** alexpilotti has joined #openstack-ironic11:17
*** Marga_ has quit IRC11:17
*** enterprisedc has joined #openstack-ironic11:23
*** pradipta_away is now known as pradipta11:32
pensuHey all, can devstack use the Ironic's external DHCP feature? Ref: https://blueprints.launchpad.net/devstack/+spec/ironic-dnsmasq-external-dhcp11:36
*** pradipta is now known as pradipta_away11:38
*** dtantsur|brb is now known as dtantsur11:46
dtantsursambetts, o/11:46
*** masco has joined #openstack-ironic11:56
*** pensu has quit IRC12:05
*** Masahiro has joined #openstack-ironic12:12
*** Masahiro has quit IRC12:15
openstackgerritVladyslav Drok proposed openstack/ironic: Do not strip 'glance://' prefix from image hrefs  https://review.openstack.org/13905712:17
*** Marga_ has joined #openstack-ironic12:18
*** bradjones has joined #openstack-ironic12:20
*** Marga_ has quit IRC12:23
*** smoriya has quit IRC12:23
*** wanghua has quit IRC12:44
openstackgerritArata Notsu proposed openstack/ironic: Correct vmware ssh power manager  https://review.openstack.org/13981012:45
*** alexpilotti has quit IRC12:46
*** rameshg87 has joined #openstack-ironic12:55
rameshg87vdrok, hi13:01
rameshg87vdrok, is there some bigger use-case that you have in mind rather than allowing to *also* mention glance://uuid instead of just uuid13:01
rameshg87ifarkas, hi13:01
*** rakesh_hs has quit IRC13:02
ifarkasrameshg87, hi13:03
rameshg87ifarkas, got a chance to read the backlog ? :)13:03
ifarkasrameshg87, yeah, I did that13:03
rameshg87ifarkas, okay to discuss comments right now ?13:04
ifarkasrameshg87, can we do it later? I am in the middle of something13:05
rameshg87ifarkas, sure ..13:05
ifarkasrameshg87, thanks13:05
*** dprince has joined #openstack-ironic13:08
*** Marga_ has joined #openstack-ironic13:19
*** foexle has joined #openstack-ironic13:19
*** Marga_ has quit IRC13:23
*** alexpilotti has joined #openstack-ironic13:25
vdrokrameshg87, hi :)13:39
vdrokrameshg87, it's about this spec - https://review.openstack.org/#/c/135276/13:39
rameshg87vdrok, i was actually wondering on rloo's statement13:39
rameshg87vdrok, ah okay13:40
vdrokrameshg87, at some point they should be just some url13:40
rameshg87vdrok, then why not mention the blueprint as part of the submittal13:40
rameshg87vdrok, i mean as part of commit msg13:40
vdrokrameshg87, idk, i can add this, but it's mentioned in dependency13:41
vdrokrameshg87, you mean implements bp ?13:41
rameshg87vdrok, yes13:42
rameshg87vdrok, i think that makes things a lot clear13:42
rameshg87vdrok, as to why we are doing this13:42
vdrokrameshg87, ok, will update13:42
rameshg87vdrok, thanks13:44
vdrokrameshg87, np :)13:44
rameshg87vdrok, and i did see you mentioned iLO in that review link https://review.openstack.org/#/c/139057/13:44
*** rushiagr is now known as rushiagr_away13:44
vdrokrameshg87, yup, its just cosmetic change for now13:45
vdrokrameshg87, just to not include it in the next one13:45
rameshg87vdrok, okay13:46
*** linggao has joined #openstack-ironic13:46
rameshg87vdrok, let me know if you need some info in that space :)13:46
vdrokrameshg87, ok, thanks :) actually there is one thing13:47
vdrokrameshg87, can ilovirtualmediaiscsi be used with gen7 server?13:47
vdrokrameshg87, or gen8 and ilo4 are minimal requirements?13:47
rameshg87vdrok, there are no features that we are using that are *only* available in gen813:48
rameshg87vdrok, but we didn't test it on gen713:48
vdrokrameshg87, ok, will try then at some point :) thanks13:49
rameshg87vdrok, somethings in the dependent library proliantutils are broken https://github.com/hpproliant/proliantutils/13:49
rameshg87vdrok, because gen7 ilo3 doesn't understand everything in that module13:49
rameshg87vdrok, we are in the process of fixing them actually13:49
vdrokrameshg87, i see, great13:49
rameshg87vdrok, will update you once we fix them if you are interested13:49
vdrokrameshg87, thanks13:50
openstackgerritSyed Ismail Faizan Barmawer proposed openstack/ironic-specs: Partition image support for agent driver  https://review.openstack.org/13736313:51
openstackgerritVladyslav Drok proposed openstack/ironic: Do not strip 'glance://' prefix from image hrefs  https://review.openstack.org/13905713:52
*** Masahiro has joined #openstack-ironic14:00
*** Masahiro has quit IRC14:04
*** rloo has joined #openstack-ironic14:04
*** Marga_ has joined #openstack-ironic14:06
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Refactoring: split test.py  https://review.openstack.org/14033914:16
*** k4n0 has quit IRC14:21
*** Marga_ has quit IRC14:22
*** Marga_ has joined #openstack-ironic14:22
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Refactoring: split test.py  https://review.openstack.org/14033914:26
jrollmorning, all14:26
*** ryanpetrello has joined #openstack-ironic14:26
dtantsurjroll, morning14:26
rloomorning jroll, afternoon dtantsur14:27
dtantsurrloo, hi :)14:27
*** Marga_ has quit IRC14:27
jrollhey rloo and dtantsur :)14:28
vdrokmorning jroll, rloo and dtantsur14:31
jroll\o vdrok14:31
rloohey, afternoon vdrok14:32
dtantsurvdrok, o/14:32
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Refactoring: split test.py  https://review.openstack.org/14033914:34
victor_lowthergoood morning14:36
dtantsurvictor_lowther, hi!14:36
* victor_lowther is caffinating14:36
jrollJoshNang: this needs a rebase: https://review.openstack.org/#/c/92627/14:38
*** athomas has quit IRC14:39
NobodyCamgood morning Ironic14:40
openstackgerritJim Rollenhagen proposed openstack/ironic: Remove python-tox from dev-quickstart docs  https://review.openstack.org/13922314:42
jrollmorning NobodyCam14:42
NobodyCammorning jroll14:42
*** athomas has joined #openstack-ironic14:47
*** rushiagr_away is now known as rushiagr14:48
*** alexpilotti_ has joined #openstack-ironic14:54
openstackgerritTan Lin proposed openstack/ironic: Add AMT-PXE-Driver to deploy cloud on PC  https://review.openstack.org/13518414:57
openstackgerritTan Lin proposed openstack/ironic: Add AMT Support with PXE Driver  https://review.openstack.org/14035114:57
*** alexpilotti has quit IRC14:57
*** alexpilotti_ is now known as alexpilotti14:57
*** ndipanov has quit IRC14:59
*** ndipanov has joined #openstack-ironic14:59
* dtantsur sees Fuel Agent as a nice holy war topic for the next meeting :)14:59
*** rushiagr is now known as rushiagr_away15:00
NobodyCamdtantsur: tonights meeting?15:00
* jroll just sent another email about that15:01
*** zz_jgrimm is now known as jgrimm15:01
jrollNobodyCam: it was last night :P15:01
dtantsurNobodyCam, oh, meeting is tonight? I thought it _was_ tonight, no?15:01
jrollit was tuesday 5am UTC15:01
NobodyCam0500 UTC on Tuesday15:01
jrollright now is tuesday 1500 UTC15:01
*** k4n0 has joined #openstack-ironic15:01
NobodyCamd'oh15:01
jroll:)15:01
dtantsuraha good :)15:01
jrollcheck out the meeting minutes, we made a plan for the state machine spec15:02
jrollor the logs, I guess15:02
dtantsurawesome \o/15:02
*** pawel-palucki has joined #openstack-ironic15:02
*** shakamunyi has joined #openstack-ironic15:04
*** shakamunyi has quit IRC15:04
*** shakamunyi has joined #openstack-ironic15:05
*** jgrimm is now known as zz_jgrimm15:08
*** lazy_prince is now known as killer_prince15:11
NobodyCamwow awesome plan..15:12
*** rushiagr_away is now known as rushiagr15:14
*** ChuckC_ has joined #openstack-ironic15:15
*** ChuckC has quit IRC15:15
*** Haomeng has joined #openstack-ironic15:20
*** ChuckC_ has quit IRC15:21
*** Haomeng|2 has quit IRC15:22
*** Marga_ has joined #openstack-ironic15:23
*** shakamunyi has quit IRC15:25
*** shakamunyi has joined #openstack-ironic15:25
rloojroll: I just read the meeting minutes. I didn't see anything about state machine logs. Oh wait, you meant look at the meeting logs ;)15:27
*** Marga_ has quit IRC15:28
jroll>.>15:28
NobodyCamrloo: it starts around half past the hour15:28
NobodyCamand morning15:28
rlooNobodyCam: yeah. I read that. I thought they had a plan for 'logs' for the state machine. I didn't know what that meant ;)15:29
NobodyCam:)15:29
rlooand morning to you NobodyCam and victor_lowther ;)15:29
*** dlpartain has joined #openstack-ironic15:30
*** alexm__ has joined #openstack-ironic15:31
rloobtw, victor_lowther, wrt the state machine spec, it'd be nice to put something at the top about how it is a spec describing the intention, not the implementation.15:31
*** zz_jgrimm is now known as jgrimm15:32
victor_lowtherYes.15:32
victor_lowtherI am updating it now.15:32
rloothx victor_lowther!15:32
victor_lowtherSpam me with your asks, lest things get +2'ed without your feedback. :)15:32
*** ndipanov_ has joined #openstack-ironic15:33
*** jerryz has quit IRC15:33
*** ndipanov has quit IRC15:34
alexm__hello guys, does Ironic support multiple disks per node? Conductor will alwyas chose the first disk I presume?15:34
jrollalexm__: correct15:35
rloovictor_lowther: I think the wolves are hungry and ready to pounce ... ;) We can always update/refine the wording if need be, later. I just want to make sure someone new reading it, understands the context.15:35
*** trown has quit IRC15:35
jrollalexm__: there's work being done to make it pick other disks, see https://review.openstack.org/#/c/138729/15:35
*** trown has joined #openstack-ironic15:36
alexm__jroll: thanks. HW Raid 1 should work seemlessly too?15:37
jrollalexm__: if you set it up ahead of time, I think it should... ironic does not configure RAID today15:38
openstackgerritJim Rollenhagen proposed openstack/ironic: Remove duplicate dependencies from dev-quickstart docs  https://review.openstack.org/13922315:45
jroll^ easy review15:45
*** Masahiro has joined #openstack-ironic15:48
*** ryanpetrello has quit IRC15:49
Shrewsjroll: one suggestion there... something i've been meaning to add myself15:49
jrollnice, good call15:50
* dtantsur brb15:50
openstackgerritJim Rollenhagen proposed openstack/ironic: Remove duplicate dependencies from dev-quickstart docs  https://review.openstack.org/13922315:51
jrollShrews: fixed15:51
jrollthanks for that15:51
Shrewsjroll: thx. i'm not 100% certain that nose is even needed anymore, but lgtm now15:52
jrollagree, but didn't want to look :P15:52
*** Masahiro has quit IRC15:53
*** ChuckC_ has joined #openstack-ironic15:54
NobodyCambrb15:56
*** ChuckC_ has quit IRC15:59
*** ndipanov_ has quit IRC16:01
*** jcoufal_ has quit IRC16:02
*** dlaube has joined #openstack-ironic16:03
*** masco has quit IRC16:03
*** pcrews has joined #openstack-ironic16:06
*** ChuckC_ has joined #openstack-ironic16:12
*** ChuckC_ is now known as ChuckC16:13
*** ndipanov_ has joined #openstack-ironic16:13
ShrewsI wonder if the state machine spec should call out changes needed for the nova driver....16:14
NobodyCamJayF: quick question. were you going to add that log line to 92627?16:17
rlooShrews: I thought it mentioned something about the nova driver?16:17
*** achanda has joined #openstack-ironic16:17
Shrewsrloo: oh yes... so short i missed it16:18
rlooShrews, jroll: either of you test 139223 for opensuse/sle?16:23
*** Marga_ has joined #openstack-ironic16:23
Shrewsrloo: not I16:24
openstackgerritVictor Lowther proposed openstack/ironic-specs: New Ironic provisioner state machine.  https://review.openstack.org/13382816:26
NobodyCamwoo hoo16:26
jrollrloo: no... don't see any reason why it wouldn't work16:26
rloojroll: famous last words ;) I'll approve anyway.16:26
jrollha, ok16:26
jrollthanks16:26
*** erwan_taf has quit IRC16:26
JayFNobodyCam: I personally was not, because at that point all three "j*" would have code in it. If you put the comment on it I'm sure JoshNang will16:28
*** Marga_ has quit IRC16:28
NobodyCamJayF: ack16:28
NobodyCamwill do16:28
*** enikanorov has quit IRC16:30
*** enikanorov has joined #openstack-ironic16:30
*** ryanpetrello has joined #openstack-ironic16:30
NobodyCamJayF: done16:31
*** subscope has quit IRC16:32
JayFadam_g: https://review.openstack.org/139770 is landing \o/ (your devstack fix)16:32
JoshNangNobodyCam: ahh good call. Will add today16:32
* jroll steps away for a while16:32
NobodyCam:)16:32
*** shakamunyi has quit IRC16:33
*** k4n0 has quit IRC16:34
devanandag'morning, all16:35
dtantsurdevananda, morning16:35
NobodyCamgood morning devananda16:36
NobodyCamoh btw state machine has 2 +2's16:36
dtantsurland LAND :D16:37
victor_lowtherwell, there is a spelling misteak to fix.16:38
*** dlpartain has quit IRC16:38
NobodyCam:-p16:38
Shrewsmisteaks are nawt aloud16:39
NobodyCamlol16:40
* devananda reads the fuel thread before coffee while wondering if that's a wise thing to do16:40
NobodyCamdevananda: I am on second cup and still not through it...16:41
NobodyCamlol it keeps growing as I read it16:41
*** ryanpetrello has quit IRC16:42
* dtantsur refrains for some time to get the other folks' opinions16:42
* Shrews has just realized that his misconfigured mail filter has deleted that entire thread16:44
victor_lowtherand Jenkins hates my table. :(16:44
*** dlpartain has joined #openstack-ironic16:44
NobodyCambrb16:44
dtantsurShrews, good filter :D16:45
Shrewslol16:45
*** krtaylor has quit IRC16:46
*** shakamunyi has joined #openstack-ironic16:47
rloovictor_lowther: I don't know if that table is necessary. I was asking about the verbs, to try to pin down what we actually will end up calling them. but that is out of scope from this spec anyway.16:47
*** dlpartain has quit IRC16:47
rloovictor_lowther: although it is a nice table ;)16:48
*** Marga_ has joined #openstack-ironic16:59
*** achanda has quit IRC17:01
*** Marga_ has quit IRC17:03
NobodyCamI like the table :)17:07
*** achanda has joined #openstack-ironic17:08
*** krtaylor has joined #openstack-ironic17:09
*** ifarkas is now known as ifarkas_afk17:10
devanandawow - 19 comments in under an hour?17:10
devanandaI think that's a record for one of our specs :)17:10
devanandarloo: as far as marking the spec informational, I will also do that in launchpad17:11
devanandarloo: when we tag it as completed, I will close it with the status "informational" rather than "implemented"17:11
devanandaooh. table of verbs. nice17:12
*** datajerk has joined #openstack-ironic17:12
*** ryanpetrello has joined #openstack-ironic17:13
*** masco has joined #openstack-ironic17:13
devanandarloo, victor_lowther: fwiw, I think the table provides a helpful reference. certainly for me, as I get to implementing that in the state machine17:15
*** afaranha has joined #openstack-ironic17:15
devanandait's easier to read and compare my code to, so I'm ++ on leaving it in the spec17:15
NobodyCam++ for leaving the table in the spec17:17
*** linggao has quit IRC17:17
*** linggao has joined #openstack-ironic17:18
rloodevananda: ok, if the table is useful for you ;)17:18
openstackgerritZhiQiang Fan proposed openstack/ironic: rename oslo.concurrency to oslo_concurrency  https://review.openstack.org/13978217:18
JayFis ^ that person on irc?17:19
rloodevananda: the table seemed like a diff representation (but same info) than what was in the diagram17:19
*** rameshg87 has quit IRC17:19
devanandarloo: yup17:19
openstackgerritVictor Lowther proposed openstack/ironic-specs: New Ironic provisioner state machine.  https://review.openstack.org/13382817:19
rloodevananda: maybe more useful to have the table under the diagram?17:20
JayFhttps://review.openstack.org/#/c/138116/ this is passing all tests and awaiting review17:20
victor_lowtherThat should have a table that Jenkins will not whine about.17:21
JoshNangJayF: will review17:23
JayFJoshNang: doesn't it make you warm and fuzzy to see the tempest job voting there17:24
JayFlol17:24
JoshNangJayF: ha very much so17:24
*** marcoemorais has joined #openstack-ironic17:26
JayFWhere are Ironic meeting logs kept?17:26
*** pensu has joined #openstack-ironic17:27
JayFsadly doesn't appear linked from https://wiki.openstack.org/wiki/Meetings/Ironic17:27
devanandaJayF: should be there ...17:28
NobodyCamJayF: http://eavesdrop.openstack.org/meetings/ironic/201417:28
devanandaJayF: "Previos Meetings"17:28
NobodyCamlink is on the bottom of the agenda17:28
* devananda updates the agenda after last night's meeting17:29
NobodyCamlol /me was all ready for the meeting tonight :(17:29
JayFd'oh17:30
devanandaNobodyCam: :(17:30
openstackgerritMerged openstack/ironic: Remove duplicate dependencies from dev-quickstart docs  https://review.openstack.org/13922317:31
NobodyCamhehe it was my bad..17:31
JoshNangso, with the split between cleaning and zapping, should they be separate specs? i'd lean towards yes, with zapping being dependent on cleaning17:32
*** penick has joined #openstack-ironic17:32
devanandavictor_lowther: tabs?17:33
JayFJoshNang: I agree17:33
victor_lowtherThat is how I would do it17:33
devanandavictor_lowther: 12 to 13 introduced a lot of tabs :(17:34
devanandahttps://review.openstack.org/#/c/133828/12..13/specs/kilo/new-ironic-state-machine.rst17:34
*** ChuckC has quit IRC17:34
victor_lowther(and deploying as well for that matter)17:34
victor_lowtherdevananda: I blame Emacs on a machine I have not finished setting up.17:34
devanandaheh17:35
devanandavictor_lowther: you had me at "i blame emacs"17:35
victor_lowtherso it did the Wrong Thing with M-x whitespace-cleanup17:35
NobodyCamthe tabs actually looked ok in the rendered version: http://docs-draft.openstack.org/28/133828/13/check/gate-ironic-specs-docs/01f36dd/doc/build/html/specs/kilo/new-ironic-state-machine.html17:35
rloomaybe we need a unit test to check for tabs :-)17:37
*** Masahiro has joined #openstack-ironic17:37
NobodyCamare tabs a bad for non code docs?17:38
victor_lowtherdetabbing is easy enough17:41
NobodyCamshould we set the spec url on https://blueprints.launchpad.net/ironic/+spec/new-ironic-state-machine17:42
*** Masahiro has quit IRC17:42
*** spandhe has joined #openstack-ironic17:43
victor_lowtherNobodyCam: done.17:43
NobodyCam:)17:43
devanandaNobodyCam: tabs in general are bad for formatting of text-based documents, whether code or otherwise17:43
NobodyCamack :)17:44
*** Marga_ has joined #openstack-ironic17:45
*** ChuckC has joined #openstack-ironic17:47
*** ryanpetrello has quit IRC17:52
*** ryanpetrello has joined #openstack-ironic17:54
openstackgerritVictor Lowther proposed openstack/ironic-specs: New Ironic provisioner state machine.  https://review.openstack.org/13382817:55
victor_lowtherNow with no tabs and rloo's nits picked.17:55
victor_lowtherand now for lunch.17:56
*** igordcard has quit IRC17:57
NobodyCamneed to run out and grab some mile for coffee... bbiafm17:58
*** david-lyle_afk is now known as david-lyle17:58
*** derekh has quit IRC18:00
*** dtantsur is now known as dtantsur|afk18:05
*** wanyen has joined #openstack-ironic18:05
wanyenHi Deva18:06
*** masco has quit IRC18:07
wanyenHi devananda, are you there?  I would like to talk to you about https://review.openstack.org/#/c/136104/18:07
devanandawanyen: hi! it has a +2 from nova PTL - what would you like to talk with me about?18:09
wanyenwe need this spec to support a node with multiple caps, e.g., secure boot, uefi, bios and tboot.18:09
devanandasure18:09
wanyencan you talk to Nova PTL when you attend next Nova meeting18:10
devanandawanyen: about what? it already has a +2 from him18:10
wanyenI would like to we need anothe +218:10
devanandawanyen: it's not his responsibility to get other +2's18:10
*** rushiagr is now known as rushiagr_away18:12
devanandawanyen: you should ask in #openstack-nova or attend their meetings. You could also post the code for review now, so that nova reviewers can see that it is a small change18:12
wanyenAre there any ironic core reviewere also Nova core reviewer?18:12
devanandawanyen: not afaik, no18:12
wanyenDeva: okay.  We will.18:13
devanandait's hard to maintain that level of activity in two active projects ...18:13
*** athomas has quit IRC18:13
wanyenThe change is needed in Nova ironic driver, so ironic own the implementation of that driver.  Right?18:14
devanandawanyen: while we are responsible for maintaining it, nova-core is responsible for reviewing and approving changes to it18:14
wanyendeva, yes I understand.18:15
wanyenAre there any changes required in Nova Ironic driver, e.g., state machine changes?18:16
wanyens/any changes/any other changes18:17
devanandapossibly. I'm going to try to minimize any impact on Nova through changes in our REST API18:17
wanyendeva, I thinking we get all the Nova ironic driver changes in one code review assuming specs can be approved soon. What do you think?18:19
devanandawanyen: I actually dont think that's a good idea. Nova-core has made it clear that they prefer small, well contained, changes -- these are easier for them to review18:20
devanandawanyen: if I understand the spec you referenced above, I think the change there will be very small18:20
devanandanot counting tests, I would expect it to be less than 100 lines18:20
*** romcheg has quit IRC18:20
* NobodyCam is back18:20
*** romcheg has joined #openstack-ironic18:21
wanyendeva, okay.  In that case, we will submit code changes for https://review.openstack.org/#/c/136104/ as soon as we can.  The changes should be small.18:21
devanandaif that's correct, and it doesn't affect anything outside of Ironic, it will probably land easily18:21
devanandanova core give a lot of leeway to changes within drivers -- as long as they don't impact any other part of nova18:21
devanandaso an approved spec with priority:None is OK18:21
devanandait means "we'll get to it, if we have time" -- and a small change takes less time to review :)18:21
wanyendeva, good.18:22
wanyendeva, tx18:22
devanandawanyen: np18:23
*** ndipanov_ is now known as ndipanov_gone18:23
*** harlowja_away is now known as harlowja18:30
*** achanda has quit IRC18:34
*** Marga_ has quit IRC18:35
*** r-daneel has joined #openstack-ironic18:35
*** Marga_ has joined #openstack-ironic18:36
*** shakamunyi has quit IRC18:39
*** Marga_ has quit IRC18:41
*** jistr has quit IRC18:49
*** pelix has quit IRC18:53
*** spandhe has quit IRC18:57
*** ifarkas_afk has quit IRC18:57
*** rushiagr_away is now known as rushiagr18:59
*** andreykurilin_ has joined #openstack-ironic18:59
*** Nisha has joined #openstack-ironic19:01
*** spandhe has joined #openstack-ironic19:04
*** Nisha_away has joined #openstack-ironic19:07
*** Nisha has quit IRC19:10
*** Marga_ has joined #openstack-ironic19:11
*** Marga_ has quit IRC19:14
*** Nisha_away has quit IRC19:15
*** Nisha has joined #openstack-ironic19:15
*** pensu has quit IRC19:16
*** bradjones has quit IRC19:16
NobodyCamsecond +2 added to new state machine spec. not +A'd just so others can re-review19:20
*** Marga_ has joined #openstack-ironic19:21
* NobodyCam wounders if it would be worth adding an "affected states" section to the spec template (for next cycle, not this one)19:24
NishaNobodyCam, devananda , jroll, JayF ....just to make sure, since the state for introspection is now INSPECTING and INSPECTFAIL, do we need to have the end point as inspect or introspect?19:25
Nisha#link https://review.openstack.org/#/c/100951/5019:25
NobodyCamNisha: line 304 of https://review.openstack.org/#/c/133828/14/specs/kilo/new-ironic-state-machine.rst19:26
jrollinspect sounds right19:26
NobodyCaminspect19:26
*** Masahiro has joined #openstack-ironic19:26
Nishajroll, NobodyCam thanks19:26
NobodyCam:)19:26
Shrewsi really hope someone makes an inspector gadget class19:26
NobodyCamlol19:27
NobodyCamgogo Shrews19:27
Nishathe CLI also to be named as node-inspect?19:27
jroll+119:28
Nishathanks jroll19:28
NishaNobodyCam, jroll i had one more ques19:28
NobodyCamsure19:28
*** bradjones has joined #openstack-ironic19:29
*** Marga_ has quit IRC19:29
NishaI have a working code for introspection ready (some minor changes  to be done as per state spec now), but the code depends on the new states. For now i have introduced the states in my code changes19:29
NishaSo for posting the code for review do i need to wait for state code to be available first?19:30
Nishaor for now i can submit the code changes as i have them today19:30
NobodyCamNisha: have you seen devananda's FSM work. https://review.openstack.org/#/c/139215/19:30
jrollyou can put the code up :)19:30
*** Masahiro has quit IRC19:30
Nishaand then later replace the states for inspection from states spec19:31
*** rushiagr is now known as rushiagr_away19:31
NishaNobodyCam, yes today meeting i saw it19:31
NobodyCam:)19:31
NishaNobodyCam, so should i base my code changes on those code changes right now? or it will be better to rebase them once devananda's work is merged19:33
*** dprince has quit IRC19:34
NobodyCamoh thats a tuff call I have only done a quick look over. I would say base the work off the devananda's but I'm really sure how close it landing it is.19:35
NobodyCam*...reallly NOT sure...19:35
Nisha:)19:36
NishaNobodyCam, Ok so i will put the code changes for review independently for now and when states code is merged will rebase my changes on top of that19:36
NobodyCamsound good... please note that in a comment or the commit message19:37
*** marcoemorais has quit IRC19:39
*** marcoemorais has joined #openstack-ironic19:40
Shrewsdevananda: Is there a reason to move the helper methods out of the ConductorManager class in 139217?19:40
*** marcoemorais has quit IRC19:40
devanandaShrews: yes. I just haven't gotten around to writing it yet19:40
devanandaShrews: see aweeks' comment on the first patch19:41
*** marcoemorais has joined #openstack-ironic19:41
*** marcoemorais has quit IRC19:41
*** marcoemorais has joined #openstack-ironic19:41
Shrewsdevananda: i see no comments on any patch19:41
devanandaNobodyCam: I think jroll would like us to land my fsm changes asap19:41
NobodyCamdevananda: I just started to take a closer look at it now19:42
NobodyCam:)19:42
Shrewsdevananda: oh, you mean the first review19:42
devanandaShrews: L121 here https://review.openstack.org/#/c/139215/1/ironic/common/fsm.py19:42
Shrewsyup19:42
*** Marga_ has joined #openstack-ironic19:44
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Refactoring: split test.py and rewrite tests for process  https://review.openstack.org/14033919:47
*** penick has quit IRC19:50
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Refactoring: split test.py and rewrite tests for process  https://review.openstack.org/14033919:51
*** marcoemorais has quit IRC19:52
*** rushiagr_away is now known as rushiagr19:52
*** marcoemorais has joined #openstack-ironic19:52
openstackgerritJosh Gachnang proposed openstack/ironic-python-agent: Use LLDP to get switch port mapping  https://review.openstack.org/9262719:59
JoshNangJayF: jroll ^19:59
JoshNangrebased19:59
JayFNobodyCam: had a vote on that too ^^19:59
NobodyCamhuh20:00
NobodyCamoh20:00
NobodyCamhehehe20:00
NobodyCamJoshNang: Thank you for the additional logging20:01
adam_gJayF, hmm. that devstack fix seems not to have solved the problem20:02
JayFadam_g: Are you sure the failure you're looking at had you change in it?20:03
JayFI think with devstack changes they have to wait for nodes to cycle and rebuild for it to take effect or smth like that20:03
*** anderbubble has joined #openstack-ironic20:03
adam_gJayF, yup. ssh key creation happens a full 3 minutes prior to nodes enrolling20:03
JayF:(20:03
*** romcheg1 has joined #openstack-ironic20:05
Shrewsdevananda: to speed things along, would you mind if i pushed a patch to your fsm change to remove the py26 import and remove comments with manager.py:<LINE> in states.py?20:05
devanandaShrews: not at all20:05
* Shrews does20:06
devanandaShrews: i'm distracted with other things right now, feel free to update that series20:06
JoshNangNobodyCam: no problem! thanks for catching that. definitely saved someone some debugging problems down the road20:06
*** spandhe has quit IRC20:07
*** Marga_ has quit IRC20:09
*** Marga_ has joined #openstack-ironic20:09
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Discover node properties using new CLI node-inspect  https://review.openstack.org/10095120:10
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Discover node properties for iLO drivers  https://review.openstack.org/10300720:13
*** jrist has quit IRC20:14
*** spandhe has joined #openstack-ironic20:16
rlooShrews: wrt the fsm and the conductor line numbers. Maybe replace with the method name for now?20:17
Shrewsrloo: i still do not like that, but if others want it, i can change it to method name20:19
*** Marga_ has quit IRC20:20
rlooShrews: I think it is useful for reviewing this code. I know what you mean, but am thinking (maybe) that the method names won't change before we transition to the new state machine.20:20
NobodyCamShrews: are you adding the logging stuff Josh H pointed out in states.py?20:20
*** andreykurilin_ has quit IRC20:20
rlooShrews: we can have a low hanging fruit for someone to delete those comments after the series has merged ;)20:20
NobodyCamworth me adding a +1 to the current review?20:21
ShrewsNobodyCam: i can, if my vm ever unfreezes20:22
NobodyCamlol :(20:23
Shrewswhich isn't looking promising, atm  :(20:23
rlooShrews: just get a new cow ;)20:23
NobodyCamrloo: http://www.wikihow.com/Have-a-Pet-Cow20:24
NobodyCamlol20:24
rlooNobodyCam: :D20:24
Shrewsugh20:25
* Shrews reboots20:25
Shrewsbrb20:25
*** jrist has joined #openstack-ironic20:26
*** rushiagr is now known as rushiagr_away20:27
*** marcoemorais has quit IRC20:34
*** marcoemorais has joined #openstack-ironic20:34
*** marcoemorais has quit IRC20:35
*** marcoemorais has joined #openstack-ironic20:41
Shrewsrloo: i can address some of your comments myself in what i'm about to push, but not all of them. i didn't write the code, and i'm pretty sure deva just copied most of it, so he may not have the answers either  :)20:41
rlooShrews -- address as many as you can. I can ping harlowja about the others. thx.20:42
*** anderbubble has quit IRC20:42
Shrewsk k20:42
rlooShrews: I'm going to try to play around with the fsm stuff. some of my questions could be cuz I don't have a clear idea yet of how it works.20:42
rlooShrews: Am I right that "we" all want this fsm stuff? (cuz there's no spec for this)20:43
Shrewsi think the exception strings should definitely be _()-ized20:43
NobodyCamrloo: there is a spec.. this is to enable the new state machine20:44
Shrewsyeah, that20:44
rlooNobodyCam: ha ha. well, that spec says there are other specs with implementation details ;)20:44
rlooNobodyCam: I just want to make sure we're following our own rules ;)20:45
jrollrloo: this is about expressing the current state machine in a sane fashion, so we can more easily evaluate the changes needed to get to the new one20:45
NobodyCamwe could file a "Ironic's current state machine lacks code" bug20:46
devanandarloo: it isn't tagged as 'implements a blueprint' -- but yues, this is all related to the state machine spec20:46
devanandarloo: I can tag it, if that will help with tracking / sanity20:46
NobodyCamI would reather not mix it in with the NEW state machine stuff20:47
devanandarloo: also, shrews is correct - i copied fsm.py from TaskFlow. you can see the full history of my changes (which I squashed before posting to gerrit) on my github, if you're interested20:47
devanandaNobodyCam: right -- and this is all prerequisite20:47
rlooyeah, I realize why we're adding the fsm, and I'm fine with it. but it is a fairly substantial change and there's no spec or anything that mentions this new feature.20:48
devanandarloo: do we actually need one? or is that just process for process' sake?20:48
NobodyCamI was half serious about the bug thing20:48
rloodevananda: I don't know if we need one. But we do ask others to write up specs, so I thought I'd ask. What do people think?20:48
rloodevananda: maybe we can just update the existing state machine spec to mention using taskflow to represent the FSM etc, and that might be good enough?20:49
devanandarloo: how about a "related-to: BP nnnnn" line in the commit messages?20:49
NobodyCamthat would work for me20:50
devanandarloo: hm. I dont think victor_lowther's spec needs to indicate exactly how the state machine is implemented, whether its borrowed from taskflow, or what ever20:50
*** anderbubble has joined #openstack-ironic20:50
rloodevananda: it doesn't. but don't you think this state machine is a feature, not a bug?20:50
devanandait does indicate that we need to have _A_ representation of a state machine20:50
devanandaand here's code to create one -- so yea, the commits should reference the spec20:50
devanandai'm convinved now :)20:50
NobodyCamlol .... /me makes a fresh pot of coffee20:51
rloodevananda: ;)20:51
*** mrda-away is now known as mrda20:51
mrdaMorning Ironic20:51
*** bradjones has quit IRC20:52
rloomornin' mrda20:54
NobodyCammorning mrda20:55
*** wanyen has quit IRC20:56
mrdao/20:57
*** Nisha has quit IRC20:57
*** dprince has joined #openstack-ironic20:57
*** marcoemorais has quit IRC20:57
*** subscope has joined #openstack-ironic20:57
rloohey harlowja, wrt your comments in https://review.openstack.org/#/c/139215. eg, 'Another change u probably want to do that I need to get a fix for...'21:01
rlooharlowja: does that mean you'll update this code?21:01
NobodyCamdevananda: would a shallow copy work here? https://review.openstack.org/#/c/139216/3/ironic/conductor/task_manager.py Line#18421:01
Shrewsdevananda: i'll add the related-to for you21:03
*** harlowja is now known as harlowja_away21:04
JayFoooh21:04
JayFsounds like I can pull ordereddict from IPA soon too :)21:05
adam_gJayF, doh, i think this is the real issue/PITA: https://bugs.launchpad.net/ironic/+bug/140088221:06
adam_gnot sure thats a bug but created a new one from the gate issue, in case21:07
JayFI mean, seems buggyish to me?21:07
JayFthe new enrollment stuff laid out in the state machine will help21:07
JayFbut it seems like for now you just need a busy loop saying Devstack isn't ready to move on until those nodes are in the resource tracker?21:08
jrollno it won't, because you'll still have to wait for nova21:08
JayFnova is the only thing with a resource tracker, right?21:09
PaulCzarI'm up to my /last/ problem with ironic ... which is that the instance can't resolve DNS while in the bootloader stage,  which means my instances can't access ironic api to tell ironic to do the iscsi stuff21:09
JayFif you're using neutron for dhcp21:10
PaulCzarsetting endpoint in keystone to IP makes it work ... but then I lose not only DNS .. but SSL21:10
adam_gJayF, yeah, polling nova's view of memory/etc as well as node count.21:10
JayFyou probably just need to have it ship along dns server ips as well21:10
openstackgerritDavid Shrewsbury proposed openstack/ironic: Add fsm.py module  https://review.openstack.org/13921521:10
PaulCzarJayF: it does!  if I spoof the MAC on a VM running ubuntu it comes up with dns working21:10
openstackgerritMerged openstack/ironic-python-agent: Use LLDP to get switch port mapping  https://review.openstack.org/9262721:11
*** anderbubble_ has joined #openstack-ironic21:11
NobodyCamPaulCzar: have you looked at https://ask.openstack.org/en/question/26278/neutron-dhcp-adding-dns/21:11
JayFPaulCzar: strange. IDK anything beyond that then, because I don't use the pxe ramdisk. I could help if it was IPA :)21:11
*** anderbubble has quit IRC21:11
*** anderbubble_ is now known as anderbubble21:11
*** harlowja_away is now known as harlowja21:12
ShrewsNobodyCam: doh! forgot the logging stuff21:12
NobodyCam:-p21:13
harlowjarloo as for 139215 i'm waiting to see where this goes first, before thinking about how to generalize it, i think thats fair with regard to the FSM stuff21:13
harlowjai have done something @ https://github.com/harlowja/automaton that might fit the needs for both projects, but it seems a little premature to start adding it to the requirements repo and such so far...21:14
ShrewsNobodyCam: where does it go??21:14
rlooharlowja: yes, that's fair. so we should just go ahead and make any changes directly to our copy?21:14
harlowjaya21:14
rlooharlowja: ok21:14
harlowjadoesn't seem like that review's even merged yet, so not sure the point until the 'vision is more clear'21:15
PaulCzarJayF: IPA requires that there's already an OS laid down on the box running the IPA right ?21:15
*** Masahiro has joined #openstack-ironic21:15
harlowjaif vision gets more clear then sure rloo we can decide what to do21:15
rlooharlowja: merging == vision is crystal clear :-)21:15
harlowjalol21:15
JayFPaulCzar: IPA is just another deploy driver. Right now they support different things (IPA only does whole disk images, and requires images from glance backed by swift)21:15
harlowjathats impossible rloo21:15
rlooharlowja: ha ha21:16
JayFPaulCzar: IPA runs inside a ramdisk just like the pxe driver stuff does; it just offloads more work to the node. i.e. IPA has a rest API that's called with a URL to an image, the agent itself downloads and images it, as opposed to pxe driver, where it mounts an iscsi volume and then lays an image down from the conductor21:16
harlowjarloo unless u and ironic folks can predict the future (if so please share some numbers with me, ha)21:16
rlooharlowja: "crystal clear" doesn't say anything about the future ;)21:17
ShrewsNobodyCam: hrm, that's not actually a small change. i'm not comfortable adding that logging stuff yet21:17
harlowjarloo fair enough :)21:17
NobodyCamShrews: ack I was just looking at that21:18
harlowjaalthough rloo if u want to go through a fine toothed comb u can go through https://github.com/harlowja/automaton and submit PRs ;)21:18
ShrewsNobodyCam: if i knew the code better...21:18
Shrewsso maybe later21:18
NobodyCam:)21:18
*** Masahiro has quit IRC21:19
rlooharlowja: what's a PR? I meant making sure our FSM reflected what our code is/was doing.21:19
*** cohn has left #openstack-ironic21:20
harlowjarloo pull request (that weird github thing)21:20
*** spandhe has quit IRC21:20
rlooharlowja: ahh. so what's the diff between harlowja/automaton and your TaskFlow stuff?21:21
harlowjaa few diffs21:21
harlowjanothing major21:21
harlowjamoved the runners to be external to the state machine they run using21:21
harlowjaand put in hierachical state machines into that one (for those that need it)21:22
harlowjaotherwise, the same21:22
harlowja* https://github.com/harlowja/automaton/blob/master/automaton/machines.py#L37021:22
*** marcoemorais has joined #openstack-ironic21:23
rlooharlowja: I'm worried about maintaining our copy of your fsm, but maybe I'll postpone the worrying and hopefully forget to worry about it.21:23
PaulCzarJayF: is there an example somewhere using IPA?   it's not on the deploying ironic documentation21:23
harlowjarloo i think the worry is justified, but need to see where this goes first21:23
harlowjaif it goes no-where, then no need to worry21:23
rlooharlowja: ok, i'll punt on the worrying then ;)21:23
harlowja:)21:23
JayFPaulCzar: I think it is in the documentation? I can't really spare the time right now to help one-on-one although I could later (tomorrow)21:24
rlooharlowja: we're going to use it...21:24
harlowjarloo thats decided already?21:24
PaulCzarJayF: Okay I'll poke around and see what I can find for myself21:24
JayFPaulCzar: or ask others in here :/21:24
harlowjarloo if so then maybe the worrying is justified, ha21:24
rlooharlowja: we just need 2 +2s and an approve ;)21:24
harlowjawell i have put up https://pypi.python.org/pypi/automaton and can iterate on that as needed, so if so it'd be nice to try to unify on a common library21:25
harlowjarloo maybe u want to be the owner of that library :-P21:25
rlooharlowja: no no no. i'm not worrying about that stuff, remember?21:25
harlowjaoh ya21:25
harlowja:)21:25
ekarlso-rloo: isonic is using a copy of automaton vs using the lib ?21:26
rlooekarlso-: well, ironic isn't using anything yet, but yeah, we're talking about this patch: https://review.openstack.org/#/c/13921521:27
harlowjaekarlso- i created automaton over the weekend, so its sorta hard for people to use it yet ;)21:27
harlowjai'm trying to be pre-emptive :-P21:27
harlowja*not releated to preemptive nuclear strikes21:27
harlowjalol21:27
PaulCzarAnyone have some time to talk me through using IPA or able to show me to some docs ?21:28
harlowjaekarlso- automaton comes from taskflow (which uses it internally to itself) and if others find something similar useful, might as well share the love/code21:29
harlowja* basically http://docs.openstack.org/developer/taskflow/types.html#module-taskflow.types.fsm21:32
JoshNangPaulCzar: sure! what questions do you have?21:32
JoshNangi'm not sure how many docs we have tbh21:32
*** marcoemorais has quit IRC21:33
*** marcoemorais has joined #openstack-ironic21:33
PaulCzarJoshNang: right now I'm using the pxe_ssh driver which is causing me grief because the initial bootloader doesn't seem to do DNS ... so I'm trying to figure out of there's a better or alternative driver that might help21:34
*** alexpilotti has quit IRC21:35
PaulCzarJoshNang: a link to a prebuild IPA loader and some quick notes on the node options to set ( equivs of -i pxe_deploy_kernel=... etc ) would probably be enough to get me in the right direction21:36
JoshNangPaulCzar: hrm, i'm not sure ipa is going to help if your ramdisk isn't getting DNS. IPA will need dns to find glance :/21:36
PaulCzarJoshNang: neutron uses the same lease for the ramdisk as it does for the actual built image right ?21:38
*** linggao has quit IRC21:38
JoshNangPaulCzar: hmm, i think so. let me check21:39
PaulCzarJoshNang: thanks.  because dns works fine one the instance is up ... so I've been assuming its a problem with the bootloader from DIB itself21:40
*** Marga_ has joined #openstack-ironic21:40
*** subscope has quit IRC21:42
JoshNangPaulCzar: yeah i'm not seeing any code that would change it's lease21:42
JoshNangbut i'm far from an expert on how the pxe driver works21:43
PaulCzarI assume with the IPA driver it still uses pxe to start the bootloader that contains the IPA ?21:44
JoshNangcorrect21:44
JoshNanghere's some extremely barebones docs (including download link) https://github.com/openstack/ironic-python-agent/tree/master/imagebuild/coreos21:45
NobodyCambrb21:45
JoshNangfor us, we use pxe to boot into ipxe, and then ipxe downloads the kernel/initrd and boots them21:46
PaulCzarJoshNang: and that pxe is outside of ironic itself ?  has nothing to do with the pxe driver ?21:50
JoshNangPaulCzar: correct. "pxe driver" is a bit confusing, as they both pxe21:51
JoshNang"pxe driver" == "write image via iSCSI using a pxe booted ramdisk"21:51
JoshNang"ipa" == "write image directly to disk after downloading from URL to pxe booted ramdisk"21:52
devanandacan we rename the pxe_* drivers to iscsi_* pls?21:52
devanandawithout, you know, breaking the world21:52
JoshNang+10021:52
PaulCzarJoshNang: right ... also the pxe driver handles the heavy lifting of managing your images/macs in /tftpboot21:52
devanandathe whole split-boot-and-deploy-interfaces seems to have been dropped lately, which is unfortunate21:52
devanandaor maybe I jsut haven't been following it?21:53
JoshNangPaulCzar: the agent driver uses the same code to handle the images/macs in tftpboot21:53
*** jrist is now known as knowncitizen21:54
*** knowncitizen is now known as jrist21:55
NobodyCamomg 3 +2's21:59
NobodyCamlol21:59
*** harlowja has quit IRC22:02
PaulCzarJoshNang: what are the -i options for the agent driver to specify the kernel/initrd ?22:05
*** ryanpetrello has quit IRC22:05
*** foexle has quit IRC22:07
JoshNangPaulCzar: should be the same as the pxe driver (and if not, i'd say that's a bug)22:07
*** andreykurilin_ has joined #openstack-ironic22:09
rloodevananda: i was going to ask about boot/deploy interfaces, cuz you wanted it for kilo1. lucas mentioned that he's too busy to work on it, and jroll offered to try to pick it up.22:10
devanandaI jhave the sense jroll is too busy, too ...22:10
*** spandhe has joined #openstack-ironic22:12
*** sdake has joined #openstack-ironic22:12
rloodevananda: what's the priority on the boot/deploy? seems like the new state stuff is higher priority?22:12
sdakehttp://sdake.io/2014/12/09/isnt-it-atomic-on-openstack-ironic-dont-you-think/22:12
sdakeenjoy ^^ ;-)22:12
*** alexpilotti has joined #openstack-ironic22:13
rloosdake, have you thought about updating/adding to the ironic docn? :-)22:14
PaulCzarJoshNang: and finally ...  the image format of the images that it supports?  It was mentioned earlier that it does whole disk ... so would that be a qcow or a raw image?22:14
*** hypnosb has joined #openstack-ironic22:14
PaulCzaror is it still the set of three image+kernel+initrd22:15
JoshNangi believe qcow's22:16
sdakewhat is a docn?22:16
rloosdake: docn -- documentation. or lack of ;)22:16
sdakeironic haas good docks22:16
sdakedocs22:16
sdakeso my thinknig is I'd like to get that blueprint impolemented22:17
sdakeso atomic really works on ironic without alot of hassle22:17
JayFPaulCzar: anything that qemu-img convert  can convert to raw :)22:17
sdake(blueprint at the end)22:17
JayFPaulCzar: are supported image types22:17
JayFlol22:17
JoshNangJayF: thanks :)22:17
hypnosbHi22:17
PaulCzarJayF: thanks :)22:18
hypnosbSo question... Does it work?22:18
devanandasdake: iiuc, this is just in VMs -- did you get devstack deploying Atomic on separate hardware?22:18
devanandahypnosb: that's a strange question22:18
rloosdake: do you know about our BP spec process?22:18
hypnosbdevananda, Asking a software project if their project works is strange?22:19
devanandahypnosb: define "it" and "works" :)22:19
hypnosbDoes it do what you say it does?22:19
devanandahypnosb: yup22:19
sdakerloo no I do not22:20
sdakeif you ahve a pointer22:20
devanandahypnosb: but that's not "does it work" -- that's "are your docs accurate"22:20
hypnosbamazing22:20
sdakedevandanda I'd assume it would work on bare metal as well22:20
hypnosbid rather have accurate docs than having it work... id go less crazy22:20
sdakerloo if you have a pointer I'll follow the process therre22:20
sdakeor an example review22:20
devanandasdake: I would assume so, too. but it's always interesting to try :)22:20
rloosdake: https://github.com/openstack/ironic-specs22:20
devanandasdake: http://specs.openstack.org/openstack/ironic-specs/specs/template.html22:21
sdakerloo cool I'll fiel that22:21
devanandasdake: https://wiki.openstack.org/wiki/Blueprints#Spec_.2B_Blueprints_lifecycle22:21
hypnosbhow is ironic different from fuel?22:21
devanandahypnosb: ehhh... very different ....22:21
devanandahypnosb: but also, not that different22:21
devanandamuch too long / loaded a question to ask me right now, as I've just spent the last four hours replyng to emails about FuelAgent22:21
rloosdake: so you file a blueprint (you did that), and then write up a specification. Otherwise that BP may sit there gathering dust.22:22
sdakerloo will do22:22
devanandahypnosb: have you read the docs for each project? it should then be clear :)22:22
sdakerloo thanks for the hand holding ;-)22:22
rloosdake: yw22:23
Shrewssdake: without me having to read the entire blog post (i'm lazy and it's almost beer-o-clock), what role is docker playing?22:23
hypnosbdevananda, ahahah nice one22:23
devanandasdake: these are *terribly* out of date, but, are still actually pretty close to graphics covering the process you didn't find22:23
devanandasdake: https://github.com/openstack/tripleo-incubator/tree/master/images22:23
sdakeshrews docker does nothing - it is about launching atomic on ironic22:24
sdakebut I am working on magnum which controls docker22:24
sdakemagnum uses ironic to launch bare metal22:24
sdakethe bare metal image needs docker (a dependency of magnum)22:24
hypnosbdoes ironic use puppet or chef?22:24
Shrewssdake: ah, ok. assumed it played a role from the 1st paragraph22:24
sdakeits just an explination of "why" you would even want atomic ;-)22:24
sdake"who needs more operating systems."22:25
Shrewsi should just RTFBP  :)22:25
sdakeits mostly documenting my steps for others22:25
sdakebut I reproduced it after I hacked together an initial implementation22:25
sdakeso it should reproduce for others as well22:25
hypnosbI thought crowbar was taking off but I guess the dell guys killed it22:25
sdakeAFAIK ironic does not use puppet or chef, but you could use heat software config resources to do that job22:26
sdakebtw, +1 from me on ironic architecture and implementation22:26
sdakehaving been through all of it, it looks like pretty hot tech22:26
hypnosbmy thought for puppet or chef was the bare metal provision... I guess its all bash script?22:26
hypnosbpython22:27
sdakehypnosb provision can be done with cloudinit using heat software config22:27
devanandawhoa, conflation of concepts here folks22:27
hypnosbsure but what about bare metal of a node?22:27
devanandasdake: you mean s/provision/parameterization/22:28
hypnosboh22:28
hypnosber22:28
hypnosbyeah idk how heat can do bare metal?22:28
devanandasdake: parameterization of an instance, after it has been deployed, can be done by chef/puppet/salt/ansible/anything22:28
devanandacloud-init + os-apply-config can do that parameterization as well22:28
devanandahypnosb: look at tripleo22:28
devanandaheat drives nova, and passes metadata through cloud-init22:29
hypnosbomg another openstack project22:29
hypnosbffs22:29
devanandaos-*-config scripts run within the instance at boot, gather the metadata, and parameterize22:29
devanandahypnosb: hi. what's with all the hate?22:29
hypnosbfrustrated, 5:30pm22:30
hypnosbill go22:30
hypnosbsorry, bye22:30
devanandahypnosb: your first words in this channel were not friendly. perhaps try again tomorrow :)22:30
sdakedevandanda yup I thnk I siaid that, just differently ;-)22:31
devanandasdake: possibly :) the part that confused me was "provision can be done with cloudinit".22:32
alexm__Hi everyone, wondering what you are using to build your bootable metal images for Ironic and if we need to do anything special to make them compatible. My ironic deployment with nova succeeds, but the node doesn’t want to boot on the written disk partition :(22:33
devanandasdake: at least in the context of ironic, provisioning means something specific, which doesn't have anything to do with cloud-init22:33
*** dprince has quit IRC22:33
devanandaalexm__: hi! we're using diskimage-builder, but in principle, images built by other tools should work too22:34
NobodyCamShrews: I think I got that logging stuff mind if I push up over 13921522:34
NobodyCamdevananda: also ^^^22:34
ShrewsNobodyCam: YES!22:34
PaulCzarJoshNang: looks like pxe driver uses pxe_deploy_kernel and agent uses deploy_kernel22:34
NobodyCamyes you mind?22:34
ShrewsNobodyCam: i mean... NO!22:34
Shrews:)22:34
NobodyCamlol22:34
NobodyCamhehehe22:34
ShrewsNobodyCam: please do it22:34
devanandasdake: nice blog post, btw22:35
alexm__devananda: thx. I used diskimage-builder to make my ubuntu and kernel+ramdisk images. Unfortunately it doesn’t seem to boot.22:35
devanandasdake: if you feel like diagraming, I'd love to have the deploy process for the pxe_* class of drivers, done using asciiflow, in our doc/source22:36
JoshNangPaulCzar: :/22:36
devanandasdake: I have this terribly old one here ... https://github.com/openstack/tripleo-incubator/blob/master/images/baremetal-deploy.odp which I haven't updated in years :(22:36
openstackgerritChris Krelle proposed openstack/ironic: Add fsm.py module  https://review.openstack.org/13921522:36
NobodyCamShrews: ^^^ I think thats kinda it22:36
devanandaalexm__: ok. I'd need more information to be able to help determine what the problem is22:36
*** ryanpetrello has joined #openstack-ironic22:37
*** harlowja has joined #openstack-ironic22:37
devanandaNobodyCam: let's remove run_iter() -- I dont think we'll ever use it (famous last words ...)22:39
sdakedevananda I dont know asciflow, and at the moment don't have time until January to dig into any new work22:39
alexm__devananda: All the service logs look okay, from the point of view of conductor and nova it was a success. Maybe there was a problem witth the pxe config. Do you have a valid one that I can compare with?22:39
harlowjadevananda in https://github.com/harlowja/automaton i split off the runner to be its own class, might help for those who don't care about using it22:39
sdakedevananda i'd be more willing if I could use imagemagick files22:39
harlowja*including run_iter()22:40
sdake(eg SVG)22:40
devanandasdake: either this https://www.websequencediagrams.com/#22:40
devanandasdake: or this: http://asciiflow.com/22:40
sdakeis asciiflow integrated into sphinx in some way?22:41
devanandaboth are text representations that can be embedded in REST files, and later edited, and which generate pictures22:41
devanandanope22:41
devanandabut the output can easily be pasted into a .rst file22:41
sdakeI see22:41
sdakeinteresting22:41
devanandasee eg. https://review.openstack.org/#/c/133828/12/specs/kilo/new-ironic-state-machine.rst22:41
sdakeI'll look into it after milestone #1 of magnum is out22:41
devanandafor several complex diagrams in asciiflow22:42
devanandasdake: ack. anyway, thanks!22:42
*** krtaylor has quit IRC22:42
sdakesure, hope the blog post attracts some more attention to ironic (and atomic)22:42
*** ryanpetrello has quit IRC22:44
NobodyCamharlowja: question: can I ask why this check: https://github.com/openstack/taskflow/blob/master/taskflow/engines/action_engine/runner.py#L16622:45
harlowjau may ask :-P22:45
NobodyCam:)22:45
harlowjaNobodyCam meh, just avoids adding on un-needed things, nothing much else there22:45
harlowjaavoid any callbacks if no debuggin on22:45
harlowjasave some cycles, blah blah :-P22:46
*** ryanpetrello has joined #openstack-ironic22:46
harlowjai know its python...22:46
harlowjaha22:46
NobodyCamok thats what I thought22:46
NobodyCam:)22:46
NobodyCamThank you22:46
harlowjanp22:46
PaulCzarJoshNang: the bootloader started ... but it crashed out to a prompt at 'Failed to start Switch Root'22:47
* devananda steps away for a break22:47
JoshNangPaulCzar: weird!22:48
PaulCzarJoshNang: that's with the prebuilt images I downloaded ... digging into the problem there appear to be some files missing in the image so I'll create images locally and try again22:49
JoshNangPaulCzar: hmmm i'm pretty sure we're using those images in the gate right now. what files are missing?22:49
JayFPaulCzar: not all ubuntu cloud images you can download come with hw drivers :)22:50
NobodyCamd'oh22:50
NobodyCamShrews: are you pushing up another 139215?22:50
PaulCzarJayF: this is still at the bootloader stage ...  its a coreos error22:51
ShrewsNo. I'm pushing a beer into my face hole and making dinner22:51
JayFPaulCzar: is that an error from booting IPA then?22:51
PaulCzaryeah22:51
NobodyCamlol.. ack I will fix my pep8 errors :-p22:51
*** mjturek has quit IRC22:52
JayFPaulCzar: try redownloading it and seeing if the md5 matches?22:52
JayFPaulCzar: there is a small race where new IPA images aren't uploaded atomically, and I bet that could've been what happened to you22:52
JayFPaulCzar: (i.e., if you download while we're uploading you can get a bad image)22:52
JayFPaulCzar: also, alternatively, build your own. It's trivial to do.22:52
PaulCzaryeah that's what I'm doing22:53
openstackgerritChris Krelle proposed openstack/ironic: Add fsm.py module  https://review.openstack.org/13921522:53
PaulCzarmd5 checksums match ...  I'll see what happens on a self built set22:57
*** jgrimm is now known as zz_jgrimm22:59
PaulCzarI assume the images are supposd to go into glance with disk-format aki/ari ?22:59
*** bigjools has quit IRC23:00
sdakeramdisk goes in as ari, kernel goes in as aki23:00
sdakebase image goes in as ami23:00
sdakethe base image must be a filesystem (use the file command to double check)23:01
sdakethe base image must be registered with ramdisk_id and kernel_id glance properties23:01
PaulCzarI'm using the same base image that I used for the pxe_ssh driver23:01
*** bigjools has joined #openstack-ironic23:02
*** bigjools has quit IRC23:02
*** bigjools has joined #openstack-ironic23:02
*** alexpilotti has quit IRC23:02
sdakedid you glance image-create --property kernel_id=xyz --proeprty ramdisk_id=xyz ?23:02
PaulCzaryessir23:02
sdakei'm out of ideas, but keep in mind I've only been using ironic for about 3 days :)23:02
*** Masahiro has joined #openstack-ironic23:04
PaulCzarsdake: as I've been discovering over the last few weeks, it all works great in devstack ...  just trying to run it in a real openstack is 'fun'23:04
sdakeI see, I haven't actually tried real hardware23:04
sdakeI only have one machine I can use, and I don't want to lose the disk on it23:04
*** shakamunyi has joined #openstack-ironic23:04
sdakerather two machines, I have my workststation where I could run a conductor and a machine that I could deploy on23:04
PaulCzaryeah, also devstack hides a ton of complexity in impossible to read shell scripts all over the place23:05
sdakethere are installation documentation that looked pretty good I think23:05
sdakeare you sure your root image is correct?23:05
sdakeyou can check by running file on it23:05
sdakeit should print out that its ext3/ext4/etc23:05
*** Masahiro has quit IRC23:08
PaulCzarsdake: yeah my problems aren't with the image itself ... its with the deployers23:10
sdakewhat driver are you using?23:10
PaulCzarthe pxe_ssh deploy images don't seem to get dns info23:10
PaulCzarand I just remembered why I wasn't going to use the agent driver ... it requries swift23:10
PaulCzarwhich is a lot of overhead for provisioning machines23:11
*** mikedillion has joined #openstack-ironic23:16
harlowjaNobodyCam devananda  http://i.imgur.com/q8Hknb9.png (Created from that review)23:19
harlowjausing http://paste.ubuntu.com/9448172/ (A clone of something similar in taskflow)23:19
harlowjauseful for looking at (and seeing if any anomalies)23:19
harlowjarloo ^23:19
harlowjafeel free to hack that up, but neat that it can be done imho23:22
*** mikedillion has quit IRC23:23
NobodyCamharlowja: wow thats hard to follow ... but really cool23:23
harlowjaNobodyCam hey, its from the FSM; can only blame the FSM :-P23:23
harlowjaonly outputs what u give it, ha23:23
NobodyCamlol :)23:23
*** mikedillion has joined #openstack-ironic23:23
harlowjai think u can add edge names to, the code above just isn't,23:23
harlowjathat might help23:23
NobodyCamalthough rebuild looks wrong there23:25
harlowjaya, thats what the graph is nice for finding out23:25
NobodyCam:)23:25
devanandaPaulCzar: this statement doesn't jive for me. what do you mean? "the pxe_ssh deploy images don't seem to get dns info"23:26
devanandaNobodyCam: hm. ya. "rebuild" is an input, not a state, in the current machine23:27
NobodyCam:)23:28
PaulCzardevananda: bad wording?   basically when its doing the pxe / bootloader thing it boots up and tries to curl the ironic-api ...  it fails saying that it can't look up the name23:28
devanandaPaulCzar: oh! shouldn't it be using the IP ?23:28
harlowja* http://i.imgur.com/SnjnZsF.png has the 'on event' names23:28
PaulCzardevananda: that would mean that keystone would need to have it as IP in the catalog23:28
harlowjafor edges ^23:28
NobodyCamharlowja: Much nicer23:28
harlowja* http://paste.ubuntu.com/9448289/23:28
PaulCzardevananda: which would among other things break SSL certs23:29
*** krtaylor has joined #openstack-ironic23:29
NobodyCamharlowja: Neat-oh TY23:29
harlowjanp23:29
PaulCzardevananda: if there was a way to tell ironic to override the endpoint and pass the IP to the bootloader instead then I'd be okay using the IP, because there's nothing scary in the payload23:30
devanandaPaulCzar: the ironic API URL is passed as a kernel cmdline, not fetched from keystone: https://github.com/openstack/diskimage-builder/blob/master/elements/deploy-ironic/init.d/80-deploy-ironic#L123:30
PaulCzardevananda: right,  but ironic sets that ... not me23:30
PaulCzarunless I'm being obtuse and there's an obvious place to override it23:30
devanandahttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe_config.template23:31
devanandahttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L7523:31
devanandaPaulCzar: config option :)23:31
NobodyCamPaulCzar: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L520 (config option)23:32
devanandaoops. NobodyCam is correct23:33
devanandaI pasted a link to the wrong option23:33
PaulCzardevananda: NobodyCam ahahahahaha dammit.  I had scoured the config looking for stuff like ironic_api ...23:34
NobodyCam:)23:34
harlowjazer0c00l u looked at any of the state machine stuff, might be interseting from a newb persepctive to see what u think23:35
zer0c00lharlowja:  link?23:37
harlowjazer0c00l https://review.openstack.org/#/c/139215/ visualized @ http://i.imgur.com/SnjnZsF.png23:38
harlowjamaybe requires to much context though23:39
*** shakamunyi has quit IRC23:39
NobodyCambrb23:40
*** jb11211 has joined #openstack-ironic23:44
jb11211Hey guys. is there anything ironic which could snapshot hosts? say if you were sharing the same hardware and wanted to grab/restore images23:44
*** shakamunyi has joined #openstack-ironic23:45
*** anderbubble has quit IRC23:46
*** jrist is now known as knowncitizen23:48
*** knowncitizen is now known as jrist_23:49
*** naohirot has joined #openstack-ironic23:50
*** jlk has joined #openstack-ironic23:50
naohirotgood morning ironic23:50
*** yuanying_ has joined #openstack-ironic23:50
*** jrist_ is now known as jrist23:50
*** jrist is now known as jrist_23:50
*** jrist_ is now known as jrist23:51
NobodyCammorning naohirot23:52
naohirotNobodyCam: Hi23:52
*** Hefeweizen has quit IRC23:52
devanandajb11211: nope23:53
*** yuanying has quit IRC23:53
*** shakamunyi has quit IRC23:54
naohirotNobodyCam: quick question, http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-service-to-use-the-bare-metal-service23:56
naohirotNobodyCam: /etc/nova/nova.conf, This configuration file must be modified on the Compute Service’s controller nodes and compute nodes.23:56
*** harlowja has quit IRC23:56
*** igordcard has joined #openstack-ironic23:57
naohirotNobodyCam: does that mean that OpenStack of a pair of controller and compute node have to be prepared for 2 set of controller and compute node at least to manage both virtual deploy and physical deploy?23:58

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