Thursday, 2019-01-10

*** macza has quit IRC00:01
*** slaweq has quit IRC00:05
*** igordc has quit IRC00:12
*** tbachman has quit IRC00:13
*** itlinux has joined #openstack-nova00:17
*** slaweq has joined #openstack-nova00:22
*** slaweq has quit IRC00:26
yonglihealex_xu, small spec 'server group' is reviewed rounds, i worry it might need few more, we are so close to feature freeze, it's quite small.  please put it on your work list00:28
yonglihe https://review.openstack.org/#/c/612255/1200:28
*** tbachman has joined #openstack-nova00:36
*** tetsuro has joined #openstack-nova00:37
*** ileixe has joined #openstack-nova00:44
*** igordc has joined #openstack-nova01:03
*** gmann_pto is now known as gmann01:07
*** tetsuro has quit IRC01:11
*** tetsuro has joined #openstack-nova01:24
*** tetsuro has quit IRC01:24
*** tetsuro has joined #openstack-nova01:25
*** igordc has quit IRC01:26
*** tetsuro has quit IRC01:30
*** macza has joined #openstack-nova01:38
*** markvoelker has joined #openstack-nova01:40
*** markvoelker has quit IRC01:45
*** gyee has quit IRC01:50
*** macza has quit IRC01:50
*** _alastor_ has joined #openstack-nova01:53
*** _alastor_ has quit IRC01:58
openstackgerritJie Li proposed openstack/nova master: Support volume-backed server rebuild in compute  https://review.openstack.org/62589301:59
*** tbachman has quit IRC02:10
*** macza has joined #openstack-nova02:17
openstackgerritMerged openstack/nova-specs master: add spec "show-server-numa-topology"  https://review.openstack.org/61225602:20
*** macza has quit IRC02:22
yongliheThanks, Jay, alex_xu, i know this might be too pushy,  it's run out of time, i wonder the server-group spec may also close enough. i'm here, could response quickly.02:22
*** Dinesh_Bhor has joined #openstack-nova02:28
*** hongbin has joined #openstack-nova02:30
*** whoami-rajat has joined #openstack-nova02:31
openstackgerritMerged openstack/nova-specs master: Propose counting quota usage from placement and API database  https://review.openstack.org/50904202:32
*** tetsuro has joined #openstack-nova02:44
*** markvoelker has joined #openstack-nova02:51
*** sapd__x has quit IRC02:53
*** mhen has quit IRC02:57
*** wolverineav has quit IRC02:58
*** mhen has joined #openstack-nova02:58
*** wolverineav has joined #openstack-nova02:59
*** wolverineav has quit IRC03:06
*** wolverineav has joined #openstack-nova03:12
*** wolverineav has quit IRC03:17
*** igordc has joined #openstack-nova03:25
*** igordc has quit IRC03:29
*** jackding has quit IRC03:44
*** jding1_ has joined #openstack-nova03:44
*** markvoelker has quit IRC03:53
*** sapd__x has joined #openstack-nova03:53
*** cfriesen has quit IRC04:04
openstackgerritJack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation  https://review.openstack.org/62070604:04
*** Dinesh_Bhor has quit IRC04:07
*** sapd__x has quit IRC04:19
*** jding1_ has quit IRC04:20
*** jackding has joined #openstack-nova04:21
*** stakeda has joined #openstack-nova04:25
*** wwriverrat has quit IRC04:32
*** ileixe has quit IRC04:32
*** tetsuro has quit IRC04:41
*** annp_ has joined #openstack-nova04:42
*** udesale has joined #openstack-nova04:50
*** Dinesh_Bhor has joined #openstack-nova04:55
*** hongbin has quit IRC04:56
*** markvoelker has joined #openstack-nova04:59
*** ileixe has joined #openstack-nova05:00
*** xek_ has joined #openstack-nova05:11
*** xek has quit IRC05:12
*** lbragstad has quit IRC05:12
*** ratailor has joined #openstack-nova05:18
*** itlinux has quit IRC05:18
*** TxGirlGeek has joined #openstack-nova05:45
openstackgerritHesam Chobanlou proposed openstack/nova master: nova.exception._clean_dict should use oslo_utils.strutils  https://review.openstack.org/62976905:47
*** _alastor_ has joined #openstack-nova05:55
openstackgerritMerged openstack/nova-specs master: Per-instance serial number (spec)  https://review.openstack.org/61253105:55
*** _alastor_ has quit IRC06:00
*** markvoelker has quit IRC06:03
*** markvoelker has joined #openstack-nova06:05
*** macza has joined #openstack-nova06:10
*** tbachman has joined #openstack-nova06:13
*** macza has quit IRC06:14
*** tbachman has quit IRC06:35
*** dave-mccowan has quit IRC06:41
*** Dinesh_Bhor has quit IRC06:41
*** Dinesh_Bhor has joined #openstack-nova06:44
*** rcernin has quit IRC06:44
*** rcernin has joined #openstack-nova06:44
*** sridharg has joined #openstack-nova06:54
*** Dinesh_Bhor has quit IRC06:58
*** ondrejme has joined #openstack-nova06:59
*** jiapei has joined #openstack-nova07:01
*** rcernin has quit IRC07:03
*** TxGirlGeek has quit IRC07:03
*** ccamacho has joined #openstack-nova07:21
*** macza has joined #openstack-nova07:27
*** sapd__x has joined #openstack-nova07:30
*** macza has quit IRC07:31
*** Luzi has joined #openstack-nova07:35
*** janki has joined #openstack-nova07:46
*** rpittau has joined #openstack-nova07:46
*** dpawlik has joined #openstack-nova07:48
*** pcaruana has joined #openstack-nova07:51
*** alex_xu has quit IRC08:05
*** brault has joined #openstack-nova08:08
*** alex_xu has joined #openstack-nova08:09
jangutter;08:13
jangutterbauzas: thanks for the review on https://review.openstack.org/567148 - did I address your query, or should I amend the spec to explicitly state "no impact in this case"?08:15
*** helenafm has joined #openstack-nova08:21
openstackgerritJie Li proposed openstack/nova master: Change compute api microversion to support for volume-backed server rebuild  https://review.openstack.org/62751808:36
*** ralonsoh has joined #openstack-nova08:37
*** markvoelker has quit IRC08:52
kashyapdansmith: Thanks for the review!08:54
kashyapdansmith: Yeah, forgot the index thingie; will fix.08:54
openstackgerritJie Li proposed openstack/nova master: Change compute api microversion to support for volume-backed server rebuild  https://review.openstack.org/62751809:06
*** janki has quit IRC09:13
*** bhagyashris has joined #openstack-nova09:13
*** logan- has quit IRC09:21
openstackgerritTakashi NATSUME proposed openstack/nova master: Add description about sort order in API ref guideline  https://review.openstack.org/62728209:21
*** erlon has joined #openstack-nova09:24
*** logan- has joined #openstack-nova09:28
*** logan- has quit IRC09:28
*** janki has joined #openstack-nova09:31
*** wolverineav has joined #openstack-nova09:36
*** derekh has joined #openstack-nova09:39
*** derekh has quit IRC09:40
*** derekh has joined #openstack-nova09:40
*** wolverineav has quit IRC09:41
openstackgerritTakashi NATSUME proposed openstack/nova stable/rocky: Add links to summit videos in user/cells.rst  https://review.openstack.org/62701909:43
*** bhagyashris has quit IRC09:48
*** markvoelker has joined #openstack-nova09:49
*** markvoelker has quit IRC09:51
*** luksky has joined #openstack-nova09:52
*** mmethot has quit IRC09:55
*** mmethot has joined #openstack-nova09:56
*** jaosorior has quit IRC10:00
*** itlinux has joined #openstack-nova10:01
*** itlinux has quit IRC10:05
*** priteau has joined #openstack-nova10:08
openstackgerritVlad Gusev proposed openstack/nova stable/queens: Do not dump all instances in the scheduler  https://review.openstack.org/62982210:10
*** dtantsur|afk is now known as dtantsur10:13
openstackgerritJie Li proposed openstack/nova master: Change api microversion to support for bfv  https://review.openstack.org/62751810:20
*** alex_xu has quit IRC10:30
openstackgerritStephen Finucane proposed openstack/nova stable/queens: Fail to live migration if instance has a NUMA topology  https://review.openstack.org/62959710:35
*** alex_xu has joined #openstack-nova10:35
*** yan0s has joined #openstack-nova10:45
*** brault has quit IRC10:52
openstackgerritLee Yarwood proposed openstack/nova stable/rocky: libvirt: Add workaround to cleanup instance dir when using rbd  https://review.openstack.org/62795810:52
*** jaosorior has joined #openstack-nova11:01
*** openstackgerrit has quit IRC11:05
*** udesale has quit IRC11:14
*** udesale has joined #openstack-nova11:14
*** ileixe has quit IRC11:20
*** openstackgerrit has joined #openstack-nova11:23
openstackgerritJie Li proposed openstack/nova master: Bump API microversion to support rebuild bfv instances  https://review.openstack.org/62751811:23
openstackgerritMerged openstack/nova-specs master: Spec to implement vRouter HW offloads  https://review.openstack.org/56714811:30
openstackgerritKashyap Chamarthy proposed openstack/nova master: docs: Secure live migration with QEMU-native TLS  https://review.openstack.org/62962711:30
kashyapstephenfin: If you have a few minutes today, appreciate your rST (& general) eyes ^11:30
stephenfinkashyap: Already done11:30
stephenfinI think11:30
kashyapDarn, did I _just_ miss your feedback?11:30
stephenfinThink so11:31
kashyapAh, yes.  I addressed a few; will address the rest11:32
kashyapstephenfin: Before I go for lunuch11:32
kashyapstephenfin: ... on that "has" vs. "have" -- I was referring to Dan's correction of my mistake at the top of the file11:32
kashyap(So I was referring to )11:32
kashyaps/to/to my own mistake/11:32
kashyapstephenfin: Thanks for the comments11:34
kashyapstephenfin: And yes -- the doc _does_ render; I built locally before pushing11:35
stephenfinkashyap: render _correctly_?11:35
*** psachin has joined #openstack-nova11:35
*** maciejjozefczyk has joined #openstack-nova11:36
*** psachin has quit IRC11:44
*** brault has joined #openstack-nova11:46
*** psachin has joined #openstack-nova11:48
*** brault has quit IRC11:50
*** brault has joined #openstack-nova11:57
*** slaweq has joined #openstack-nova11:58
*** ratailor has quit IRC12:00
*** itlinux has joined #openstack-nova12:01
*** itlinux has quit IRC12:06
*** sapd__x has quit IRC12:07
*** sridharg has quit IRC12:07
*** rpittau is now known as rpittau|lunch12:10
*** mrch_ has joined #openstack-nova12:10
*** luksky has quit IRC12:15
*** cdent has joined #openstack-nova12:16
*** sridharg has joined #openstack-nova12:17
jaypipesyonglihe: morning.12:23
jaypipesyonglihe: or evening for you :)12:23
jaypipesyonglihe: I'm close on the server group spec. will review shortly.12:23
jaypipesyonglihe: you might want to reach out to gmann since he was -1 on the spec12:24
jaypipesyonglihe: ok, re-reviewing. Ghanshyam's suggestions are very good. If you could make those small changes, I'll +W it.12:29
jaypipesalex_xu: ^^12:29
*** slaweq has quit IRC12:38
cdentjaypipes: you wanna kick https://review.openstack.org/#/c/629299/ in , make cfriesen happy12:41
*** nehaalhat has quit IRC12:43
*** ShilpaSD has quit IRC12:44
*** nnsingh has quit IRC12:44
*** priteau has quit IRC12:48
jaypipescdent: reviewed.12:48
cdentthanks12:48
jaypipescdent: I would like at the very least a more descriptive comment and a link to some docs.12:49
cdentseems reasonable12:49
jaypipesty sir12:49
*** moshele has joined #openstack-nova12:49
cdentjaypipes: you might also have some thoughts on https://review.openstack.org/#/c/626055/ , which seems to be a "why aren't you using placement for this" spec12:50
cdentof course given the date, I'm not sure it matters12:50
jaypipescdent: ack, thanks for the heads up, appreciated.12:51
*** nnsingh has joined #openstack-nova12:55
*** ShilpaSD has joined #openstack-nova12:55
*** rpittau|lunch is now known as rpittau13:09
*** luksky has joined #openstack-nova13:16
*** diga has joined #openstack-nova13:18
kashyapstephenfin: Yes, it does.  Sorry, that's what I meant.13:20
kashyapLet me show you a quick screenshot13:20
kashyapstephenfin: Here: http://kashyapc.fedorapeople.org/TLS-rST-rendering-just-for-stephenfin.png13:22
kashyapstephenfin: Ah, here is the official rendering: http://logs.openstack.org/27/629627/2/check/openstack-tox-docs/dc52e37/html/admin/secure-live-migration-with-qemu-native-tls.html13:24
*** derekh has quit IRC13:25
*** maciejjozefczyk has quit IRC13:42
*** priteau has joined #openstack-nova13:43
*** dpawlik has quit IRC13:43
*** dpawlik has joined #openstack-nova13:44
alex_xujaypipes: let me ping yonglihe13:44
yonglihei'm here13:44
yongliheworking on that13:44
jaypipesheya :)13:44
yonglihehi,  good morning13:45
alex_xuhah, cool13:45
*** maciejjozefczyk has joined #openstack-nova13:47
stephenfinkashyap: Ah, all good so13:49
stephenfinOne thing I do spot is the note here http://logs.openstack.org/27/629627/2/check/openstack-tox-docs/dc52e37/html/admin/secure-live-migration-with-qemu-native-tls.html#performing-the-migration13:49
* kashyap clicks13:49
stephenfinIndent _is_ off for that13:49
stephenfin(presumably, anyway)13:49
kashyapstephenfin: Ah, that's why it didn't render cleanly13:49
kashyapGood catch13:50
gibinova meeting starts in 10 minutes in #openstack-meeting13:50
*** cfriesen has joined #openstack-nova13:50
kashyapstephenfin: On that ~~~ vs ---, I checked the existing pattern and went with it13:51
stephenfinyup, agreed13:51
stephenfinI did the same13:51
kashyapstephenfin: Since we're going to "fix it", should I change it to ---?13:51
stephenfineh, we'll do it all at once, I suspect13:51
stephenfinlocal consistency first13:51
kashyapOr stay consistent for now, and do a bulk thingie later?13:51
kashyapYep13:51
kashyapI'll let it be for now.13:51
*** priteau has quit IRC13:54
kashyapstephenfin: Please remind me again -- on using lower case "nova" vs. upper case?13:54
stephenfinkashyap: https://docs.openstack.org/doc-contrib-guide/writing-style/openstack-components.html13:54
* cdent loves that stephenfin loves hypertext13:55
* kashyap clicks13:55
*** lbragstad has joined #openstack-nova13:56
*** moshele has quit IRC13:56
kashyapstephenfin: It doesn't tell _why_13:56
openstackgerritYongli He proposed openstack/nova-specs master: add 'show-server-group' spec  https://review.openstack.org/61225513:56
kashyapI think it's in the "Service Project Naming"13:56
*** rpittau_ has joined #openstack-nova13:56
stephenfinkashyap: For that, you'll have to ask #openstack-doc. I don't actually recall13:56
kashyap[quote]13:57
kashyapThe history of this decision is that the documentation contributors wanted the least amount of cognitive overhead when writing and reviewing. Learning rules about case can be difficult across multiple projects with hundreds of documentation contributors and thousands of changes and additions. Lowercase for project names as a rule is then easiest to review and enforce at this scale and growth13:57
*** maciejjozefczyk has quit IRC13:57
kashyappattern.13:57
kashyap[/quote]13:57
kashyapSo it's to lower the cognitive overhead, at the cost of "impure English".  Yikes!13:57
cdentwhy do you think nova should be uppered?13:58
kashyapThat's also part of concentration & focus: using the correct capitalization, punctuation, etc. at the right place13:58
* cdent hates extraneous upper casing13:58
kashyapcdent: It's a proper noun13:58
*** lbragstad has joined #openstack-nova13:58
cdentnot all proper nouns are upper cased13:58
kashyapTrue, though.  But in this case "intuition" says so :D13:59
*** rpittau has quit IRC13:59
kashyapAnd "Nova" is a special noun13:59
kashyapBut anyway, I'll go with the rule, now that I know the history of it.13:59
kashyapcdent: Random internet person says this: https://grammar.yourdictionary.com/capitalization/rules-for-capitalizing-proper-nouns.html14:00
stephenfinkashyap: Where was that quote from?14:00
cdentkashyap: my response to that is "meh" ;)14:00
stephenfinYou might add it to that page I linked, if you find the time14:00
kashyapstephenfin: Ha!  I read the doc buried in the doc -- https://governance.openstack.org/tc/reference/service-project-naming.html14:00
stephenfinahh14:01
stephenfincool :)14:01
kashyapI wonder what else are we doing in the name of "lowering cognitive overhead".  There's a time and place for it14:01
kashyapBut not with proper nouns.  Haha14:01
* kashyap stops trolling14:01
jangutter"openstack: The meeting name has been set to 'nova' " <---- shouldn't that be "OpenStack: The meeting name has been set to 'Nova' " ?14:01
kashyapjangutter: Heh, there the bot is called 'openstack'; so in that context it's OK :D14:02
* stephenfin sees what jangutter did there14:02
jangutterkashyap: hehehe, it's just the Baader-Meinhof Phenomenon14:03
jaypipesalex_xu: feel free to send https://review.openstack.org/#/c/612255/ along14:04
*** priteau has joined #openstack-nova14:04
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script  https://review.openstack.org/61416714:04
*** Luzi has quit IRC14:05
*** rpittau_ is now known as rpittau14:06
alex_xujaypipes: done14:06
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal  https://review.openstack.org/59160714:06
stephenfinbauzas: You think https://blueprints.launchpad.net/nova/+spec/libvirt-neutron-sriov-livemigration is something you could revisit today?14:07
stephenfinand maybe jaypipes, for the first time? ^14:08
stephenfinI ask because I have reviewed most of the implementation, and it's pretty straightforward. Also, it's a long time bugbear of mine14:08
jaypipesack. will get to that shortly stephenfin14:09
stephenfinthank you sir14:09
openstackgerritStephen Finucane proposed openstack/nova stable/queens: Fail to live migration if instance has a NUMA topology  https://review.openstack.org/62959714:11
*** efried has joined #openstack-nova14:12
*** maciejjozefczyk has joined #openstack-nova14:12
*** derekh has joined #openstack-nova14:15
*** jmlowe has quit IRC14:16
openstackgerritMerged openstack/nova-specs master: add 'show-server-group' spec  https://review.openstack.org/61225514:19
yonglihealex_xu, Jay,  thanks. Have a good one.14:20
kashyapstephenfin: Is a single back-tick OK here: :oslo.config:option:`libvirt.live_migration_with_native_tls`14:20
kashyapOr does it require a double back-tick?14:20
stephenfinsingle14:20
stephenfindouble is only used for literals14:21
stephenfinsingle backticks means "default role", which is configurable but defaults to italics14:21
stephenfinsingle backticks without the :xyz: prefix for an explicit role14:21
kashyapstephenfin: I see, thanks for the education.14:21
*** TxGirlGeek has joined #openstack-nova14:23
kashyapstephenfin: One last English nit-pick: you suggest "commas" in the place of em-dash, e.g. look at point 2 in the "Prerequisites" section: http://logs.openstack.org/27/629627/2/check/openstack-tox-docs/dc52e37/html/admin/secure-live-migration-with-qemu-native-tls.html14:25
stephenfinyup?14:26
kashyapIt makes sense to retain that there.14:26
kashyapBecause the content between the em-dashes can also be used in brackets, _or_ two commas.14:26
stephenfinkashyap: True, em dash ~= comma14:27
stephenfinbut I tend to see them used more for asides14:27
kashyapSure.14:27
stephenfinsomething not directly related to the main sentence14:27
stephenfini.e.14:27
kashyapIn this case I've used it in place of two commas (which is also used for bracketed content)14:27
stephenfinI was on the way home, driving on the motorway, when I saw a cat14:28
stephenfinvs.14:28
stephenfinI was on the way home -- a needless endeavour, it turned out -- when I saw a cat14:28
stephenfinI don't know if that's correct, but that's how _I've_ done it :)14:29
kashyapHehe, I see your point there14:29
kashyapBut, in this case:14:29
kashyap"A pre-configured TLS environment—i.e. CA, server, and client certificates, their file permissions, et al—must be “correctly” configured (typically by an installer tool) on all relevant compute nodes."14:29
*** mvkr has quit IRC14:29
jaypipeskashyap: em-dash ftw.14:29
stephenfinfair :)14:29
kashyapReplacing those em-dashes with more commas is fugly14:29
jaypipesstephenfin: :P14:29
kashyapI can use brackets, though.14:29
stephenfinJust space them out either side and I'm happy (not sure if that's correct but it's tough to read otherwise)14:29
stephenfinjaypipes: I blame asettle14:30
jaypipesheh :)14:30
kashyapstephenfin: Haha, you know ... I have read the literature on the _spacing_ too14:30
kashyapstephenfin: When the doc is rendered, a space between two em-dashes attract too much attention to the em-dash iself.14:30
kashyaps/iself/itself/14:30
kashyapSee the fine point? :D14:30
kashyapThat was also a conscious (and consistent) choice throughout the doc14:31
kashyapBut if you insist, you know, in the name of civility I will happily add the spaces (consistently through out) :-)14:31
stephenfin:D14:31
kashyapstephenfin: I don't joke when I say I take my English seriously :D14:31
stephenfinNoted :D14:32
kashyapSorry, if I'm taxing your brain.  I like these pedantics, I know you do, too.14:32
jangutterkasyap: obligatory https://imgs.xkcd.com/comics/third_way.png14:32
* stephenfin thinks kashyap missed his chance to work with #openstack-doc when it was still a normal team :) Nice bunch14:32
kashyapstephenfin: I dislike the past tense :-(14:33
kashyapThe team no longer exists now?  And all docs are taken care of by relavant components?14:33
*** jmlowe has joined #openstack-nova14:33
*** belmoreira has quit IRC14:34
stephenfinStill there but only for things like the contributor guide that don't have a home elsewhere14:34
kashyapjangutter: Two spaces after a period is, of course, the correct thing.  It gives the brain a "breathing room" between the sentences, dammit.14:34
* stephenfin sharpens pitchfork14:35
kashyapstephenfin: I see.  But I can still enjoy working on the Nova docs to my heart's content14:35
kashyapstephenfin: LOL, I know your view on that14:35
jangutterkashyap: Them's fighting words.14:35
*** dpawlik has quit IRC14:35
*** dpawlik has joined #openstack-nova14:36
edleafekashyap: I like how the iPhone settled it. You type two spaces, and it inserts a period and a single space. :)14:42
*** mdbooth has joined #openstack-nova14:43
jangutterOne day, someone's going to add a small piece of code to lint the spaces after a period in doc8 and it will trigger World War III.14:44
*** takashin has quit IRC14:46
adriancjaypipes: Hi, ive been working with sean-k-mooney on enabling live-migration with SR-IOV, it would be great if you could add this to your review queue.14:46
*** takashin has joined #openstack-nova14:47
jaypipesadrianc: it's the next one in my list.14:47
gibimelwitt: FYI On the today's meeting stephenfin and I agreed that the bp https://blueprints.launchpad.net/nova/+spec/fix-reschedule-up-calls is good to go for Stein but as no other cores was present I haven't approved the bp yet.14:47
kashyapedleafe: Heh14:47
kashyapedleafe: Didn't know that; /me doesn' want to put more cash in Apple's bank :D14:47
adriancjaypipes, thanks :)14:47
*** jchhatbar has joined #openstack-nova14:51
*** tbachman has joined #openstack-nova14:54
*** janki has quit IRC14:54
*** jmlowe has quit IRC14:58
*** jmlowe has joined #openstack-nova14:59
stephenfinadrianc: finished reviewing the stack of changes and spec. Ping me if there's anything else to do14:59
sean-k-mooneyi see  stephenfin has corrected my grammer hehe15:00
stephenfinsean-k-mooney: I had to leave _some_ comments ;)15:00
stephenfinemphasis on "none of which matter that much unless you have to rework" though15:00
*** jchhatbar is now known as janki15:00
*** itlinux has joined #openstack-nova15:01
sean-k-mooneystephenfin: ill wait for jaypipes to finish and i can quickly resping or i can submig a follow up pacth with the cleanup which ever suits15:01
*** takashin has left #openstack-nova15:01
adriancstephenfin: ack thank you, will review.15:01
openstackgerritKashyap Chamarthy proposed openstack/nova master: docs: Secure live migration with QEMU-native TLS  https://review.openstack.org/62962715:02
*** TxGirlGeek has quit IRC15:03
*** itlinux has quit IRC15:06
*** TxGirlGeek has joined #openstack-nova15:06
*** TxGirlGeek has quit IRC15:10
*** efried has quit IRC15:11
*** efried has joined #openstack-nova15:11
kashyapDarn, one last thing to fix, the ".. note::" indentation15:12
*** mrch_ has quit IRC15:14
*** mvkr has joined #openstack-nova15:15
openstackgerritKashyap Chamarthy proposed openstack/nova master: docs: Secure live migration with QEMU-native TLS  https://review.openstack.org/62962715:16
*** awaugama has joined #openstack-nova15:17
*** igordc has joined #openstack-nova15:20
*** _alastor_ has joined #openstack-nova15:27
*** efried has quit IRC15:27
*** _alastor_ has quit IRC15:31
*** priteau has quit IRC15:31
*** efried has joined #openstack-nova15:34
kashyapdansmith: When you have a spare minute, as promised (with even a nice +2 on it), the doc patch: http://logs.openstack.org/27/629627/4/check/openstack-tox-docs/5bc0591/html/admin/secure-live-migration-with-qemu-native-tls.html15:41
kashyapdansmith: It is also revised and corrected as thoroughly as I could (and the back-and-forth on the review shows it, I hope) :-)15:42
kashyap(For convenience, here's the review: https://review.openstack.org/#/c/629627/4)15:42
dansmithkashyap: yeah I know, I've gotten the notifications, just have stuff to do this morning before I get to it15:42
kashyapstephenfin: Thanks for the quick turn around15:42
kashyapdansmith: Ah, didn't know you follow those notifications.  On IRC, I follow the "explicit is better than implicit" principle :-)15:43
kashyapIt can wait until your morning tasks are done, so long as it doesn't miss the BP-Approvals deadline.15:44
*** ccamacho has quit IRC15:45
*** ccamacho has joined #openstack-nova15:46
*** priteau has joined #openstack-nova15:47
mnaserkashyap: that's a pretty sweet doc patch.15:51
kashyapmnaser: Thanks :-)15:51
mnaseri'm going to try and work up some openstack ansible patches which allow setting tls config15:52
kashyapmnaser: Cool; do post a URL here when you get around to it15:52
mnaserwe have a full pki so switching this on and replacing it should be straightforward, will do15:52
kashyapmnaser: When I dev-tested it, I painfully configured it by hand!  Look at the scrollbar: https://kashyapc.fedorapeople.org/Native-TLS/Setup-for-NBD-and-migration-streams-over-TLS.rst.txt15:52
mnaserOSA doesn't have the PKI, our deployments do, but i'm sure we can figure out a way to make it easily available inside it, somehow15:53
*** hongbin has joined #openstack-nova15:53
kashyapIt's from-scratch, though, set up in virtual environments15:53
dansmithkashyap: I'm fine with approving the blueprint at this point15:53
*** slaweq has joined #openstack-nova15:53
kashyapmnaser: Do you know about the freeipa.org project?15:53
mnaserkashyap: freeipa is exactly what we run :)15:53
kashyapmnaser: It does all the Cert Lifecycle handling.  (In the distant past, /me used to work on one of its PKI components :-))15:53
mnaserso i was thinking of getting certs and using certmonger to keep em up to date too15:53
kashyapdansmith: Nice, thank you15:53
jaypipessean-k-mooney: done.15:53
jaypipesadrianc: ^15:54
kashyapmnaser: Yep; 'certmonger' handles the much-needed renewal aspect15:54
sean-k-mooneyjaypipes: cool ill assume i have rework to do so ill respin15:54
jaypipessean-k-mooney: just typos/grammar nits.15:54
mnaseryep, so i'd probably wire it up to that, somehow15:54
jaypipessean-k-mooney: fix those up and I'm +2.15:54
sean-k-mooneycool ill do dose now so thanks for reviewing15:55
kashyapmnaser: Sometimes, I just write a no-frills PKI setup with GnuTLS -- that's hacky, but I like the minimalism of it15:55
mnaserkashyap: freeipa has simplified pki so much15:56
kashyap(That's only for dev environments, though.  Normally, you'd want an "enterprise PKI" :D)15:56
mnaserand it has a ton of neat things like we're using SSHFP + DNSSEC so for clients that aren't enrolled, they can use SSHFP to verify validity of server when logging in15:57
kashyapmnaser: Yep, recall.  Components of FreeIPA were part of my first project at Red Hat 10 years ago :-)15:57
mnaseraka never hitting "yes" blindly anymore15:57
kashyaps/recal/I recall/15:57
*** munimeha1 has joined #openstack-nova15:57
kashyapmnaser: Didn't follow it lately; good to know it also handles DNNSEC15:59
*** _alastor_ has joined #openstack-nova15:59
mnaserkashyap: yeah so you can have signed zones, and then freeipa will create SSHFP records on enrollment, ssh clients lookup these records when connecting and make sure the server is who it claims to be :) pretty neat15:59
kashyapVery nifty16:00
*** maciejjozefczyk has quit IRC16:03
*** ratailor has joined #openstack-nova16:05
*** belmoreira has joined #openstack-nova16:06
*** udesale has quit IRC16:06
*** cfriesen has quit IRC16:14
*** TxGirlGeek has joined #openstack-nova16:14
*** cfriesen has joined #openstack-nova16:14
*** dpawlik has quit IRC16:17
melwittdansmith, gibi, jaypipes: looks like we are a go for approving https://blueprints.launchpad.net/nova/+spec/support-qemu-native-tls-for-live-migration now?16:17
dansmithyup16:17
melwittI also wanted to get consensus on approving this one too https://blueprints.launchpad.net/nova/+spec/fix-reschedule-up-calls16:18
*** slaweq has quit IRC16:19
dansmithis matt really going to work on that?16:19
dansmithif so, I'm fine with it.. looks like it's just a token bp because of the rpc changes or something16:20
melwittyeah, I'm not sure if he'll have time given cross-cell resize16:21
jaypipesmelwitt: sorry, Melanie, I haven't had a chance to look at that.16:21
melwittbut I noticed the bp when I went through the list of things targeted to stein16:22
*** jmlowe has quit IRC16:25
*** diga has quit IRC16:28
*** efried has quit IRC16:28
*** efried has joined #openstack-nova16:30
*** jmlowe has joined #openstack-nova16:32
*** ccamacho has quit IRC16:33
*** itlinux has joined #openstack-nova16:34
*** slaweq has joined #openstack-nova16:36
*** macza has joined #openstack-nova16:37
*** macza_ has joined #openstack-nova16:40
*** macza has quit IRC16:42
*** helenafm has quit IRC16:47
*** slaweq has quit IRC16:48
efriedjackding: Hi there. Do you know who is planning to implement https://review.openstack.org/#/c/620959/ and on what kind of timeline?16:49
efriedjackding: bp/cpu-model-selection that is16:50
*** imacdonn has quit IRC16:51
*** imacdonn has joined #openstack-nova16:51
*** pchavva has joined #openstack-nova16:54
jackdingefried: I don't know the timeline but I believe a Intel team is looking into this.16:55
efriedjackding: Heh, just so. I guess that's why I'm being asked about it.16:55
jackdingefried: yeh I don't think they have sorted out the details16:56
*** luksky has quit IRC16:56
openstackgerritJack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling  https://review.openstack.org/61669216:57
kashyapstephenfin: Hope my response answers your question there: https://review.openstack.org/#/c/625216/5/nova/virt/libvirt/driver.py@23017:02
*** yan0s has quit IRC17:03
melwittdansmith: I'm thinking we can just approve the up calls one and then I'll ask matt next week if he's planning to work on it, if not I'll just defer it back out17:04
dansmithmelwitt: okay17:05
*** rpittau has quit IRC17:07
openstackgerritArtom Lifshitz proposed openstack/nova master: Doc: rebuild can result in SHUTOFF VM state  https://review.openstack.org/62990417:09
*** igordc has quit IRC17:10
stephenfinjaypipes: Happy to +2 this assuming sean-k-mooney provides a follow up that handles the outstanding issues? https://review.openstack.org/60511617:11
stephenfinsean-k-mooney: I assume you're planning doing that :)17:11
stephenfin*to do17:11
*** dtantsur is now known as dtantsur|afk17:12
sean-k-mooneyits half done17:12
sean-k-mooneymetting delayed me finishing it17:12
*** ratailor has quit IRC17:12
sean-k-mooneystephenfin: jaypipes the only real change to me is do "git commit" or "git commit --amend" when its done which will be later today im just about to have something quick to eat and look at the extra nits from artom.17:15
jaypipesstephenfin: yes, I'm happy to +2 after sean-k-mooney removes the Sean-speak from it :P17:15
jaypipessean-k-mooney: I'm also grabbing some lunch, no worires :)17:16
artom"worires"? Sean-speak, eh?17:16
artom>;)17:16
dansmithoh gawd, jay caught it17:22
*** gyee has joined #openstack-nova17:24
sean-k-mooneyyeah im slowly infulencing ye all :)17:24
sean-k-mooneybe back in 30 mins or so17:26
*** fragatina has joined #openstack-nova17:27
*** psachin has quit IRC17:28
kashyapIs Gerrit down?17:29
kashyap"The server is temporarily unable to service your request due to maintenance downtime or capacity problems. Please try again later."17:30
kashyapWhoops, this says it's just me: https://downforeveryoneorjustme.com/review.openstack.org17:30
cdentme too17:30
melwittthe infra folks just restarted gerrit17:30
melwittshould be back soon I think17:30
kashyapmelwitt: Ah, thanks!17:31
kashyapYes, it's back now17:31
*** priteau has quit IRC17:33
melwittdansmith: do you agree that this bp should have a spec at least because it's proposing to add a new flavor extra spec? or do you think it would be ok specless? https://blueprints.launchpad.net/nova/+spec/libvirt-disable-pmu-for-realtime-guests17:33
dansmithwill look in a sec17:34
melwittthanks. just want a sanity check17:34
kashyapstephenfin: Worth respinning for the nit, and losing the +2s?  Instead of a follow-up?17:34
kashyap(For this, I mean: https://review.openstack.org/#/c/625216/)17:34
stephenfinkashyap: narp. Follow up17:35
stephenfin+W'd17:35
stephenfinassuming melwitt didn't want to review it? ^17:35
kashyapAh, okay, then.  Thank you.  I do have a follow-up patch in prep for the deprecation notes, etc.17:35
kashyapFWIW, Mel did review the "mini spec" (blueprint), though.  So I know she's onboard with the idea  :-)17:37
kashyapAlright, I get dinner, then.17:37
dansmithmelwitt: I dunno, I have a couple questions about the functionality, which might mean spec. Not sure if we always require a spec for new extra_specs, but generally they mean features which need some discussion17:39
dansmithmelwitt: sucks to say that on the spec deadline day, but this was also filed three days ago, so I dunno17:39
sean-k-mooneydansmith: it was a small feature i was just pulling in form work i planned to do in train. i can write a mini spec if people want but i also wont be upset if we defer17:40
dansmithsean-k-mooney: if it's train, then spec and defer17:40
dansmithimho17:41
dansmithoh, this isn't targeted at stein? I figured that's why melwitt was asking17:41
sean-k-mooneyit is targeted to stien17:41
melwittwell, sean-k-mooney added it to the etherpad https://etherpad.openstack.org/p/nova-stein-blueprint-spec-freeze near the bottom so I was trying to make sure it got some consideration17:42
sean-k-mooneybut down stream it was not planned to be worked on until next cycle17:42
sean-k-mooneyi just taugh ti would have time to do it since it is small17:42
dansmithsean-k-mooney: okay I don't see that in lo17:42
dansmith*lp17:42
melwittyeah, I think only the nova-drivers group can target it17:43
dansmithsean-k-mooney: are you saying that merely having the device exposed incurs the perf penalty?17:43
sean-k-mooneyso ya ill write a spec a and propose in train when the window reopens17:43
sean-k-mooneyyes17:43
dansmithif so, that sucks, but also I don't really like having the tristate where =none could mean anything17:43
dansmithIMHO, =none should mean =false, although I'm sure you're going to say that it may or may not be enabled today by default,17:44
sean-k-mooneythe over all latency cost of virtualiation was a minium latency hit of 60 micro second this redused it to 50 micoro second when turning it off17:44
dansmithbut regardless, seems better to be more deterministic to me17:44
sean-k-mooneylibvirt automaticaly tunes it on of you use cpu_mode=host-passthorugh and turns it off otherwise17:44
sean-k-mooneyso i was just tryign to keep that behavior17:45
dansmithsounds confusing to me17:45
dansmithlibvirt or our libvirt driver?17:45
sean-k-mooneywell if the pmu is not present things like the linux perf tool dont work fully so i dont want disable by defualt17:45
sean-k-mooneylibvirt well qemu17:46
dansmithokay17:46
sean-k-mooneyif no element is defiend qemu make the decion based on if cpu=host or some other value17:46
sean-k-mooneybasically my plan was if you dont set the extra spec we would not set the libivrt xml value which is what we do today and if you do set it we set the element to what you set explictly17:48
dansmithyeah, understand17:49
dansmiththere's some benefit to that for sure, it just sucks to have that be ill-defined I think17:50
dansmithbut definitely needs to be documented17:50
sean-k-mooneyyep so do we want to leave this to train or woudl it be suffient to document it in the singel patch that impements the feature17:51
*** moshele has joined #openstack-nova17:51
*** panda is now known as panda|off17:53
*** janki has quit IRC17:54
*** janki has joined #openstack-nova17:54
*** igordc has joined #openstack-nova18:00
*** derekh has quit IRC18:00
*** TxGirlGeek has quit IRC18:02
*** TxGirlGeek has joined #openstack-nova18:03
*** nicolasbock has joined #openstack-nova18:03
*** cfriesen has quit IRC18:08
*** moshele has quit IRC18:13
*** sridharg has quit IRC18:15
nicolasbockHi. Is it possible to PXE boot a Nova instance? Looking at https://review.openstack.org/#/c/133254/ this possibility was at least considered at some point, but abandoned in the Kilo cycle. The current Nova master doesn't seem to support PXE booting (or specifying a network device as boot device) if I understand the code correctly. But I wanted to make sure that I am not missing something.18:18
sean-k-mooneynicolasbock: i belive you have to enable the boot menu via the image metadata and then hit f118:19
sean-k-mooneyin the vnc console to select pxe booting18:19
*** mvkr has quit IRC18:19
nicolasbockAh ok18:19
nicolasbockSo there is still a manual step involved18:19
sean-k-mooneyya i dont think you can set it to auto pxe boot but i may be wronge18:20
nicolasbockIn other words, Nova can not configure the libvirt domain to try booting from net.18:20
nicolasbockOk18:20
nicolasbockThanks for the info sean-k-mooney !18:20
*** janki has quit IRC18:20
sean-k-mooneyperhaps someone else can answer the netboot question but not that i am aware of18:21
*** ralonsoh has quit IRC18:22
*** luksky has joined #openstack-nova18:24
*** brault has quit IRC18:25
openstackgerritsean mooney proposed openstack/nova-specs master: Add spec for sriov live migration  https://review.openstack.org/60511618:26
sean-k-mooneyjaypipes: artom stephenfin bauzas ^ nits addressed18:26
*** brault has joined #openstack-nova18:27
*** brault has quit IRC18:31
openstackgerritsean mooney proposed openstack/nova-specs master: Add spec for sriov live migration  https://review.openstack.org/60511618:36
openstackgerritsean mooney proposed openstack/nova-specs master: Add spec for sriov live migration  https://review.openstack.org/60511618:37
sean-k-mooneyok i had a tab and a repated "as" which i also just fixed that really should be done now18:39
*** moshele has joined #openstack-nova18:39
sean-k-mooneyhttps://review.openstack.org/#/c/605116/10..13/specs/stein/approved/libvirt-neutron-sriov-livemigration.rst will show you the delta from the previous version ye reviewed18:40
*** wolverineav has joined #openstack-nova18:48
*** wolverineav has quit IRC18:48
*** wolverineav has joined #openstack-nova18:48
*** moshele has quit IRC18:58
jaypipessean-k-mooney: +2 from me. stephenfin can send it on its merry way in a bit.18:59
sean-k-mooneyjaypipes: well he could if he was a specs core but he is not19:00
jaypipesah, then melwitt can :)19:00
sean-k-mooneyjaypipes: thank you however for your +219:00
sean-k-mooneynope neither is she19:00
jaypipeswha?!19:00
sean-k-mooneyjaypipes: dansmith: and bauzas  can19:01
jaypipesok, bauzas it is then, since he's reviewed that already :)19:01
melwittyeah, I think you could +W it yourself though with a note that bauzas was previously +2 before nit fixing19:01
sean-k-mooneyjaypipes: nova-specs-core is a very select group https://review.openstack.org/#/admin/groups/302,members :)19:01
melwittin case bauzas is gone for the day19:02
*** moshele has joined #openstack-nova19:02
jaypipesmelwitt: k, I will give bauzas a little while and will +W it along after waiting a bit.19:04
melwittk, sounds good19:04
*** brault has joined #openstack-nova19:10
*** brault has quit IRC19:14
*** mvkr has joined #openstack-nova19:21
*** cfriesen has joined #openstack-nova19:24
*** brault has joined #openstack-nova19:31
*** brault has quit IRC19:36
*** cdent has quit IRC19:38
*** moshele has quit IRC19:50
*** READ10 has joined #openstack-nova20:00
*** hongbin has quit IRC20:24
*** erlon has quit IRC20:32
*** jmlowe has quit IRC20:44
*** READ10 has quit IRC20:47
*** openstackgerrit has quit IRC20:50
*** hongbin has joined #openstack-nova20:56
*** mlavalle has joined #openstack-nova21:00
*** whoami-rajat has quit IRC21:01
*** jmlowe has joined #openstack-nova21:07
*** moshele has joined #openstack-nova21:38
artomBooting an instance with cpu_policy=dedicated (and no other NUMA-y stuff) and 1GB of RAM in the flavor results in a rather bizarre libvirt message21:41
artomtotal memory for NUMA nodes (0x100000) should equal RAM size (0x200000)21:41
*** moshele has quit IRC21:42
artomEven if <memory unit='KiB'>1024</memory> and <numa> <cell id='0' cpus='0' memory='1024' unit='KiB'/> </numa>21:42
artomSame thing but with 2GB is fine.21:42
artomEither I'm missing something way obvious, or there's a bug :/21:43
*** pcaruana has quit IRC21:46
artomsean-k-mooney ^^ if your weird sleep schedule makes you still be here. You always know about these things ^_^21:47
sean-k-mooneyartom i wasnt but i am now22:05
sean-k-mooneyartom: adding hw:cpu_policy=dedicated create an implict numa toplogy as if you also set hw:numa_nodes=122:06
sean-k-mooneythat message however is strange22:08
*** openstackgerrit has joined #openstack-nova22:10
openstackgerritInbar Stolberg proposed openstack/nova master: Fixes server rebuild if NumaTopologyFilter is used with CPU pinning  https://review.openstack.org/62964622:10
artomsean-k-mooney, right, I knew about the implicit NUMA topology22:13
artomBut... Nova's XML is correct, as far as I can tell, units and everything22:13
sean-k-mooneyso i cant find that error upstream22:13
sean-k-mooneywhat raised it22:13
artomsean-k-mooney, it's from libvirt itsel22:13
artomFor all I know it's a bug in qemu/kvm22:13
*** awaugama has quit IRC22:23
*** brault has joined #openstack-nova22:34
*** hongbin has quit IRC22:36
*** brault has quit IRC22:38
*** rcernin has joined #openstack-nova22:38
*** itlinux has quit IRC22:55
*** etp has quit IRC22:57
*** etp has joined #openstack-nova22:59
*** gyee has quit IRC23:11
sean-k-mooneyjaypipes: im just about to log off for the night. looks like bauzas didnt come back online. are you still ok to +w https://review.openstack.org/#/c/605116/?23:12
*** tbachman has quit IRC23:13
jaypipessean-k-mooney: yup23:13
sean-k-mooneythanks :)23:13
jaypipessean-k-mooney: done23:14
*** etp has quit IRC23:14
sean-k-mooneynight all o/23:15
*** tbachman has joined #openstack-nova23:15
*** etp has joined #openstack-nova23:16
openstackgerritMerged openstack/nova-specs master: Add spec for sriov live migration  https://review.openstack.org/60511623:19
*** edmondsw has quit IRC23:23
*** edleafe- has joined #openstack-nova23:23
*** edleafe has quit IRC23:24
*** edleafe- is now known as edleafe23:24
*** priteau has joined #openstack-nova23:27
*** priteau has quit IRC23:32
*** erlon has joined #openstack-nova23:33
*** _alastor_ has quit IRC23:34
*** luksky has quit IRC23:36
*** edmondsw has joined #openstack-nova23:39
*** brault has joined #openstack-nova23:39
*** brault has quit IRC23:43
*** macza_ has quit IRC23:57

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