| *** Swami has quit IRC | 00:01 | |
| *** brinzhang has joined #openstack-nova | 00:12 | |
| *** tetsuro has joined #openstack-nova | 00:22 | |
| *** gyee has quit IRC | 00:29 | |
| *** takashin has joined #openstack-nova | 00:29 | |
| *** zhanglong has joined #openstack-nova | 00:41 | |
| alex_xu | cfriesen: the qemu will take care the live migration for vTPM? | 00:48 |
|---|---|---|
| *** TuanDA has joined #openstack-nova | 01:04 | |
| openstackgerrit | fupingxie proposed openstack/nova master: Don't recreate inst_base on source when using rbd backend in resize https://review.openstack.org/605590 | 01:04 |
| zhanglong | alex_xu: what's the give scenario about your issue? to migrate [vtpm vm] to [another host], or to migrate [vtpm vm] to [another non-vtpm vm]? | 01:15 |
| openstackgerrit | fupingxie proposed openstack/nova master: Don't recreate inst_base on source when using rbd backend in resize https://review.openstack.org/605590 | 01:16 |
| *** mrsoul has joined #openstack-nova | 01:30 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: os-xenapi(n-rp): add traits for vgpu n-rp https://review.openstack.org/604269 | 01:32 |
| *** spatel has joined #openstack-nova | 01:34 | |
| alex_xu | zhanglong: migrate vtpm vm to another host | 01:35 |
| alex_xu | zhanglong: live migrate | 01:36 |
| *** k_mouza has quit IRC | 01:38 | |
| *** jdillaman has quit IRC | 01:42 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:43 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 01:45 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041 | 01:45 |
| *** jdillaman has joined #openstack-nova | 01:45 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: os-xenapi(n-rp): add traits for vgpu n-rp https://review.openstack.org/604269 | 01:45 |
| *** k_mouza has joined #openstack-nova | 01:48 | |
| *** k_mouza has quit IRC | 01:51 | |
| *** k_mouza has joined #openstack-nova | 01:51 | |
| *** mhen has quit IRC | 01:55 | |
| *** mhen has joined #openstack-nova | 01:57 | |
| *** k_mouza has quit IRC | 01:58 | |
| *** macza has joined #openstack-nova | 01:58 | |
| *** k_mouza has joined #openstack-nova | 01:58 | |
| *** k_mouza has quit IRC | 02:02 | |
| *** macza has quit IRC | 02:02 | |
| *** k_mouza has joined #openstack-nova | 02:05 | |
| *** k_mouza has quit IRC | 02:10 | |
| zhanglong | alex_xu: OK. So to ensure [source Asymmetric key] only migrate to [trusted destination vTPM], it is requested that [destination vTPM] use AIK signed key. | 02:11 |
| zhanglong | alex_xu: And there are several solutions to solve this issue.see paper <vTPM: Virtualizing the Trusted Platform Module>. | 02:12 |
| alex_xu | zhanglong: are you saying we don't have enough support in kvm to support live migrate with vTPM? | 02:19 |
| alex_xu | zhanglong: btw, i'm revewing this spec https://review.openstack.org/571111, try to figure out the live migration whether is the scope of that proposal | 02:22 |
| zhanglong | alex_xu: KVM? vm is related to QEMU, not KVM. | 02:24 |
| zhanglong | alex_xu: As I know , the process of [live migrate with vTPM] is related to vm manager e.g. nova, and can be decoupling with host. :) | 02:27 |
| alex_xu | zhanglong: yea, sorry, i'm just saying the generic word, i say kvm is mean to all those stuff qemu, kvm. i know qemu is about device emulation, kvm is kernel module... | 02:28 |
| zhanglong | alex_xu: mm :) | 02:29 |
| *** k_mouza has joined #openstack-nova | 02:29 | |
| *** zigo has quit IRC | 02:29 | |
| *** k_mouza has quit IRC | 02:33 | |
| *** k_mouza has joined #openstack-nova | 02:43 | |
| *** hongbin has joined #openstack-nova | 02:45 | |
| *** k_mouza has quit IRC | 02:45 | |
| *** lbragstad has joined #openstack-nova | 02:45 | |
| *** k_mouza has joined #openstack-nova | 02:49 | |
| *** k_mouza has quit IRC | 02:52 | |
| *** psachin has joined #openstack-nova | 02:53 | |
| *** hshiina has joined #openstack-nova | 02:56 | |
| *** Dinesh_Bhor has quit IRC | 02:57 | |
| *** Dinesh_Bhor has joined #openstack-nova | 03:05 | |
| *** dave-mccowan has quit IRC | 03:05 | |
| *** tetsuro has quit IRC | 03:06 | |
| *** tetsuro_ has joined #openstack-nova | 03:06 | |
| *** macza has joined #openstack-nova | 03:18 | |
| *** macza has quit IRC | 03:22 | |
| *** k_mouza has joined #openstack-nova | 03:43 | |
| *** k_mouza has quit IRC | 03:47 | |
| *** k_mouza has joined #openstack-nova | 03:48 | |
| *** yikun has quit IRC | 03:49 | |
| *** k_mouza has quit IRC | 03:52 | |
| *** udesale has joined #openstack-nova | 03:54 | |
| *** k_mouza has joined #openstack-nova | 03:55 | |
| *** Dinesh_Bhor has quit IRC | 03:56 | |
| *** k_mouza has quit IRC | 03:59 | |
| *** spatel has quit IRC | 04:00 | |
| *** k_mouza has joined #openstack-nova | 04:01 | |
| *** hongbin has quit IRC | 04:01 | |
| *** macza has joined #openstack-nova | 04:04 | |
| *** k_mouza has quit IRC | 04:05 | |
| *** smcginnis has quit IRC | 04:06 | |
| *** k_mouza has joined #openstack-nova | 04:07 | |
| *** macza has quit IRC | 04:10 | |
| *** k_mouza has quit IRC | 04:10 | |
| *** macza has joined #openstack-nova | 04:10 | |
| *** macza has quit IRC | 04:15 | |
| *** macza has joined #openstack-nova | 04:16 | |
| *** macza has quit IRC | 04:20 | |
| *** lbragstad has quit IRC | 04:21 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:29 | |
| *** icey has quit IRC | 04:30 | |
| *** breton has quit IRC | 04:30 | |
| *** icey has joined #openstack-nova | 04:30 | |
| *** k_mouza has joined #openstack-nova | 04:52 | |
| *** k_mouza has quit IRC | 04:56 | |
| *** macza has joined #openstack-nova | 04:57 | |
| *** macza has quit IRC | 05:02 | |
| *** k_mouza has joined #openstack-nova | 05:06 | |
| *** toabctl has joined #openstack-nova | 05:07 | |
| *** k_mouza has quit IRC | 05:09 | |
| *** k_mouza has joined #openstack-nova | 05:12 | |
| *** janki has joined #openstack-nova | 05:15 | |
| *** pcaruana has joined #openstack-nova | 05:15 | |
| *** k_mouza has quit IRC | 05:17 | |
| *** k_mouza has joined #openstack-nova | 05:21 | |
| *** k_mouza has quit IRC | 05:24 | |
| *** ratailor has joined #openstack-nova | 05:25 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova master: Add note for sending SIGHUP at rolling upgrade steps https://review.openstack.org/609910 | 05:25 |
| *** k_mouza has joined #openstack-nova | 05:27 | |
| *** k_mouza has quit IRC | 05:31 | |
| *** dr_gogeta86 has quit IRC | 05:32 | |
| *** k_mouza has joined #openstack-nova | 05:44 | |
| cfriesen | alex_xu: not sure if it's libvirt or qemu, but nova doesn't need to worry about it. | 05:45 |
| alex_xu | cfriesen: cool, i got it | 05:46 |
| *** k_mouza has quit IRC | 05:47 | |
| *** k_mouza has joined #openstack-nova | 05:50 | |
| *** k_mouza has quit IRC | 05:53 | |
| openstackgerrit | Merged openstack/nova master: Add volume_type field to BlockDeviceMapping object https://review.openstack.org/604687 | 05:54 |
| openstackgerrit | Merged openstack/nova master: Replace openSUSE experimental check with newer version https://review.openstack.org/609467 | 05:54 |
| openstackgerrit | Merged openstack/nova master: Add compute version 36 to support ``volume_type`` https://review.openstack.org/579360 | 05:54 |
| openstackgerrit | Merged openstack/nova master: Add compute API validation for when a volume_type is requested https://review.openstack.org/605573 | 05:56 |
| *** hshiina has quit IRC | 06:00 | |
| *** hshiina_ has joined #openstack-nova | 06:00 | |
| *** k_mouza has joined #openstack-nova | 06:01 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/nova master: Remove dead code from policy module https://review.openstack.org/609591 | 06:02 |
| *** k_mouza has quit IRC | 06:03 | |
| *** tetsuro_ has quit IRC | 06:06 | |
| *** k_mouza has joined #openstack-nova | 06:07 | |
| *** k_mouza has quit IRC | 06:10 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398 | 06:10 |
| *** k_mouza has joined #openstack-nova | 06:14 | |
| *** k_mouza has quit IRC | 06:16 | |
| *** janki has quit IRC | 06:19 | |
| *** k_mouza has joined #openstack-nova | 06:20 | |
| *** k_mouza has quit IRC | 06:24 | |
| *** dpawlik has joined #openstack-nova | 06:30 | |
| *** k_mouza has joined #openstack-nova | 06:33 | |
| *** janki has joined #openstack-nova | 06:34 | |
| *** k_mouza_ has joined #openstack-nova | 06:36 | |
| *** k_mouza has quit IRC | 06:38 | |
| *** k_mouza_ has quit IRC | 06:41 | |
| *** k_mouza has joined #openstack-nova | 06:41 | |
| *** k_mouza has quit IRC | 06:44 | |
| *** Dinesh_Bhor has quit IRC | 06:45 | |
| *** k_mouza has joined #openstack-nova | 06:45 | |
| *** k_mouza has quit IRC | 06:47 | |
| *** k_mouza has joined #openstack-nova | 06:48 | |
| *** maciejjozefczyk has joined #openstack-nova | 06:51 | |
| *** k_mouza_ has joined #openstack-nova | 06:52 | |
| *** k_mouza has quit IRC | 06:52 | |
| *** k_mouza_ has quit IRC | 06:56 | |
| *** k_mouza has joined #openstack-nova | 06:58 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398 | 06:58 |
| *** cfriesen has quit IRC | 07:00 | |
| *** k_mouza has quit IRC | 07:02 | |
| *** rcernin has quit IRC | 07:03 | |
| *** k_mouza has joined #openstack-nova | 07:05 | |
| *** alexchadin has joined #openstack-nova | 07:09 | |
| *** k_mouza has quit IRC | 07:10 | |
| *** k_mouza has joined #openstack-nova | 07:13 | |
| *** k_mouza has quit IRC | 07:17 | |
| *** helenafm has joined #openstack-nova | 07:25 | |
| *** k_mouza has joined #openstack-nova | 07:27 | |
| *** alexchadin has quit IRC | 07:27 | |
| *** alexchadin has joined #openstack-nova | 07:27 | |
| *** Luzi has joined #openstack-nova | 07:29 | |
| *** k_mouza has quit IRC | 07:29 | |
| *** maciejjozefczyk has quit IRC | 07:32 | |
| *** k_mouza has joined #openstack-nova | 07:33 | |
| *** k_mouza has quit IRC | 07:36 | |
| *** k_mouza_ has joined #openstack-nova | 07:36 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:37 | |
| *** k_mouza_ has quit IRC | 07:40 | |
| *** k_mouza has joined #openstack-nova | 07:44 | |
| *** ralonsoh has joined #openstack-nova | 07:48 | |
| *** k_mouza has quit IRC | 07:48 | |
| *** hshiina_ has quit IRC | 07:51 | |
| *** k_mouza has joined #openstack-nova | 08:04 | |
| *** takashin has left #openstack-nova | 08:06 | |
| *** tssurya has joined #openstack-nova | 08:07 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Use tempest-pg-full https://review.openstack.org/609954 | 08:07 |
| *** k_mouza has quit IRC | 08:09 | |
| *** rpittau has quit IRC | 08:09 | |
| *** rpittau has joined #openstack-nova | 08:10 | |
| *** rpittau has quit IRC | 08:10 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:11 | |
| *** rpittau has joined #openstack-nova | 08:11 | |
| *** macza has joined #openstack-nova | 08:18 | |
| *** ttsiouts has joined #openstack-nova | 08:19 | |
| *** macza has quit IRC | 08:19 | |
| *** macza has joined #openstack-nova | 08:19 | |
| *** k_mouza has joined #openstack-nova | 08:20 | |
| *** ttsiouts has quit IRC | 08:22 | |
| *** ttsiouts_ has joined #openstack-nova | 08:22 | |
| *** macza has quit IRC | 08:24 | |
| *** k_mouza has quit IRC | 08:25 | |
| openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960 | 08:29 |
| *** zigo has joined #openstack-nova | 08:30 | |
| *** bauzas is now known as PapaOurs | 08:30 | |
| *** jangutter has joined #openstack-nova | 08:31 | |
| *** brinzhang has quit IRC | 08:32 | |
| *** brinzhang has joined #openstack-nova | 08:32 | |
| *** gibi is now known as giblet | 08:36 | |
| *** k_mouza_ has joined #openstack-nova | 08:40 | |
| *** derekh has joined #openstack-nova | 08:41 | |
| *** ttsiouts_ has quit IRC | 08:42 | |
| *** ttsiouts has joined #openstack-nova | 08:42 | |
| *** k_mouza_ has quit IRC | 08:43 | |
| *** k_mouza has joined #openstack-nova | 08:43 | |
| gmann | alex_xu: melwitt i feel we should restrict the null and empty string in volume_type which are going to be ignored on cinder side - https://review.openstack.org/#/c/606398/13 | 08:43 |
| *** dpawlik has quit IRC | 08:46 | |
| *** slaweq has quit IRC | 08:46 | |
| *** ttsiouts has quit IRC | 08:47 | |
| *** k_mouza has quit IRC | 08:47 | |
| *** k_mouza has joined #openstack-nova | 08:50 | |
| *** dpawlik has joined #openstack-nova | 08:51 | |
| *** k_mouza_ has joined #openstack-nova | 08:53 | |
| *** k_mouza has quit IRC | 08:54 | |
| *** psachin has quit IRC | 08:55 | |
| *** ttsiouts has joined #openstack-nova | 08:56 | |
| *** k_mouza_ has quit IRC | 08:57 | |
| *** janki has quit IRC | 08:58 | |
| *** janki has joined #openstack-nova | 08:58 | |
| *** k_mouza has joined #openstack-nova | 09:01 | |
| *** janki has quit IRC | 09:03 | |
| *** k_mouza has quit IRC | 09:04 | |
| *** janki has joined #openstack-nova | 09:04 | |
| *** janki has quit IRC | 09:05 | |
| *** janki has joined #openstack-nova | 09:05 | |
| *** k_mouza has joined #openstack-nova | 09:07 | |
| *** k_mouza_ has joined #openstack-nova | 09:10 | |
| *** k_mouza has quit IRC | 09:11 | |
| *** k_mouza has joined #openstack-nova | 09:12 | |
| *** k_mouza_ has quit IRC | 09:14 | |
| *** ttsiouts has quit IRC | 09:16 | |
| *** ttsiouts has joined #openstack-nova | 09:16 | |
| *** k_mouza has quit IRC | 09:17 | |
| *** ttsiouts_ has joined #openstack-nova | 09:18 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/nova master: Remove dead code from policy module https://review.openstack.org/609591 | 09:18 |
| *** ttsiouts has quit IRC | 09:21 | |
| *** k_mouza has joined #openstack-nova | 09:25 | |
| *** k_mouza has quit IRC | 09:28 | |
| *** k_mouza has joined #openstack-nova | 09:28 | |
| *** k_mouza has quit IRC | 09:31 | |
| *** k_mouza has joined #openstack-nova | 09:31 | |
| *** dtantsur|afk is now known as dtantsur | 09:33 | |
| *** gnuoy has quit IRC | 09:35 | |
| *** k_mouza_ has joined #openstack-nova | 09:35 | |
| *** k_mouza has quit IRC | 09:35 | |
| *** ttsiouts_ has quit IRC | 09:37 | |
| *** gnuoy has joined #openstack-nova | 09:37 | |
| *** gnuoy has quit IRC | 09:38 | |
| *** gnuoy has joined #openstack-nova | 09:38 | |
| *** ttsiouts has joined #openstack-nova | 09:39 | |
| *** k_mouza_ has quit IRC | 09:40 | |
| *** psachin has joined #openstack-nova | 09:45 | |
| *** s10 has joined #openstack-nova | 09:47 | |
| *** gnuoy has quit IRC | 09:50 | |
| *** ttsiouts has quit IRC | 09:53 | |
| *** imacdonn has quit IRC | 09:53 | |
| *** imacdonn has joined #openstack-nova | 09:53 | |
| *** Luzi has quit IRC | 09:55 | |
| *** gnuoy has joined #openstack-nova | 09:55 | |
| *** slaweq has joined #openstack-nova | 09:56 | |
| *** Dinesh_Bhor has quit IRC | 09:57 | |
| *** ttsiouts has joined #openstack-nova | 09:57 | |
| *** zhanglong has quit IRC | 09:58 | |
| *** ttsiouts has quit IRC | 10:01 | |
| *** brinzhang has quit IRC | 10:02 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Change the default values of XXX_allocation_ratio https://review.openstack.org/602803 | 10:10 |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Use new ``initial_xxx_allocation_ratio`` CONF https://review.openstack.org/602804 | 10:10 |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration https://review.openstack.org/609995 | 10:10 |
| *** TuanDA has quit IRC | 10:13 | |
| *** hoangcx has quit IRC | 10:17 | |
| nehaalhat_ | sean-k-mooney: Hi | 10:25 |
| sean-k-mooney | nehaalhat_: hi | 10:25 |
| nehaalhat_ | sean-k-mooney: want to discuss regarding patch: https://review.openstack.org/#/c/581218/ | 10:25 |
| sean-k-mooney | sure | 10:26 |
| nehaalhat_ | sean-k-mooney: the log messages that are mentioned in commit message of above patch, are logged from keystoneauth but with logger name: 'novaclient.v.client'. As these logs are from keyatoneauth the logger name should be like 'keystoneauth.session'' | 10:28 |
| nehaalhat_ | 'novaclient.v2.client' | 10:29 |
| *** psachin has quit IRC | 10:33 | |
| sean-k-mooney | ok | 10:33 |
| nehaalhat_ | sean-k-mooney: I have tried your suggestion that: set default_log_levels=novaclient=DEBUG in cinder.conf and then checked logs, but the logs related to nova are getting logged using 'novaclient.v2.client' logger name only. | 10:33 |
| sean-k-mooney | right they should have novaclient.v2.client name | 10:34 |
| sean-k-mooney | were are the keyston auth logs your are trying to suppress being generagted | 10:35 |
| *** Dinesh_Bhor has joined #openstack-nova | 10:35 | |
| *** udesale has quit IRC | 10:36 | |
| sean-k-mooney | they should only have the keystoneauth.session logger name if they are being logged in teh libary | 10:36 |
| sean-k-mooney | if the log function is called in the nova client code they should be novaclient.v2.client | 10:36 |
| nehaalhat_ | sean-k-mooney: yes the logger is initialized in novaclient only, so it is taking name as 'novaclient.v2.client'. You can refer this logs: http://paste.openstack.org/show/731959/ that i am getting for cinder to novaclient interaction | 10:39 |
| nehaalhat_ | sean-k-mooney: In the logs you can see the logs are getting logged in keystoneauth library | 10:40 |
| nehaalhat_ | but with logger name 'novaclient.v2.client' | 10:41 |
| *** jpena|off has quit IRC | 10:41 | |
| sean-k-mooney | do you know where in novaclient it is calling log.debug | 10:42 |
| *** jpena|off has joined #openstack-nova | 10:42 | |
| nehaalhat_ | sean-k-mooney: No | 10:43 |
| *** tbachman has quit IRC | 10:44 | |
| *** psachin has joined #openstack-nova | 10:49 | |
| sean-k-mooney | nehaalhat_: it looks like they are bing logged in the keystone auth lib but that means its not internally creating its own logger | 10:51 |
| nehaalhat_ | sean-k-mooney: yes it is being logged in keystoneauth lib | 10:52 |
| sean-k-mooney | nehaalhat_: i think you should be making you change here instead https://github.com/openstack/python-novaclient/blob/003ac57d9af74aa4658a7bf6cc6b6b3bafa58c11/novaclient/client.py#L167 | 10:53 |
| nehaalhat_ | sean-k-mooney: But identity related logs are logged using logger name 'keystoneauth.session' and the are also logged in 'keystone auth' lob | 10:54 |
| sean-k-mooney | actully that is the only place the logger is used in that function. | 10:54 |
| nehaalhat_ | sean-k-mooney: you are suggesting to make changes here: https://github.com/openstack/python-novaclient/blob/003ac57d9af74aa4658a7bf6cc6b6b3bafa58c11/novaclient/client.py#L167 right? | 10:58 |
| sean-k-mooney | i was but looking at how the code works and the keystonauth adapter.py the client is working differntely then i had taught | 11:02 |
| sean-k-mooney | i think you can leave you chage as it is | 11:02 |
| nehaalhat_ | sean-k-mooney: ok..thanks | 11:03 |
| sean-k-mooney | that said i think the api for the keystone auth adapter is incorerct. im not sure it should allow passing a logger at all | 11:03 |
| sean-k-mooney | nehaalhat_: the other option you have is to creat a new logger and pass it in | 11:04 |
| nehaalhat_ | sean-k-mooney: ok | 11:04 |
| sean-k-mooney | so you could create a novaclient.v2.client.auth logger and pass that in | 11:05 |
| sean-k-mooney | then people could use the python loging filtering to choose to filter it out or not | 11:05 |
| nehaalhat_ | sean-k-mooney: ok will check | 11:07 |
| *** ttsiouts has joined #openstack-nova | 11:08 | |
| sean-k-mooney | coool you would then set novaclient.v2.client.auth to info in cinder to not log the debug logs form the keystone auth adapter | 11:08 |
| *** Dinesh_Bhor has quit IRC | 11:10 | |
| nehaalhat_ | sean-k-mooney: what if i want logs related to debug | 11:11 |
| sean-k-mooney | novaclient.v2.client.auth would only be the kestoneauth logs your are currently uncondtionally removing | 11:12 |
| sean-k-mooney | so if you have novaclient.v2.client at debug setting novaclient.v2.client.auth to info will only filter out the auth debug logs | 11:13 |
| sean-k-mooney | the keystone auth adapter does not fall back to its own logging if you don tpass a logger in http://git.openstack.org/cgit/openstack/keystoneauth/tree/keystoneauth1/adapter.py#n129 | 11:14 |
| *** ttsiouts has quit IRC | 11:16 | |
| *** ttsiouts has joined #openstack-nova | 11:17 | |
| *** ttsiouts has quit IRC | 11:21 | |
| *** ttsiouts has joined #openstack-nova | 11:23 | |
| *** ttsiouts has quit IRC | 11:27 | |
| *** psachin has quit IRC | 11:28 | |
| *** trident has joined #openstack-nova | 11:31 | |
| *** psachin has joined #openstack-nova | 11:34 | |
| *** dave-mccowan has joined #openstack-nova | 11:38 | |
| *** ttsiouts has joined #openstack-nova | 11:39 | |
| *** k_mouza has joined #openstack-nova | 11:44 | |
| *** tbachman has joined #openstack-nova | 11:48 | |
| *** k_mouza has quit IRC | 11:48 | |
| *** tbachman_ has joined #openstack-nova | 11:53 | |
| *** tbachman has quit IRC | 11:53 | |
| *** tbachman_ is now known as tbachman | 11:53 | |
| sean-k-mooney | are the wsgi script for placement generated the same way we generate the the nova-compute etc command scripts using setuptool entrypoints? | 11:58 |
| *** jaypipes is now known as leakypipes | 11:58 | |
| sean-k-mooney | specifcally https://github.com/openstack/nova/blob/master/setup.cfg#L78 gets processed by setuptools to generate /var/www/cgi-bin/nova/nova-placement-api ? | 12:00 |
| *** nehaalhat_ has quit IRC | 12:01 | |
| sean-k-mooney | ah they are generated by pbr | 12:01 |
| *** ttsiouts has quit IRC | 12:03 | |
| *** ttsiouts has joined #openstack-nova | 12:03 | |
| jaosorior | sean-k-mooney: hey, if you have some time, can you check the review from yesterday https://review.openstack.org/#/c/609591/ ? | 12:04 |
| *** tbachman has quit IRC | 12:08 | |
| *** ttsiouts has quit IRC | 12:08 | |
| *** nehaalhat_ has joined #openstack-nova | 12:10 | |
| *** ttsiouts has joined #openstack-nova | 12:11 | |
| *** ratailor has quit IRC | 12:17 | |
| *** hoonetorg has joined #openstack-nova | 12:24 | |
| *** psachin has quit IRC | 12:45 | |
| leakypipes | mordred: Monty, would you be a dear and review https://review.openstack.org/#/c/581218/ please? Both mriedem and myself are kinda lost as to what is going on there and need your feedback on the impact this would have in SDKs. | 12:45 |
| leakypipes | aspiers: BTW, it may be a little while until I can review https://review.openstack.org/#/c/609779/1/specs/stein/approved/amd-sev-libvirt-support.rst... it's cause me to open a number of new browser tabs as I read about that technology. :) | 12:46 |
| leakypipes | aspiers: something that negatively affected my goal for yesterday. shame on you. (https://twitter.com/jaypipes/status/1050394398163443713) :P | 12:47 |
| aspiers | leakypipes: no problem :) although if it's any consolation, I know very little about it myself, but I don't think a deep understanding is really necessary anyway ... | 12:48 |
| aspiers | haha :) | 12:48 |
| aspiers | there are browser extensions for that, you know | 12:50 |
| aspiers | although depressingly the one I use has just vanished from my Chrome :-/ | 12:51 |
| leakypipes | aspiers: well, I actually *do* want to read (or take other action on) all those tabs... just takes a while! :) | 12:51 |
| leakypipes | aspiers: just takes some good ol' fashioned time to do it | 12:52 |
| aspiers | ah well, then you need a way to move them onto a todo list | 12:52 |
| aspiers | https://getpocket.com/ is pretty handy, but then so is a text file full of URLs | 12:52 |
| aspiers | or if you want nicer hyperlinks, try this https://github.com/aspiers/desktop-config/blob/master/lib/browser/userscripts/page-id-helpers/README.md | 12:53 |
| leakypipes | aspiers: rock on brother. (though you just added yet another tab to my browser, damn you Adam!) | 12:58 |
| *** mriedem has joined #openstack-nova | 12:58 | |
| aspiers | X-p | 12:59 |
| leakypipes | aspiers: btw, just curious, is SUSE planning on deploying an AMD-based cloud? or do you have customers that want to use EPYCs instead of Xeons? | 12:59 |
| aspiers | I'm guessing it's for customers eventually but I don't really have a clue - not involved in the business discussions thank goodness :) | 13:00 |
| aspiers | our partnership with AMD goes way back | 13:00 |
| leakypipes | ack | 13:00 |
| aspiers | IIRC we were the first enterprise Linux to support AMD, or something like that | 13:00 |
| aspiers | mind you, we're obviously very close with Intel too | 13:01 |
| leakypipes | gotcha | 13:01 |
| mriedem | well, i guess we have to decide if the volume_type support when creating a server should be the same as cinder's volume create api https://review.openstack.org/#/c/606398/ and allow passing volume_type='' or volume_type=None, or restrict that in the compute API | 13:04 |
| mriedem | so that if you pass a volume_type, it has to be non-null and minLength: 1 | 13:04 |
| mriedem | i'm personally fine keeping parity with cinder's API since people really just want the proxy here | 13:05 |
| *** elod has quit IRC | 13:06 | |
| *** elod has joined #openstack-nova | 13:06 | |
| *** dtantsur is now known as dtantsur|afk | 13:07 | |
| leakypipes | mriedem: how about --volume-type=force:True? That would mesh well with our long tradition of force APIs. | 13:07 |
| leakypipes | mriedem: you could even add a --volume-type=force:None to make things super clear. | 13:08 |
| mriedem | i would like it to pass it over flavor extra spec myself, | 13:08 |
| mriedem | but we don't all get what we want | 13:08 |
| leakypipes | mriedem: scheduler hint for volume type --hint force-volume-type=None | 13:08 |
| mriedem | or encoded scheduler hitn | 13:08 |
| mriedem | *hint | 13:08 |
| mriedem | yes | 13:08 |
| leakypipes | mriedem: beat you to it. | 13:08 |
| *** eharney has joined #openstack-nova | 13:10 | |
| giblet | :) | 13:10 |
| mordred | leakypipes: looking | 13:13 |
| *** markvoelker has quit IRC | 13:15 | |
| leakypipes | mordred: thanks man. it was mriedem who also wanted your opinion on that, which I concurred with. | 13:16 |
| mordred | leakypipes: I believe the patch looks fine - and I'm happy to see people are excited about the split-loggers option :) | 13:16 |
| mriedem | ? https://review.openstack.org/#/c/581218/ ? | 13:18 |
| *** ttsiouts has quit IRC | 13:20 | |
| leakypipes | mordred: what about sean-k-mooney's first comment here? https://review.openstack.org/#/c/581218/5/novaclient/v2/client.py | 13:20 |
| leakypipes | mordred: is that a concern? | 13:20 |
| mriedem | jackding: you know your reboot/rebuild check the network for unbound ports patch? | 13:21 |
| leakypipes | mriedem: yes, that one. you had commented on there: "-1 for the missing lower-constraints bump, but I'd also like to see Monty +1 this before we merge it since I'm not really following all of this." | 13:21 |
| mriedem | that's checking the ports right after we refreshed them from neutron, | 13:21 |
| mriedem | can we avoid the list_ports in your check method? | 13:21 |
| mriedem | and just use the instance.info_cache? | 13:21 |
| mriedem | that would help some of my concern | 13:21 |
| *** munimeha1 has joined #openstack-nova | 13:21 | |
| mriedem | leakypipes: yar | 13:21 |
| mriedem | if you want split logs, pass a scheduler hint | 13:22 |
| leakypipes | mriedem: aye, matey. | 13:22 |
| *** ttsiouts has joined #openstack-nova | 13:22 | |
| leakypipes | mriedem: lol. thx moleman. | 13:22 |
| *** mriedem is now known as hansmoleman | 13:22 | |
| leakypipes | there he is! | 13:22 |
| hansmoleman | https://deadhomersociety.files.wordpress.com/2016/11/thisisaknife.gif?w=655 | 13:23 |
| *** lbragstad has joined #openstack-nova | 13:23 | |
| hansmoleman | gmann: do you have a patch to cap max microversions for tempest in devstack for rocky yet? | 13:24 |
| hansmoleman | nope, i'll do that | 13:25 |
| *** erlon has joined #openstack-nova | 13:28 | |
| openstackgerrit | sean mooney proposed openstack/nova master: harden placement init under wsgi https://review.openstack.org/610034 | 13:30 |
| sean-k-mooney | ^ that may be dumb but its my attempt to fix part of a downstream bug in triplo that i dont have an enviornment to reproduce and or time to reporduce. | 13:33 |
| hansmoleman | comments inline | 13:34 |
| hansmoleman | efried: what's the process now for placement bug fixes? | 13:34 |
| hansmoleman | fix in nova and cherry pick to placement or vice versa? | 13:34 |
| sean-k-mooney | oh right placement is moving... | 13:35 |
| *** munimeha1 has quit IRC | 13:38 | |
| mordred | leakypipes, hansmoleman: +1 left - I agree with sean-k-mooney's followup comment | 13:39 |
| *** tssurya has quit IRC | 13:39 | |
| *** tssurya has joined #openstack-nova | 13:40 | |
| PapaOurs | hansmoleman: AFAIUC, we need two changes | 13:40 |
| PapaOurs | hansmoleman: and just waiting for the placement one to be merged before merging the nova one | 13:41 |
| PapaOurs | but not Depends-On | 13:41 |
| *** dansmith is now known as SteelyDan | 13:41 | |
| *** stephenfin is now known as finucannot | 13:41 | |
| finucannot | hansmoleman: What would it cost me to get you to look at https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/queens+topic:bug/1746393 again? | 13:42 |
| finucannot | hansmoleman: It's not something I'm going to be able to test with the Intel NFV CI but I did manually verify it | 13:42 |
| *** irclogbot_0 has joined #openstack-nova | 13:44 | |
| PapaOurs | does someone remember the link for querying all the OpenStack project repos ? | 13:45 |
| hansmoleman | codesearch.openstack.org | 13:46 |
| PapaOurs | hansmoleman: <3 | 13:46 |
| hansmoleman | finucannot: how far back to plan on trying to get those backports? has that code just always been broken? | 13:47 |
| hansmoleman | that stack seems extremely risky to me | 13:47 |
| hansmoleman | especially when we have 0 CI | 13:47 |
| *** irclogbot_0 has quit IRC | 13:49 | |
| hansmoleman | s/to plan/do you plan/ | 13:49 |
| finucannot | hansmoleman: Ideally back to when support for emulator threads was first added (Ocata) but Ocata is em now and I haven't even looked at the practicality of Pike. Queens would be good enough | 13:50 |
| finucannot | Plus, Pike is just security/data loss bugs now anyway, right? | 13:50 |
| hansmoleman | no | 13:50 |
| hansmoleman | EM means we no longer have phases like that | 13:50 |
| hansmoleman | https://docs.openstack.org/project-team-guide/stable-branches.html#maintenance-phases | 13:51 |
| finucannot | The more you know | 13:51 |
| sean-k-mooney | leakypipes: ya i proably should have submitted it against placement. | 13:52 |
| finucannot | But yeah, given how specific this is, whatever risk there is is going limited to select use cases | 13:52 |
| sean-k-mooney | leakypipes: ill abandon after the ci results are in i mainly pushed to make sure this did not make anything else explode | 13:52 |
| finucannot | That said, I have no idea how else to mitigate that risk besides the unit tests, manual testing that's already been done and downstream QE we'll be doing for this | 13:52 |
| *** irclogbot_0 has joined #openstack-nova | 13:53 | |
| *** rpittau has quit IRC | 13:54 | |
| leakypipes | sean-k-mooney: get with cdent in #openstack-placement since he's got a patch series already that is cleaning up the placement DB stuff. | 13:54 |
| leakypipes | sean-k-mooney: Chris isn't here atm, but just grab him whenevs | 13:54 |
| *** munimeha1 has joined #openstack-nova | 13:56 | |
| sean-k-mooney | leakypipes: sure. im likely not going to be working on this till next week. im in munich for new heir orentation from sunday to wednesday so i wont get much done until thrusday | 13:56 |
| leakypipes | ack, no worries. | 13:57 |
| *** mlavalle has joined #openstack-nova | 13:57 | |
| leakypipes | sean-k-mooney: btw, is "new heir orentation" something to do with the royal family? | 13:57 |
| leakypipes | :P | 13:57 |
| sean-k-mooney | also apparently i joinded redhat 90days ago as of monday. | 13:57 |
| leakypipes | sean-k-mooney: congrats! :) | 13:57 |
| PapaOurs | sean-k-mooney: I'm sure you'll love sales talks | 13:58 |
| *** irclogbot_0 has quit IRC | 13:59 | |
| sean-k-mooney | PapaOurs: apparently we are not ment to bring our laptops to the 2 days of training... | 13:59 |
| *** irclogbot_0 has joined #openstack-nova | 13:59 | |
| PapaOurs | sean-k-mooney: tbh I don't remember mine | 13:59 |
| sean-k-mooney | also i miss octoberfest in munich by a week | 13:59 |
| PapaOurs | I mean, whether I used my laptop | 13:59 |
| PapaOurs | but you'll get a free fedora | 14:00 |
| PapaOurs | hum | 14:00 |
| PapaOurs | looks like we're discussing corp here | 14:00 |
| PapaOurs | shit | 14:00 |
| PapaOurs | fortunately, I'm hidden behind my nick | 14:00 |
| sean-k-mooney | :) | 14:00 |
| *** efried is now known as fried_rice | 14:05 | |
| *** cfriesen has joined #openstack-nova | 14:06 | |
| fried_rice | hansmoleman: Same fix (as far as possible) with same change-id in both nova and placement | 14:07 |
| melwitt | 14:08 | |
| *** eharney has quit IRC | 14:09 | |
| hansmoleman | fried_rice: does order of repos matter? | 14:10 |
| hansmoleman | placement first so it's not lost after the extraction? | 14:11 |
| *** smcginnis has joined #openstack-nova | 14:11 | |
| fried_rice | I don't think so | 14:11 |
| fried_rice | Like, propose it to both at the same time, but I don't think it matters if one merges first as long as they both merge. | 14:11 |
| hansmoleman | that's kind of weird | 14:12 |
| hansmoleman | if one merges and the other has changes | 14:12 |
| hansmoleman | i think of it like backports | 14:12 |
| hansmoleman | you want to start in the clean target first and then cherry pick (backport) to nova | 14:13 |
| hansmoleman | but what do i know | 14:13 |
| hansmoleman | SteelyDan: re our conversation the other day about attaching volumes while resized, apparently it's fine once you revert, the volume attached while the server is in VERIFY_RESIZE state continues to be attached when you revert | 14:16 |
| *** tbachman has joined #openstack-nova | 14:16 | |
| hansmoleman | now, i'm using the fake driver in devstack so i can have 2 computes on a single node, | 14:16 |
| hansmoleman | so i'm not sure that attached volume is actually still in the guest... | 14:16 |
| SteelyDan | hansmoleman: ugh | 14:17 |
| hansmoleman | i don't have a 2-node devstack with libvirt handy | 14:17 |
| *** irclogbot_0 has quit IRC | 14:18 | |
| fried_rice | hansmoleman: I guess if you want to be that strict about it, sure. But which would you merge first? The nova side so you can test it fully? | 14:21 |
| hansmoleman | as in devstack runs? | 14:22 |
| hansmoleman | that probably makes more sense... | 14:22 |
| fried_rice | Okay. | 14:23 |
| hansmoleman | cfriesen: do you guys care about this? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/queens+topic:bug/1746393 | 14:23 |
| fried_rice | hansmoleman: To drop a placement fix at this point, you need to propose it to both the nova and placement repositories with the same change-id, but merge the nova side first. | 14:24 |
| fried_rice | There, it's official. | 14:24 |
| *** lbragstad is now known as elbragstad | 14:25 | |
| hansmoleman | it's not official until it's engraved in stone tablets | 14:25 |
| sean-k-mooney | fried_rice: so for https://review.openstack.org/#/c/610034/ i need to just cherrypick it ot placement too | 14:25 |
| fried_rice | sean-k-mooney: afaik there's no actual cherry pick between different repositories. But in spirit, yes. | 14:26 |
| fried_rice | sean-k-mooney: And -W it until ^ merges. | 14:27 |
| sean-k-mooney | fried_rice: you have to manually add the other repo as a remote | 14:27 |
| hansmoleman | finucannot: i guess i still feel that backport is really more about a feature than a bug | 14:27 |
| hansmoleman | it's an optimization thing isn't it? | 14:27 |
| sean-k-mooney | if placement was extracted correctly with its git history it shoudl work but ya | 14:27 |
| hansmoleman | i.e. when cpu pinning was added, or emulator thread policy, people didn't think about them being used together all the way, so it was less optimal, | 14:28 |
| hansmoleman | and that's fixed since rocky | 14:28 |
| fried_rice | sean-k-mooney: Having them under the same change-id ought to be sufficient. If you included the commit hash, you would have to also include the repo name for that commit. | 14:28 |
| hansmoleman | but doesn't mean we need to backport that to queens | 14:28 |
| fried_rice | sean-k-mooney: But they won't match exactly, if for no other reason than the file names. | 14:28 |
| sean-k-mooney | fried_rice: right | 14:29 |
| sean-k-mooney | hansmoleman: we may backport it downstram but it depens i dont think we need to backport to queens upstream | 14:30 |
| finucannot | hansmoleman: Not sure, to be honest. Guess that comes down to interpretation | 14:31 |
| sean-k-mooney | the downstream but was reported against rocky so that is likely all that would be useful to backport to in anycase | 14:31 |
| hansmoleman | i've -1ed the bottom queens backport then | 14:31 |
| hansmoleman | if you want my official opinion | 14:31 |
| finucannot | Heh. Fair :) | 14:32 |
| sean-k-mooney | hansmoleman: wait which patch | 14:32 |
| hansmoleman | https://review.openstack.org/#/c/588570/ | 14:32 |
| *** munimeha1 has quit IRC | 14:32 | |
| sean-k-mooney | oh i was talking about the placement one im not sure about that one | 14:33 |
| hansmoleman | too late | 14:33 |
| hansmoleman | you said red hat doesn't care so i get to -1 | 14:33 |
| SteelyDan | reading the bug, that seems like a performance feature to me | 14:33 |
| SteelyDan | is there some correctness aspect to it, or purely optimized layout/ | 14:33 |
| sean-k-mooney | this bug https://bugs.launchpad.net/nova/+bug/1744965 just reading | 14:34 |
| openstack | Launchpad bug 1744965 in OpenStack Compute (nova) "'emulator_threads_policy' doesn't work with 'vcpu_pin_set'" [Undecided,Fix released] - Assigned to Stephen Finucane (stephenfinucane) | 14:34 |
| finucannot | The BZ that was based on is probably more useful https://bugzilla.redhat.com/show_bug.cgi?id=1534669 | 14:35 |
| openstack | bugzilla.redhat.com bug 1534669 in openstack-nova "emulator_threads_policy needs improvement when hyper threading is enabled" [Medium,On_qa] - Assigned to sfinucan | 14:35 |
| finucannot | (see comment 1. That's a bug, IMO) | 14:35 |
| sean-k-mooney | finucannot: what was your fix to just give both thread spiblivgs to the emultor threads | 14:36 |
| finucannot | sean-k-mooney: Nah. We stopped applying the vCPU policies to emulator threads | 14:37 |
| sean-k-mooney | finucannot: the polices dotn change teh number of thread you get pinned too howerver | 14:37 |
| finucannot | They kind of do. isolate doesn't cause the VM to consume more cores but the thread siblings are marked as unusable | 14:38 |
| sean-k-mooney | that is different | 14:39 |
| *** rnoriega has quit IRC | 14:39 | |
| sean-k-mooney | if the vcpu_pin_set has 6 cores and you request a vm with 6 vcpus and 1 emulator thread it should not be shcudled to that host as it cant fit | 14:39 |
| finucannot | Correct | 14:39 |
| finucannot | If vcpu_pin_set has 7 cores though, it should be scheduled | 14:40 |
| sean-k-mooney | yes | 14:40 |
| sean-k-mooney | so what is the bug | 14:40 |
| finucannot | We actually needed 8 | 14:41 |
| *** markvoelker has joined #openstack-nova | 14:41 | |
| sean-k-mooney | why | 14:41 |
| finucannot | because the policy meant for vcPUx was being incorrectly applied to emulator threads | 14:41 |
| finucannot | *vCPUs | 14:41 |
| sean-k-mooney | even if we applied it it should not matter what thread policy would chage it | 14:41 |
| finucannot | huh? | 14:41 |
| sean-k-mooney | isolate is not meant to consider thread outside of the vcpu_pin_set so even if you have hyper threading it should not be an issue | 14:42 |
| finucannot | Exactly, but the implementation was buggy | 14:43 |
| *** mrch has quit IRC | 14:43 | |
| *** bnemec is now known as beekneemech | 14:44 | |
| finucannot | It was saying "because we're using pairs of thread siblings for these vCPUs, we should do that for emulator threads too" | 14:44 |
| *** eharney has joined #openstack-nova | 14:44 | |
| sean-k-mooney | finucannot: ok ya that is wrong. is thei your change to handeling differnet lent sibling sets | 14:45 |
| finucannot | So if you were using the require policy, that would require two cores for emulator threads (the latter wasn't use though, I think) | 14:45 |
| openstackgerrit | Chuck Short proposed openstack/os-traits master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610065 | 14:46 |
| sean-k-mooney | you are intended to be able to ask for 1 core and say threading policy require | 14:46 |
| finucannot | Yup, that was broken too | 14:46 |
| sean-k-mooney | you are alos ment to be able to use prefer on host that dont have hyper threading | 14:47 |
| finucannot | yep | 14:47 |
| sean-k-mooney | finucannot: so if we backported it to queens are you also going to backport it to pike | 14:49 |
| sean-k-mooney | finucannot: that is where the downstream bug was reported | 14:49 |
| finucannot | I'd like to but you've to do one before the other | 14:50 |
| openstackgerrit | Chuck Short proposed openstack/os-vif master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610068 | 14:50 |
| hansmoleman | persisting limits and requested_destination in a request spec seems like a bad idea... http://paste.openstack.org/show/731972/ | 14:50 |
| openstackgerrit | Jose Castro Leon proposed openstack/nova master: Fix get_device_path from network mounted volume https://review.openstack.org/590188 | 14:52 |
| sean-k-mooney | finucannot: so i dont think there is anything harmful in the backport so i guess its fine but im not sure it qualifes under the backport policy | 14:53 |
| finucannot | Yup, seems to be the general consensus, heh | 14:54 |
| sean-k-mooney | queens would be pahse 2 right so its not a security fix and its not a critical prioity bug https://docs.openstack.org/murano/pike/contributor/stable_branches.html | 14:55 |
| sean-k-mooney | oh thats mruanos one... | 14:55 |
| *** spatel has joined #openstack-nova | 14:55 | |
| finucannot | sean-k-mooney: https://docs.openstack.org/project-team-guide/stable-branches.html#maintenance-phases | 14:55 |
| sean-k-mooney | oh ya they changed the with extended maintaince | 14:56 |
| melwitt | hm, why isn't the nova-lvm job running anymore... | 14:58 |
| melwitt | oh nvm, it only runs on libvirt changes | 14:58 |
| hansmoleman | ah yes: cold migrate to a specified host, confirm the resize, then live migrate w/o specifying a host, kablammo | 14:59 |
| hansmoleman | reqspec strikes agin | 14:59 |
| hansmoleman | *again | 14:59 |
| openstackgerrit | Chuck Short proposed openstack/osc-placement master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610074 | 15:00 |
| sean-k-mooney | hansmoleman: im guessign we persist a host of somthing that we should have deleted in the confirm step | 15:00 |
| hansmoleman | i know exactly what it is | 15:01 |
| hansmoleman | i just needed to confirm | 15:01 |
| *** maciejjozefczyk has quit IRC | 15:01 | |
| sean-k-mooney | actully for moving py35 to py37 job on stein+ that maeans that the minium version of python for stein becomes 36 right | 15:02 |
| sean-k-mooney | wew would no longer be testing 35 so it cant be the minium anymore | 15:02 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Deprecate the nova-console service https://review.openstack.org/610075 | 15:02 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Deprecate the nova-xvpvncproxy service https://review.openstack.org/610076 | 15:02 |
| *** jmlowe has quit IRC | 15:03 | |
| cfriesen | hansmoleman: we had to modify some of that code due to supporting some other features, so I don't think we'd care about a backport. | 15:04 |
| hansmoleman | yippee https://bugs.launchpad.net/nova/+bug/1797580 | 15:06 |
| openstack | Launchpad bug 1797580 in OpenStack Compute (nova) "NoValidHost during live migration after cold migrating to a specified host" [High,Triaged] - Assigned to Matt Riedemann (mriedem) | 15:06 |
| *** awestin1 has joined #openstack-nova | 15:11 | |
| *** irclogbot_0 has joined #openstack-nova | 15:15 | |
| *** dpawlik has quit IRC | 15:17 | |
| *** jmlowe has joined #openstack-nova | 15:18 | |
| *** gyee has joined #openstack-nova | 15:19 | |
| *** irclogbot_0 has quit IRC | 15:21 | |
| *** irclogbot_0 has joined #openstack-nova | 15:21 | |
| fried_rice | leakypipes: A schema like this? https://review.openstack.org/#/c/579289/12/nova_powervm/virt/powervm/inventory_schema.yaml | 15:23 |
| hansmoleman | huh osc doesn't have instance action CLI support? | 15:26 |
| SteelyDan | hansmoleman: I think it does | 15:26 |
| SteelyDan | pretty sure I was using it the other day | 15:26 |
| hansmoleman | don't see it here https://docs.openstack.org/python-openstackclient/latest/cli/command-list.html | 15:26 |
| hansmoleman | server event list? | 15:27 |
| hansmoleman | oh there it is | 15:27 |
| hansmoleman | ok, and because my live migration fails but the server isn't put into error status, i can't see the fault | 15:29 |
| hansmoleman | so i have to use the instance action list | 15:29 |
| hansmoleman | or the migration status | 15:29 |
| hansmoleman | which probably isn't in osc | 15:29 |
| *** ttsiouts has quit IRC | 15:31 | |
| sean-k-mooney | im going to call it a day early(for me). is there anything i should review before next week? | 15:32 |
| sean-k-mooney | if not i proably wont be on irc until thrudsay | 15:33 |
| *** sean-k-mooney has quit IRC | 15:36 | |
| *** mtreinish has joined #openstack-nova | 15:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for bug 1797580 https://review.openstack.org/610088 | 15:39 |
| openstack | bug 1797580 in OpenStack Compute (nova) "NoValidHost during live migration after cold migrating to a specified host" [High,Triaged] https://launchpad.net/bugs/1797580 - Assigned to Matt Riedemann (mriedem) | 15:39 |
| hansmoleman | SteelyDan: to fix ^ i'm thinking we should just not persist the RequestSpec.requested_destination | 15:39 |
| hansmoleman | similar to how we've fixed a few related things | 15:39 |
| hansmoleman | requested_destination should be per operation and not saved | 15:39 |
| SteelyDan | aye | 15:39 |
| leakypipes | fried_rice: yup. | 15:41 |
| leakypipes | fried_rice: but that describes the entire provider descriptor file. | 15:41 |
| fried_rice | leakypipes: That schema describes the entire file as originally designed. I'll mod it to be more generic. | 15:42 |
| fried_rice | and include versioning and whatnot | 15:43 |
| *** dpawlik has joined #openstack-nova | 15:47 | |
| *** irclogbot_0 has quit IRC | 15:50 | |
| *** markvoelker has quit IRC | 15:51 | |
| *** dpawlik has quit IRC | 15:51 | |
| *** markvoelker has joined #openstack-nova | 15:52 | |
| hansmoleman | cfriesen: i asked jackding about this earlier, and just left a comment in https://review.openstack.org/#/c/603844/, but it seems we shouldn't need to list the ports again in that new method to check if there are failed port bindings, | 15:54 |
| hansmoleman | b/c we just refreshed the instance info cache before calling that, | 15:54 |
| hansmoleman | which lists the ports for that server | 15:54 |
| hansmoleman | so if we can just rely on the cache, it's a much less heavy cahnge | 15:54 |
| hansmoleman | *change | 15:54 |
| *** irclogbot_0 has joined #openstack-nova | 15:55 | |
| *** derekh has quit IRC | 16:01 | |
| *** finucannot is now known as stephenfin | 16:01 | |
| *** panda has quit IRC | 16:01 | |
| *** erlon has quit IRC | 16:02 | |
| *** panda has joined #openstack-nova | 16:02 | |
| openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware: OVA and StrOpt images as VM templates https://review.openstack.org/609736 | 16:03 |
| *** janki has quit IRC | 16:04 | |
| *** jangutter has quit IRC | 16:05 | |
| *** alexchadin has quit IRC | 16:06 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't persist RequestSpec.requested_destination https://review.openstack.org/610098 | 16:06 |
| melwitt | tssurya: is it cool if I assign this bug to myself for the quota thing? I see you are currently assigned https://bugs.launchpad.net/nova/+bug/1771810 | 16:07 |
| openstack | Launchpad bug 1771810 in OpenStack Compute (nova) "Quota calculation connects to all available cells" [Undecided,New] - Assigned to Surya Seetharaman (tssurya) | 16:07 |
| hansmoleman | SteelyDan: easy peasy https://review.openstack.org/#/c/610098/ | 16:07 |
| melwitt | tssurya: (if you're not working on it, that is) | 16:07 |
| SteelyDan | I'll be the judge of easy peasy | 16:09 |
| SteelyDan | hansmoleman: did you ever look at artom's patch from yesterday? | 16:09 |
| hansmoleman | i got scared | 16:10 |
| hansmoleman | it's starred, but on | 16:10 |
| hansmoleman | *no | 16:11 |
| hansmoleman | i'll scan it quick | 16:12 |
| *** s10 has quit IRC | 16:15 | |
| hansmoleman | SteelyDan: artom: comments inline | 16:18 |
| hansmoleman | seems fine, | 16:18 |
| hansmoleman | i haven't looked at tests yet | 16:18 |
| hansmoleman | going to lunch and then a date with the mrs | 16:18 |
| *** hansmoleman is now known as mriedem_away | 16:18 | |
| jungleboyj | Has anyone seen issues with Cloudinit not working after changing the instances_path ? Have a packstack where I changed that and now neither Bionic or Xenial set up their networking on boot. | 16:19 |
| *** irclogbot_0 has quit IRC | 16:21 | |
| melwitt | I didn't think instances_path could affect how cloudinit works. does anyone else know if it would be expected to affect cloudinit? | 16:21 |
| SteelyDan | potentially if using configdrive and the proper image isn't loaded? | 16:23 |
| *** irclogbot_0 has joined #openstack-nova | 16:23 | |
| SteelyDan | I'd think more likely that something else got broken along the way | 16:23 |
| cfriesen | hansmoleman: I'm not self.network_api.get_instance_nw_info() will do the right thing, but it's worth taking a closer look. | 16:25 |
| *** ShilpaSD has quit IRC | 16:25 | |
| cfriesen | mriedem_away: ^ | 16:25 |
| *** irclogbot_0 has quit IRC | 16:27 | |
| *** fried_rice is now known as fried_rolls | 16:27 | |
| jungleboyj | SteelyDan: Ok. I was trying to look to see if there were errors in the libvirt logs but didn't see anything. | 16:27 |
| SteelyDan | jungleboyj: it's a reach.. you'd need to have pointed it somewhere so it found an image, but just the wrong one.. like I say, I'd look elsewhere than just the instances_path change | 16:28 |
| jungleboyj | SteelyDan: Ok. Will do. Thanks. | 16:28 |
| *** helenafm has quit IRC | 16:30 | |
| *** irclogbot_4 has joined #openstack-nova | 16:35 | |
| *** tbachman has quit IRC | 16:36 | |
| *** tbachman has joined #openstack-nova | 16:36 | |
| *** tbachman has quit IRC | 16:41 | |
| *** irclogbot_4 has quit IRC | 16:42 | |
| openstackgerrit | Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | 16:42 |
| *** spatel has quit IRC | 16:48 | |
| *** Bhujay has joined #openstack-nova | 17:07 | |
| *** irclogbot_0 has joined #openstack-nova | 17:09 | |
| *** SpamapS has quit IRC | 17:13 | |
| *** SpamapS has joined #openstack-nova | 17:15 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517 | 17:17 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517 | 17:17 |
| *** artom is now known as temka | 17:18 | |
| *** tssurya has quit IRC | 17:22 | |
| *** ralonsoh has quit IRC | 17:28 | |
| *** tbachman has joined #openstack-nova | 17:31 | |
| *** tbachman has quit IRC | 17:36 | |
| *** tbachman has joined #openstack-nova | 17:45 | |
| *** Bhujay has quit IRC | 17:50 | |
| *** irclogbot_0 has quit IRC | 17:51 | |
| *** rnoriega has joined #openstack-nova | 18:14 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517 | 18:34 |
| *** eharney has quit IRC | 18:34 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Fixed concurrent access to direct io test file https://review.openstack.org/515091 | 18:34 |
| *** eharney has joined #openstack-nova | 18:45 | |
| *** kmalloc is now known as needscoffee | 18:48 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Zuul: Update barbican experimental job https://review.openstack.org/610141 | 18:54 |
| *** pooja-jadhav has joined #openstack-nova | 19:03 | |
| *** pooja_jadhav has quit IRC | 19:05 | |
| *** tbachman has quit IRC | 19:10 | |
| melwitt | fried_rolls, edmondsw: could one of y'all please help triage this powervm bug? https://bugs.launchpad.net/nova/+bug/1771538 | 19:10 |
| openstack | Launchpad bug 1771538 in OpenStack Compute (nova) "PowerVM config drive path is not secure" [Undecided,New] | 19:10 |
| *** fried_rolls is now known as fried_rice | 19:10 | |
| fried_rice | melwitt: looking. | 19:10 |
| melwitt | it was found via a bandit scan | 19:10 |
| *** pooja_jadhav has joined #openstack-nova | 19:10 | |
| fried_rice | curse those bandits | 19:11 |
| melwitt | just need someone who knows about powervm to ack it or nak it accordingly | 19:11 |
| *** nehaalhat_ has quit IRC | 19:11 | |
| edmondsw | looking | 19:11 |
| melwitt | heh | 19:12 |
| edmondsw | seems legitimate | 19:12 |
| fried_rice | seems legit | 19:12 |
| edmondsw | jinx | 19:12 |
| *** jmlowe has quit IRC | 19:12 | |
| fried_rice | edmondsw: You doing the needful? | 19:12 |
| *** pooja-jadhav has quit IRC | 19:13 | |
| edmondsw | fried_rice I'll try to take a quick stab at it. Can't spend much time today | 19:13 |
| fried_rice | edmondsw: I just meant marking the bug report | 19:14 |
| edmondsw | yeah, I just want to check the code real quick and then will mark it confirmed | 19:14 |
| *** irclogbot_4 has joined #openstack-nova | 19:25 | |
| *** irclogbot_4 has quit IRC | 19:30 | |
| *** irclogbot_4 has joined #openstack-nova | 19:31 | |
| *** irclogbot_4 has quit IRC | 19:36 | |
| *** irclogbot_4 has joined #openstack-nova | 19:37 | |
| *** irclogbot_4 has quit IRC | 19:43 | |
| *** irclogbot_0 has joined #openstack-nova | 19:44 | |
| *** irclogbot_0 has quit IRC | 19:44 | |
| *** irclogbot_4 has joined #openstack-nova | 19:51 | |
| *** needscoffee is now known as kmalloc | 20:20 | |
| *** eharney has quit IRC | 20:40 | |
| *** irclogbot_4 has quit IRC | 20:47 | |
| *** irclogbot_4 has joined #openstack-nova | 20:47 | |
| *** mriedem_away is now known as mriedem | 20:51 | |
| *** irclogbot_4 has quit IRC | 20:54 | |
| *** irclogbot_4 has joined #openstack-nova | 20:54 | |
| melwitt | mriedem: this bug looks in your wheelhouse https://bugs.launchpad.net/nova/+bug/1788619 | 20:54 |
| openstack | Launchpad bug 1788619 in Cinder "disk cachemodes should be restricted with multiattached volumes" [High,New] | 20:54 |
| edleafe | \ | 20:55 |
| * edleafe curses his fat fingers | 20:55 | |
| mriedem | melwitt: sounds like it hasn't been tested | 20:56 |
| mriedem | # Shareable disks like for a multi-attach volume need to have the | 20:56 |
| mriedem | # driver cache disabled. | 20:56 |
| mriedem | if getattr(conf, 'shareable', False): | 20:56 |
| mriedem | conf.driver_cache = 'none' | 20:56 |
| *** dave-mccowan has quit IRC | 20:57 | |
| *** irclogbot_4 has quit IRC | 20:58 | |
| mriedem | invalidated the bug | 20:58 |
| melwitt | mriedem: ah, thanks | 20:58 |
| *** irclogbot_4 has joined #openstack-nova | 20:58 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Handle IndexError in _populate_neutron_binding_profile https://review.openstack.org/610163 | 20:59 |
| *** temka has quit IRC | 21:01 | |
| *** spatel has joined #openstack-nova | 21:03 | |
| *** jamesdenton has quit IRC | 21:24 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: Use tempfile for powervm config drive https://review.openstack.org/610174 | 21:29 |
| edmondsw | fried_rice ^ quick attempt at a fix, untested as yet | 21:31 |
| *** PapaOurs is now known as bauzas | 21:32 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix NoneType error in _notify_volume_usage_detach https://review.openstack.org/609518 | 21:39 |
| openstackgerrit | Merged openstack/nova master: Skip _remove_deleted_instances_allocations if compute is new https://review.openstack.org/609552 | 21:47 |
| *** irclogbot_4 has quit IRC | 21:53 | |
| *** irclogbot_4 has joined #openstack-nova | 21:54 | |
| *** jistr has quit IRC | 21:55 | |
| *** jistr has joined #openstack-nova | 21:56 | |
| *** jistr has quit IRC | 22:05 | |
| *** mriedem is now known as mriedem_away | 22:05 | |
| *** jistr has joined #openstack-nova | 22:06 | |
| *** spatel has quit IRC | 22:11 | |
| *** pcaruana has quit IRC | 22:20 | |
| *** zer0c00l has joined #openstack-nova | 22:22 | |
| *** elbragstad has quit IRC | 22:56 | |
| *** spatel has joined #openstack-nova | 22:57 | |
| *** irclogbot_4 has quit IRC | 22:58 | |
| *** tbachman has joined #openstack-nova | 23:01 | |
| *** spatel has quit IRC | 23:01 | |
| *** spatel has joined #openstack-nova | 23:03 | |
| *** spatel has quit IRC | 23:08 | |
| *** elbragstad has joined #openstack-nova | 23:15 | |
| *** elbragstad has quit IRC | 23:15 | |
| *** jmlowe has joined #openstack-nova | 23:18 | |
| *** owalsh_ has joined #openstack-nova | 23:18 | |
| *** tbachman has quit IRC | 23:18 | |
| *** owalsh has quit IRC | 23:21 | |
| *** slaweq has quit IRC | 23:32 | |
| *** mlavalle has quit IRC | 23:42 | |
| *** gyee has quit IRC | 23:52 | |
| *** artom has joined #openstack-nova | 23:54 | |
| *** spatel has joined #openstack-nova | 23:55 | |
| *** spatel has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
