Monday, 2020-03-30

*** tetsuro has joined #openstack-nova00:25
*** tetsuro_ has joined #openstack-nova00:28
*** tetsuro has quit IRC00:31
*** brinzhang has joined #openstack-nova00:45
*** yingjun has joined #openstack-nova01:05
*** yingjun has quit IRC01:05
*** ociuhandu has joined #openstack-nova01:06
*** ociuhandu has quit IRC01:09
*** ociuhandu has joined #openstack-nova01:10
*** ociuhandu has quit IRC01:20
*** ociuhandu has joined #openstack-nova01:21
*** ociuhandu has quit IRC01:25
*** Liang__ has joined #openstack-nova01:29
*** brinzhang_ has joined #openstack-nova01:31
*** zhanglong has joined #openstack-nova01:32
*** brinzhang has quit IRC01:35
*** zhanglong has quit IRC01:36
*** zhanglong has joined #openstack-nova01:38
*** sapd__x has quit IRC01:41
*** Liang__ is now known as LiangFang01:45
*** larainema has joined #openstack-nova01:57
*** sapd__x has joined #openstack-nova02:12
openstackgerritMerged openstack/nova master: Expose instance action event details out of the API  https://review.opendev.org/69443002:24
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in limits policies  https://review.opendev.org/71576002:24
*** sapd1 has joined #openstack-nova02:26
*** ociuhandu has joined #openstack-nova02:34
openstackgerritGhanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy  https://review.opendev.org/71576102:36
*** ociuhandu has quit IRC02:39
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in limits policies  https://review.opendev.org/71576002:48
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in limits policies  https://review.opendev.org/71576002:50
openstackgerritGhanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy  https://review.opendev.org/71576102:50
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in os-agents policies  https://review.opendev.org/70164802:59
openstackgerritGhanshyam Mann proposed openstack/nova master: Pass the actual target in os-agents policy  https://review.opendev.org/70164903:00
openstackgerritBrin Zhang proposed openstack/nova master: Add instance actions v284 samples test  https://review.opendev.org/70625103:02
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in limits policies  https://review.opendev.org/71576003:07
openstackgerritGhanshyam Mann proposed openstack/nova master: Correct limits policy check_str  https://review.opendev.org/71567203:07
openstackgerritGhanshyam Mann proposed openstack/nova master: Combine the limits policies in single place  https://review.opendev.org/71567803:07
openstackgerritGhanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies  https://review.opendev.org/71567403:08
openstackgerritGhanshyam Mann proposed openstack/nova master: Introduce scope_types in limits policy  https://review.opendev.org/71568003:08
*** psachin has joined #openstack-nova03:09
*** vishalmanchanda has joined #openstack-nova03:24
*** mkrai has joined #openstack-nova03:27
*** zhanglong has quit IRC03:28
*** zhanglong has joined #openstack-nova03:28
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in limits policies  https://review.opendev.org/71576003:32
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in limits policies  https://review.opendev.org/71576003:32
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in limits policies  https://review.opendev.org/71576003:33
openstackgerritGhanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy  https://review.opendev.org/71576103:33
openstackgerritKevin Zhao proposed openstack/nova master: libvirt: change UEFI check to handle AArch64 better  https://review.opendev.org/71431103:39
*** evrardjp has quit IRC04:03
*** evrardjp has joined #openstack-nova04:10
*** ociuhandu has joined #openstack-nova04:14
*** ociuhandu has quit IRC04:24
*** ociuhandu has joined #openstack-nova04:24
*** zhanglong has quit IRC04:27
*** zhanglong has joined #openstack-nova04:28
*** ociuhandu has quit IRC04:30
*** evrardjp has quit IRC04:36
*** evrardjp has joined #openstack-nova04:36
*** ratailor has joined #openstack-nova05:16
*** ociuhandu has joined #openstack-nova05:24
*** links has joined #openstack-nova05:26
*** udesale has joined #openstack-nova05:28
*** ociuhandu has quit IRC05:34
*** ociuhandu has joined #openstack-nova05:35
*** ociuhandu has quit IRC05:40
*** ociuhandu has joined #openstack-nova05:44
*** ociuhandu has quit IRC05:54
*** ociuhandu has joined #openstack-nova05:56
*** ociuhandu has quit IRC06:00
*** udesale_ has joined #openstack-nova06:23
*** dpawlik has joined #openstack-nova06:23
*** udesale has quit IRC06:25
*** udesale_ has quit IRC06:27
*** xek has joined #openstack-nova06:29
*** zhanglong has quit IRC06:30
*** xek_ has joined #openstack-nova06:33
*** xek has quit IRC06:33
*** zhanglong has joined #openstack-nova06:34
*** ociuhandu has joined #openstack-nova06:35
*** ircuser-1 has quit IRC06:40
*** ociuhandu has quit IRC06:40
openstackgerritKevin Zhao proposed openstack/nova master: [DNM] Run tempest jobs on Arm64 needed fix  https://review.opendev.org/71456406:42
*** xek_ has quit IRC06:49
*** xek_ has joined #openstack-nova06:50
*** ociuhandu has joined #openstack-nova07:05
*** ociuhandu has quit IRC07:09
*** tesseract has joined #openstack-nova07:15
*** ociuhandu has joined #openstack-nova07:19
openstackgerritSylvain Bauza proposed openstack/nova master: FUP for Add a placement audit command  https://review.opendev.org/71579707:22
*** maciejjozefczyk has joined #openstack-nova07:22
bauzasstephenfin: morning, when you're up, I just provided a FUP based on your comments for https://review.opendev.org/#/c/670112 : https://review.opendev.org/#/c/715797/107:23
bauzasI'd appreciate if you could review those soon07:23
*** zhanglong has quit IRC07:24
*** tosky has joined #openstack-nova07:25
*** ociuhandu has quit IRC07:25
*** zhanglong has joined #openstack-nova07:32
*** rpittau|afk is now known as rpittau07:34
*** arxcruz|off is now known as arxcruz07:35
*** ociuhandu has joined #openstack-nova07:52
*** ralonsoh has joined #openstack-nova07:53
*** ociuhandu has quit IRC07:57
*** ociuhandu has joined #openstack-nova08:01
*** jangutter has joined #openstack-nova08:12
*** zhanglong has quit IRC08:13
*** zhanglong has joined #openstack-nova08:19
*** tkajinam has quit IRC08:30
*** derekh has joined #openstack-nova08:33
*** dtantsur|afk is now known as dtantsur08:34
*** nightmare_unreal has joined #openstack-nova08:37
*** udesale has joined #openstack-nova08:38
nightmare_unrealhey like we have assertFlavorMatchesAllocation() do we have opposite of it ?08:41
nightmare_unreallike where assertflavor not match08:41
*** martinkennelly has joined #openstack-nova08:46
openstackgerritJohn Garbutt proposed openstack/nova master: Add stub unified limits driver  https://review.opendev.org/71213708:50
*** rcernin has quit IRC09:00
*** ociuhandu has quit IRC09:03
*** ociuhandu has joined #openstack-nova09:09
*** ociuhandu has quit IRC09:13
*** LiangFang has quit IRC09:29
luyaostephenfin: Hi, your comments are addressed. :)  https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/support-live-migration-with-virtual-persistent-memory09:32
*** sapd__x has quit IRC09:34
*** tkajinam has joined #openstack-nova09:41
*** sapd1_x has joined #openstack-nova09:45
openstackgerritjayaditya gupta proposed openstack/nova master: Support for --overwrite flag for nova-manage placement heal_allocations command Closes-Bug:#1868997  https://review.opendev.org/71539509:56
*** zhanglong has quit IRC09:56
*** zhanglong has joined #openstack-nova10:02
*** tetsuro_ has quit IRC10:04
*** mkrai has quit IRC10:14
openstackgerritSylvain Bauza proposed openstack/nova master: FUP for Add a placement audit command  https://review.opendev.org/71579710:18
bauzasstephenfin: thanks for noting the pep8 issue. fixed ^10:18
openstackgerritAndreas Jaeger proposed openstack/nova-specs master: Update hacking for Python3  https://review.opendev.org/71565010:20
*** ociuhandu has joined #openstack-nova10:21
openstackgerritAndreas Jaeger proposed openstack/os-vif master: Update hacking for Python3  https://review.opendev.org/71565110:21
*** mkrai has joined #openstack-nova10:27
gibibauzas: hi! Does https://review.opendev.org/#/c/715797/ solves the bug 1793569 or we need other things still?10:32
openstackbug 1793569 in OpenStack Compute (nova) "Add placement audit commands" [Wishlist,In progress] https://launchpad.net/bugs/1793569 - Assigned to Sylvain Bauza (sylvain-bauza)10:32
bauzasgibi: just a FUP10:33
bauzasgibi: the one that fixes the issue is https://review.opendev.org/#/c/670112/1410:33
bauzashence the Related-Bug10:33
gibibauzas: bah, you are right. my bad10:33
bauzasjust in case people want to backport the fixes10:33
bauzasthey could merge both10:33
gibiI just looked at the bug and saw that it is still open then found the FUP10:34
gibibut now I see that the main patch is still on the gate10:34
bauzasnp10:35
*** mkrai has quit IRC10:36
*** mkrai_ has joined #openstack-nova10:36
openstackgerritGuo Jingyu proposed openstack/nova master: WIP: nova supports password encrypted VNC  https://review.opendev.org/62233610:37
openstackgerritGuo Jingyu proposed openstack/nova master: WIP: nova supports password encrypted VNC  https://review.opendev.org/62233610:42
*** zhanglong has quit IRC10:45
*** yaawang has joined #openstack-nova10:53
*** rpittau is now known as rpittau|bbl10:56
*** jangutter has quit IRC11:14
*** dklyle has quit IRC11:15
*** jangutter has joined #openstack-nova11:16
*** jangutter has quit IRC11:20
stephenfinbauzas: done11:30
*** artom has joined #openstack-nova11:38
gibigmann: I have a question in https://review.opendev.org/#/c/715071/211:47
*** mkrai_ has quit IRC11:51
*** shyamb has joined #openstack-nova11:55
openstackgerritMerged openstack/nova master: Add a placement audit command  https://review.opendev.org/67011212:01
openstackgerritMerged openstack/nova master: Add instance actions v284 samples test  https://review.opendev.org/70625112:01
openstackgerritMerged openstack/nova master: Add test coverage of existing instance usage log policies  https://review.opendev.org/71508012:01
*** lpetrut has joined #openstack-nova12:02
*** bbowen has joined #openstack-nova12:05
*** macz_ has joined #openstack-nova12:09
*** macz_ has quit IRC12:10
*** macz_ has joined #openstack-nova12:11
*** jangutter has joined #openstack-nova12:12
*** shyamb has quit IRC12:16
bauzasstephenfin: ta12:37
*** mkrai_ has joined #openstack-nova12:43
*** rpittau|bbl is now known as rpittau12:46
*** ociuhandu has quit IRC12:48
*** nweinber has joined #openstack-nova12:48
*** redrobot has quit IRC12:55
*** Guest43440 has joined #openstack-nova12:56
*** Guest43440 is now known as redrobot12:58
*** ratailor has quit IRC13:00
*** ociuhandu has joined #openstack-nova13:00
*** Luzi has joined #openstack-nova13:02
*** ociuhandu has quit IRC13:05
*** ociuhandu has joined #openstack-nova13:10
*** nweinber_ has joined #openstack-nova13:12
*** mriedem has joined #openstack-nova13:13
*** irclogbot_0 has quit IRC13:13
*** f0o has quit IRC13:15
*** nweinber has quit IRC13:15
*** tinwood has quit IRC13:15
*** amodi has joined #openstack-nova13:16
*** tinwood has joined #openstack-nova13:17
*** f0o has joined #openstack-nova13:18
*** irclogbot_3 has joined #openstack-nova13:18
*** mkrai_ has quit IRC13:21
openstackgerritMerged openstack/nova master: Introduce scope_types in os-instance-usage-audit-log  https://review.opendev.org/71508213:22
*** Luzi has quit IRC13:26
*** ociuhandu has quit IRC13:27
*** mgariepy has quit IRC13:30
openstackgerritMatt Riedemann proposed openstack/nova master: Use scatter/gather in HostAPI.compute_node_get  https://review.opendev.org/68644213:33
gmanngibi: checking13:42
gibigmann: hu13:42
gibihi13:42
gibiI might miss some stlying thing in those policy descriptions13:42
*** mgariepy has joined #openstack-nova13:43
gmanngibi: let me check generated doc.13:44
gmanngibi: it is generated multiline  - https://4ea5dff2d36d88968673-539e9689cff44bf991ce02b5200fbfd6.ssl.cf5.rackcdn.com/715071/2/check/openstack-tox-docs/fcd5a65/docs/configuration/policy.html13:48
gmanngibi: let me fix that. thanks for catching that13:48
dansmithbrinzhang_: are you working on the changes to that set or should I?13:48
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in os-hypervisors policies  https://review.opendev.org/71507113:50
gibigmann: thanks. fyi, there is more than one such wrap in the hypervisor patch13:50
gmanngibi: ^^ did but i need to rebase it for test_policy.13:51
gibigmann: ack13:51
*** xek has joined #openstack-nova13:56
*** lseki has joined #openstack-nova13:57
*** xek_ has quit IRC13:57
*** hrw has joined #openstack-nova14:00
hrwmorning14:00
hrwwhy openstack cli has to be so weird14:02
hrw'nova volume-attach' but no 'openstack volume attach'. 'openstack volume list' but no 'nova volume-list'14:03
hrwargh14:03
*** lpetrut has quit IRC14:13
dansmithgibi: 2.84 got taken since friday, so I have to rebase and re-version that set again14:14
dansmithgibi: I think it'd be good to work on lining those up to avoid that churn14:14
*** lpetrut has joined #openstack-nova14:15
*** beekneemech is now known as bnemec14:15
gibidansmith: ack. I won't start rebasing other's patches top of each other as I don't know which patch should be the bottom.14:16
dansmithgibi: even if we just identify all the people competing for numbers and order them so we don't keep rebasing and re-versioning until we're closer would be helpful14:17
*** francoisp has joined #openstack-nova14:17
gibiI see: https://review.opendev.org/#/c/708436/ and https://review.opendev.org/#/c/701430/ and https://review.opendev.org/#/c/693828/ as API changes. But lyarwood's patch is still on 2.8314:20
gibidansmith:  as far as I see stephenfin's https://review.opendev.org/#/c/708436/ patch is up to date and I'm +2 of if some other can can review then that can be the next14:20
gibinvm14:21
gibi2.85 is the next14:21
dansmithyeah, that's out of date now14:21
gibiso none of the current API patches is up to date14:21
dansmithand I just started on 2.8514:21
*** TxGirlGeek has joined #openstack-nova14:21
dansmithargh14:21
gibistephenfin, lyarwood : do you plan to update your patches to 2.85 soon?14:22
dansmithgibi: should I keep reversioning for 2.85 or do you want one of those to go first?14:22
stephenfingibi: Yeah, I'm respinning it at the moment14:22
stephenfinlyarwood is out 'til Friday though so I guess his will be a while14:23
gibistephenfin: ack, then you and dansmith are up for the 2.8514:23
stephenfinhow ready is your patch, dansmith?14:23
gibidansmith: stephenfin would need a second core (besides me) to merge on 2.8514:24
dansmithstephenfin: slightly less than yours I imagine, since it was already +2, but it's fairly close I think14:24
stephenfinokay, I'll push this now so14:27
openstackgerritStephen Finucane proposed openstack/nova master: api: Add framework for extra spec validation  https://review.opendev.org/70464314:27
openstackgerritStephen Finucane proposed openstack/nova master: api: Add microversion 2.84, extra spec validation  https://review.opendev.org/70843614:27
openstackgerritStephen Finucane proposed openstack/nova master: docs: Add documentation for flavor extra specs  https://review.opendev.org/71003714:27
stephenfinand go review that PUT volume attachments patch14:27
gibistephenfin: you have nothing to loose if you already made the change to 2.8514:28
stephenfingibi: Yup, fair. That second patch needs your +2 _yet again_ so :(14:29
gibiI have to drop offline untill 16:00 UTC but I will be back14:29
*** ociuhandu has joined #openstack-nova14:34
*** mkrai has joined #openstack-nova14:35
*** dklyle has joined #openstack-nova14:40
gmanndansmith: replied on volumeId as optional comment- https://review.opendev.org/#/c/693828/20/nova/api/openstack/compute/volumes.py@44714:46
*** klippo has left #openstack-nova14:46
gmanndansmith: what if request is for both swap as well as update delete flag ?14:46
openstackgerritStephen Finucane proposed openstack/nova master: Add new default roles in os-instance-usage-audit-log policies  https://review.opendev.org/71508514:46
openstackgerritStephen Finucane proposed openstack/nova master: Pass the actual target in os-instance-usage-audit-log policy  https://review.opendev.org/71508914:46
gmanni think that is brinzhang_ question also ? but not sur14:46
gmannsure14:46
dansmithgmann: okay got it14:49
*** ociuhandu has quit IRC14:52
*** hrw has left #openstack-nova14:53
*** iurygregory has quit IRC14:54
*** iurygregory has joined #openstack-nova14:56
*** links has quit IRC14:56
*** yoctozepto has quit IRC15:07
*** yoctozepto has joined #openstack-nova15:08
*** toabctl has quit IRC15:08
*** ab-a has quit IRC15:08
*** arxcruz has quit IRC15:09
*** obre has quit IRC15:09
*** damien_r has quit IRC15:10
*** aarents has quit IRC15:10
*** arxcruz has joined #openstack-nova15:10
*** ab-a has joined #openstack-nova15:11
*** cmurphy has quit IRC15:13
dansmithartom: around?15:14
*** hrw has joined #openstack-nova15:18
artomdansmith, sup15:19
openstackgerritMarcin Juszkiewicz proposed openstack/nova master: Feature matrix: update AArch64 information  https://review.opendev.org/71597915:19
hrwfew new entries. mostly volumes related15:20
dansmithartom: if I wanted to support changing a volume attachment tag, I'd need to make a call to the compute node? I haven't looked at all, but what is the procedure when volume attach specifies a tag?15:20
dansmithartom: could I reasonably just change it on the BDM and expect the virt driver to pick it up on the next reboot or something?15:20
artomdansmith, I believe just saving the new tag to the DB is enough15:20
dansmithoh?15:21
artomLemme refresh my memory on how we actually build metadata15:21
openstackgerritGhanshyam Mann proposed openstack/nova master: Add test coverage of existing hypervisors policies  https://review.opendev.org/71502915:22
openstackgerritGhanshyam Mann proposed openstack/nova master: Introduce scope_types in os-hypervisors  https://review.opendev.org/71503615:22
*** cmurphy has joined #openstack-nova15:22
openstackgerritMarcin Juszkiewicz proposed openstack/nova master: Feature matrix: update AArch64 information  https://review.opendev.org/71597915:23
hrwmissed 'nova boot'15:23
artomdansmith, ah, no, we have to call down to the virt driver to rebuild the instance metadata15:24
dansmithokay, that's what I thought15:25
*** ociuhandu has joined #openstack-nova15:26
*** ab-a has quit IRC15:26
gmannhrw: firs example is from python-novaclient which is python client only for nova  (from starting) and has cmd like volume-attach etc. every project has their separate python client.  Now we have new unified CLI for the whole openstack which is openstackclient and making all project CLI in standard way for example 'openstack volume attach'.15:29
*** ab-a has joined #openstack-nova15:30
gmannnot all project has finished the new client migration so both are maintained  for now.15:30
*** mkrai has quit IRC15:32
*** jgwentworth is now known as melwitt15:33
*** ociuhandu has quit IRC15:34
*** iurygregory has quit IRC15:35
*** ociuhandu has joined #openstack-nova15:35
*** iurygregory has joined #openstack-nova15:35
*** nightmare_unreal has quit IRC15:36
hrwgmann: I am aware of migration. it started in ocata times or earlier iirc ;D15:38
gmannyeah, it still going on :). hope to make it commnity goal for V cycle if it happens. it is second try15:39
*** andreykurilin has quit IRC15:39
*** obre has joined #openstack-nova15:39
*** mkrai has joined #openstack-nova15:40
hrwgmann: I like building openstack, solving some aarch64 architecture related issues. but I really dislike using it. two ways of using cli, commands having missing subcommands, missing sane features...15:40
hrwgmann: 'openstack hypervisor list' but no 'hypervisor up' - have to find 'service somethingsomething --up'15:41
hrw'openstack volumes list' but 'nova volume-attach'15:41
hrwand not found yet 'openstack create image --from-url'15:42
*** ab-a has quit IRC15:45
*** rajinir has quit IRC15:45
*** bcm has quit IRC15:45
*** donnyd has quit IRC15:45
*** cz3 has quit IRC15:45
*** ttx has quit IRC15:45
*** vdrok has quit IRC15:45
*** masterpe has quit IRC15:45
*** rpittau has quit IRC15:45
*** Anticimex has quit IRC15:45
gmannits openstack image create.15:45
*** ab-a has joined #openstack-nova15:45
*** masterpe has joined #openstack-nova15:45
*** rajinir has joined #openstack-nova15:45
*** bcm has joined #openstack-nova15:45
*** donnyd has joined #openstack-nova15:45
*** cz3 has joined #openstack-nova15:45
*** ttx has joined #openstack-nova15:45
*** vdrok has joined #openstack-nova15:45
*** rpittau has joined #openstack-nova15:45
*** Anticimex has joined #openstack-nova15:45
*** andreykurilin has joined #openstack-nova15:46
gmannbut yeah, if you compare naming convention for both its different.15:46
*** damien_r has joined #openstack-nova15:47
gmanni do not know which one is best though i hardly use CLI:)15:47
hrwgmann: 'openstack image create' allows to create image from file.15:47
hrwwhich sucks when you fetch 1GB image from US to pipe it through thin pipe back to US15:48
*** masterpe has quit IRC15:48
hrwgmann: I prefer to use cli rather than horizon ;D15:48
*** aarents has joined #openstack-nova15:49
hrwasymetric home networks suck15:49
*** damien_r has quit IRC15:52
*** damien_r has joined #openstack-nova15:54
donnydglance used to be able to be told to go download an image for you. not sure if this is still a thing it can do15:58
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in os-hypervisors policies  https://review.opendev.org/71600315:58
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in os-hypervisors policies  https://review.opendev.org/71507115:59
openstackgerritGhanshyam Mann proposed openstack/nova master: Pass the actual target in os-hypervisors policy  https://review.opendev.org/71507416:00
* hrw out16:01
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in os-agents policies  https://review.opendev.org/70164816:02
openstackgerritGhanshyam Mann proposed openstack/nova master: Pass the actual target in os-agents policy  https://review.opendev.org/70164916:03
openstackgerritGhanshyam Mann proposed openstack/nova master: Add new default roles in limits policies  https://review.opendev.org/71576016:10
openstackgerritGhanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy  https://review.opendev.org/71576116:10
*** rpittau is now known as rpittau|afk16:12
*** mkrai has quit IRC16:15
huaqiangstephenfin: I saw your comments on the WIP 'pcpuset alternative' patch,16:21
huaqiangNow I dropped the 'cpu_pinning' approach. I am still working on the them, and I'll send out all patches16:22
huaqiangI'll send out all of them tomorrow16:22
stephenfinPerfect (y)16:22
*** kevinz has quit IRC16:22
*** tesseract has quit IRC16:28
*** udesale has quit IRC16:29
*** evrardjp has quit IRC16:36
*** evrardjp has joined #openstack-nova16:36
*** Anticimex has quit IRC16:43
stephenfingibi: Do we need to worry about notification responses changing? Are those versioned? https://review.opendev.org/#/c/715362/1/doc/notification_samples/instance-live_migration_rollback_dest-start.json16:43
*** Anticimex has joined #openstack-nova16:44
stephenfin(it's changing because of https://review.opendev.org/#/c/715362/1/nova/compute/manager.py@a8644)16:44
*** prometheanfire has quit IRC16:45
*** masterpe has joined #openstack-nova16:47
*** psachin has quit IRC16:53
gibistephenfin: is it just the content but not the structure? (I'm still reading that patch)17:01
*** xek has quit IRC17:02
*** xek has joined #openstack-nova17:03
*** dtantsur is now known as dtantsur|afk17:06
*** derekh has quit IRC17:09
*** edleafe has joined #openstack-nova17:10
*** edleafe has left #openstack-nova17:11
gibistephenfin: replied in https://review.opendev.org/#/c/715362 No worries about the notificatins. The format are versioned sure but the task_state value change is not part of the contract, just the structure17:14
gibistephenfin: +2 on the flavor extra spec verification again.17:18
gibistephenfin: but the doc patch on top failed the functional test17:19
gibistephenfin: and that feels relevant17:19
gibihttps://07b92d57b5f1a612a74b-4cebd80e32fd7e7586df4103c9478cce.ssl.cf5.rackcdn.com/710037/6/check/nova-tox-functional-py36/c7d4b1e/testr_results.html17:19
gibiI just don't know how17:21
*** Sundar has joined #openstack-nova17:21
Sundardansmith, gibi: Can one of you give W+1 for the patches in https://review.opendev.org/#/q/project:openstack/nova+bp/nova-cyborg-interaction+status:open ? Thanks17:22
dansmithSundar: I haven't reviewed those to +2 status yet and I'm working on something else right now17:23
dansmithbut it's on my list of course17:23
gibistephenfin: found it, left a -1 with details17:23
*** prometheanfire has joined #openstack-nova17:23
*** ociuhandu has quit IRC17:24
gibiSundar: I'm +2 all the way, so we have to wait for dansmith or others to +217:24
gibidansmith: can I do something for you before I finish for today?17:25
dansmithgibi: I'm just working on all the changes to that d-o-t series.. should I rebase against stephenfin's 2.85 patch as 2.86? If we're not going to strictly order them then I'll wait until that one merges17:26
dansmithgibi: I think an etherpad with an intended line of succession would be appropriate at this point that cores try to hold to17:26
gmanngibi: updated the policy description  for this - https://review.opendev.org/#/c/715071/417:27
gibidansmith: thanks. If I know who will be the second for for stephenfin's series then I would put that first in the order as that is ready17:28
gibilately efried reviewed stephenfin's patch besides me so we are missing a core there17:29
efriedsomething I can look at quickly gibi?17:29
dansmithokay, not sure what you're saying, so I will just push this up at 2.85 since I had already started tat re-version, and will hope to have some coordination17:30
gibiefried: stephenfin's extra spec validation series is mostly ready (there are a small test issue due to microversion bump)17:30
gibiefried: but I totally understand if that is a big piece to chew17:30
efriedoh, yeah, I never gave that a thorough review in the first place, just had one isolated comment on something I noticed out of context.17:30
gibidansmith: even if I put stephen first, we dont have a second core for that series17:30
dansmithI was assuming he was first17:31
gibidansmith: will you review stephenfin's series then?17:31
dansmithgibi: I have zero context on it, and need to finish this one, finish the cyborg, etc17:32
dansmithso, maybe, but it won't be soon17:32
gibidansmith: totally understand this is why I did want to put thing in order as I don't know who will +A stephenfin series and without that it would block the whole queue17:32
gibis/ did want/did not want/17:32
dansmithwell, putting them in order helps communicate importance, but it also just helps the rest of the queue burning time re-versioning a lot17:33
dansmithi.e. time they can spend reviewing17:33
dansmithsurely melwitt could review stephenfin's series?17:33
gibiif somebody signs up for it then I'm OK to put stephenfin's first.17:34
gibibut I feel we are in cycles17:34
dansmithwell, we had a bunch of support for the d-o-t series,17:34
dansmithand it's already at 2.85, so we could put that at the front too17:35
gibidansmith: I can review d-o-t first thing in the morning17:35
dansmithokay, I will have it up before long17:35
gibidansmith: so let's put that first and stephen next, and lyarwood 3rd17:35
dansmithokay17:35
gibidansmith: thanks17:35
gibiand thank you being patient. this situation is obviously new to me and I'm not up to speed yet to coordinate the release.17:36
dansmiththe d-o-t one is also just one patch to claim the microversion17:36
dansmithso anything not version related could be done in a fup to avoid blocking the queue further17:36
gibidansmith: good point17:37
dansmithbut I think the extra specs one is multiple before the microversion right?17:37
gibithe second patch is the microversion in the extra spec, so yes17:37
dansmithtwo, okay17:37
gibiI'm leave for today. See you tomorrow17:38
dansmitho/17:39
gibio/17:39
*** lpetrut has quit IRC17:40
*** Sundar has quit IRC17:42
*** ociuhandu has joined #openstack-nova17:59
*** ralonsoh has quit IRC18:17
*** vishalmanchanda has quit IRC18:18
*** francoisp has quit IRC18:23
*** ociuhandu has quit IRC18:48
*** amodi has quit IRC19:07
*** maciejjozefczyk has quit IRC19:13
openstackgerritGhanshyam Mann proposed openstack/nova master: Add test coverage of existing lock server policies  https://review.opendev.org/71605719:14
*** ociuhandu has joined #openstack-nova19:25
*** ociuhandu has quit IRC19:29
*** luyao has quit IRC19:31
openstackgerritDan Smith proposed openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination  https://review.opendev.org/69382819:34
openstackgerritDan Smith proposed openstack/nova master: Separate update and swap volume policies  https://review.opendev.org/71119419:34
*** amodi has joined #openstack-nova19:35
*** larainema has quit IRC19:41
*** mugsie has quit IRC19:51
*** mugsie has joined #openstack-nova19:54
*** dpawlik has quit IRC19:59
*** martinkennelly has quit IRC20:03
*** dpawlik has joined #openstack-nova20:08
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix unlock server policy to be admin_or_owner  https://review.opendev.org/71607120:08
*** dpawlik has quit IRC20:13
*** macz_ has quit IRC20:14
openstackgerritMerged openstack/nova master: Add new default roles in os-instance-usage-audit-log policies  https://review.opendev.org/71508520:20
openstackgerritMerged openstack/nova master: Pass the actual target in os-instance-usage-audit-log policy  https://review.opendev.org/71508920:20
*** macz_ has joined #openstack-nova20:27
openstackgerritVictor Coutellier proposed openstack/python-novaclient master: Microversion 2.83 - Add more filters for the nova list command  https://review.opendev.org/71308920:35
*** xek has quit IRC20:35
*** mlavalle has joined #openstack-nova21:05
*** mriedem has left #openstack-nova21:40
*** nweinber_ has quit IRC21:45
*** jangutter has quit IRC21:46
*** rcernin has joined #openstack-nova21:56
*** slaweq has quit IRC22:08
*** tosky has quit IRC22:20
openstackgerritGhanshyam Mann proposed openstack/nova master: Add test coverage of existing lock server policies  https://review.opendev.org/71605722:36
*** tetsuro has joined #openstack-nova23:15
*** tetsuro has quit IRC23:28
*** ircuser-1 has joined #openstack-nova23:28
gmanndansmith: we are not updating 'tag' in same version ? - https://review.opendev.org/#/c/693828/2123:29
*** tetsuro has joined #openstack-nova23:41
*** tetsuro has quit IRC23:43
*** macz_ has quit IRC23:45
*** alex_xu has joined #openstack-nova23:49
brinzhang_gmann: hi, https://review.opendev.org/#/c/693828/20/nova/api/openstack/compute/volumes.py@44723:53
brinzhang_gmann: dansmith: I mean, the 'volumeId' is optional now, should we support to update the swaping volume's delete_ontermination?23:54
gmannbrinzhang_: it's not optional, it is kept same way it was.  but yes dansmith latest PS does the same swap+update if requested23:55
brinzhang_In the spec, We have discussed that if both volumeId and delete_on_terminnation are specified, it returns 400. But now volumeId is not optional, so do we support changing the delete_on_termination property of the swapping volume?23:55
gmannyeah23:56
brinzhang_yes, that's what I want to say, I will reply in step :)23:56
gmannbasically this tests - https://review.opendev.org/#/c/693828/21/nova/tests/unit/api/openstack/compute/test_volumes.py@122123:57
gmannbrinzhang_: see i read your mind :)23:57
brinzhang_yeah, gmann, thanks23:57
brinzhang_that day I used my phone, typing is not easy, so writing is a bit simple, thank you very much for your explanation.23:58
*** tetsuro has joined #openstack-nova23:58

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