Thursday, 2018-05-10

*** srart has joined #openstack-ironic00:02
*** gyee has quit IRC00:08
*** cjloader has joined #openstack-ironic00:13
*** cjloader has quit IRC00:17
*** akhilaki has quit IRC00:32
*** mjturek has joined #openstack-ironic00:39
*** mjturek has quit IRC00:40
*** liuzz has joined #openstack-ironic00:41
*** mikal_ has joined #openstack-ironic00:45
*** mikal has quit IRC00:49
*** zhangfei has joined #openstack-ironic01:03
*** HoloIRCUser1 has joined #openstack-ironic01:06
*** hshiina has joined #openstack-ironic01:06
*** akhilaki has joined #openstack-ironic01:07
*** akhilaki has quit IRC01:08
*** HoloIRCUser1 has quit IRC01:12
*** HoloIRCUser1 has joined #openstack-ironic01:13
*** HoloIRCUser1 is now known as rloo01:14
*** rloo has left #openstack-ironic01:15
*** cjloader has joined #openstack-ironic01:15
*** cjloader has quit IRC01:20
*** sapd has joined #openstack-ironic01:24
*** phuongnh has joined #openstack-ironic01:27
openstackgerritMerged openstack/ironic master: Install OSC during quickstart  https://review.openstack.org/56706101:31
*** tiendc has joined #openstack-ironic02:04
*** cjloader has joined #openstack-ironic02:18
*** cjloader has quit IRC02:23
*** links has joined #openstack-ironic02:28
*** tiendc has quit IRC03:10
*** tiendc has joined #openstack-ironic03:12
*** cjloader has joined #openstack-ironic03:13
*** cjloader has quit IRC03:18
*** links has quit IRC03:43
*** fragatina has quit IRC03:55
*** fragatina has joined #openstack-ironic03:56
*** gyankum has joined #openstack-ironic03:56
*** gyan_ has joined #openstack-ironic03:56
*** fragatina has quit IRC04:00
*** rbudden has quit IRC04:09
*** hamzy has joined #openstack-ironic04:38
*** links has joined #openstack-ironic04:49
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow up to inspect wait implementation  https://review.openstack.org/56694804:55
*** marios has joined #openstack-ironic04:56
*** rcernin has joined #openstack-ironic05:05
*** jtomasek has joined #openstack-ironic05:15
*** cjloader has joined #openstack-ironic05:17
*** marios has quit IRC05:18
*** marios has joined #openstack-ironic05:18
*** cjloader has quit IRC05:22
*** fragatina has joined #openstack-ironic05:22
*** fragatina has quit IRC05:23
*** fragatina has joined #openstack-ironic05:23
*** UForgotten has quit IRC05:25
*** jmccrory has quit IRC05:28
*** jmccrory has joined #openstack-ironic05:28
*** UForgotten has joined #openstack-ironic05:29
*** jhesketh has quit IRC05:58
*** hjensas has quit IRC05:58
*** rajinir has quit IRC06:05
*** fragatin_ has joined #openstack-ironic06:17
*** fragati__ has joined #openstack-ironic06:18
*** fragatina has quit IRC06:20
*** fragatin_ has quit IRC06:21
*** fragati__ has quit IRC06:27
*** trungnv has quit IRC06:38
*** trungnv has joined #openstack-ironic06:38
*** phuongnh has quit IRC06:39
*** phuongnh has joined #openstack-ironic06:40
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/queens: Cap hacking to avoid gate failure  https://review.openstack.org/56748106:43
*** rbartal has joined #openstack-ironic06:45
*** trungnv has quit IRC06:49
*** trungnv has joined #openstack-ironic06:49
*** pcaruana has joined #openstack-ironic06:51
*** threestrands_ has quit IRC06:58
*** kiseok7 has quit IRC07:18
*** tesseract has joined #openstack-ironic07:24
*** zhangfei has quit IRC07:24
pas-halet's merge this to unblock queens gates ^07:26
*** trungnv has quit IRC07:32
*** racedo_ has joined #openstack-ironic07:32
*** trungnv has joined #openstack-ironic07:32
*** zhangfei has joined #openstack-ironic07:37
*** arxcruz is now known as arxcruz|off07:46
*** lucas-afk is now known as lucasagomes07:55
*** livelace has joined #openstack-ironic07:57
*** mgoddard has joined #openstack-ironic08:00
*** jaganathan has joined #openstack-ironic08:02
*** e0ne has joined #openstack-ironic08:03
*** toabctl has quit IRC08:04
*** erolg has joined #openstack-ironic08:06
*** dougsz has joined #openstack-ironic08:10
*** livelace has quit IRC08:10
*** jhesketh has joined #openstack-ironic08:29
*** tiendc has quit IRC08:31
*** jovial[m] has quit IRC08:36
*** mkrai has quit IRC08:36
*** csmart has quit IRC08:41
*** retr0h has quit IRC08:41
*** derekh has joined #openstack-ironic08:45
*** jhesketh has quit IRC08:47
*** jhesketh has joined #openstack-ironic09:06
*** johnthetubaguy has quit IRC09:09
*** johnthetubaguy has joined #openstack-ironic09:10
*** csmart has joined #openstack-ironic09:29
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add system boot mode support  https://review.openstack.org/52610109:39
*** d0ugal_ has quit IRC09:45
*** d0ugal has joined #openstack-ironic09:45
*** d0ugal has quit IRC09:45
*** d0ugal has joined #openstack-ironic09:45
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Simulator processes return rc on exit  https://review.openstack.org/56754109:47
openstackgerritAija proposed openstack/sushy master: Create mappings for System Indicator LED  https://review.openstack.org/48537610:16
*** mkrai has joined #openstack-ironic10:23
*** retr0h has joined #openstack-ironic10:23
*** jovial[m] has joined #openstack-ironic10:24
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add an option to bind simulators to a local interface  https://review.openstack.org/56754910:28
phuongnhhi mgoddard10:32
phuongnhI just want to create a custom traits for node so I have looked in the test code and decided to use this code:10:32
phuongnhhttps://github.com/openstack/ironic/blob/master/ironic/tests/unit/api/controllers/v1/test_node.py#L54210:32
phuongnhmgoddard, and I changed to:10:32
phuongnhobjects.TraitList.create(task.context, node.id, ['CUSTOM_1'])10:32
phuongnhand insert in this line:10:32
phuongnhhttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/irmc/inspect.py#L24510:32
phuongnhjust before node.save()10:32
phuongnhbut this line of code does not work, ['CUSTOM_1'] is not found in node info. What should I do?10:32
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387910:34
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add system boot mode support  https://review.openstack.org/52610110:36
*** milan has joined #openstack-ironic10:42
mgoddardhi phuongnh. Where are you expecting the traits to be seen?10:49
mgoddardopenstack baremetal node list?10:49
mgoddardthey should be shown in openstack baremetal node show <node>, if you use a sufficient api version10:50
phuongnhoh, thanks10:51
phuongnhI used ironic node-show <node>10:51
phuongnh:)10:51
mgoddardI think that would also work10:52
mgoddardbut you need to use a sufficient version10:53
mgoddard1.3710:53
phuongnh:) I see10:54
*** rh-jelabarre has joined #openstack-ironic10:56
*** MattMan has quit IRC11:04
*** MattMan has joined #openstack-ironic11:04
*** zhangfei has quit IRC11:05
*** lucasagomes is now known as lucas-hungry11:09
*** dprince has joined #openstack-ironic11:16
*** rcernin has quit IRC11:31
TheJuliagood morning everyone11:42
*** bfournie has quit IRC11:44
*** bfournie has joined #openstack-ironic11:44
openstackgerritJulia Kreger proposed openstack/python-ironic-inspector-client stable/queens: Gate fix: Cap hacking to avoid gate failure  https://review.openstack.org/56756711:49
*** bfournie has quit IRC11:49
openstackgerritJulia Kreger proposed openstack/python-ironicclient stable/queens: Gate fix: Cap hacking to avoid gate failure  https://review.openstack.org/56756811:49
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/queens: Gate fix: Cap hacking to avoid gate failure  https://review.openstack.org/56756911:49
openstackgerritJulia Kreger proposed openstack/ironic-inspector stable/queens: Gate fix: Cap hacking to avoid gate failure  https://review.openstack.org/56757011:49
phuongnhgood morning TheJulia11:49
openstackgerritJulia Kreger proposed openstack/ironic-lib stable/queens: Gate fix: Cap hacking to avoid gate failure  https://review.openstack.org/56757111:49
phuongnhTheJulia: I have added a [RFE] in storyboard, please have a look if you have free time :)  https://storyboard.openstack.org/#!/story/200199911:50
TheJuliapas-ha: seems rloo is going to have to approve that one today, I'll try to remind her11:51
TheJuliawhat is this free time you speak of? ;)11:51
TheJuliaphuongnh: I don't understand your rfe. Anything prepended with custom is allowed to be added as a trait11:52
*** dsneddon has quit IRC11:53
phuongnhTheJulia: you mean I do not have to create a [RFE], right?11:53
TheJuliano you don't. We have a structural limit of 50 traits to prevent abuse and overloading the scheduler11:54
phuongnhthanks, I will remove that RFE :)11:54
TheJuliabut anything prepended with CUSTOM or that is defined in os-traits is permitted11:54
phuongnhI got it, thanks11:55
*** dsneddon has joined #openstack-ironic11:56
*** lucas-hungry is now known as lucasagomes11:56
jrollmorning ironic11:56
jrollooooo this is a fun one11:58
jrollhttps://bugs.launchpad.net/nova/+bug/176981711:58
openstackLaunchpad bug 1769817 in OpenStack Compute (nova) "Ironic serial console doesn't get disabled when instance is deleted" [Undecided,Confirmed]11:58
jrollit appears we don't tear the console down ironic-side during tear-down either: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L89311:58
jrollalso makes a fun process question: if we want a fix in nova and ironic, do we just duplicate the bug in storyboard?12:00
sambetts|afkI belive thats what would have happened to existing bugs that had mulitple projects including ironic associated with them12:01
*** sambetts|afk is now known as sambetts12:02
* jroll files https://storyboard.openstack.org/#!/story/200200012:02
jrolloooo I got a good number12:02
sambetts:-P12:02
TheJulianice!12:05
* TheJulia goes and finds quiet place to work on slide decks12:07
*** phuongnh has quit IRC12:10
*** rbudden has joined #openstack-ironic12:13
*** trown|outtypewww is now known as trown12:15
jrollfix inbound now12:15
openstackgerritJim Rollenhagen proposed openstack/ironic master: Tear down console during unprovisioning  https://review.openstack.org/56757712:16
jrollwill need to backport that all the way to ocata12:16
jrollblah, I always forget I don't have stable privs: https://review.openstack.org/#/c/567481/12:16
patchbotpatch 567481 - ironic (stable/queens) - Cap hacking to avoid gate failure12:16
*** bfournie has joined #openstack-ironic12:21
*** AlexeyAbashkin has joined #openstack-ironic12:28
*** AlexeyAbashkin has quit IRC12:37
TheJuliajroll: I need to send a few emails this week... I'll try to remember that as well12:44
*** jiapei has joined #openstack-ironic12:44
jrollTheJulia: thanks :)12:44
*** AlexeyAbashkin has joined #openstack-ironic12:47
*** AlexeyAbashkin has quit IRC12:48
*** v12aml has quit IRC12:50
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add system boot mode support  https://review.openstack.org/52610112:53
*** rloo has joined #openstack-ironic12:55
*** hshiina is now known as hshiina|afk12:58
openstackgerritMerged openstack/ironic master: Fix W504 errors  https://review.openstack.org/56700912:59
*** jcoufal has joined #openstack-ironic13:01
*** mjturek has joined #openstack-ironic13:03
*** pcaruana|afk| has joined #openstack-ironic13:03
*** NotPcaruana has joined #openstack-ironic13:04
*** gyankum has quit IRC13:05
*** gyan_ has quit IRC13:05
*** Goneri has joined #openstack-ironic13:06
openstackgerritSam Betts proposed openstack/ironic master: [WIP] Direct deploy serve HTTP images from conductor  https://review.openstack.org/56547413:08
*** v12aml has joined #openstack-ironic13:14
TheJuliaoh joy, newer coreos takes a while to boot13:21
TheJuliaipa starting around 550 seconds in13:21
sambetts\o/13:23
sambettsle sigh... centos7 seems to have python package conflicts breaking our CI :( http://192.133.156.18/74/565474/7/check/ironic-dsvm-tempest-ironic-cimc-current-centos-7-cimc/795f310/logs/devstacklog.txt.gz#_2018-05-10_13_18_05_58013:25
TheJuliaI'm thinking we need to all go on a long cruise and ignore computers for a week or three.13:26
sambettsI agree...13:27
*** jistr is now known as jistr|mtg13:32
*** v12aml has quit IRC13:32
*** NotPcaruana has quit IRC13:34
*** pcaruana|afk| has quit IRC13:34
*** cjloader has joined #openstack-ironic13:38
openstackgerritMerged openstack/ironic master: Fix E305 Errors  https://review.openstack.org/56702513:40
*** v12aml has joined #openstack-ironic13:46
*** links has quit IRC13:46
*** dmellado has joined #openstack-ironic13:47
*** rbartal_ has joined #openstack-ironic13:48
*** rbartal has quit IRC13:50
*** r-daneel has joined #openstack-ironic14:08
openstackgerritMerged openstack/ironic master: Fix W605 Errors  https://review.openstack.org/56702614:12
openstackgerritMerged openstack/ironic master: Silence F405 errors  https://review.openstack.org/56702714:12
*** cjloader has quit IRC14:13
*** cjloader has joined #openstack-ironic14:13
jrollrloo: since you weren't here at the time, this is a nasty one that needs reviews: https://review.openstack.org/56757714:14
patchbotpatch 567577 - ironic - Tear down console during unprovisioning14:14
rloojroll: OH. ok, will wait for tests to pass first.14:15
jrollthey're still running? :|14:15
rloojroll: all i see is red but that's 3rd party14:16
* jroll should probably note the test update in commit message too14:16
jrollright14:16
*** jistr|mtg is now known as jistr14:17
jrollugh, now I'm reading where else we stop the console, why do I need to check myself if it's running14:17
jrolland handle notifications and such14:17
jrollwe suck at driver interfaces14:18
jrollnever mind, that patch isn't ready :(14:18
rloojroll: ok, off my list now :)14:19
*** baha has joined #openstack-ironic14:21
*** rajinir has joined #openstack-ironic14:23
*** rbartal_ has quit IRC14:33
openstackgerritJim Rollenhagen proposed openstack/ironic master: Tear down console during unprovisioning  https://review.openstack.org/56757714:35
openstackgerritJim Rollenhagen proposed openstack/ironic master: Tear down console during unprovisioning  https://review.openstack.org/56757714:55
jrollok this is much better now14:55
*** rpioso|afk is now known as rpioso14:55
rpiosoGood morning14:55
TheJuliagood morning rpioso14:56
rpiosoTheJulia: :)14:56
*** mgoddard has quit IRC15:05
openstackgerritMerged openstack/ironic master: Gate fix: Cap hacking to avoid gate failure  https://review.openstack.org/56709915:07
openstackgerritMerged openstack/ironic stable/queens: Cap hacking to avoid gate failure  https://review.openstack.org/56748115:07
*** e0ne has quit IRC15:07
*** mgoddard has joined #openstack-ironic15:09
*** jiapei has quit IRC15:09
TheJuliaWow we have many repos15:11
*** alex_xu has quit IRC15:24
*** v12aml has quit IRC15:27
*** v12aml has joined #openstack-ironic15:29
*** jaganathan has quit IRC15:30
*** prometheanfire has joined #openstack-ironic15:34
prometheanfireso... the rados stuff15:34
prometheanfiregiven swift removing support for that url format, that means that the swift_account_type conf option does nothing and should be removed15:35
openstackgerritMerged openstack/ironic-inspector master: Fix pycodestyle warnings/errors now visible with hacking 1.1.0  https://review.openstack.org/56706615:35
openstackgerritMerged openstack/ironic master: Follow up to inspect wait implementation  https://review.openstack.org/56694815:35
openstackgerritMerged openstack/ironic master: BIOS Settings: Add BIOSInterface  https://review.openstack.org/50779315:36
*** v12aml has quit IRC15:36
*** srart has quit IRC15:37
*** srart has joined #openstack-ironic15:38
*** v12aml has joined #openstack-ironic15:40
*** bdodd has quit IRC15:44
*** pcaruana has quit IRC15:52
jrollprometheanfire: cool with me - I wonder if this makes it a multi-cycle thing to deprecate it though :(15:53
TheJuliagiven how long it has been broken, I kind of see the deprecation policy as not really working in this case15:54
jrollhas it been broken long? I thought this just released15:55
TheJuliaI thought it required ocata client code to still work15:56
TheJuliaunless there was just no need for the client to be updated for cycles and the breaking change landed this cycle.15:56
TheJuliaI'll have to actually look deeper, I glanced at it yesterday and got pulled onto something else a few times :(15:57
jrolloh, it was made in ocata15:57
jrollkill it with fire15:57
* TheJulia makes mental note to try and discuss it with some folks internally tomorrow15:57
clarkbI'm not caught up on email, I take it TheJulia's ask that the client not care so much went nowhere?16:02
prometheanfirejroll: ya, that's my worry (multi-cycle stuff)16:02
TheJuliaAs of this morning, I didn't see any reply16:02
prometheanfireit requires newton swiftclient to work16:02
TheJuliabut I've not looked back since  I'm in prep mode for summit/forum16:02
prometheanfireocata is where the break occured16:03
jrollit's about ceph trying to emulate swift features, I've got no problem with the client staying as-is given ceph can support it16:03
prometheanfireya, no need to use a new client, that'd be more of a pain16:04
*** racedo_ has quit IRC16:05
*** racedo_ has joined #openstack-ironic16:05
*** racedo_ has quit IRC16:05
* TheJulia sighs16:05
*** racedo_ has joined #openstack-ironic16:06
*** racedo_ has quit IRC16:06
*** racedo_ has joined #openstack-ironic16:07
prometheanfirenew client meanting not-swiftclient16:07
*** racedo_ has quit IRC16:07
*** racedo_ has joined #openstack-ironic16:07
clarkbI think the ask was to update swiftclient to accomodate less strict urls16:08
*** racedo_ has quit IRC16:08
prometheanfireit was16:08
prometheanfireand they (notmyname) said no16:08
clarkbah ok that is what I was behind on16:09
prometheanfireswift cares about swift only16:09
TheJulia$WORDS16:09
prometheanfire:D16:10
clarkbTheJulia: summit prep probably a good idea :P /me wanders off to do some of that too16:10
TheJuliaIt is far from ideal that is their stance since it ultimately harms the overall community, but if they wish to play the game of lock-in... there is not really much we can do.16:10
prometheanfireI expect this to fail spectacularly16:10
openstackgerritMatthew Thode proposed openstack/ironic master: Remove endpoint_type from configuration  https://review.openstack.org/56762816:10
prometheanfirebut that's all the refrences I think that needed to be updated16:11
TheJuliaI guess there is a fifth option... Someone go about the path of decoupling ceph from swift in ironic, and we take a whole other route, but that is a longer term thing that maybe is worth doing after the initial tearout16:11
prometheanfireTheJulia: ya, that was my view too, it's a valid opinion, but coming from swiftstack, sounds like lockin too16:11
NobodyCamGood Morning Ironic'ers16:13
*** gyee has joined #openstack-ironic16:13
TheJuliaprometheanfire: I guess in my mind, it feels like intentional lock-in, and I'm feel like given my perceptions of their focus that even if we pressed and had a CI job for it, we would have had to go down the same route of ripping it out... just sooner :(16:16
TheJuliaThe optics of abrupt feature rip-out are just... bad.16:17
*** marios has quit IRC16:17
prometheanfireagreed16:21
*** sambetts is now known as sambetts|afk16:21
* TheJulia goes back to slide deck because the end of it is in sight16:21
sambetts|afknight all16:22
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Mark xclarity password as secret  https://review.openstack.org/56763716:22
sambetts|afko/16:22
TheJuliagoodnight sambetts|afk16:22
* TheJulia sighs16:22
*** baha has quit IRC16:23
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Mark xclarity password as secret  https://review.openstack.org/56763916:25
*** fragatin_ has joined #openstack-ironic16:25
*** erolg has quit IRC16:26
jrollTheJulia: fwiw we aren't removing the feature, just requiring a config option set on the ceph side16:30
jrolland ripping out the config on our side as it is no longer needed16:31
TheJuliayeah, I just picked up on that checking the thread16:31
jroll:)16:31
clarkbjroll: any idea how difficult it is to change that setting on an existing ceph install?16:33
clarkb(I have no idea myself)16:33
jrollclarkb: not a clue16:34
*** pcaruana has joined #openstack-ironic16:36
*** milan has quit IRC16:36
*** trown is now known as trown|lunch16:39
prometheanfireclarkb: not hard, using ceph-ansible just take the config from the bug and rerun the ceph-deply playbook16:42
clarkbprometheanfire: and users don't all have to update their usage of the api?16:43
clarkbrpobably not if the old urls are still allowed16:43
*** pcaruana has quit IRC16:44
prometheanfireclarkb: that's a good question, not sure if the old format will continue to work16:44
*** e0ne has joined #openstack-ironic16:49
*** dprince has quit IRC16:50
*** dsneddon_ has joined #openstack-ironic16:51
*** dsneddon_ has quit IRC16:53
*** prometheanfire has quit IRC16:57
*** dsneddon_ has joined #openstack-ironic16:58
*** prometheanfire has joined #openstack-ironic16:58
*** dougsz has quit IRC16:59
*** dsneddon_ has quit IRC16:59
*** dsneddon_ has joined #openstack-ironic16:59
*** derekh has quit IRC17:00
*** lucasagomes is now known as lucas-afk17:00
*** mgoddard has quit IRC17:07
*** cjloader has quit IRC17:09
*** jtomasek has quit IRC17:10
*** dprince has joined #openstack-ironic17:16
jrollok, zuul is happy with this one: https://review.openstack.org/#/c/567577/17:18
patchbotpatch 567577 - ironic - Tear down console during unprovisioning17:18
jrollTheJulia: bit of a security thing there so we should merge that sooner than later17:18
TheJuliajroll: yah :)17:27
*** jaganathan has joined #openstack-ironic17:31
*** ianychoi_ has joined #openstack-ironic17:32
*** ianychoi has quit IRC17:35
*** baha has joined #openstack-ironic17:37
*** trown|lunch is now known as trown17:37
*** mgoddard has joined #openstack-ironic17:42
*** rcernin has joined #openstack-ironic17:52
*** sekharvajjula has joined #openstack-ironic17:52
*** dsneddon has quit IRC17:52
*** ChandraS has quit IRC17:54
*** tesseract-RH has joined #openstack-ironic18:08
*** tesseract has quit IRC18:09
*** tesseract-RH has quit IRC18:11
*** tesseract has joined #openstack-ironic18:11
*** liuzz_ has joined #openstack-ironic18:15
*** masuberu has joined #openstack-ironic18:16
*** liuzz has quit IRC18:17
*** alineb has quit IRC18:18
*** mjturek has quit IRC18:19
*** masber has quit IRC18:19
openstackgerritMatthew Thode proposed openstack/ironic master: Remove endpoint_type from configuration  https://review.openstack.org/56762818:22
*** fragatin_ has quit IRC18:31
*** mgoddard has quit IRC18:33
*** e0ne has quit IRC18:33
*** alineb has joined #openstack-ironic18:45
*** tesseract has quit IRC18:49
*** rcernin has quit IRC18:50
*** fragatina has joined #openstack-ironic18:54
*** fragatina has quit IRC18:54
*** fragatina has joined #openstack-ironic18:55
*** livelace has joined #openstack-ironic19:01
*** rcernin has joined #openstack-ironic19:09
*** milan has joined #openstack-ironic19:14
*** rcernin has quit IRC19:15
*** jcoufal has quit IRC19:16
TheJuliajroll: comment on that patch19:19
jrollthanks, gimme a few19:19
TheJuliajroll: thanks, my only issue is with the reno to keep down edits... esp since we need to backport it19:29
*** jaganathan has quit IRC19:39
*** mjturek has joined #openstack-ironic19:44
*** hoonetorg has quit IRC19:46
*** karimull has joined #openstack-ironic19:49
karimullhi can get used case scenario on how the exta-spec traits are used in ironic node configuration19:50
*** prometheanfire has quit IRC19:52
*** prometheanfire has joined #openstack-ironic19:53
*** hoonetorg has joined #openstack-ironic20:00
TheJuliado you mean traits?20:01
karimullyes20:01
*** dprince has quit IRC20:01
TheJuliatraits are not used yet, but will eventually be translated, where appropriate, to driver level actions such as setting the boot mode, since a node can support multiple boot modes20:02
TheJuliaraid is another case where it would be useful20:02
karimullok..20:02
jrollthey are used for scheduling, I believe20:02
jrollso you could set a node with a trait of CUSTOM_HAS_BITCOIN_MINER, and set the same in the flavor, and they'll be matched up20:03
TheJuliaThat as well, but that is not node configuration yet20:03
karimullwhen they say get_traits() in nova does it mean..getting from the remote node or from instance_info/traits file?20:03
jrollTheJulia: reading is hard20:03
TheJuliajroll: indeed :(20:03
TheJuliakarimull: instance_info traits data should be the traits that are requested by the user for the instance20:04
jrollkarimull: the resource tracker calls get_traits() in the ironic driver, to find which traits are set on the node20:04
*** prometheanfire has quit IRC20:04
jrollduring deploy, the ironic driver puts the traits from the flavor into instance_info/traits for the node20:05
*** milan has quit IRC20:05
karimulldeploy means..during ironic node enrollment is that correct?20:06
TheJuliano20:06
karimullnova boot?20:06
*** prometheanfire has joined #openstack-ironic20:06
TheJuliaenrollment is when someone adds a baremetal node to ironic, deploy equates to nova boot20:06
karimulleven during enrollment  we should be able to pass traits right?..20:07
karimullin future once implemented how do we know that particular bare metal server has that traits that user has requested?20:08
jrollthe traits which an operator defines on a node go in node.traits, and can be added at enrollment20:09
jrollthe traits which a user has requested go in instance_info/traits20:09
karimullfor example if I have SGX enabled functionality on a baremetal server.. and how does the placement api compares with user requested CUSTOM_SGX traits with real configuration20:10
jroll20:03:36           jroll | so you could set a node with a trait of CUSTOM_HAS_BITCOIN_MINER, and set the same in the flavor, and they'll be matched up20:11
karimullok..so the operator has to make sure that particular node has this traits on the BMS20:11
jrollyes20:11
karimulland nova / placement-api will compare instance_info/traits with node.traits to filter out the nodes20:14
jrollit will compare the traits in flavor extra_specs with node.traits to find a node. then while booting the instance on that node, it will put the traits from the flavor extra_specs in node.instance_info/traits20:15
karimullgot it..20:15
karimullI wanted to add a ironic node with trusted_boot traits ..but a node is considered trusted_boot only after user image is up and attestation is done..20:17
karimullin this case as an operator how can indicate this particular node is trusted?20:17
TheJuliaso that gets to be a bit complex20:17
jrollthat attestation process is done outside of ironic, right?20:18
karimullyes20:18
TheJuliaThe trusted_boot concept is signed kernel, any attestation is completely outside of ironic, and we never know about it20:18
karimullbut they is client running on irnoc20:18
jrollso when that is complete, you can add the trait to the node20:18
jrollbefore making it available to nova20:18
TheJuliaproblem is...20:18
TheJuliaattestation is from the OS running20:18
karimulllike do an update ..yes that would do20:18
TheJuliaafter deployment20:19
jrollyes, `openstack baremetal node add trait CUSTOM_TRUSTED_BOOT` or so20:19
jrollTheJulia: I think s/attestation/measurement/20:19
karimullcool..thanks..20:20
TheJuliawell both are post-deployment things. measurement would be the launch environmment enforcing the policy. attesttion is stating to the attestation server what the measurement is20:20
jroller, you're right20:21
karimullyes both are post deployment after tboot procedure20:21
jrollI guess I assumed it was being boot once manually for attestation20:21
TheJuliaall that ironic can help with advertisment wise to nova for scheduling... is the capacity to support20:21
jrolland then it was good to go to be re-deployed20:21
TheJuliaand20:21
TheJuliathe fact that the TPM is owned by an attestation server20:21
TheJuliabut that last part would have to be operator interaction with the hardware depending on attestation solution used/desired20:22
TheJuliaso the tpm ownership can, at least with OpenCIT cross deployments so it is a do it once action20:22
karimullyes ... once we make sure attestation is done..operator intervention is required20:23
TheJuliaI believe the same is true with keylime, but i'm not 100% sure20:23
jrollhow we're planning on doing secure boot things is to do all the attestation stuff before we install the server at it's site20:23
jrollso I guess I assumed that's what was happening20:23
karimullin secure boot does the signature validation is done after deployment?20:24
TheJuliaso the UEFI ramdisk _should_ be able to verify the signatures on the bootloader/boot loader elements20:25
TheJuliaafter that, measurement validations can take place with a policy, but tboot is needed or... something else to also tie into the boot loader20:26
TheJuliato enable further measurment/validation where required20:26
* TheJulia is kind of glad we're having this discussion20:26
* jroll is just getting confused20:27
TheJuliajroll: it is a confusing topic20:27
* TheJulia suspects whiskey would improve it20:28
jrollTheJulia: oh I know, it's been on my mind this week20:28
karimullwe are planning to resolve an issue where user wants to tag a node with trusted_boot .before deployment of user image..like in initial phase of initramfs20:29
TheJuliakarimull: what would that entail from your perspective?20:30
karimullwe are investigating ..if this possible..because to compare the whitelist with measure values we have to do our side the node..and we don't know ..like it can be done20:31
karimullout*20:32
TheJuliaso if a user were to submit a measurement policy in with the image to be deployed, or possibly even into nova, I think it might be feasible to automate that20:32
TheJuliawe historically have resisted any attempts to edit the user supplied image, but that needs to be house in with something like the tboot extension for grub20:32
TheJuliabut checking existing values for the hardware seems feasible, we've discussed adding a security_interface concept20:33
karimullmeasurement policy with image in the sense like image meta data?20:34
karimullthat seems to be good idea20:34
TheJuliaWell, I'm not entirely sure that is feasible because the measurement policy is an in-filesystem element ultimately and can have measurements for libraries and binaries inside the system20:35
TheJuliaThe system can't attest if it cant read the policy, but it could be stored in configuration drive data and the booting OS could put it in place. The booting guest will then still need to reboot I believe20:36
karimullhmm..need to look into openCIT on how they are doing this comparison. From my understanding they get required PCR values and compare against the measure value20:36
TheJuliakarimull: they use tboot + an agent you  have to install post deployment20:37
*** jaganathan has joined #openstack-ironic20:37
karimullok..20:37
TheJuliaI've been thinking the agent can be installed, but I don't think I've found the agent source out there20:37
TheJuliaI also haven't looked too hard20:38
TheJuliaI spoke with keylime folks, and they have a different approach, I just ton't remember all of the details. They are also only TPM1.1 on x86, not TPM2 and x86_6420:38
karimullwe have to get the source and compile it ourselfs..thatz what I figured it out20:38
TheJuliakarimull: you might be figuring out why I've not written a security interface specification for ironic yet :)20:39
karimullthat would really help us20:39
karimulladmin12320:40
TheJuliaif we could understand your usecases/workflows, that would help us understand logical hook points20:40
TheJuliaat least, what you can share into the public universe that makes sense in your case20:40
karimullok..i'm also in inital phase of understanding the used case...but mean while wanted to know how traits in Ironic works..so landed here...we start posting my developmets from now onwars.20:41
karimullthanks for the info ....will comeback with more updates and questions...20:42
TheJuliakarimull: your welcome to bring it up and discuss it with us during our weekly meeting, there are a number of other intereted individuals20:42
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Fix gate and bump CoreOS version to latest stable.  https://review.openstack.org/56609420:43
TheJuliajroll: bumped the core count ^^^ since it really helps CoreOS fire up faster inside a VM20:43
karimullsure will be there in next meeting..20:43
jrollTheJulia: cool, were those failures timeouts on the last one?20:44
TheJuliajroll: best as I can tell... it took 550+ seconds before it would have even gotten to starting to fire up IPA20:44
jrollTheJulia: oof, you were so close, see my comment20:44
TheJuliaoof?20:45
jrolllike, oof, I fell20:45
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Fix gate and bump CoreOS version to latest stable.  https://review.openstack.org/56609420:45
jrolla word not an acronym :)20:45
jroll+2, thanks!20:45
TheJuliajroll: lol20:45
jroll:P20:45
openstackgerritMerged openstack/virtualbmc master: Revert "Add Serial-over-LAN (SOL) support"  https://review.openstack.org/56664620:46
openstackgerritMerged openstack/virtualbmc master: Unrevert usage example inconsistency fix in docs  https://review.openstack.org/56718620:47
openstackgerritJim Rollenhagen proposed openstack/ironic master: Tear down console during unprovisioning  https://review.openstack.org/56757720:54
jrollTheJulia: rloo: this is ready for y'all again ^20:54
rloojroll: ok20:54
*** trown is now known as trown|outtypewww20:58
*** Goneri has quit IRC21:01
*** baha has quit IRC21:10
*** hemna_ has quit IRC21:18
openstackgerritWill Szumski proposed openstack/ironic-python-agent master: Try empty password for ata security-unlock in GenericHardwareManager  https://review.openstack.org/55775521:29
*** livelace has quit IRC21:58
*** bfournie has quit IRC22:05
openstackgerritWill Szumski proposed openstack/ironic-python-agent master: Try empty password for ata security-unlock in GenericHardwareManager  https://review.openstack.org/55775522:05
*** hemna_ has joined #openstack-ironic22:10
*** rbudden has quit IRC22:28
*** jaypipes_ has joined #openstack-ironic22:42
*** jaypipes has quit IRC22:42
*** vdrok has quit IRC22:42
*** vdrok has joined #openstack-ironic22:43
*** jaypipes__ has joined #openstack-ironic22:48
*** jaypipes_ has quit IRC22:49
*** bfournie has joined #openstack-ironic22:50
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Fix gate and bump CoreOS version to latest stable.  https://review.openstack.org/56609422:50
*** jaganathan has quit IRC23:04
*** rloo has quit IRC23:14
*** srart has quit IRC23:32
*** liuzz has joined #openstack-ironic23:40
*** liuzz_ has quit IRC23:40
*** rajinir has quit IRC23:43
*** srart has joined #openstack-ironic23:53
*** rpioso is now known as rpioso|afk23:59

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