Tuesday, 2017-06-13

*** crushil has quit IRC00:00
*** crushil has joined #openstack-ironic00:01
*** shobhar__ has quit IRC00:01
*** MattMan has quit IRC00:01
*** MattMan has joined #openstack-ironic00:02
*** crushil has quit IRC00:07
*** crushil has joined #openstack-ironic00:07
*** yolanda has quit IRC00:14
*** rama_y has quit IRC00:19
*** rbartal has quit IRC00:20
*** crushil has quit IRC00:25
*** rbartal has joined #openstack-ironic00:33
*** ijw has joined #openstack-ironic00:33
*** yolanda has joined #openstack-ironic00:37
*** zhurong has joined #openstack-ironic00:44
openstackgerritWang KaiFeng proposed openstack/ironic master: Make ip address of socat console configurable  https://review.openstack.org/47320100:44
*** baoli has joined #openstack-ironic00:49
*** boris-42_ has quit IRC00:59
*** phuongnh has joined #openstack-ironic01:03
openstackgerritMerged openstack/python-ironicclient master: Updated from global requirements  https://review.openstack.org/47291401:03
*** ijw has quit IRC01:05
*** pmannidi has quit IRC01:06
*** bfournie has joined #openstack-ironic01:09
openstackgerritMerged openstack/ironic-specs master: Trivial typos fix  https://review.openstack.org/47263601:12
*** litao__ has joined #openstack-ironic01:15
*** pmannidi has joined #openstack-ironic01:19
*** ijw has joined #openstack-ironic01:23
*** ijw has quit IRC01:23
*** rloo has quit IRC01:33
*** yolanda has quit IRC01:37
*** SerenaFeng has joined #openstack-ironic01:50
*** SerenaFeng has quit IRC01:58
*** SerenaFeng has joined #openstack-ironic02:00
*** joe___ has joined #openstack-ironic02:03
*** jo_______ has quit IRC02:03
*** zhurong has quit IRC02:07
openstackgerritMerged openstack/ironic-lib master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/47256302:07
*** tellesnobrega has quit IRC02:07
*** tuanluong has joined #openstack-ironic02:13
*** tellesnobrega has joined #openstack-ironic02:20
mikalIs there a standard way of getting self signed ssl certs working with IPA without having to rebake the IPA image for every deployment?02:25
*** baoli has quit IRC02:26
*** baoli has joined #openstack-ironic02:27
*** baoli has quit IRC02:28
*** chlong has quit IRC02:42
*** baoli has joined #openstack-ironic02:57
*** baoli has quit IRC02:58
*** baoli has joined #openstack-ironic02:59
*** baoli has quit IRC03:03
*** ijw has joined #openstack-ironic03:03
*** ijw has quit IRC03:07
*** rbudden has quit IRC03:43
*** SerenaFeng has quit IRC03:44
*** zhurong has joined #openstack-ironic03:51
mikalAhhh, found the flag in Ocata. Its not in Newton though...03:56
*** gcb has quit IRC03:58
*** baoli has joined #openstack-ironic04:00
openstackgerritMichael Still proposed openstack/ironic-python-agent stable/newton: Configure and use SSL-related requests options  https://review.openstack.org/47367704:00
*** baoli has quit IRC04:06
*** jaganathan_ has joined #openstack-ironic04:13
*** zhurong has quit IRC04:27
*** links has joined #openstack-ironic04:32
*** zhurong has joined #openstack-ironic04:33
*** adisky__ has joined #openstack-ironic04:42
*** rbartal has quit IRC04:46
*** mvpnitesh has joined #openstack-ironic04:50
*** udesale has joined #openstack-ironic04:52
*** oanson has quit IRC04:59
*** oanson_ has joined #openstack-ironic04:59
*** oanson_ is now known as oanson04:59
*** FrankJ has joined #openstack-ironic05:23
*** amotoki_away is now known as amotoki05:27
*** gcb has joined #openstack-ironic05:30
*** krtaylor has quit IRC05:33
*** ccamacho has quit IRC05:34
*** FrankJ has quit IRC05:37
*** root has joined #openstack-ironic05:38
*** root is now known as Guest8712005:38
*** root4 has quit IRC05:42
*** jaganathan_ has quit IRC05:43
*** gcb has quit IRC05:46
*** gcb has joined #openstack-ironic05:47
*** krtaylor has joined #openstack-ironic05:51
openstackgerritMerged openstack/ironic-inspector master: Remove pbr warnerrors in favor of sphinx check  https://review.openstack.org/47256205:53
*** yolanda has joined #openstack-ironic05:54
*** ChubYann has quit IRC05:54
*** mjura has joined #openstack-ironic05:56
*** ccamacho has joined #openstack-ironic06:03
*** baoli has joined #openstack-ironic06:03
*** rcernin has joined #openstack-ironic06:05
*** pcaruana has joined #openstack-ironic06:06
*** ccamacho has quit IRC06:07
*** baoli has quit IRC06:08
*** mvpnitesh has quit IRC06:20
*** mvpnitesh has joined #openstack-ironic06:20
*** SerenaFeng has joined #openstack-ironic06:22
*** amarao has joined #openstack-ironic06:23
*** milan has joined #openstack-ironic06:34
*** tesseract has joined #openstack-ironic06:41
openstackgerritparesh sao proposed openstack/ironic-specs master: Out-of-band Boot from Volume using UEFI iSCSI Boot.  https://review.openstack.org/20733706:56
*** dyasny has quit IRC06:57
*** jaganathan_ has joined #openstack-ironic06:58
*** Nisha_Agarwal has joined #openstack-ironic06:58
*** SerenaFeng has quit IRC06:59
openstackgerritparesh sao proposed openstack/ironic-specs master: Out-of-band Boot from Volume using UEFI iSCSI Boot.  https://review.openstack.org/20733707:02
*** baoli has joined #openstack-ironic07:04
openstackgerritTovin Seven proposed openstack/ironic master: Enable OSProfiler support in Ironic  https://review.openstack.org/35814207:06
milanmorning Ironic! :)07:06
* milan early bird today, sorry ;)07:06
*** baoli has quit IRC07:08
Nisha_Agarwalmilan, o/07:10
Nisha_Agarwalgood morning07:10
milanhey Nisha_Agarwal! morning! :)07:10
Nisha_Agarwal:) could you please look into RFE and put your ideas there?07:10
Nisha_Agarwalsorry to ask you as soon as you came in :D07:10
*** rbartal has joined #openstack-ironic07:11
*** hjensas has quit IRC07:11
*** ccamacho has joined #openstack-ironic07:12
*** jaganathan_ has quit IRC07:14
*** jaganathan has joined #openstack-ironic07:14
*** rama_y has joined #openstack-ironic07:15
*** dyasny has joined #openstack-ironic07:18
*** SerenaFeng has joined #openstack-ironic07:18
*** jaosorior_away is now known as jaosorior07:18
milanNisha_Agarwal, yeah on my way to read through, no worries ;)07:23
*** bcornec1 has joined #openstack-ironic07:25
*** yolanda has quit IRC07:28
openstackgerritNguyen Hung Phuong proposed openstack/ironic master: Add functional API tests for volume connector and volume target  https://review.openstack.org/47371707:29
*** SerenaFeng has quit IRC07:29
*** jtomasek has quit IRC07:29
*** SerenaFeng has joined #openstack-ironic07:30
*** SerenaFeng has quit IRC07:30
*** racedo has joined #openstack-ironic07:30
openstackgerritAnshul Jain proposed openstack/proliantutils master: Adds manager support for redfish API's.  https://review.openstack.org/47371907:31
*** Nisha_Agarwal has quit IRC07:32
*** Nisha_Agarwal has joined #openstack-ironic07:32
*** SerenaFeng has joined #openstack-ironic07:33
*** SerenaFeng has quit IRC07:33
*** SerenaFeng has joined #openstack-ironic07:34
*** SerenaFeng has quit IRC07:37
*** fragatina has joined #openstack-ironic07:37
*** priteau has joined #openstack-ironic07:38
*** Nisha_Agarwal has quit IRC07:40
*** ralonsoh has joined #openstack-ironic07:45
openstackgerritWang KaiFeng proposed openstack/ironic master: Make IP address of socat console configurable  https://review.openstack.org/47320107:52
*** zhurong has quit IRC07:55
*** jtomasek has joined #openstack-ironic07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** e0ne has joined #openstack-ironic08:01
*** aarefiev_afk is now known as aarefiev08:07
aarefievmorning all!08:07
*** mvpnitesh has quit IRC08:12
*** SerenaFeng has joined #openstack-ironic08:15
*** lucas-afk is now known as lucasagomes08:17
lucasagomesmorning all08:17
*** samodid has joined #openstack-ironic08:18
milanmorning aarefiev, lucasagomes! :)08:19
aarefievmorning milan, lucasagomes08:19
* milan requires more coffee08:20
* milan .get_coffee() == futures.NEAR08:21
lucasagomeshah (-:08:27
lucasagomeskill -SIGCOFFEE milan08:28
*** skazi_ has joined #openstack-ironic08:29
*** ppiela_ has joined #openstack-ironic08:31
*** ppiela has quit IRC08:34
*** l_jacob has joined #openstack-ironic08:37
*** makowals has quit IRC08:38
*** dtantsur|afk is now known as dtantsur08:40
dtantsurmorning ironic08:41
*** makowals has joined #openstack-ironic08:41
*** mgoddard has joined #openstack-ironic08:42
*** derekh has joined #openstack-ironic08:47
*** sambetts|afk is now known as sambetts08:54
milanlucasagomes, lol08:59
*** l_jacob has quit IRC08:59
milanmorning dtantsur! :)08:59
lucasagomesdtantsur, hey there08:59
milanhaving had a coffee I realise I'm hungry08:59
dtantsuro/09:00
*** SerenaFeng has quit IRC09:04
*** zhurong has joined #openstack-ironic09:06
*** skazi_ has quit IRC09:15
*** skazi has quit IRC09:15
*** skazi_ has joined #openstack-ironic09:15
*** skazi has joined #openstack-ironic09:16
*** skazi_ has quit IRC09:16
*** skazi_ has joined #openstack-ironic09:16
*** priteau has quit IRC09:16
*** rama_y has quit IRC09:19
dtantsurmilan: are you in the office today?09:22
milandtantsur, affirmative, some tea content is present too09:26
milanaarefiev, thx for the +2 :D09:26
dtantsurmilan: cool! I'll be there after lunch09:27
milan\o/ :D09:27
milantoday seems like a coffee day!09:27
dtantsuryep09:27
aarefiev:)09:27
sambettsdtantsur: any idea why we're now seeing lots of http://paste.openstack.org/show/612380/ in the Ironic conductor logs09:27
milanmorning sambetts! :)09:28
sambettsdtantsur: none of the enabled drivers or interfaces include these drivers09:28
dtantsursambetts: hmmm.. some import is not correct?09:28
tuanluongmorning ironic, aarefiev lucasagomes milan dtantsur sambetts09:29
aarefievo/09:29
lucasagomeso/09:29
dtantsursambetts: actually, I'm confused.. do you have an example?09:29
sambettsdtantsur: just looking through some logs for a patch of mine and saw it http://logs.openstack.org/61/364861/5/check/gate-tempest-dsvm-ironic-pxe_ipmitool-postgres-ubuntu-xenial-nv/cced33e/logs/screen-ir-cond.txt.gz#_Jun_12_17_47_35_70349709:30
dtantsursambetts: please file a bug, we need to look into it09:31
dtantsurthere are a lot of these DriverLoadErrors all over the place09:31
dtantsurI wonder if it's oslo.log's fault09:31
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC  https://review.openstack.org/46286109:33
*** e0ne has quit IRC09:35
sambettsdtantsur: the driver load errors come from https://github.com/openstack/ironic/blob/master/ironic/drivers/pxe.py#L91 which means that these driver classes are being loaded some how09:36
sambettsdtantsur: I wondered if the hardware types code was loading them some how by mistake09:37
sambettseven though the hardware type for oneview and ilo aren't enabled :/09:37
*** mvpnitesh has joined #openstack-ironic09:38
*** skazi_ has quit IRC09:39
*** e0ne has joined #openstack-ironic09:39
*** makowals has quit IRC09:43
*** mvpnitesh has quit IRC09:46
*** mvpnitesh has joined #openstack-ironic09:46
sambettsdtantsur: https://bugs.launchpad.net/ironic/+bug/169765709:47
openstackLaunchpad bug 1697657 in Ironic "Ironic drivers are accidentally being loaded even when unconfigured" [Undecided,New]09:47
*** skazi_ has joined #openstack-ironic09:48
dtantsurthanks09:48
sambettsdtantsur: the second example of the log is really weird...09:48
dtantsurthis looks like some logging issue, I'm not sure why these DriverLoadError mix with normal logging09:48
*** makowals has joined #openstack-ironic09:49
sambettsbut the DriverLoadErrors shouldn't even be being thrown :/09:50
*** skazi_ has quit IRC09:51
dtantsuryep09:52
dtantsurboth are the issues here, I guess09:52
*** dtantsur is now known as dtantsur|brb09:58
milanmorning tuanluong! :)09:58
*** mvpnitesh has quit IRC10:04
*** mvpnitesh has joined #openstack-ironic10:04
*** baoli has joined #openstack-ironic10:07
*** milan is now known as milan|bbl10:07
*** baoli has quit IRC10:12
*** zhurong has quit IRC10:20
*** oanson has quit IRC10:21
openstackgerritAnshul Jain proposed openstack/proliantutils master: Adds manager support for redfish API's.  https://review.openstack.org/47371910:22
*** phuongnh has quit IRC10:22
*** oanson has joined #openstack-ironic10:24
openstackgerritSam Betts proposed openstack/ironic master: Receive and store agent version on heartbeat  https://review.openstack.org/36486110:42
*** zhurong has joined #openstack-ironic10:43
*** dtantsur|brb is now known as dtantsur10:53
*** ccamacho is now known as ccamacho|lunch10:55
dtantsurany ilo folks around? is there any known issues with pxe_ilo and HP BL460?10:56
*** ricardoas has quit IRC11:04
*** lucasagomes is now known as lucas-hungry11:08
*** ricardoas has joined #openstack-ironic11:09
*** jkilpatr has joined #openstack-ironic11:10
openstackgerritAnshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems  https://review.openstack.org/46878311:15
*** ijw has joined #openstack-ironic11:19
*** ijw has quit IRC11:23
*** udesale has quit IRC11:27
openstackgerritMerged openstack/ironic master: Make default_boot_option configurable in devstack  https://review.openstack.org/47329411:37
*** bfournie has quit IRC11:37
*** rbudden has joined #openstack-ironic11:39
openstackgerritAparna proposed openstack/proliantutils master: Redfish: Add support for Virtual Media Operations  https://review.openstack.org/47379811:41
sambettsdtantsur: thanks for adding me to that nova patch11:41
dtantsursure11:41
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/47290211:45
*** deadnull has joined #openstack-ironic11:45
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Deprecate removing old status and disable it by default  https://review.openstack.org/47265611:45
*** tellesnobrega has quit IRC11:46
dtantsurinspector folks, could you please approve example.conf regeneration? :) https://review.openstack.org/#/c/472655/11:46
patchbotpatch 472655 - ironic-inspector - Regenerate example.conf11:46
*** wagiel has joined #openstack-ironic11:54
*** litao__ has quit IRC11:55
wagielHi Openstackers, can anyone share ipa files with dracut-network included, I have used all combinations in dib, none seems to work11:55
*** chlong has joined #openstack-ironic11:57
*** jheroux has joined #openstack-ironic12:00
*** dprince has joined #openstack-ironic12:02
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add docs for node.resource_class and flavor creation  https://review.openstack.org/34508112:04
*** tuanluong has quit IRC12:04
*** baoli has joined #openstack-ironic12:08
TheJuliadtantsur: I don't seem to remember the ilo driver supporting hardware enclosures with multiple devices in it that utilizes a central ilo management address.12:09
*** bfournie has joined #openstack-ironic12:10
*** baoli has quit IRC12:13
TheJuliadtantsur: granted, that was a while back, but it may still be a constraint12:13
*** trown|outtypewww is now known as trown12:14
*** ccamacho|lunch is now known as ccamacho12:14
*** openstackgerrit has quit IRC12:18
*** lucas-hungry is now known as lucasagomes12:19
*** zhurong has quit IRC12:19
*** ppiela has joined #openstack-ironic12:20
*** hjensas has joined #openstack-ironic12:20
*** hjensas has quit IRC12:20
*** hjensas has joined #openstack-ironic12:20
*** tellesnobrega has joined #openstack-ironic12:22
*** ppiela_ has quit IRC12:23
*** wagiel has quit IRC12:32
*** baoli has joined #openstack-ironic12:46
*** baoli has quit IRC12:47
*** baoli has joined #openstack-ironic12:48
*** l_jacob has joined #openstack-ironic12:49
dtantsurTheJulia: it seems to be on the list of supported devices though, assuming that BL460 is the same as BL460c12:55
*** sasha2 has joined #openstack-ironic12:57
*** rloo has joined #openstack-ironic13:05
*** Goneri has joined #openstack-ironic13:13
*** openstackgerrit has joined #openstack-ironic13:16
openstackgerritJulia Kreger proposed openstack/ironic master: Logic for skipping deployment with BFV  https://review.openstack.org/45424313:16
*** jcoufal has joined #openstack-ironic13:17
*** racedo has quit IRC13:18
*** mvpnitesh has quit IRC13:22
*** racedo has joined #openstack-ironic13:24
TheJuliadtantsur: *shrugs* It has been a long time since I've touched that kind of gear13:29
dtantsurI see, thanks13:29
TheJuliadtantsur: I seem to remember someone mentioning to me that it did support individual addresses for each iLO, along with a central address13:29
TheJuliaThings like moonshot though, didn't have individual iLOs at all, hence dual bus bridging in the ipmi driver.13:30
dtantsuryeah, I recall something like that as well13:30
* TheJulia looks at open ironic-ui patches, and wonders if it even makes sense to have a meeting this week13:31
*** jaosorior has quit IRC13:32
*** jaosorior has joined #openstack-ironic13:32
*** links has quit IRC13:33
ppielaTheJulia: I am happy to skip this week's meeting13:34
* TheJulia gets a good chuckle since a former boss of her's from like 15 years ago found bifrost13:35
dtantsur:)13:35
*** jaypipes has joined #openstack-ironic13:38
*** fragatina has quit IRC13:39
*** fragatina has joined #openstack-ironic13:40
*** l_jacob has quit IRC13:45
rloohi and good morning, esp to two of my favorite people TheJulia, dtantsur13:48
rlooand also, cuz TheJulia, dtantsur, have a few min to explain your comments in https://review.openstack.org/#/c/412397/49/ironic/objects/base.py?13:48
patchbotpatch 412397 - ironic - Add version column (MERGED)13:48
dtantsurmorning rloo. I think so, yeah13:49
rloodtantsur: L19313:49
*** skazi_ has joined #openstack-ironic13:49
*** ricardoas has quit IRC13:50
dtantsurrloo: tl;dr: you're convering an object, then just throw the change away, if it's not needed13:50
dtantsurrloo: why not first check whether it's needed (line 193), then only convert (line 192) if the condition holds?13:50
rloodtantsur: right, the only time i convert and throw away, is IF the target version is the same as the DB version13:50
rloodtantsur: no, always want to convert to the latest version13:50
dtantsurok, so why not check it first?13:50
dtantsurI mean, the code is fine, just the ordering is a bit unusual..13:51
rloodtantsur: so, I always want to convert the object to the latest version that the service is aware of, regardless of the version in the DB13:51
rloodtantsur: the question is whether we want to keep around the changes used in the conversion13:51
dtantsuryeah.. why convert just to undo it one line later..13:51
rloodtantsur: the changes are needed if we end up saving the object in the latest version. if we save it in the target version -- we don't need to save those changes13:52
dtantsurrloo: yes, but you throw them away at line 196 at some conditions13:52
rloodtantsur: oh. so the 'undo' is *only* removing the list of changes that the object keeps track of. the object itself has the actual fields/changes.13:52
rloodtantsur: let me explain this way. the object has fields as we expect, that correspond to the db columns/fields. the object also has an internal list it keeps of what was changed in that object.13:53
vdroklate morning everyone@!13:53
dtantsurAAAAAHHHH13:53
rloodtantsur: and when the object is saved in the DB, the code looks at this internal 'changed' field, and only those things are actually saved.13:53
rloodtantsur: yeah, tricky. and not obvious unless you understand how the versioned objects work.13:54
rloodtantsur: i can add docs here if you want, but it is used throughout.13:54
dtantsurrloo: I wonder if our code may change something based on the changes we're not going to save.. and then we end up with inconsistent state13:54
dtantsurlike 'if node.a == 42: node.b = ...'13:55
rloodtantsur: yeah, i wondered too. i tested it, although it is possible i missed a case.13:55
dtantsurwith node.a not going to be saved13:55
* dtantsur is not sure13:55
rloodtantsur: so if old version has node.a, new version has node.b (eg renamed a to b)13:56
*** rama_y has joined #openstack-ironic13:56
dtantsuryep. and imagine some code works, assuming that node.b is going to be saved13:56
dtantsurlike, node.c is set based on node.b13:56
dtantsurthis is pure speculation, I'm not sure it actually happens13:56
rloodtantsur: if old version in db, this would change to node.b. if not pinned, we'd keep the changes around. and then if it got saved, it would be saved in latest version with node.b and node.a modified in db.13:56
rloodtantsur: in your example, i'm assuming node.c is latest version, node.b also in latest, or just old version?13:57
dtantsurI'm worried about changes that may be consequence of the converted changes13:58
dtantsurlike node.a has value None in old version, and 42 in new version13:58
rloodtantsur: yeah, so it is up to the code that has the changes, to deal with it when doing the conversion.13:58
dtantsurand then we have code somewhere that does 'if node.a is not None: node.b = something'13:59
openstackgerritMerged openstack/ironic-python-agent stable/newton: [imagebuild] support requirements referring to /opt/stack/new  https://review.openstack.org/47351813:59
dtantsurthen node.a is NOT saved, but node.b is13:59
* dtantsur jumps on another meeting, sigh14:00
rloodtantsur: 1. saving in new version. node.a = 42, node.b = something, changed = a, b, saved a, b in latest version14:00
*** dyasny has quit IRC14:00
*** dyasny has joined #openstack-ironic14:01
rloodtantsur: 2. save in target/old version. node.a=42, node.b = something, changed = b, <code-to-convert-to-target-version-needs-to-address-what-to-do-with-node.b/.a changes>... something is saved to DB in target version14:02
dtantsurah, I see. this is worth carefully documenting14:03
rloodtantsur: yeah, i can document for developers, what to watch out for. do you want me to doc in eg spec, about this 'subtle' code here wrt the changed fields?14:04
*** hjensas has quit IRC14:05
dtantsurrloo: spec != docs. feel free to, but I mostly care about dev docs.14:05
rloodtantsur: ok, i'll add to dev docs, and add more comments to that code.14:06
dtantsurthanks!14:06
rloothank you! i want this code to be clear for future generations :-)14:06
TheJuliaGood morning rloo14:07
rlooTheJulia: hi! ^^ don't know if that helps with your comment too14:07
rlooTheJulia: at L188 of https://review.openstack.org/#/c/412397/49/ironic/objects/base.py14:07
patchbotpatch 412397 - ironic - Add version column (MERGED)14:07
TheJuliarloo: reading your comment, and will then go back to the buffer14:08
TheJuliaMakes sense to me14:13
TheJuliaI guess I have the different concern, but i also hope we write smarter code in the future around database transitions between versions :)14:17
rlooTheJulia: ok, on to the future!14:18
TheJuliaBut, as long as someone walks the happy path for an upgrade, I suspect all we be okay with the current plan. :)14:18
rlooTheJulia: /me hopes!14:19
*** rbartal has quit IRC14:21
openstackgerritMerged openstack/pyghmi master: Do not reuse an expired session  https://review.openstack.org/46513114:22
TheJuliahope(morehope(allthehope(hope())))14:28
rlooTheJulia: :D14:31
jtannerre-paste from last night: Does anyone know why root_gb is required for provisioning when local_gb is the only key that inspection adds? Cannot validate driver deploy. Some parameters were missing in node's instance_info. Missing are: ['root_gb']: MissingParameterValue: Cannot validate driver deploy. Some parameters were missing in node's instance_info. Missing14:33
jtanner6:30 PM are: ['root_gb']14:33
TheJuliaroot_gb is if your attempting to write a partition image14:36
vdrokjtanner: root_gb will be added during boot process if booted through nova14:37
jtannerin the bifrost case though, we have no nova14:37
jtannerso my qcow image is probably affecting this?14:37
jtanneri'm using the centos atomic image, which might explain it14:38
dtantsuratomic images are whole-disk IIRC14:38
vdrokthe root_gb gets used during check_image_size14:39
vdrokto determine if the image will fit the disk, but it's iscsi driver iirc14:39
vdrokjtanner: are you using iscsi driver?14:40
jtanneri'm using whatever defaults i got through bifrost14:41
jtannerthis is how i'm defining the nodes https://github.com/jctanner/bifrost/blob/BAREMETAL_DEMOS/tools/setupnodes.py#L106-L12814:42
jtannerthey are auto-registered through the pxe+ipa process14:42
*** jaganathan has quit IRC14:45
TheJuliajtanner: when registered, is the driver showing as agent_ipmitool?14:51
TheJuliaor pxe_ipmitool?14:51
openstackgerritVladyslav Drok proposed openstack/ironic master: Remove times.dbm prior to test run  https://review.openstack.org/47386714:51
jtanneragent_ipmitool14:52
openstackgerritDan Smith proposed openstack/ironic master: Discover hosts while waiting for hypervisors to show up in devstack  https://review.openstack.org/47386814:52
*** Nisha_Agarwal has joined #openstack-ironic14:52
TheJuliajtanner: I don't remember root_gb being required, and I'm fairly sure I've provisioned without it in the past.  Any chance you can or did paste an entire node-show output?14:53
jtanneri can, one sec14:53
openstackgerritDan Smith proposed openstack/ironic master: Discover hosts while waiting for hypervisors to show up in devstack  https://review.openstack.org/47386814:53
TheJuliajtanner: local_gb is mainly for nova scheduling on to the node, fwiw14:54
jtannerhttps://paste.fedoraproject.org/paste/mFZuniw-VDy6B2cxOn3ARw14:54
*** mjura has quit IRC14:55
*** amarao has quit IRC14:55
TheJuliajtanner: in instance_info, you have a the kernel and ramdisk keys defined14:57
jtannermistake?14:57
TheJuliayou don't need those in instance_info if your working with a whole disk image14:57
TheJuliaif it is a whole disk image, you also don't need image_type14:58
jtannerError: Cannot validate image information for node 0abc4b7c-e626-447b-bcd0-673718c0b728 because one or more parameters are missing from its instance_info. Missing are: ['ramdisk', 'kernel', 'image_source'] (HTTP 500)14:58
jtannerthat's why i added those14:58
TheJuliaephemeral_gb, preserve_ephemeral14:59
jtanneri don't think i set image_type14:59
TheJulia"one or more"14:59
jtannerhttps://github.com/jctanner/bifrost/blob/BAREMETAL_DEMOS/tools/setupnodes.py#L106-L12814:59
TheJuliano, looks like the code is setting image_type Oo14:59
TheJuliayeah, nuke 118 and 119, and it should behave properly15:00
TheJuliaanupn: I think we're going to skip the ui meeting today.  Any objections?15:01
TheJuliacrushil is also not here today it seems15:01
* jtanner does a clean and retries15:02
*** ricardoas has joined #openstack-ironic15:03
ricardoasmorning, ironic!15:03
*** milan|bbl has quit IRC15:03
*** skazi_ has quit IRC15:04
*** joe_____ has joined #openstack-ironic15:05
*** joe___ has quit IRC15:08
*** tesseract has quit IRC15:10
TheJuliagood morning ricardoas15:11
*** hjensas has joined #openstack-ironic15:11
*** hjensas has quit IRC15:11
*** hjensas has joined #openstack-ironic15:11
*** tellesnobrega has quit IRC15:17
jtannerhow do you restart vbmc/upstart in the bifrost vagrant box?15:23
jtannerinitctl restart  ?15:24
TheJuliaI think it is just a running service, so service $blah restart might do it15:25
TheJuliabut I've never mucked with it on that level15:25
*** jaganathan has joined #openstack-ironic15:27
jtanner"vbmc: unrecognized service" ... shrug, i'll hack it later15:27
jtannerit appears as though the verification phase hangs for a while and eventually rolls the state back to "enrolled" if the ipmi device is unreachable15:28
*** jkilpatr_ has joined #openstack-ironic15:29
*** mikal has quit IRC15:29
TheJuliayup :(15:30
openstackgerritMerged openstack/ironic-inspector master: Regenerate example.conf  https://review.openstack.org/47265515:30
*** mikal has joined #openstack-ironic15:31
jtanner"wait call-back" ... that's new. maybe i'm making progress15:31
*** jkilpatr has quit IRC15:31
TheJuliayeah, wait callback is the state where it is waiting for the agent to boot15:34
TheJuliappiela: email cancelling the ui meeting sent to the mailing list15:36
ppielaTheJulia: thx15:36
TheJuliaI'm going to run into town for an errand, I'll be back in ~2 hours15:36
rama_yTheJulia, ppiela, thx for the info here about the ui meeting.15:40
NobodyCamGood Morning Ironic'ers :)15:41
rama_ygood morning NobodyCam!15:44
NobodyCamGood morning rama_y :)15:45
*** jcoufal has quit IRC15:50
openstackgerritNisha Agarwal proposed openstack/ironic master: Fixes typeerror for py3.5  https://review.openstack.org/47388815:51
*** deadnull has quit IRC15:52
*** d0ugal has quit IRC15:53
Nisha_Agarwaldtantsur, i have a question on virtualbmc15:54
dtantsursure15:56
dtantsurmorning NobodyCam15:56
Nisha_Agarwaldtantsur, lucasagomes from ironic we use ipmitool command for issuing ipmi commands.....but from pyghmi its not ipmitool command....then how does the CI gates for pxe work through virtualbmc?15:56
NobodyCamMorning dtantsur :)15:56
dtantsurNisha_Agarwal: both pyghmi and ipmitool talk the IPMI protocol. I'm not sure what confuses you..15:56
jlvillalGood morning Ironic15:57
*** jaosorior has quit IRC15:57
*** jaosorior has joined #openstack-ironic15:58
*** jaosorior is now known as jaosorior_away15:58
lucasagomesNisha_Agarwal, what dtantsur said. But, note that pyghmi implements an IPMI client AND server15:58
rama_ygood morning jlvillal15:58
lucasagomesvirtualbmc was created on top of the server bits15:58
jlvillal:)15:59
*** jcoufal has joined #openstack-ironic15:59
lucasagomesNobodyCam, jlvillal rloo morning :D15:59
jlvillal:)15:59
lucasagomesTheJulia, rama_y morning  too :D15:59
Nisha_Agarwallucasagomes, ok15:59
rloohi lucasagomes, jlvillal, Nisha_Agarwal, rama_y, NobodyCam16:00
NobodyCamGood Morning lucasagomes jlvillal rloo Nisha_Agarwal16:00
*** e0ne has quit IRC16:00
jlvillal:)16:00
*** SerenaFeng has joined #openstack-ironic16:01
Nisha_AgarwalGood Morning NobodyCam lucasagomes rloo rama_y jlvillal :)16:01
NobodyCam:)16:02
*** ccamacho has quit IRC16:03
*** hamzy has quit IRC16:04
*** trown is now known as trown|lunch16:04
rama_yhi, nice to hear from all :)16:16
jlvillalTheJulia: What are the bare-metal node requirements for doing boot from volume? Any special hardware needed?16:16
dtantsursee you tomorrow, folks16:17
jlvillalTheJulia: My team may want to do some testing of it.16:17
jlvillaldtantsur: ciao16:17
dtantsurjlvillal: nice!16:17
jlvillaldtantsur: "may" ;)  Have to see what happens here.16:17
jlvillaldtantsur: But we have customers who are interested. So good chance.16:18
dtantsurIIUC there are no requirements beyond using iPXE16:18
jlvillaldtantsur: Great. That makes it easy16:18
* jlvillal heads to get something to drink16:18
*** dtantsur is now known as dtantsur|afk16:19
*** samodid has quit IRC16:20
*** jaosorior_away has quit IRC16:23
*** SerenaFeng has quit IRC16:34
*** jkilpatr_ has quit IRC16:39
*** jkilpatr has joined #openstack-ironic16:39
* sambetts -> home 16:40
sambettsnight all16:40
*** sambetts is now known as sambetts|afk16:40
*** rcernin has quit IRC16:41
*** derekh has quit IRC16:43
*** lucasagomes is now known as lucas-afk16:50
openstackgerritRamamani Yeleswarapu proposed openstack/python-ironicclient master: Add OSProfiler support to Ironicclient  https://review.openstack.org/36233116:52
*** tuanla_fujitsu has joined #openstack-ironic16:59
*** fragatina has quit IRC17:00
*** tuanla__ has joined #openstack-ironic17:00
*** amotoki is now known as amotoki_away17:01
*** jkilpatr has quit IRC17:02
*** ralonsoh has quit IRC17:03
*** dsneddon is now known as dsneddon_afk_bia17:03
TheJuliajlvillal: no special requirements as long as the environment boots via ipxe, for now :)17:03
jlvillalTheJulia: Thanks!17:04
*** jkilpatr has joined #openstack-ironic17:04
openstackgerritNisha Agarwal proposed openstack/ironic master: py3.5:Workaround fix for forcing virtualbmc installation with pip2  https://review.openstack.org/47391317:05
*** fragatina has joined #openstack-ironic17:06
*** mgoddard has quit IRC17:07
*** fragatina has quit IRC17:08
*** fragatina has joined #openstack-ironic17:09
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Add support for editing node's resource_class  https://review.openstack.org/46385217:11
*** ianychoi has joined #openstack-ironic17:15
*** baoli has quit IRC17:17
*** samodid has joined #openstack-ironic17:17
*** hamzy has joined #openstack-ironic17:23
*** trown|lunch is now known as trown17:32
*** krtaylor has quit IRC17:35
*** tuanla__ has quit IRC17:37
*** tuanla_fujitsu has quit IRC17:37
*** baoli has joined #openstack-ironic17:49
*** joe_____ has quit IRC17:52
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: [WIP][DNM] Set boot device on a node  https://review.openstack.org/47392617:53
*** fragatina has quit IRC17:57
*** bcornec1 has quit IRC18:00
*** crushil has joined #openstack-ironic18:04
*** joe_____ has joined #openstack-ironic18:04
*** joe_____ has quit IRC18:04
*** joe_____ has joined #openstack-ironic18:05
*** rama_y has quit IRC18:13
jtannerwould there be any iscsi related logging on the bifrost/ironic host when a node is being provisioned?18:16
*** athomas has quit IRC18:16
*** krtaylor has joined #openstack-ironic18:20
*** rcernin has joined #openstack-ironic18:26
*** ChubYann has joined #openstack-ironic18:32
*** ijw has joined #openstack-ironic18:32
*** jcoufal has quit IRC18:37
TheJuliajtanner: using agent_ipmitool, afaik no18:40
*** athomas has joined #openstack-ironic18:40
*** Nisha_Agarwal has quit IRC18:41
jtannermeaning it won't log, or it won't be using iscsi at all?18:46
TheJuliashouldn't be using iscsi at all18:46
jtanneroh, is the agent just streaming over http?18:46
TheJuliaagent, or what we call direct mode in the driver composition world (I can share a link for that), downloads over http and streams to disk18:46
TheJuliayup18:47
jtannerk, that makes more sense now18:47
jtannerthanks18:47
jtannerthe nested vm's cpu has been goin nuts for a while ... i can only presume it's extracting/writing/etc18:48
TheJulialikely :)18:48
*** dsneddon_afk_bia has quit IRC18:48
jtannerzomg, i think i finally have my first successful deploy of an atomic image18:59
NobodyCamjtanner: ++ w00t :)18:59
*** baoli has quit IRC19:01
TheJulia\o/19:06
*** vinsh has joined #openstack-ironic19:13
*** krtaylor has quit IRC19:20
*** dsneddon_afk_bia has joined #openstack-ironic19:20
*** marlinc has joined #openstack-ironic19:22
*** tellesnobrega has joined #openstack-ironic19:24
*** adisky__ has quit IRC19:26
*** ijw has quit IRC19:28
*** dprince has quit IRC19:30
*** baoli has joined #openstack-ironic19:33
*** dsneddon_afk_bia is now known as dsneddon19:34
openstackgerritAnup Navare proposed openstack/ironic-ui master: Allow selecting interfaces while enrolling nodes  https://review.openstack.org/47395719:41
*** racedo has quit IRC19:49
*** krtaylor has joined #openstack-ironic20:00
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/ocata: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44863220:03
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/ocata: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44863220:05
*** priteau has joined #openstack-ironic20:06
*** rama_y has joined #openstack-ironic20:14
*** jkilpatr has quit IRC20:15
*** vinsh has quit IRC20:15
*** samodid has quit IRC20:16
*** baoli has quit IRC20:18
*** baoli has joined #openstack-ironic20:23
*** mjturek has joined #openstack-ironic20:25
*** crushil has quit IRC20:27
*** jkilpatr has joined #openstack-ironic20:31
mjturekTheJulia: sorry I've been afk this week. I'm out of town but I might have should have some time tonight for updating the wirein patch. Just want to make sure I'm not stepping on anything you're working on20:31
*** vinsh has joined #openstack-ironic20:32
*** baoli has quit IRC20:33
TheJuliamjturek: I've been working on the wire-in patch this afternoon, should have an update posted in the next little bit20:45
TheJuliamjturek: sorry for the delay, I was on the phone for a... long.. while.20:45
mjturekokay no problem! sorry about that20:45
*** ianychoi has quit IRC20:46
TheJuliano worries!20:46
TheJuliamjturek: well, maybe hour or two.  My next call is in 10 minutes20:50
TheJulia:(20:50
mjturekouuuch20:50
*** Goneri has quit IRC20:53
*** alex_xu has quit IRC20:56
*** sasha2 has quit IRC20:56
*** alex_xu has joined #openstack-ironic20:57
*** sasha2 has joined #openstack-ironic20:59
*** mjturek has quit IRC20:59
*** jheroux has quit IRC21:02
*** baoli has joined #openstack-ironic21:10
*** krtaylor has quit IRC21:13
*** vinsh has joined #openstack-ironic21:14
openstackgerritJulia Kreger proposed openstack/ironic master: Wire in storage interface attach/detach operations  https://review.openstack.org/40629021:17
openstackgerritJulia Kreger proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332421:19
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node  https://review.openstack.org/47392621:20
*** vinsh has quit IRC21:27
jlvillalTheJulia, I left a small comment in the "Wire in storage.."21:31
jlvillalTheJulia, I will work on a full review in a bit.21:32
openstackgerritJulia Kreger proposed openstack/ironic master: Logic for skipping deployment with BFV  https://review.openstack.org/45424321:32
TheJuliajlvillal: huh... I thought I changed21:34
jlvillalTheJulia, I've said that myself before ;)21:34
TheJuliaoh, I must have bee looking at an older set of cached data21:35
TheJuliajlvillal: fixing21:35
openstackgerritJulia Kreger proposed openstack/ironic master: Wire in storage interface attach/detach operations  https://review.openstack.org/40629021:37
openstackgerritJulia Kreger proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332421:38
openstackgerritJulia Kreger proposed openstack/ironic master: Logic for skipping deployment with BFV  https://review.openstack.org/45424321:39
openstackgerritJulia Kreger proposed openstack/ironic master: Add CRUD notification objects for volume connector and volume target  https://review.openstack.org/46393021:40
openstackgerritJulia Kreger proposed openstack/ironic master: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458621:41
*** fragatina has joined #openstack-ironic21:41
TheJuliajlvillal: rebases above :)21:44
jlvillalTheJulia, Thanks21:44
TheJuliajlvillal: I only rebases half of the patches, but most of them begin getting far removed in terms of the volume connection info stuff21:46
TheJuliaFar removed from what I updated.  I don't expect I'll be able to revise patches tomorrow since I'll be doing a thing21:47
jlvillalTheJulia, Makes sense. Thanks.21:47
*** fragatina has quit IRC21:48
*** oanson_ has joined #openstack-ironic21:50
*** fragatina has joined #openstack-ironic21:51
*** ElCoyote_ has left #openstack-ironic21:52
*** krtaylor has joined #openstack-ironic21:53
*** oanson has quit IRC21:53
*** oanson_ is now known as oanson21:53
*** rcernin has quit IRC21:54
*** bfournie has quit IRC21:54
*** fragatina has quit IRC21:55
*** ElCoyote_ has joined #openstack-ironic21:56
*** priteau has quit IRC22:05
*** sasha2 has quit IRC22:16
*** ijw has joined #openstack-ironic22:20
*** jaganathan has quit IRC22:23
openstackgerritDan Smith proposed openstack/ironic master: Discover hosts while waiting for hypervisors to show up in devstack  https://review.openstack.org/47386822:45
*** fragatina has joined #openstack-ironic22:52
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/47290222:54
*** fragatina has quit IRC23:01
openstackgerritMerged openstack/ironic master: Move port object creation to conductor  https://review.openstack.org/46993123:03
*** vinsh has joined #openstack-ironic23:05
*** vinsh has quit IRC23:05
*** ijw has quit IRC23:07
*** bfournie has joined #openstack-ironic23:10
*** bfournie has quit IRC23:11
*** baoli has quit IRC23:20
jtanneri've been noticing that if you try to spin up too many (>2) vms within the bifrost nested network, the pxeboot will fail on some of them with "no bootable disk" errors23:28
jtannerthe workaround is to simply restart them one by one23:28
*** joe_____ has quit IRC23:32
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Put tests in correct location for ironic/api/controllers/v1/  https://review.openstack.org/46265823:38
*** fragatina has joined #openstack-ironic23:58

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