Thursday, 2017-08-03

*** rloo has quit IRC00:01
*** hw_wutianwei has joined #openstack-ironic00:01
*** http_GK1wmSU has joined #openstack-ironic00:03
*** http_GK1wmSU has left #openstack-ironic00:05
*** zhangfei has joined #openstack-ironic00:12
*** rama_y has quit IRC00:14
*** baoli has joined #openstack-ironic00:16
*** baoli has quit IRC00:17
*** yangyapeng has joined #openstack-ironic00:19
*** baoli has joined #openstack-ironic00:22
*** yangyapeng has quit IRC00:24
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/48800500:31
*** rwsu has quit IRC00:38
*** litao__ has joined #openstack-ironic00:56
*** crushil has quit IRC00:57
*** phuongnh has joined #openstack-ironic01:03
*** rloo_ has quit IRC01:06
*** hao has joined #openstack-ironic01:07
*** phuongnh has quit IRC01:08
*** mat128 has joined #openstack-ironic01:08
*** crushil has joined #openstack-ironic01:15
*** phuongnh has joined #openstack-ironic01:17
*** yangyapeng has joined #openstack-ironic01:18
*** yangyape_ has joined #openstack-ironic01:20
*** yangyapeng has quit IRC01:23
*** namnh has joined #openstack-ironic01:31
*** mat128 has quit IRC01:31
openstackgerritRuby Loo proposed openstack/ironic master: Remove WARNING from pin_release_version's help  https://review.openstack.org/49027001:34
*** mat128 has joined #openstack-ironic01:36
openstackgerritMerged openstack/python-ironicclient master: Update the documentation link for doc migration  https://review.openstack.org/48991701:36
*** mat128 has quit IRC01:44
openstackgerritWang KaiFeng proposed openstack/bifrost master: no_proxy does not take effect on install  https://review.openstack.org/49027701:51
openstackgerritWang KaiFeng proposed openstack/bifrost master: no_proxy does not take effect on install  https://review.openstack.org/49027701:53
jlvillalihrachys, So far 4 patches x 2 test runs, have all passed. Trying another test run on all four patches.01:54
ihrachysjlvillal, hm, which patch do you test again? I don't have a fix for that :)01:55
jlvillalihrachys, https://review.openstack.org/#/c/490139/ was what I am depending on01:56
patchbotpatch 490139 - neutron - Degrade log message for missing fanout exchange to...01:56
jlvillalihrachys, One of the test patches: https://review.openstack.org/49024001:57
patchbotpatch 490240 - ironic - WIP/DNM: Test Neutron patch #201:57
jlvillalihrachys, But maybe red herring...01:58
ihrachysjlvillal, well it won't solve the issue :)01:58
ihrachysit just degrades an error log level01:58
ihrachysthe actual problem is in oslo.messaging messing up connection pool when the exception is raised01:59
*** mtanino has quit IRC01:59
*** ihrachys has quit IRC01:59
*** ihrachys has joined #openstack-ironic02:00
jlvillalihrachys, Ah, thanks02:00
ihrachysjlvillal, sorry, logged off because pressed wrong button :)02:00
masbergood morning, sorry to bother you but my baremetal server can't see my pxe server during boot time and I don't know what to do. Could someone please help me troubleshooting?02:01
jlvillalihrachys, Thanks!02:02
jlvillalihrachys, Now that I read the patch I see :)02:04
*** mat128 has joined #openstack-ironic02:04
*** rbudden has quit IRC02:06
*** rbartal has quit IRC02:07
openstackgerritWang KaiFeng proposed openstack/bifrost master: Update .gitignore  https://review.openstack.org/49028202:08
*** jrist has joined #openstack-ironic02:11
*** gcb has quit IRC02:15
*** gcb has joined #openstack-ironic02:16
*** Nisha_Agarwal has joined #openstack-ironic02:18
*** baoli has quit IRC02:36
openstackgerritMerged openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/48800502:39
*** tiendc has joined #openstack-ironic02:47
*** diablo_rojo has quit IRC02:51
*** Nisha_ has joined #openstack-ironic02:56
*** ihrachys has quit IRC02:58
*** Nisha_Agarwal has quit IRC02:59
openstackgerritTuan Luong-Anh proposed openstack/ironic master: Secure boot support for irmc-pxe driver  https://review.openstack.org/48047703:02
openstackgerritTuan Luong-Anh proposed openstack/ironic master: Secure boot support for irmc-pxe driver  https://review.openstack.org/48047703:02
*** rbartal has joined #openstack-ironic03:04
*** diablo_rojo has joined #openstack-ironic03:08
*** masuberu has joined #openstack-ironic03:18
*** masber has quit IRC03:22
*** tuanluong has joined #openstack-ironic03:30
*** Nisha_ has quit IRC03:32
*** udesale has joined #openstack-ironic03:34
openstackgerritRuby Loo proposed openstack/ironic master: Rolling upgrades related dev documentation  https://review.openstack.org/41943903:38
*** yangyapeng has joined #openstack-ironic03:39
*** yangyapeng has joined #openstack-ironic03:41
openstackgerritTony Breeds proposed openstack/ironic master: Testing reno 2.5.0  https://review.openstack.org/49031503:41
*** yangyape_ has quit IRC03:43
*** yangyapeng has quit IRC03:46
openstackgerritMerged openstack/ironic master: Add new dbsync command with first online data migration  https://review.openstack.org/40855603:48
openstackgerritShangXiao proposed openstack/ironic master: [Trivialfix]Fix typos in ironic  https://review.openstack.org/49031903:48
*** links has joined #openstack-ironic03:54
openstackgerritTony Breeds proposed openstack/ironic master: Testing reno 2.5.0  https://review.openstack.org/49032704:02
*** adisky__ has joined #openstack-ironic04:03
*** aparnav has joined #openstack-ironic04:05
openstackgerritMerged openstack/ironic master: Removed unnecessary setUp() call in unit tests  https://review.openstack.org/48967004:06
openstackgerritHironori Shiina proposed openstack/ironic master: iRMC: Support volume boot for iRMC virtual media boot interface  https://review.openstack.org/46791104:26
*** sasha2 has quit IRC04:34
*** yangyapeng has joined #openstack-ironic04:43
*** rbartal has quit IRC04:44
*** rama_y has joined #openstack-ironic04:45
*** hel0tsp has joined #openstack-ironic04:47
*** mat128 has quit IRC04:48
*** yangyape_ has joined #openstack-ironic04:48
*** yangyapeng has quit IRC04:51
openstackgerritHironori Shiina proposed openstack/ironic master: iRMC: Add documentation for remote volume boot  https://review.openstack.org/48962004:55
*** hel0tsp has quit IRC05:05
*** udesale__ has joined #openstack-ironic05:05
*** hel0tsp has joined #openstack-ironic05:08
*** konetzed has joined #openstack-ironic05:08
*** udesale has quit IRC05:08
konetzedcan an ironic node only have one capability, as in "properties/capabilities='boot_mode:uefi'"05:09
*** diablo_rojo has quit IRC05:15
konetzedThere seems to be no way to do something like 'ironic node-update <node-uuid> add properties/capabilities='disk_label:gpt' properties/capabilities='boot_mode:uefi'" as last one always wins, using mitaka release05:23
*** marios has joined #openstack-ironic05:30
*** robcresswell has quit IRC05:41
*** dims has quit IRC05:52
*** dims has joined #openstack-ironic05:54
*** livelace has joined #openstack-ironic05:58
*** hel0tsp has quit IRC05:58
*** mjura_ has joined #openstack-ironic06:00
*** marios has quit IRC06:01
*** marios has joined #openstack-ironic06:02
*** mjura has joined #openstack-ironic06:04
*** mjura_ has quit IRC06:05
*** mrunge has left #openstack-ironic06:06
*** jaganathan has joined #openstack-ironic06:09
*** marios has quit IRC06:11
*** marios has joined #openstack-ironic06:11
*** hoonetorg has quit IRC06:12
*** rcernin has joined #openstack-ironic06:19
*** hoonetorg has joined #openstack-ironic06:25
tuanluonghello konetzed. ironic node-update $NODE add properties/capabilities='boot_mode:uefi, disk_label:gpt'06:27
*** ralonsoh has joined #openstack-ironic06:29
*** rbartal has joined #openstack-ironic06:30
*** hel0tsp has joined #openstack-ironic06:33
*** makowals has joined #openstack-ironic06:38
*** ChubYann has quit IRC06:40
*** pcaruana has joined #openstack-ironic06:48
*** alineb has quit IRC06:51
*** jaganathan has quit IRC06:51
*** jaganathan has joined #openstack-ironic06:52
openstackgerritMateusz Kowalski proposed openstack/ironic-python-agent master: BMC can be configured using different lan channel  https://review.openstack.org/48067906:58
*** gcb has quit IRC07:05
*** jtomasek has joined #openstack-ironic07:09
*** Dinesh_Bhor has quit IRC07:10
openstackgerritMerged openstack/bifrost master: Fix tools/vagrant_dev_env/vagrant.yml  https://review.openstack.org/48695407:13
*** Dinesh_Bhor has joined #openstack-ironic07:16
pas-hamorning ironic :)07:17
*** tesseract has joined #openstack-ironic07:17
*** udesale__ has quit IRC07:18
*** udesale__ has joined #openstack-ironic07:19
*** ccamacho has joined #openstack-ironic07:20
openstackgerritWang KaiFeng proposed openstack/bifrost master: no_proxy does not take effect on install  https://review.openstack.org/49027707:28
*** mgoddard has joined #openstack-ironic07:33
*** aarefiev_afk is now known as aarefiev07:35
*** priteau has joined #openstack-ironic07:39
*** robcresswell has joined #openstack-ironic07:42
openstackgerritMerged openstack/ironic master: [devstack] add support for running behind tls-proxy  https://review.openstack.org/46983607:43
*** mgoddard has quit IRC07:52
*** hshiina has joined #openstack-ironic08:01
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters  https://review.openstack.org/47882508:06
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for cinderclient  https://review.openstack.org/47617108:06
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for neutronclient  https://review.openstack.org/47617008:06
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Rework keystone auth for glance  https://review.openstack.org/46772808:06
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Fix swiftclient creation  https://review.openstack.org/47617308:06
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Use adapters for inspectorclient  https://review.openstack.org/47617208:06
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Introduce keystoneauth adapters for clients  https://review.openstack.org/46962708:06
openstackgerritMerged openstack/bifrost master: Update .gitignore  https://review.openstack.org/49028208:07
*** mgoddard has joined #openstack-ironic08:11
*** derekh has joined #openstack-ironic08:18
*** openstackgerrit has quit IRC08:18
*** udesale has joined #openstack-ironic08:18
*** udesale__ has quit IRC08:20
*** openstackgerrit has joined #openstack-ironic08:22
openstackgerritWang KaiFeng proposed openstack/bifrost master: no_proxy does not take effect on install  https://review.openstack.org/49027708:22
*** phuongnh has quit IRC08:24
*** phuongnh has joined #openstack-ironic08:27
*** yangyapeng has joined #openstack-ironic08:27
*** lucas-afk is now known as lucasagomes08:28
*** fxpester has joined #openstack-ironic08:28
*** yangyap__ has joined #openstack-ironic08:29
*** yangyape_ has quit IRC08:32
*** milan has joined #openstack-ironic08:32
*** yangyapeng has quit IRC08:32
*** jaganathan has quit IRC08:33
*** jaganathan has joined #openstack-ironic08:34
milanmorning Ironic! :)08:40
vdrokmorning ironic, milan pas-ha08:44
milanmorning pas-ha, vdrok! :)08:44
milan#pixiesay -mflexing coffeeeeee! :D08:45
PixieBootsᕙʕ⇀ᴥ⇀ʔᕗ: coffeeeeee! :D08:45
*** rama_y_ has joined #openstack-ironic09:01
*** e0ne has joined #openstack-ironic09:01
*** rama_y has quit IRC09:03
*** dims has quit IRC09:04
openstackgerritTao Li proposed openstack/ironic-specs master: Add service management API for ironic  https://review.openstack.org/47121709:05
*** arnewiebalck has joined #openstack-ironic09:06
*** dims has joined #openstack-ironic09:08
openstackgerritTao Li proposed openstack/ironic-specs master: Add service management API for ironic  https://review.openstack.org/47121709:33
*** stendulker has joined #openstack-ironic09:36
*** sambetts|afk is now known as sambetts09:38
*** Noam_ has joined #openstack-ironic09:45
Noam_Hi guys, i need some help regarding scheduling based on resource classes09:46
Noam_i have looked here https://docs.openstack.org/project-install-guide/baremetal/draft/configure-nova-flavors.html#Scheduling based on resource classes09:46
Noam_and it does not seem to work09:46
*** jaganathan has quit IRC09:46
Noam_using devstack master09:46
*** jaganathan has joined #openstack-ironic09:47
*** strigazi_OFF has quit IRC09:49
*** strigazi_OFF has joined #openstack-ironic09:50
sambettsHi Noam_, what failures are you seeing?, the resource classes work is quite a new feature so I can try to help out the best I can09:50
*** etingof has quit IRC09:51
Noam_ERROR nova.scheduler.client.report [None req-030ec9a7-4299-47e0-bd38-b9d40721c210 admin admin] Failed to retrieve allocation candidates from placement API for filters {}. Got 400: <htm Aug 03 12:40:11 qa-cld-m1-neo4 nova-scheduler[4057]:  <head> Aug 03 12:40:11 qa-cld-m1-neo4 nova-scheduler[4057]:   <title>400 Bad Request</title> Aug 03 12:40:11 qa-cld-m1-neo4 nova-scheduler[4057]:  </head> Aug 03 12:40:11 qa-cld-m1-neo4 nov09:53
sambettsplease use paste.openstack.org when pasting logs, they don't format well in IRC :)09:54
Noam_http://paste.openstack.org/show/617389/09:54
sambettsNoam_: that looks like an error from the placements-api, can you see anything in the placement-api service's logs09:55
Noam_http://paste.openstack.org/show/617390/09:55
*** yangyap__ has quit IRC09:56
*** yangyapeng has joined #openstack-ironic09:57
Noam_cant find any errors there09:58
*** yangyape_ has joined #openstack-ironic10:00
*** yangyapeng has quit IRC10:01
*** mrtenio has joined #openstack-ironic10:04
sambettsNoam_: can you paste.openstack.org the schdeduler logs before that error, there may be some warnings that will point us in the right direction10:04
mrtenioGood Morning Ironic!10:04
Noam_sure few min ill reproudce10:05
sambettsNoam_: actually I believe I know what the issue is, you are missing the Ironic resource class from your flavor10:05
sambettsnova flavor-key my-baremetal-flavor set resources:CUSTOM_<IRONIC_NODE_RESOURCE_CLASS>=110:06
sambettsNoam_: you have uncovered a case of bad error handling in nova though10:07
sambettsNoam_: I'm going to open a bug about it10:07
Noam_http://paste.openstack.org/show/617390/ this is how my flavor defined10:08
sambettsNoam_: I can see in the properties section the ironic resource class is missing10:08
*** etingof has joined #openstack-ironic10:09
*** dtantsur|afk is now known as dtantsur10:10
dtantsurmorning ironic10:10
sambettsHey dtantsur10:10
Noam_oh10:10
Noam_this custom prefix is requirede?10:10
sambettsNoam_: that is the part that links the ironic node to the flavor10:11
sambettsthe custom prefix is required because its an ironic specific resources entry10:11
*** yangyapeng has joined #openstack-ironic10:12
dtantsurnot sure I can recommend using resource classes until we get a successful CI run with them10:12
sambetts+1 definatly not from prod yet, but this is a devstack env10:13
sambettss/from/for10:13
*** hao has quit IRC10:13
dtantsurah, ok10:14
dtantsurjust making it clear: I'm not sure myself it works the way we expect it too10:14
dtantsurhelp is welcome: https://review.openstack.org/47696810:14
patchbotpatch 476968 - ironic - [devstack] use resource classes by default10:14
Noam_is this correct wat http://paste.openstack.org/show/617395/?10:15
*** yangyape_ has quit IRC10:15
sambettshaha yeah me neither, the error we've just debugged is the first of a number of weird errors I'm sure we're going to see10:15
dtantsurNoam_: check the patch I posted, this is how it's supposed to work (but it does not for me)10:15
mrtenioMorning dtantsur. I am working with fellypefca on the patch https://review.openstack.org/#/c/483430. About your comment about PXE interface. Since PXE is not a driver, I moved the pxe.rst to source/admin instead of source/admin/driver. But since it is not a driver, where should I reference it? Or I don't need to do that? thanks10:15
patchbotpatch 483430 - ironic - Add a flag to always perform persistent boot on PX...10:15
dtantsurmrtenio: it's fine to keep it in source/admin/driver, as it's driver-related. just please call it "boot interface", not "driver"10:16
sambettsdtantsur: it makes me wonder if we should have a admin/interfaces.rst because with hardware decomp the standard admin/drivers page is not quite in the right format10:17
sambettsdtantsur: also we're going to need to start documenting the hardware types soon too10:17
sambettsI can begin to put together the doc infra for those if you want10:17
mrteniohmm, that makes sense.10:18
dtantsursambetts: "driver" is fine with me, as stuff there ends up related to the "driver" field on Node10:18
dtantsurI'm not found of multiplying the directories in docs, but I won't block it either I guess10:18
dtantsurs/found/fond/ ofc10:18
sambettsdtantsur: hmmm, just thinking that eventually when we document hardware types multiple hardware types will want to link to the same interfaces10:18
dtantsursambetts: I think we can put interfaces to the "driver" dir as well. for user point of view they are drivers, after all10:19
sambettsyeah that makes sense10:19
mrtenioI am keeping it in the driver dir then. If it comes to changing its location, 'will do it. thank you dtantsur sambetts.10:21
sambettsawesome thanks mrtenio, so the only change required is the rename it from "pxe driver" to "pxe boot interface"10:22
mrteniook :)10:22
aparnavhi dtantsur, Regarding your comment in the patch https://review.openstack.org/#/c/422572/26. We wanted to keep `update_firmware` as a generic clean step with different modes of firmware update since it will be easier for the user as the arguments are similar for both the modes.10:23
aparnav What do you think about this?10:23
patchbotpatch 422572 - ironic - Support hpsum based `update_firmware` for iLO drivers10:23
dtantsuraparnav: hi! they are not similar.. they work differently and accept different arguments. Why confuse users?10:23
dtantsurwhat's the benefit of keeping the same name?10:23
*** yangyapeng has quit IRC10:25
aparnavdtantsur, One reason is the end result is updating the firmware.10:29
aparnav Also the existing update_firmware has the support to allow different mode of update.10:29
dtantsurwell, yes. but the underlying code is all different, and it is visible to a user10:30
dtantsurputting aside that in-band get work longer, the arguments are slightly different10:31
dtantsurit's not a matter of simply changing mode to hpsum10:31
* dtantsur brb10:32
Noam_sambetts , dtantsur this what im getting now10:32
Noam_http://paste.openstack.org/show/617399/10:32
Noam_btw im using real hw10:33
sambettsNoam_: have you set the resource_class field on your nodes in Ironic? and do you have nodes in Ironic that are avaible10:34
Noam_http://paste.openstack.org/show/617400/10:36
sambettsNoam_: can you do export IRONIC_API_VERSION=latest and rerun that ironic node-show10:37
Noam_http://paste.openstack.org/show/617401/10:38
*** phuongnh has quit IRC10:40
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Finalize migration to keystoneauth adapters  https://review.openstack.org/47882510:43
*** jkilpatr has quit IRC10:44
sambettsNoam_: you havn't set the resource_class on the ironic baremetal node, https://docs.openstack.org/project-install-guide/baremetal/draft/configure-nova-flavors.html, you need to do all the setups we describe in that documentation to enable this feature10:46
aparnavdtantsur, I have stated the comparitive argument structures for both modes here: http://paste.openstack.org/show/617402/10:46
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC  https://review.openstack.org/46286110:50
*** udesale__ has joined #openstack-ironic10:52
*** udesale has quit IRC10:53
*** paresh has joined #openstack-ironic10:54
pareshvsaienk0, Hi10:55
pareshvsaienk0, In CI gate facing issue in keystone, failing with "iso8601.iso8601.ParseError: Unable to parse date string '2017-08-02T10:53:06.000000UTC+00:00'"10:57
*** Nisha_Agarwal has joined #openstack-ironic10:57
openstackgerritMark Goddard proposed openstack/ironic master: [PoC]: General object version conversion framework  https://review.openstack.org/49042310:58
dtantsuraparnav: this is what I call different :) also keep in mind that "ilo" one accepts a list, while "hpsum" only accepts one item (IIRC)10:59
aparnavdtantsur, yea true. 'hpsum' accepts one element list.11:00
*** Nisha_Agarwal has quit IRC11:01
aparnavdtantsur, From the user's pov I also thought that it will be easier to have one stop API for `update_firmware`11:01
dtantsuraparnav: I'm not sure that's the case. They anyway have to provide a step name. And they anyway have to understand the difference in arguments between the two modes..11:02
*** Nisha_Agarwal has joined #openstack-ironic11:02
aparnavdtantsur, hmmm11:03
*** vmud213 has joined #openstack-ironic11:05
aparnavdtantsur, probably there's also plans to add more modes, hypothetically ``hybrid`` where the argument variations are exhasutive IMO11:06
dtantsuraparnav: users can request two steps.. anyway, that's all valid, but these steps still work differently and accept different arguments11:07
*** udesale__ has quit IRC11:08
dtantsurmore opinions welcome ^^^11:08
*** jaianshu has joined #openstack-ironic11:09
*** Nisha_ has joined #openstack-ironic11:13
*** Nisha_Agarwal has quit IRC11:13
*** amarao has joined #openstack-ironic11:14
aparnavdtantsur, okay. will think over it. thank you11:15
*** stendulker_ has joined #openstack-ironic11:15
*** Nisha_Agarwal has joined #openstack-ironic11:16
*** Nisha_ has quit IRC11:16
*** jaganathan has quit IRC11:16
*** jkilpatr has joined #openstack-ironic11:17
*** stendulker has quit IRC11:18
*** stendulker_ has quit IRC11:19
dtantsurnp11:20
*** zhangfei has quit IRC11:20
*** amarao1 has joined #openstack-ironic11:21
Noam_<dtantsur> <sambetts> Hi still not geting it to work http://paste.openstack.org/show/617404/11:23
sambettsNoam_: what error messages are you seeing this time?11:24
dtantsurNoam_: LGTM, not sure why it does not work (it also does not work in the CI, jaypipes promised to look into it later)11:24
*** jaianshu has quit IRC11:25
*** deray has joined #openstack-ironic11:25
Noam_http://paste.openstack.org/show/617406/11:25
deraygood morning Ironic'ers!11:27
sambettsNoam_: in your nova.conf set use_baremetal_filters=False11:30
sambettsNoam_: then restart nova compute11:30
*** fellypefca has joined #openstack-ironic11:31
fellypefcaGood morning, Ironic :)11:32
dtantsurmorning fellypefca, deray11:33
derayfellypefca, dtantsur, o/11:34
Noam_sambetts: it is working now thanks11:35
fellypefcaderay, dtantsur \o11:36
dtantsurNoam_: oh, you can deploy with resource classes now?11:37
Noam_yes11:38
sambettssounds like the problem in the gate is probably the use_baremetal_filters True11:38
sambettsI'll ping jaypipes11:39
dtantsursambetts: this is an important finding then. and btw, we need to deprecate these filters now, they're useless with resource classes11:39
dtantsuredleafe: may be of interest for you ^^^11:39
derayif anybody has some free time please have a look at: free11:39
deraysorry ^^^11:40
derayhttps://review.openstack.org/#/c/346683/11:40
patchbotpatch 346683 - ironic - Refresh firmware versions on firmware update11:40
dtantsurputting on my list, will try to get to it today/tomorrow11:40
openstackgerritStenio Araujo proposed openstack/ironic master: Add a flag to always perform persistent boot on PXE interface  https://review.openstack.org/48343011:41
fellypefcadtantsur, sambetts ^ ^ :)11:41
deraydtantsur, oh gr8. thx11:41
dtantsurputting this one too (oh my, so many patches :)11:42
fellypefcadtantsur haha ok!11:43
mrteniothanks :)11:43
*** yangyapeng has joined #openstack-ironic11:45
*** lucasagomes is now known as lucas-hungry11:48
*** paresh has quit IRC11:48
*** Nisha_Agarwal has quit IRC11:48
*** yangyapeng has quit IRC11:48
*** yangyapeng has joined #openstack-ironic11:49
aarefievmilan: hey, are you around11:50
* dtantsur wonders the same, as he wants coffee :D11:51
aarefievlol11:51
openstackgerritHao Li proposed openstack/ironic-lib master: Retry when execute partprobe cmd failed  https://review.openstack.org/44360411:52
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Skip warning when changing target_raid_config  https://review.openstack.org/48016111:53
aarefievdtantsur: btw please add https://review.openstack.org/#/c/463768/11:53
patchbotpatch 463768 - ironic-inspector - Use declarative reader/writer separation11:53
aarefievto your looong list11:53
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add test for set/unset node target_raid_config  https://review.openstack.org/48062311:57
*** litao__ has quit IRC11:59
*** namnh has quit IRC12:01
*** xavierr has joined #openstack-ironic12:02
*** hao has joined #openstack-ironic12:02
xavierrgood morning ironic :)12:05
*** makowals_ has joined #openstack-ironic12:07
*** makowals has quit IRC12:07
fellypefcaxavierr hey o/12:08
*** dprince has joined #openstack-ironic12:12
*** tuanluong has quit IRC12:14
*** rbudden has joined #openstack-ironic12:16
*** makowals_ has quit IRC12:16
*** makowals has joined #openstack-ironic12:17
*** adisky__ has quit IRC12:21
*** makowals_ has joined #openstack-ironic12:22
*** makowals has quit IRC12:23
dtantsuraarefiev: it's there, just not quite on the top12:28
dtantsurmorning xavierr12:29
*** larivee has left #openstack-ironic12:29
*** baoli has joined #openstack-ironic12:30
*** makowals_ has quit IRC12:36
*** makowals has joined #openstack-ironic12:36
sambettsdtantsur: I think we'll need this: https://review.openstack.org/#/c/490459/12:38
patchbotpatch 490459 - openstack-dev/devstack - Disable baremetal sched filters when using resourc...12:38
*** vmud213 has quit IRC12:41
*** gcb has joined #openstack-ironic12:41
openstackgerritHugo Nicodemos proposed openstack/ironic master: Introduce hpOneView and ilorest to OneView  https://review.openstack.org/46812912:41
openstackgerritSaulo-Aislan proposed openstack/ironic stable/newton: Removes agent mixin from oneview drivers  https://review.openstack.org/49009512:42
nicodemosmorning xavierr, fellypefca, dtantsur, ironic! :)12:43
fellypefcanicodemos o/12:44
fxpestercan ironic integrate with glance without swift ? only directly with glance12:47
edleafedtantsur: just getting my first cup of coffee12:49
edleafedtantsur: the baremetal filters should be working in Pike12:49
* dtantsur on mtg, sorry12:49
edleafedtantsur: in Queens the ram/disk/cpu should be removed12:49
edleafedtantsur: no worries12:49
milanaarefiev, hey :D yeah I'm here12:50
sambettsedleafe: the ExactRamFilter vomed here because of the use of resource classes http://paste.openstack.org/show/617406/12:51
*** cdearborn has joined #openstack-ironic12:53
*** udesale has joined #openstack-ironic12:53
*** Goneri has joined #openstack-ironic12:54
*** lucas-hungry is now known as lucasagomes12:54
aarefievmilan: sorry, I'm here too :) about your concerns on https://review.openstack.org/#/c/462861/12/ironic_inspector/worker/manager.py12:58
patchbotpatch 462861 - ironic-inspector - Introduce oslo.messaging RPC12:58
*** aparnav has quit IRC12:58
*** milan has quit IRC12:58
*** _milan_ has joined #openstack-ironic12:58
edleafesambetts: if the flavor used is anything like http://paste.openstack.org/show/617395/, then it's *not* using resource classes.13:00
edleafesambetts: it's just zeroing out the standard resources. It needs an entry like 'resources:CUSTOM_CLASS_NAME=1'13:01
sambettsedleafe: it isn't, the flavor looked like this: http://paste.openstack.org/show/617404/13:01
sambettsso it was correct as far as I could tell13:01
sambettsdisabling the filters made it all work13:04
*** sasha21 has joined #openstack-ironic13:04
*** Goneri has quit IRC13:05
*** links has quit IRC13:06
*** deray has quit IRC13:10
*** Goneri has joined #openstack-ironic13:12
*** mjturek has joined #openstack-ironic13:13
edleafesambetts: yes, the flavor looks correct. I'm not sure that disabling *all* the filters is necessary, though13:14
*** rloo has joined #openstack-ironic13:14
sambettsedleafe: we're leaving the stock nova filters on, just disabling the custom ones we turn on the ironic case13:14
edleafesambetts: updating the CONF for baremetal_enabled_filters to remove the cpu/ram/disk filters should be a better path13:14
sambettswe only need what nova loads as the stock filters for ironic now13:15
sambettsor should do anyway, that is one of the points for introducing ironic resource classes13:15
sambettsto make vm nova conf and ironic nova conf the same13:15
sambettsso that running in envs with both vms and bms scheduling works as expected13:16
edleafesambetts: I'm a little confused (I told you I was on my first cup of coffee!)13:17
*** mjturek has quit IRC13:17
edleafesambetts: Don't the stock nova filters also filter on ram/cpu/disk?13:17
Noam_https://github.com/openstack-dev/devstack/blob/master/lib/nova_plugins/hypervisor-ironic#L45 maybe someone will push commit to disable it13:18
sambettsNoam_: done13:18
sambettsedleafe: not ExactRam match that is required for the case before resource classes when flavor ram has to exactly == resource provders free ram13:18
sambettsnova default for ram do if free_ram >= asked for ram13:19
sambettss/do/does13:19
edleafesambetts: ah, ok13:20
edleafesambetts: so MEMORY_MB of 0 will pass13:20
sambettsyes, with the default filters, but with the exact match filters the ironic nodes still populate ram in nova, so then 0 != 256gb so it fails13:21
sambettsso thats why we need to disable those filters now13:22
sambettsthe whole use_baremetal_filters = True option in the nova config can probably be deprecated tbh13:22
*** udesale has quit IRC13:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Serve boot configs from Ironic API  https://review.openstack.org/36943813:23
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Implement Driver API for dynamic boot config  https://review.openstack.org/39439813:23
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Add RPC API to get the boot config  https://review.openstack.org/39439913:23
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: DNM test enabled dynamic iPXE  https://review.openstack.org/39194313:23
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Allow not writing pxe configs to disk  https://review.openstack.org/39439713:23
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Convert iPXE boot config to proper Jinja template  https://review.openstack.org/39346213:23
*** jcoufal has joined #openstack-ironic13:23
*** udesale has joined #openstack-ironic13:23
edleafesambetts: are there any other baremetal filters that may be needed that aren't resource-specific? IOW, are we removing some selective process by disabling all baremetal filters?13:24
sambettsedleafe: only additional filters I think are the Exact ones https://github.com/openstack/nova/blob/master/nova/conf/scheduler.py#L28413:25
sambettsvs enabled_filters here: http://logs.openstack.org/68/476968/5/check/gate-tempest-dsvm-ironic-pxe_ipmitool-postgres-ubuntu-xenial-nv/8c9583f/logs/etc/nova/nova.conf.txt.gz13:26
*** udesale has quit IRC13:28
edleafesambetts: ok, thanks13:28
*** marios has quit IRC13:29
*** mtanino has joined #openstack-ironic13:29
*** marios has joined #openstack-ironic13:30
xavierrfolks, may you +A https://review.openstack.org/#/c/485798 ? it has two +2 already :)13:31
patchbotpatch 485798 - ironic - Add hardware type for HPE OneView13:31
*** mjturek has joined #openstack-ironic13:32
*** SerenaFe_ has joined #openstack-ironic13:41
*** crushil has quit IRC13:48
*** jaypipes has joined #openstack-ironic13:49
openstackgerritDao Cong Tien proposed openstack/ironic master: Documentation for irmc hardware type  https://review.openstack.org/49048313:50
openstackgerritMerged openstack/ironic master: [Trivialfix]Fix typos in ironic  https://review.openstack.org/49031913:52
*** tiendc has quit IRC13:55
openstackgerritMerged openstack/ironic master: Avoid double ".img" postfix of image file path in devstack installation  https://review.openstack.org/45241313:55
dtantsursambetts, edleafe, not sure I understand the outcome: should we deprecate these Exact filters now?13:57
sambettsdtantsur: first outcome is disable the filters in devstack if using resource classes, not sure on the long term13:57
dtantsurokay, lemme update my patch13:58
sambettsdtantsur: https://review.openstack.org/#/c/490459/13:58
patchbotpatch 490459 - openstack-dev/devstack - Disable baremetal sched filters when using resourc...13:58
*** mkovacik__ has joined #openstack-ironic13:58
sambettsI guess you just need to add an depends-on13:58
*** rloo has quit IRC13:58
*** _milan_ has quit IRC13:59
*** rloo has joined #openstack-ironic13:59
openstackgerritDmitry Tantsur proposed openstack/ironic master: [devstack] use resource classes by default  https://review.openstack.org/47696813:59
*** crushil has joined #openstack-ironic13:59
dtantsuryep, done ^^13:59
sambettsawesome, we'll see if it passes13:59
*** marios has quit IRC14:01
openstackgerritMerged openstack/ironic master: Revert "Wait until iDRAC is ready before out-of-band cleaning"  https://review.openstack.org/48131814:01
edleafedtantsur: the exact filters should be removed in Queens, so deprecate now sounds good14:01
rloohello and good morning sambetts, dtantsur, edleafe14:01
*** krtaylor has quit IRC14:02
sambettso/ rloo14:02
*** alezil has quit IRC14:03
*** marios has joined #openstack-ironic14:03
rloodtantsur: btw, this failed, and when i looked it was due to a port.physical_network test I think. so am guessing it has to do with the API version thing: https://review.openstack.org/#/c/484405/14:04
patchbotpatch 484405 - ironic (stable/ocata) - Force InnoDB engine on interfaces table14:04
*** alezil has joined #openstack-ironic14:05
sambettsrloo: yup, I left a comment on it regarding that14:05
*** hshiina has quit IRC14:05
sambettsdtantsur's patch should fix that14:05
dtantsurrloo: sigh.. I'll take a look in a few moments14:05
rloosambetts: it merged14:05
sambettsoh did it fail again?!14:06
rloosambetts: that's why i'm pinging dtantsur. unless the timing was off...14:06
rloothe recheck of that patch was done about 1 min after dmitry's patch merged; that should have been enough time?14:07
rloojlvillal just did a recheck so we could also wait for that...14:07
sambettsrloo: ah it looks like the problem is that dtantsur's fix has merged into the out-of-tree ironic tempest plugin14:08
sambettshttp://logs.openstack.org/05/484405/1/check/gate-tempest-dsvm-ironic-pxe_ipmitool-postgres-ubuntu-xenial-nv/fbd2874/console.html#_2017-08-01_14_06_21_46621914:08
dtantsurwut?14:08
sambettsacutally I might be reading that wrong :/14:08
dtantsurit looks like my patch did not work...14:09
dtantsurmaybe it was not picked yet indeed?14:09
sambettstrying to remember where in the devstack logs the tempest plugin is pulled in from master14:10
sambettsso I can try to see the commit ID14:11
jlvillalsambetts, TEMPEST_PLUGIN I think14:14
jlvillalsambetts, https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L188-L19614:15
* jlvillal goes to make breakfast14:15
sambettsyeah, I'm trying to see where it gets processed in the run to actaully clone it14:15
jlvillalsambetts, I mean: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L233-L24114:16
jlvillalOther one was inspector14:16
* dtantsur tries to clarify whether we have to branch next week actually...14:16
openstackgerritXavier proposed openstack/ironic master: Introduce hpOneView and ilorest to OneView  https://review.openstack.org/46812914:17
openstackgerritXavier proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687214:17
*** udesale has joined #openstack-ironic14:19
*** mjura has quit IRC14:21
sambettsjlvillal: this is what I was looking for http://logs.openstack.org/05/484405/1/check/gate-tempest-dsvm-ironic-pxe_ipmitool-postgres-ubuntu-xenial-nv/fbd2874/logs/devstacklog.txt.gz#_2017-08-01_13_58_17_571 but unfortunatly pip install doesn't log which commit ID it has cloned14:22
openstackgerritFellype Cavalcante proposed openstack/ironic master: Add a flag to always perform persistent boot on PXE interface  https://review.openstack.org/48343014:22
*** athomas has quit IRC14:23
*** athomas has joined #openstack-ironic14:26
openstackgerritChason Chan proposed openstack/ironic-inspector master: Update the documentation link for doc migration  https://review.openstack.org/48507114:29
rloodtantsur: no, branch next week? cuz of rc1? yikes. yeah, let us know what that would mean :-(14:29
dtantsurrloo: I've asked on the ML, let's see14:30
rlooironic doesn't follow the release cycle-with-milestones?, so I don't think so14:30
rloodtantsur: yeah, we're release:cycle-with-intermediary14:32
sambettsyeah I think we're good because we do to rcs we just do releases and then tag a stable from whatever release we're on at the time pike is released14:32
*** Noam_ has quit IRC14:32
rloodtantsur: https://releases.openstack.org/reference/release_models.html#cycle-with-intermediary14:33
*** rbartal has quit IRC14:34
*** rama_y_ has quit IRC14:35
dtantsurI know, but we still follow the release cadence.. and the email I've recieved clearly said that our type should also branch next week. Let's see..14:36
openstackgerritXavier proposed openstack/ironic master: Introduce hpOneView and ilorest to OneView  https://review.openstack.org/46812914:36
openstackgerritHugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers  https://review.openstack.org/46842814:37
TheJuliagood morning folks14:39
rlooMorning TheJulia!14:40
*** dyasny has joined #openstack-ironic14:41
dtantsurmorning TheJulia!14:44
edleaferloo: pushed the streamlined version of https://review.openstack.org/#/c/487954/ - just for you!14:48
patchbotpatch 487954 - nova - Handle addition of new nodes/instances in ironic f...14:48
rlooedleafe: all those other readers/reviewers out there thank you too! :)14:48
*** slagle has quit IRC14:48
*** slagle has joined #openstack-ironic14:50
edleaferloo: :)14:50
*** rpioso has joined #openstack-ironic14:52
*** mat128 has joined #openstack-ironic14:53
openstackgerritRushil Chugh proposed openstack/ironic-specs master: Add spec for XClarity Driver  https://review.openstack.org/47692214:54
*** hel0tsp has quit IRC14:57
*** krtaylor has joined #openstack-ironic15:00
*** mat128 has quit IRC15:02
rpiosodtantsur: Can the interfaces supported by a hardware type be changed from release to release of ironic, including the default interfaces?  How would that be handled during upgrade?15:08
*** priteau has quit IRC15:09
sambettsrpioso: there would need to be a deprecation period as with changing any default behaviour in Ironic15:10
sambettsrpioso: so for one release a hardware type would need to remain the same defaults with a deprecation warning and the new default as an optional interface, and then change it in the next release15:11
*** rama_y has joined #openstack-ironic15:13
konetzedgetting an error/warning in ironic mitika that says something like whole disk image isnt supported for uefi, is this something not supported by ironic in any version?15:13
rlooTheJulia: (just picking on you). in the rolling upgrades process, there is a bit about 'api load balancing draining' (when upgrading ironic-api). i'm not sure it is worth mentioning -- operators would/should know how to deal with that. wdyt? http://docs-draft.openstack.org/65/416565/23/check/gate-ironic-docs-ubuntu-xenial/4226e88//doc/build/html/admin/upgrade-guide.html#api-load-balancer-draining15:15
TheJuliathey should if they are using load balancing, since it would be dependent upon their hardware or software and the running configuration15:16
TheJuliaI would say "we recommend that you redirect requests to the newer API services and drain off of the api servers that have not yet been upgraded." and kind of leave it at that.15:17
*** MattMan has quit IRC15:18
rlooTheJulia: ok, so you think I should leave that in. will do.15:18
*** MattMan has joined #openstack-ironic15:18
rpiososambetts: How is that deprecation expressed in the hardware type source code?15:18
sambettsrpioso: I expect that it'll need to be in the __init__ function for the hardware_type15:20
*** krtaylor has quit IRC15:22
TheJuliakonetzed: So that was likely a bad check that I think we fixed in the newton release (I seem to remember something about it, since whole disk images can contain the partition required for UEFI boot, the constraint is of course, setting the bios to be in uefi boot mode which is likely a vendor driver specific feature most of the time.15:25
konetzedTheJulia: thanks, i will look at upgrading to the newton release :D15:26
TheJuliakonetzed: likely a good idea at a minimum, since we don't support mitaka anymore15:27
NobodyCamGood Morning Ironic'ers :)15:27
*** dprince has quit IRC15:27
*** mkovacik__ is now known as milan15:28
milanmorning NobodyCam! :)15:28
konetzedTheJulia: I figured that would be the answer ;) Trying to get a moonshot up and running with ironic.  BTW keep up the good work my teams are loving Ironic!15:28
NobodyCamhey hey Good Morning milan :)15:29
NobodyCamalmost Friday!15:29
milan:)15:29
milanyup :D15:29
TheJuliakonetzed: are you using ipmi based drivers?15:32
konetzedTheJulia: yes15:32
TheJuliagood, since the ilo driver doesn't support the moonshot series :)15:32
* TheJulia really wishes she was able to leave HPE with a moonshot somehow....15:33
TheJuliawould have made the perfect multi-node... needing tons of power to run lab.15:34
konetzedTheJulia: just got one with 45 of the m510 cartridges15:34
TheJuliakonetzed: nice!15:35
TheJuliaI had an early model that I was playing with. I think they the lowest level cartridges available. :(15:36
jlvillaldtantsur, sambetts rloo: I see the stable/ocata patch still failed :(15:36
*** baoli has quit IRC15:36
rloojlvillal: :-(15:37
*** baoli has joined #openstack-ironic15:38
*** dprince has joined #openstack-ironic15:40
*** kiseok7 has quit IRC15:43
* sambetts --> home 15:45
sambettsnigth all15:45
*** sambetts is now known as sambetts|afk15:45
*** kiseok7 has joined #openstack-ironic15:46
*** ccamacho has quit IRC15:47
*** e0ne has quit IRC15:47
*** penick has joined #openstack-ironic15:49
dtantsurjlvillal: yep :(15:49
dtantsurrpioso: sorry, was out. I guess your question was answered?15:49
*** aarefiev is now known as aarefiev_afk15:53
*** dyasny has quit IRC15:53
*** penick has quit IRC15:53
*** amarao2 has joined #openstack-ironic15:54
*** amarao1 has quit IRC15:54
*** amarao2 has quit IRC15:55
*** udesale has quit IRC15:55
*** dyasny has joined #openstack-ironic15:55
*** amarao1 has joined #openstack-ironic15:56
*** amarao1 has quit IRC16:01
*** mtanino has quit IRC16:01
*** rpioso has quit IRC16:05
*** Nisha_Agarwal has joined #openstack-ironic16:06
rloodoes anyone know how to locally generate some part (eg doc/source/admin) of the docs?16:06
dtantsurI doubt it's possible16:07
*** amarao has quit IRC16:07
rlooboo hoo16:07
*** hel0tsp has joined #openstack-ironic16:07
*** marios has quit IRC16:08
*** rcernin has quit IRC16:12
*** krtaylor has joined #openstack-ironic16:12
*** SerenaFe_ has quit IRC16:14
*** fellypefca has quit IRC16:16
rloodtantsur: wrt L134, should I delete or reword as suggestion: https://review.openstack.org/#/c/416565/15..23/doc/source/admin/upgrade-guide.rst16:17
patchbotpatch 416565 - ironic - Rolling upgrade procedure documentation16:17
dtantsurrloo: I'd remove it, but I'm clearly biased16:18
rloodtantsur: oh, if you are biased, then i should change to a suggestion.16:18
rloodtantsur: clearly you don't advise it. but there could be conflicting dependencies?16:19
rloooh, i'll remove it. someone could add it in later.16:20
dtantsurrloo: if there are, our partial grenade will be broken, no?16:20
dtantsurwe don't deploy our CI in separate venvs, nor in containers16:20
*** vishwana_ has joined #openstack-ironic16:20
rloodtantsur: ah, good point. at least, it works with ubuntu :)16:20
dtantsurright :) the only potential trouble makers are upper caps. I think we not try to avoid them16:22
rloodtantsur: thx. we can wait to see what real live experience sez.16:23
*** vishwanathj has quit IRC16:24
*** ccamacho has joined #openstack-ironic16:24
*** jtomasek has quit IRC16:25
dtantsuryep16:25
*** ccamacho has quit IRC16:26
openstackgerritHugo Nicodemos proposed openstack/ironic master: Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687216:27
openstackgerritRuby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation  https://review.openstack.org/41656516:29
openstackgerritIlya Etingof proposed openstack/virtualbmc master: multiprocess server, socket-based management cli tool  https://review.openstack.org/48887416:30
*** rama_y has quit IRC16:30
dtantsurrloo, TheJulia, so he's the apparent outcome: we must have the final release before Aug 24, and we're advised to do it somewhat earlier16:31
TheJuliaof course :)16:32
rloodtantsur, TheJulia: I was hoping to have a very-close-to-final candidate by Aug 14, or by mid-week16:33
*** Goneri has quit IRC16:33
rloodtantsur, TheJulia: polishing, finishing touches after that. which include mostly (I hope) doc changes.16:33
dtantsurthis is a good plan16:34
rloodtantsur, TheJulia: The only 'big' things left that need serious reviewing are IPA REST API versioning and refactoring of code to access other services.16:35
rloobesides gate failure issues of course :-(16:35
rlooanything else?16:35
rloodtantsur: we might want to try to land the new hw types; i haven't looked at any of those patches though.16:36
dtantsuryeah, to be honest I care about new hw types even more than about IPA API16:36
rloodtantsur: oh, would be nice to get the supported power states API done too but I haven't looked at that either16:36
dtantsurwe're going to advertise the driver composition this release, we better provide more of them16:36
dtantsurrloo: should be easy, I'll try to get to it16:36
rloodtantsur: ah, didn't realize that. we should have put it in this week's priorities. ok, will try to look at them today.16:36
dtantsuralso these small vendor things that we never have time for...16:36
rloodtantsur: huh? i don't have time for them :) and :(16:37
dtantsurI'm helping landing a few of them.. we don't need everyone to pile on it16:37
rloodtantsur: could you check, after we cut stable/pike, are they going to be ok with us making doc changes in stable/pike?16:37
rloothx dtantsur! (no piling from me then)16:38
rloobbiab, need to get food...16:38
dtantsurrloo: I have an answer: backporting the docs is fine16:38
dtantsuras long as the docs actually apply to the stable branch, of course16:38
*** rama_y has joined #openstack-ironic16:39
*** oomichi has quit IRC16:41
*** lucasagomes is now known as lucas-afk16:42
*** Nisha_ has joined #openstack-ironic16:43
*** Nisha_Agarwal has quit IRC16:43
*** oomichi has joined #openstack-ironic16:44
*** sergek_ has quit IRC16:47
*** sergek_ has joined #openstack-ironic16:48
*** yufei has joined #openstack-ironic16:51
openstackgerritDao Cong Tien proposed openstack/ironic master: Documentation for irmc hardware type  https://review.openstack.org/49048316:52
*** hel0tsp has left #openstack-ironic16:53
*** diablo_rojo has joined #openstack-ironic16:54
*** pcaruana has quit IRC16:54
openstackgerritDao Cong Tien proposed openstack/ironic master: Documentation for irmc hardware type  https://review.openstack.org/49048316:55
rloodtantsur: good, then we can punt the docs til the end :)16:55
dtantsuras long as we don't forget about them completely..16:55
*** Goneri has joined #openstack-ironic16:56
dtantsuranyway, time to go. g'night all16:57
rloodtantsur: won't. i. don't. think.16:57
rloodtantsur: good night!16:57
*** dtantsur is now known as dtantsur|afk16:57
*** derekh has quit IRC16:58
openstackgerritIhar Hrachyshka proposed openstack/ironic master: DNM check neutron fix for grenade job  https://review.openstack.org/49059417:00
*** Nisha_ has quit IRC17:00
*** Nisha_away has joined #openstack-ironic17:00
*** rcernin has joined #openstack-ironic17:02
*** mgoddard has quit IRC17:03
*** sasha21 has quit IRC17:09
*** baoli has quit IRC17:10
*** etingof has quit IRC17:11
*** yufei has quit IRC17:16
*** Nisha_away has quit IRC17:17
* jlvillal hopes it fixes the issue17:18
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DNM check neutron fix for grenade job #1  https://review.openstack.org/49059417:19
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DNM check neutron fix for grenade job #1  https://review.openstack.org/49060317:20
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DNM check neutron fix for grenade job #2  https://review.openstack.org/49060417:20
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DNM check neutron fix for grenade job #3  https://review.openstack.org/49060517:20
*** Nisha_away has joined #openstack-ironic17:20
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DNM check neutron fix for grenade job #4  https://review.openstack.org/49060617:20
*** sasha2 has joined #openstack-ironic17:22
*** etingof has joined #openstack-ironic17:27
*** baoli has joined #openstack-ironic17:29
*** milan has quit IRC17:33
*** tesseract has quit IRC17:49
*** dyasny_ has joined #openstack-ironic17:56
*** baoli has quit IRC17:58
*** dyasny has quit IRC18:00
Nisha_awayvdrok, hi18:02
Nisha_awayvdrok, wrt to ques on https://review.openstack.org/#/c/422572/18:02
patchbotpatch 422572 - ironic - Support hpsum based `update_firmware` for iLO drivers18:02
Nisha_awayupdate_firmware via ilo is OOB operation actually and doesnt require any functionality from ramdisk18:03
Nisha_awayvdrok, ^^18:03
Nisha_awayvdrok, while hpsum is a software needs to be included in the ramdisk to perform firmware update18:04
*** baoli has joined #openstack-ironic18:07
*** jlabarre has joined #openstack-ironic18:11
*** ralonsoh has quit IRC18:12
TheJuliarloo: I'll take a look at the IPA stuff in a few hours. Out of spoons at the moment :(18:12
rlooTheJulia: thx. although dtantsur|afk said he sees the new hw types as higher priority.18:13
rlooTheJulia: having said that, I have reviewed the IPA versioning stuff, and I believe it is mostly all there. I didn't see anything majorly wrong with the patches. so should be doable in the next week.18:14
TheJuliaNisha_away: vdrok likely already left for the day, did you put those comments on to the review?18:14
TheJuliarloo: yeah, I saw that and I think I've looked at two hw type patches so far today :)18:15
Nisha_awayTheJulia, no. I will add.18:15
TheJuliaNisha_away: thanks!18:15
rlooTheJulia: thx! I think we should be good, barring any major bugs...18:15
TheJuliaOr gates melting18:16
rlooTheJulia: or freezing :)18:19
TheJuliaOr, going on vacation18:19
*** rpioso has joined #openstack-ironic18:22
rloo:)18:24
*** mat128 has joined #openstack-ironic18:33
*** rama_y has quit IRC18:33
*** rama_y has joined #openstack-ironic18:35
TheJuliarama_y: quick question, have you had a chance to revise the ironic-ui patch that I commented on ?yesterday?18:47
rama_yHi TheJulia, I have a revision ready; will post it soon. Thanks18:47
*** livelace has quit IRC18:49
TheJuliaAwesome!18:51
TheJuliaThanks!18:51
*** masber has joined #openstack-ironic18:57
*** ChubYann has joined #openstack-ironic18:58
*** masuberu has quit IRC18:59
*** racedo has quit IRC19:00
*** racedo has joined #openstack-ironic19:00
*** Sukhdev has joined #openstack-ironic19:13
*** jcoufal has quit IRC19:15
*** racedo has quit IRC19:15
*** racedo has joined #openstack-ironic19:17
*** Sukhdev_ has joined #openstack-ironic19:19
*** tellesnobrega has joined #openstack-ironic19:20
*** racedo_ has joined #openstack-ironic19:21
tellesnobregahey guys, I need to test the integration between sahara and ironic, but sadly I don't have physical nodes to work with. I saw somewhere that it is possible to use vms to simulate the physical nodes for ironic, can you point to some documentation that explains how that is done?19:21
tellesnobregathanks19:21
*** racedo has quit IRC19:22
*** baoli has quit IRC19:27
*** baoli has joined #openstack-ironic19:28
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DNM check neutron fix for grenade job  https://review.openstack.org/49059419:29
*** mat128 has quit IRC19:30
*** Nisha_away has quit IRC19:30
TheJuliatellesnobrega: we don't explicitly have documentation on this.  Our devstack script sets up a selectable number of "baremetal" vms uses virtualbmc to manage the VM nodes via ipmi,19:31
TheJuliayou would likely want to use the devstack script to setup ironic anyway for your integration testing19:31
tellesnobregaTheJulia, basically a devstack deploy will do the trick?19:34
tellesnobregaI have a packstack installation running, but if devstack is easier I will switch to that19:34
*** Sukhdev has quit IRC19:36
*** Sukhdev_ is now known as Sukhdev19:36
tellesnobregathanks19:36
rlootellesnobrega: don't know if this helps or not, wrt devstack + ironic: https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#deploying-ironic-with-devstack19:36
tellesnobregarloo, I bet it will, thanks19:37
*** ijw has joined #openstack-ironic19:38
TheJuliarloo: Thanks!19:39
rlooTheJulia: yw. (for what?)19:39
TheJuliarloo: for getting the link while I was distracted reviewing a driver patch19:40
rlooTheJulia: :)19:40
TheJuliarloo: looks like the ipa version change needs a rebase :(19:47
rlooTheJulia: oh. I think both ipa and ironic patches need updates. they aren't ready to land yet.19:48
TheJuliaAdded to my review queue anyway19:48
rlooTheJulia: I think it will land next week... (my crystal ball and all)19:49
TheJuliamost likely19:49
TheJuliahopefully I'll have the spoons to review a lot next week :)19:49
* rloo gives TheJulia thousands of spoons19:53
TheJuliaBadCub claims that I have a drawer someplace with an endless supply....19:53
* rloo wonders how BadCub would know...19:54
TheJuliaI worked with some stuff downstream at HP that was super stressful for a while and BadCub was my human shield during that time.19:55
rlooTheJulia: oh, that rings a bell... I remember BadCub mentioning those wonderful times19:56
*** harlowja has quit IRC19:57
TheJuliaYes. Wonderful would not begin to describe what I went through during the summit in paris.19:57
TheJuliaAt least, with the time zone difference, I was able to have a decent dinner most nights.19:57
rloo:(20:00
*** milan has joined #openstack-ironic20:00
*** sasha2 has quit IRC20:04
*** sasha2 has joined #openstack-ironic20:05
* BadCub hears name and magically appears20:15
BadCubdid I hear mention of spoons and good times?20:16
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Refactoring the firewall  https://review.openstack.org/47183120:18
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Allow periodics to terminate inspector  https://review.openstack.org/48790920:18
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver  https://review.openstack.org/46644820:18
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack  https://review.openstack.org/46871220:18
*** baoli has quit IRC20:22
*** hamzy has quit IRC20:37
*** amarao has joined #openstack-ironic20:39
*** Sukhdev_ has joined #openstack-ironic20:41
rloohi BadCub :)20:45
*** jkilpatr has quit IRC20:48
*** ihrachys has joined #openstack-ironic20:50
ihrachysjlvillal, looks like DNMs suggest the fix actually works?20:50
jlvillalihrachys: It does. I've done about 5 test passes and things are working.20:52
jlvillalihrachys: There was one grenade failure in https://review.openstack.org/#/c/490606/120:52
patchbotpatch 490606 - ironic - DNM check neutron fix for grenade job #420:52
ihrachysthat's another job20:53
jlvillalihrachys: But I left a comment that it failed during package installation. So it didn't get to the failure we have been seeing.20:53
ihrachysack20:53
ihrachysI will seek for second +2 then20:53
jlvillalihrachys: I have four of my own test jobs :)20:53
jlvillalihrachys: Thanks. I'll let you know if I see any failures20:53
jlvillalihrachys: Very much appreciated all of your work on this!!!20:53
ihrachysone day you'll scratch our back20:54
ihrachys:)20:54
jlvillal:)20:54
openstackgerritJulia Kreger proposed openstack/bifrost master: Ignore DIB failures that generate output files  https://review.openstack.org/49005620:56
*** milan has quit IRC20:58
*** hao has quit IRC21:01
*** krtaylor has quit IRC21:02
*** amarao has quit IRC21:02
*** Sukhdev_ has quit IRC21:03
*** ijw has quit IRC21:04
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Set target raid configuration on a node  https://review.openstack.org/48320521:05
*** ijw has joined #openstack-ironic21:05
*** crushil has quit IRC21:10
rloojlvillal: i really need to make notes about this. wrt grenade multinode/upgrade. it doesn't do a full upgrade, right? it stops at: old ironic-api, old ironic-conductor, new ironic-conductor pinned-to-ocata21:10
rloojlvillal: i wanted to add the 'ironic-dbsync online_data_migrations' call, but can't do that until the upgrade is done.21:11
*** xavierr has quit IRC21:11
rloowere there any plans to do more testing in grenade multinode? or maybe the infrastructure doesn't allow it. I am thinking, 'upgrade both conductors, upgrade API all pinned, run tests.21:12
*** Sukhdev_ has joined #openstack-ironic21:12
rloojlvillal: ah. i think we need to add ironic-dbsync online_data_migrations call to our devstack setup...21:13
jlvillalrloo: Uh, I don't know about adding more things. Maybe. It is already a pretty long job time wise.21:15
*** harlowja has joined #openstack-ironic21:15
rloojlvillal: ok. something for me to think about maybe, or discuss at PTG. it only makes sense to make that call when upgrading, if starting with fresh db, there is nothing to migrate.21:16
rlooedleafe: wrt 487954, that's why i think it would be ok to change L537 to get the node from the cache itself. if for whatever reason it isn't there, then your check for None is good.21:19
*** sasha2 has quit IRC21:30
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Set target raid configuration on a node  https://review.openstack.org/48320521:39
*** dprince has quit IRC21:54
*** cdearborn has quit IRC21:54
jlvillalihrachys: Darn, the Neutron patch failed in the gate :(  I'll do a recheck once the whole job fails.22:00
ihrachysjlvillal, how did it fail?22:01
ihrachyswe can rebase it to speed up things22:01
jlvillalihrachys: http://logs.openstack.org/05/490305/1/gate/gate-neutron-dsvm-functional-ubuntu-xenial/1bccf56/22:01
ihrachysok that's a known22:01
ihrachyswe have some test case killing functional test workers which results in that patterns22:02
jlvillalihrachys: Ah, so you have a flaky gate too :(22:02
jlvillalihrachys: I feel your pain.22:02
ihrachyshah22:02
openstackgerritJohn L. Villalovos proposed openstack/ironic-tempest-plugin master: Script to extract the ironic_tempest_plugin/ directory  https://review.openstack.org/48976222:06
*** jlabarre has quit IRC22:12
*** krtaylor has joined #openstack-ironic22:17
*** krtaylor has quit IRC22:23
*** krtaylor has joined #openstack-ironic22:27
*** krtaylor has quit IRC22:31
*** Sukhdev_ has quit IRC22:34
*** Sukhdev_ has joined #openstack-ironic22:43
*** diablo_rojo has quit IRC22:54
*** rcernin has quit IRC22:58
*** mjturek has quit IRC23:04
*** ijw has quit IRC23:04
*** ijw has joined #openstack-ironic23:05
*** ihrachys_ has joined #openstack-ironic23:06
*** ihrachys has quit IRC23:06
*** sasha2 has joined #openstack-ironic23:07
jlvillalTheJulia, rloo: A proposal bot patch to stable/pike ironic-lib:  https://review.openstack.org/#/c/488638/23:08
patchbotpatch 488638 - ironic-lib (stable/pike) - Update UPPER_CONSTRAINTS_FILE for stable/pike23:08
jlvillalAnd: https://review.openstack.org/#/c/488776/ and https://review.openstack.org/#/c/488773/23:09
patchbotpatch 488776 - python-ironicclient (stable/pike) - Update UPPER_CONSTRAINTS_FILE for stable/pike23:09
patchbotpatch 488773 - python-ironic-inspector-client (stable/pike) - Update UPPER_CONSTRAINTS_FILE for stable/pike23:09
*** ijw has quit IRC23:09
jlvillalTheJulia, rloo: never mind. Link isn't yet valid :(23:11
*** ijw has joined #openstack-ironic23:14
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: Enable OSProfiler support in Ironic - follow-up  https://review.openstack.org/49067523:14
*** ijw has quit IRC23:15
*** Goneri has quit IRC23:20
*** ijw has joined #openstack-ironic23:25
*** Goneri has joined #openstack-ironic23:27
*** baoli has joined #openstack-ironic23:27
*** baoli has quit IRC23:32
*** Goneri has quit IRC23:33
*** hamzy has joined #openstack-ironic23:33
*** rpioso has quit IRC23:35
*** ricardoas has quit IRC23:44
*** tellesnobrega has quit IRC23:44
*** tellesnobrega has joined #openstack-ironic23:45

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