Monday, 2017-10-23

*** acormier has quit IRC00:01
*** edmondsw has joined #openstack-nova00:02
*** acormier has joined #openstack-nova00:02
*** thorst has joined #openstack-nova00:03
openstackgerritMichael Still proposed openstack/nova master: Move shred to privsep.  https://review.openstack.org/49553700:06
openstackgerritMichael Still proposed openstack/nova master: Move xend existence probes to privsep.  https://review.openstack.org/49553800:06
openstackgerritMichael Still proposed openstack/nova master: Move the idmapshift binary into privsep.  https://review.openstack.org/49554100:06
openstackgerritMichael Still proposed openstack/nova master: Move loopback setup and removal to privsep.  https://review.openstack.org/49566400:06
openstackgerritMichael Still proposed openstack/nova master: Move nbd commands to privsep.  https://review.openstack.org/50035100:06
openstackgerritMichael Still proposed openstack/nova master: Move kpartx calls to privsep.  https://review.openstack.org/50035400:06
openstackgerritMichael Still proposed openstack/nova master: Move blkid calls to privsep.  https://review.openstack.org/50039800:06
*** edmondsw has quit IRC00:06
*** acormier has quit IRC00:06
*** thorst has quit IRC00:07
*** thorst has joined #openstack-nova00:07
*** salv-orlando has quit IRC00:07
*** salv-orlando has joined #openstack-nova00:08
*** thorst has quit IRC00:08
*** yangyape_ has quit IRC00:17
*** yangyapeng has joined #openstack-nova00:18
*** rodolof has joined #openstack-nova00:20
*** yangyapeng has quit IRC00:22
*** acormier has joined #openstack-nova00:28
*** edmondsw has joined #openstack-nova00:33
*** acormier has quit IRC00:36
*** acormier has joined #openstack-nova00:36
*** rodolof has quit IRC00:38
*** hieulq has joined #openstack-nova00:43
*** armax has joined #openstack-nova00:45
*** cheneydc has joined #openstack-nova00:46
*** huanxie has joined #openstack-nova00:54
*** edmondsw has quit IRC00:56
*** phuongnh has joined #openstack-nova01:01
*** armax has quit IRC01:02
*** armax has joined #openstack-nova01:02
*** armax has quit IRC01:03
*** armax has joined #openstack-nova01:03
*** armax has quit IRC01:03
*** armax has joined #openstack-nova01:04
*** armax has quit IRC01:04
*** armax has joined #openstack-nova01:05
*** armax has quit IRC01:05
*** armax has joined #openstack-nova01:06
*** armax has quit IRC01:06
*** salv-orl_ has joined #openstack-nova01:07
*** Shunli has joined #openstack-nova01:07
*** salv-orlando has quit IRC01:10
*** thorst has joined #openstack-nova01:11
*** thorst has quit IRC01:11
*** litao__ has joined #openstack-nova01:17
*** armax has joined #openstack-nova01:18
*** armax has quit IRC01:19
*** yangyapeng has joined #openstack-nova01:20
*** liusheng has quit IRC01:21
openstackgerritjichenjc proposed openstack/nova master: Refactor placement version check  https://review.openstack.org/51249701:25
*** hongbin has joined #openstack-nova01:26
*** suresh12 has quit IRC01:28
*** fried_rice is now known as efried01:32
*** suresh12 has joined #openstack-nova01:39
openstackgerritjichenjc proposed openstack/nova master: Refactor placement version check  https://review.openstack.org/51249701:39
*** psachin has joined #openstack-nova01:39
*** jichen has joined #openstack-nova01:40
*** trungnv has joined #openstack-nova01:40
*** suresh12 has quit IRC01:45
*** acormier_ has joined #openstack-nova01:48
*** acormier has quit IRC01:52
*** suresh12 has joined #openstack-nova01:53
*** vladikr has joined #openstack-nova01:54
*** edmondsw has joined #openstack-nova01:55
*** edmondsw has quit IRC01:59
*** liusheng has joined #openstack-nova02:04
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Create specs directory for Rocky  https://review.openstack.org/51410102:13
*** Tom____ has joined #openstack-nova02:15
*** gcb has joined #openstack-nova02:15
*** liusheng has quit IRC02:22
*** yikun_jiang has quit IRC02:22
*** liusheng has joined #openstack-nova02:22
*** hshiina has joined #openstack-nova02:22
*** yikun has joined #openstack-nova02:23
*** xinliang has quit IRC02:23
*** salv-orl_ has quit IRC02:25
*** suresh12 has quit IRC02:25
*** salv-orlando has joined #openstack-nova02:25
*** cheneydc has quit IRC02:26
*** acormier_ has quit IRC02:32
*** acormier has joined #openstack-nova02:32
*** xinliang has joined #openstack-nova02:35
*** acormier has quit IRC02:37
*** suresh12 has joined #openstack-nova02:40
*** salv-orlando has quit IRC02:43
*** salv-orlando has joined #openstack-nova02:44
*** chyka has joined #openstack-nova02:45
*** suresh12 has quit IRC02:45
*** masber has quit IRC02:47
*** acormier has joined #openstack-nova02:53
openstackgerritjichenjc proposed openstack/nova master: Add more functional test for placement.aggregates  https://review.openstack.org/51410902:53
*** thorst has joined #openstack-nova03:02
*** vladikr has quit IRC03:04
*** vladikr has joined #openstack-nova03:05
*** dave-mccowan has quit IRC03:05
*** thorst has quit IRC03:08
*** chyka has quit IRC03:11
openstackgerritYikun Jiang proposed openstack/nova master: Add migration_get_by_uuid in db api.  https://review.openstack.org/51142103:15
*** hieulq has quit IRC03:23
*** hieulq has joined #openstack-nova03:23
*** suresh12 has joined #openstack-nova03:25
*** nicolasbock has quit IRC03:26
*** suresh12_ has joined #openstack-nova03:27
*** acormier has quit IRC03:28
*** acormier has joined #openstack-nova03:29
*** suresh12 has quit IRC03:30
*** acormier has quit IRC03:30
*** acormier has joined #openstack-nova03:30
*** masber has joined #openstack-nova03:33
*** sree has joined #openstack-nova03:37
*** vladikr has quit IRC03:38
*** vladikr has joined #openstack-nova03:39
*** edmondsw has joined #openstack-nova03:43
*** edmondsw has quit IRC03:47
*** udesale has joined #openstack-nova03:49
*** Tom____ has quit IRC03:51
*** Tom____ has joined #openstack-nova03:51
*** hshiina has quit IRC03:54
*** Tom____ has quit IRC03:56
*** hshiina has joined #openstack-nova03:57
*** mdnadeem has joined #openstack-nova03:59
*** acormier has quit IRC03:59
*** vladikr has quit IRC03:59
*** armax has joined #openstack-nova04:00
*** acormier has joined #openstack-nova04:00
*** vladikr has joined #openstack-nova04:01
*** acormier has quit IRC04:05
*** armax has quit IRC04:09
*** penick has joined #openstack-nova04:11
*** claudiub|2 has joined #openstack-nova04:11
*** hongbin has quit IRC04:12
*** yamamoto has joined #openstack-nova04:16
*** Dinesh_Bhor has joined #openstack-nova04:19
*** bhagyashris has joined #openstack-nova04:20
*** syqian has joined #openstack-nova04:28
*** suresh12_ has quit IRC04:33
*** vks1 has joined #openstack-nova04:37
*** suresh12 has joined #openstack-nova04:37
*** penick has quit IRC04:58
*** jaosorior has joined #openstack-nova05:02
*** claudiub|2 has quit IRC05:03
*** Tom____ has joined #openstack-nova05:08
*** vladikr has quit IRC05:11
*** diga has joined #openstack-nova05:24
*** claudiub|2 has joined #openstack-nova05:25
*** suresh12 has quit IRC05:27
*** takashin_ has joined #openstack-nova05:30
*** suresh12 has joined #openstack-nova05:30
*** sree_ has joined #openstack-nova05:31
*** sridharg has joined #openstack-nova05:31
*** sree_ is now known as Guest3893205:31
*** edmondsw has joined #openstack-nova05:31
*** takashin has quit IRC05:32
*** sree has quit IRC05:34
*** suresh12 has quit IRC05:34
*** yangyapeng has quit IRC05:36
*** edmondsw has quit IRC05:36
*** yangyapeng has joined #openstack-nova05:36
*** yangyapeng has quit IRC05:41
*** trinaths has joined #openstack-nova05:49
*** spectr has joined #openstack-nova05:52
*** niraj_singh has quit IRC05:54
*** Oku_OS-away is now known as Oku_OS06:09
*** udesale__ has joined #openstack-nova06:09
*** udesale has quit IRC06:10
*** sahid has joined #openstack-nova06:11
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix an example in "Delete Assisted Volume Snapshot"  https://review.openstack.org/51412806:11
*** udesale has joined #openstack-nova06:12
*** edand has joined #openstack-nova06:13
*** udesale__ has quit IRC06:14
Dinesh_Bhorcfriesen: Hi, you there? I want to discuss about this: https://review.openstack.org/#/c/512990/06:16
*** namnh has joined #openstack-nova06:16
*** josecastroleon has joined #openstack-nova06:17
namnhlyarwood: hi lee, do you know a etherpad for skip-upgrade(FFU) at sydney?06:18
*** rqw has joined #openstack-nova06:20
*** rqw has left #openstack-nova06:21
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2)  https://review.openstack.org/40895506:23
openstackgerritTakashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2)  https://review.openstack.org/40896406:23
*** Eran_Kuris has joined #openstack-nova06:24
*** andreas_s has joined #openstack-nova06:26
openstackgerritYushiro FURUKAWA proposed openstack/nova master: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648706:32
*** yamamoto has quit IRC06:33
*** josecastroleon has quit IRC06:37
*** ociuhandu has joined #openstack-nova06:39
*** avolkov has joined #openstack-nova06:46
*** yamamoto has joined #openstack-nova06:52
*** rodolof has joined #openstack-nova06:53
*** Tom____ has quit IRC06:53
*** karthiks has joined #openstack-nova06:53
*** Tom____ has joined #openstack-nova06:54
openstackgerritHuan Xie proposed openstack/nova master: VGPU: Define vgpu resource class  https://review.openstack.org/51118006:57
*** Tom_____ has joined #openstack-nova06:57
*** Tom____ has quit IRC06:58
*** pcaruana has joined #openstack-nova07:00
*** jangutter has joined #openstack-nova07:01
*** slaweq has joined #openstack-nova07:04
*** josecastroleon has joined #openstack-nova07:05
*** andreas_s has quit IRC07:09
*** andreas_s has joined #openstack-nova07:09
*** ircuser-1 has joined #openstack-nova07:10
*** yamamoto_ has joined #openstack-nova07:11
*** yamamoto has quit IRC07:13
*** andreas_s has quit IRC07:14
*** edmondsw has joined #openstack-nova07:19
*** edmondsw has quit IRC07:24
*** andreas_s has joined #openstack-nova07:24
*** andreas_s has quit IRC07:25
*** andreas_s has joined #openstack-nova07:25
lyarwoodnamnh: I don't no sorry07:26
*** jistr is now known as jistr|mtgs07:26
*** tesseract has joined #openstack-nova07:27
*** yamamoto_ has quit IRC07:29
*** yamamoto has joined #openstack-nova07:29
*** hshiina has quit IRC07:30
*** sshwarts has joined #openstack-nova07:31
openstackgerritMerged openstack/nova stable/ocata: Account for compute.metrics.update in legacy notification whitelist  https://review.openstack.org/50944007:36
openstackgerritMerged openstack/nova stable/pike: Add _wait_for_action_fail_completion to InstanceHelperMixin  https://review.openstack.org/50992207:37
openstackgerritMerged openstack/nova stable/pike: Add recreate test for live migrate rollback not cleaning up dest allocs  https://review.openstack.org/50992307:37
*** AlexeyAbashkin has joined #openstack-nova07:37
*** jpena|off is now known as jpena07:43
*** yamamoto has quit IRC07:43
*** rtjure has joined #openstack-nova07:45
*** Tom_____ has quit IRC07:46
*** salv-orlando has quit IRC07:50
*** salv-orlando has joined #openstack-nova07:50
*** salv-orl_ has joined #openstack-nova07:55
*** yamamoto has joined #openstack-nova07:57
*** salv-orlando has quit IRC07:58
*** Eran_Kuris has quit IRC07:59
*** yamamoto has quit IRC08:03
openstackgerritMichael Still proposed openstack/nova master: Move nbd commands to privsep.  https://review.openstack.org/50035108:03
openstackgerritMichael Still proposed openstack/nova master: Move kpartx calls to privsep.  https://review.openstack.org/50035408:03
openstackgerritMichael Still proposed openstack/nova master: Move blkid calls to privsep.  https://review.openstack.org/50039808:03
*** AlexeyAbashkin has quit IRC08:12
*** AlexeyAbashkin has joined #openstack-nova08:12
*** Eran_Kuris has joined #openstack-nova08:14
*** rtjure has quit IRC08:14
*** Tom____ has joined #openstack-nova08:16
*** owalsh_ is now known as owalsh08:18
*** rtjure has joined #openstack-nova08:20
*** ralonsoh has joined #openstack-nova08:22
*** finucannot is now known as stephenfin08:23
*** rtjure has quit IRC08:25
*** rtjure has joined #openstack-nova08:30
openstackgerritChris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates  https://review.openstack.org/51352608:31
*** derekh has joined #openstack-nova08:32
*** spectr has quit IRC08:35
*** rtjure has quit IRC08:35
*** lucas-afk is now known as lucasagomes08:36
*** gszasz has joined #openstack-nova08:36
*** yamamoto has joined #openstack-nova08:36
*** yamamoto has quit IRC08:38
*** gszasz_ has joined #openstack-nova08:39
*** diga has quit IRC08:40
*** rtjure has joined #openstack-nova08:40
*** gszasz has quit IRC08:40
*** rtjure has quit IRC08:44
*** dtantsur|afk is now known as dtantsur08:45
*** vks1 has quit IRC08:48
*** spectr has joined #openstack-nova08:49
*** yamamoto has joined #openstack-nova08:50
openstackgerritsahid proposed openstack/nova master: libvirt: rework _get_vpcu_total to return online pCPUs  https://review.openstack.org/50973008:53
openstackgerritsahid proposed openstack/nova master: libvirt: set emulator threads on isolated host cpus  https://review.openstack.org/51089708:53
*** rtjure has joined #openstack-nova08:54
*** yikun_jiang has joined #openstack-nova08:57
*** yikun has quit IRC08:58
*** rtjure has quit IRC08:59
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Change Subnet.dhcp_server to contain multiple IPs  https://review.openstack.org/50910709:01
*** josecastroleon has quit IRC09:02
*** yamamoto has quit IRC09:03
*** josecastroleon has joined #openstack-nova09:04
*** jmlowe_ has joined #openstack-nova09:07
*** rtjure has joined #openstack-nova09:07
*** edmondsw has joined #openstack-nova09:08
*** jmlowe has quit IRC09:08
*** vks1 has joined #openstack-nova09:09
*** derekh has quit IRC09:11
*** edmondsw has quit IRC09:12
*** rtjure has quit IRC09:12
*** yamamoto has joined #openstack-nova09:13
*** derekh has joined #openstack-nova09:13
*** sambetts|afk is now known as sambetts09:15
*** rtjure has joined #openstack-nova09:17
*** ygl has joined #openstack-nova09:17
yglhi all09:17
yglcan some one point me to the exact code the libvirt uses to launch a vm on kvm hypervisor in nova ?09:18
*** rtjure has quit IRC09:21
*** claudiub has joined #openstack-nova09:23
yglcan some one point me to the exact code the libvirt uses to launch a vm on kvm hypervisor in nova ?09:24
ygldoes it use qemu-kvm binary or virt-install ?09:25
*** claudiub|2 has quit IRC09:26
*** mvk has quit IRC09:27
*** Shunli has quit IRC09:28
*** rtjure has joined #openstack-nova09:29
openstackgerritsahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/49745709:29
*** vvargaszte has joined #openstack-nova09:30
*** spectr has quit IRC09:31
*** spectr has joined #openstack-nova09:31
stephenfinbauzas: How much do you know about gate jobs?09:33
*** vvargaszte has quit IRC09:33
*** rtjure has quit IRC09:34
*** salv-orl_ has quit IRC09:36
*** tyrefors has quit IRC09:37
*** salv-orlando has joined #openstack-nova09:37
*** rtjure has joined #openstack-nova09:41
*** salv-orlando has quit IRC09:41
*** takashin_ has left #openstack-nova09:41
*** sdague has joined #openstack-nova09:44
*** rtjure has quit IRC09:46
openstackgerritAlex Xu proposed openstack/nova master: Add _get_provider_ids_with_any_trait method  https://review.openstack.org/51118409:49
openstackgerritAlex Xu proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters  https://review.openstack.org/47976609:49
openstackgerritAlex Xu proposed openstack/nova master: Add get_resource_provider_with_any_resource method  https://review.openstack.org/51419709:49
openstackgerritAlex Xu proposed openstack/nova master: Add ProviderSummaryList object  https://review.openstack.org/51419809:49
openstackgerritAlex Xu proposed openstack/nova master: Add more caches for the ProviderSummary  https://review.openstack.org/51419909:49
openstackgerritAlex Xu proposed openstack/nova master: Add AggregatedProviderSummary and AggregatedProviderSummaryList objects  https://review.openstack.org/51420009:49
*** Eran_Kuris has quit IRC09:50
*** Eran_Kuris has joined #openstack-nova09:50
*** andreas_s has quit IRC09:51
*** andreas_s has joined #openstack-nova09:52
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352609:52
*** rtjure has joined #openstack-nova09:52
*** gcb has quit IRC09:56
*** rtjure has quit IRC09:57
*** yamamoto has quit IRC09:58
*** yamamoto has joined #openstack-nova09:59
*** gcb has joined #openstack-nova10:00
*** andreas_s has quit IRC10:00
*** Guest38932 has quit IRC10:00
*** andreas_s has joined #openstack-nova10:01
*** mvk has joined #openstack-nova10:02
bauzasstephenfin: hi, sorry about the delay10:02
bauzasstephenfin: I have a plumber at home10:02
bauzas(not mario FTW)10:02
bauzashow can I help you ?10:02
*** rtjure has joined #openstack-nova10:03
*** spectr has quit IRC10:04
*** yamamoto has quit IRC10:06
*** udesale has quit IRC10:06
*** andreas_s has quit IRC10:06
*** yamamoto has joined #openstack-nova10:06
sahidygl: we use qemu-kvm, but that through libvirt10:06
*** Guest7994 has quit IRC10:06
yglsahid: thanks very much10:07
*** Tom____ has quit IRC10:08
*** rtjure has quit IRC10:08
*** andreas_s has joined #openstack-nova10:16
*** rtjure has joined #openstack-nova10:16
stephenfinbauzas: I _think_ I might be OK for now but stay away from the mushrooms til I know for sure ;)10:17
*** spectr has joined #openstack-nova10:21
*** rtjure has quit IRC10:21
*** namnh has quit IRC10:22
*** syqian_ has joined #openstack-nova10:22
*** syqian__ has joined #openstack-nova10:24
*** syqian has quit IRC10:25
*** andreas_s has quit IRC10:25
*** rtjure has joined #openstack-nova10:25
*** vks1 has quit IRC10:26
*** gmann is now known as gmann_afk10:26
*** syqian_ has quit IRC10:27
*** andreas_s has joined #openstack-nova10:30
*** rtjure has quit IRC10:30
*** Tom____ has joined #openstack-nova10:33
*** andreas_s has quit IRC10:35
*** ygl has quit IRC10:35
*** rtjure has joined #openstack-nova10:35
*** Tom____ has quit IRC10:37
*** yamamoto_ has joined #openstack-nova10:37
*** yangyapeng has joined #openstack-nova10:38
*** andreas_s has joined #openstack-nova10:39
*** rtjure has quit IRC10:40
*** yamamoto has quit IRC10:40
*** gszasz_ has quit IRC10:41
*** andreas_s has quit IRC10:42
*** andreas_s has joined #openstack-nova10:42
*** vks1 has joined #openstack-nova10:42
*** yangyapeng has quit IRC10:43
*** yangyapeng has joined #openstack-nova10:44
*** spectr has quit IRC10:45
*** spectr has joined #openstack-nova10:47
*** yamamoto_ has quit IRC10:48
*** yamamoto has joined #openstack-nova10:50
*** Qiming has quit IRC10:50
openstackgerritMerged openstack/nova stable/pike: Add live.migration.force.complete to the legacy notification whitelist  https://review.openstack.org/50887710:50
openstackgerritMerged openstack/nova stable/ocata: Make conductor create InstanceAction in the proper cell  https://review.openstack.org/51130710:50
openstackgerritMerged openstack/nova stable/ocata: Target context for build notification in conductor  https://review.openstack.org/51132510:50
openstackgerritMerged openstack/nova master: propagate OSError to MigrationPreCheckError  https://review.openstack.org/46977910:51
openstackgerritMerged openstack/nova stable/ocata: Handle keypair not found from metadata server using cells  https://review.openstack.org/50095410:51
*** jichen_ has joined #openstack-nova10:51
*** jichen has quit IRC10:52
*** jichen_ is now known as jichen10:52
*** Qiming has joined #openstack-nova10:54
*** rtjure has joined #openstack-nova10:55
*** yamamoto has quit IRC10:55
*** yamamoto has joined #openstack-nova10:55
*** vladikr has joined #openstack-nova10:56
*** jichen has quit IRC10:57
*** cdent has joined #openstack-nova10:57
*** gcb has quit IRC10:58
*** Qiming has quit IRC10:58
*** rtjure has quit IRC10:59
*** gszasz has joined #openstack-nova11:00
efriedalex_xu Good evening, you still around?11:00
*** jichen has joined #openstack-nova11:00
*** Qiming has joined #openstack-nova11:01
*** sean-k-mooney has joined #openstack-nova11:02
*** dave-mccowan has joined #openstack-nova11:03
*** rtjure has joined #openstack-nova11:04
*** nicolasbock has joined #openstack-nova11:07
*** rtjure has quit IRC11:09
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103511:13
*** yassine has joined #openstack-nova11:16
*** cdent has quit IRC11:16
*** yassine is now known as Guest4026411:16
*** cdent has joined #openstack-nova11:16
*** rtjure has joined #openstack-nova11:17
*** lucasagomes is now known as lucas-hungry11:17
*** gszasz has quit IRC11:17
*** smatzek has joined #openstack-nova11:20
*** rtjure has quit IRC11:22
*** tyrefors has joined #openstack-nova11:26
*** rtjure has joined #openstack-nova11:28
*** yamamoto_ has joined #openstack-nova11:30
*** yamamoto_ has quit IRC11:31
*** rtjure has quit IRC11:32
*** yamamoto has quit IRC11:32
*** yamamoto has joined #openstack-nova11:32
*** litao__ has quit IRC11:34
*** jpena is now known as jpena|lunch11:36
*** phuongnh has quit IRC11:37
*** rtjure has joined #openstack-nova11:38
*** nicolasbock has quit IRC11:39
*** lpetrut has joined #openstack-nova11:39
*** yikun has joined #openstack-nova11:39
*** yikun_jiang has quit IRC11:42
*** erlon has joined #openstack-nova11:42
*** dtantsur is now known as dtantsur|brb11:42
*** rodolof has quit IRC11:43
*** rtjure has quit IRC11:43
*** rodolof has joined #openstack-nova11:43
*** rtjure has joined #openstack-nova11:48
*** penick has joined #openstack-nova11:49
*** penick has quit IRC11:49
efriedcdent Ready for a lunchtime spitballing session?11:51
cdentone moment please caller11:51
*** yamamoto has quit IRC11:52
*** nicolasbock has joined #openstack-nova11:52
*** rtjure has quit IRC11:53
*** yamamoto has joined #openstack-nova11:53
openstackgerritEric Fried proposed openstack/nova master: Include /resource_providers/uuid/allocations link  https://review.openstack.org/49982611:55
*** yamamoto has quit IRC11:58
cdentefried: k, was just finishing up an email about correcting a draft of my will. what’s up?11:58
efriedheh, that seems fairly important.11:58
*** rtjure has joined #openstack-nova11:58
*** Tom____ has joined #openstack-nova11:59
efriedcdent I was reviewing your 'limit allocation candidates' patch and it occurred to me that the 'randomize' conf setting should apply whether we're limiting or not.11:59
efriedLeft a comment to that effect, but wanted to brainstorm with you and see if you agree.11:59
*** thorst has joined #openstack-nova11:59
* cdent thinks11:59
cdentI’m guessing you’re thinking: if we’re going to randomize at all, it should be whether there is a limit or not, and if it going to happen it should happen in one place, not in multiple clients?12:00
*** thorst_ has joined #openstack-nova12:00
efriedcdent Javol12:00
efriedWere we randomizing from the scheduler somehow?12:01
*** thorst_ has quit IRC12:01
cdentwe were leaving todos about needing to decide if/how to select from too many candidates12:01
efriedOh, then yeah, IMO this is that decision, and this is the sensible place to do it.12:02
cdentthe belief is that most people will not want to randomize, as that will tend to spread, which is apparently not a common desire (I don’t actually know how certain or true that is)12:02
cdentyeah, I think you’re right.12:02
efriedOkay, that answers the other point of whether we should default to True - sounds like no.12:03
*** rtjure has quit IRC12:03
*** liusheng has quit IRC12:03
cdentI think we should, but apparently that would change behaviors in an unacceptable way12:03
efriedEven though the order of candidates is undefined today.12:04
cdentand bias in favor of the user rather than ops or something12:04
*** thorst has quit IRC12:04
cdentit is undefined, but semi-static12:04
efriedBut if we've ascertained that the general preference is pack vs. spread, we should indeed preserve that behavior.12:04
cdentthe database will do its own ordering, with some bias12:04
*** salv-orlando has joined #openstack-nova12:04
*** thorst has joined #openstack-nova12:04
efriedWhich will be deterministic, but not really predictable by a human.12:04
cdenthowever, I expect that the complexity of the queries thes days, and the inclusion of usage informatin throws off a lot of that ordering12:05
cdentbut it was deemed an important part of the plan when discussed in denver12:05
*** nicolasbock has quit IRC12:05
efriedcdent Also, just a point of curiosity, were we planning to use the limit param from the scheduler?  I would think so - it should be fairly simple to come up with a number that would always give us enough candidates.  And that number would be pretty small, too, like on the order of a dozen or so.12:06
cdentso I reckon the change is fairly simple: if not limit and conf thing true: random.shuffle(allocs)12:06
*** nicolasbock has joined #openstack-nova12:06
cdentefried: yes, that’s eventual hope: that the scheduler would retrieve much less data12:06
cdentsome math of needed instances X needed alternates X fudge factor12:07
efriedYeah, cause at this point we haven't chosen a cell yet, right?12:07
efriedOnce we pick a cell we get down to three alternates (per instance)12:08
cdentthe cell is chosen based on the “claimed” destination12:08
cdentthree isn’t static is it, that comes from conf, yeah?12:08
efriededleafe ^ ?12:08
efriedIn any case, that would make the math simple.  #alternates * #instances12:08
cdentneeded x retries x cells12:08
efriedOkay.  Not critical right now, but it's on the radar - cool.12:09
cdentif we don’t include X cells, we may not end up with enough space to pick from12:09
cdentyeah, on the radar - cool12:09
cdentglad you though of that12:09
*** rtjure has joined #openstack-nova12:10
cdentefried: if you want another thing to do with with your copious free time, I’ve got some microversion_parse changes that need some thinking about and there are few people who are aware of all the bits of context, you might be a good reviewer: https://review.openstack.org/#/c/496212/ and https://review.openstack.org/#/c/495356/12:11
cdentno rush on that (at all) just stuff I’m sort of rolling around for the cool radar12:11
*** josecastroleon has quit IRC12:11
efriedFor my part, I got started on numbered groupings in GET /allocation_candidates12:11
efriedWe've got like a 5-way race to placement microversion 1.11.12:12
cdentIs it only 5?12:13
*** josecastroleon has joined #openstack-nova12:14
*** rtjure has quit IRC12:15
*** huanxie has quit IRC12:16
*** lucas-hungry is now known as lucasagomes12:18
*** weshay|bbiab is now known as weshay|ruck12:18
*** rtjure has joined #openstack-nova12:21
*** edmondsw has joined #openstack-nova12:21
*** vivsoni has quit IRC12:21
*** edmondsw has quit IRC12:21
*** vivsoni has joined #openstack-nova12:22
*** edmondsw has joined #openstack-nova12:22
*** josecastroleon has quit IRC12:22
*** mariusv has quit IRC12:25
*** mariusv has joined #openstack-nova12:26
*** rtjure has quit IRC12:26
*** gszasz has joined #openstack-nova12:28
*** claudiub|2 has joined #openstack-nova12:30
*** claudiub has quit IRC12:33
*** yamamoto has joined #openstack-nova12:36
*** nicolasbock has quit IRC12:39
*** jpena|lunch is now known as jpena12:40
*** ociuhandu has quit IRC12:40
*** pcaruana has quit IRC12:41
*** edand has quit IRC12:41
*** lyan has joined #openstack-nova12:43
efrieddtantsur|brb I'd like to be invited to the discussion of ironic traits vs. configurables if possible.  Thanks!12:44
*** yamamoto has quit IRC12:46
*** nicolasbock has joined #openstack-nova12:46
*** vivsoni_ has joined #openstack-nova12:46
*** vivsoni has quit IRC12:46
openstackgerritJan Zerebecki proposed openstack/nova master: Only log not correcting allocation once per period  https://review.openstack.org/50826212:47
openstackgerritjichenjc proposed openstack/nova master: Remove duplicate error info  https://review.openstack.org/51071912:48
*** pchavva has joined #openstack-nova12:49
*** penick has joined #openstack-nova12:55
*** huanxie has joined #openstack-nova12:56
*** edand has joined #openstack-nova12:56
cdentmaybe we should put traits in etcc and make them watchable12:56
cdentjk/212:56
cdents/etcc/etcd/12:57
*** chyka has joined #openstack-nova12:57
*** lennyb has quit IRC12:59
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project  https://review.openstack.org/51430913:01
*** chyka has quit IRC13:01
*** yamamoto has joined #openstack-nova13:02
*** jdillaman has joined #openstack-nova13:02
*** lennyb has joined #openstack-nova13:05
*** mriedem has joined #openstack-nova13:07
*** penick has quit IRC13:10
bauzasefried: traits are not boolean13:12
efriedbauzas In the sense that they're True if they're present and False if they're absent, kinda?13:12
bauzasefried: turning off a capability means we remove the trait13:12
efriedoh, no, I'm pretty sure that's the entire bone of contention here.13:13
bauzaslemme respond on the ML thread13:13
efriedThough to be precise, you're correct: if we remove a *capability* (which is different from turning off a setting), we should remove the trait from that RP.13:13
efriedbauzas Which is consistent with what I said, though I should have been explicit about it.13:13
cdentremoving and turning off not the same thing13:13
efriedRight.13:14
efriedIf I make it so the RP can no longer do the thing, that should come with removing the trait.13:14
efriedIf the RP is still capable of doing the thing, and I just switch it off, I should leave the trait in place.13:14
efriedBut even capable vs. on/off is a sticky thing with ironic.13:15
*** salv-orl_ has joined #openstack-nova13:15
efriedShould more precisely say "capable of being turned on during spawn".13:15
bauzascdent: a trait just means "we support that capability"13:16
bauzasif we don't support it, then we remove the trait13:16
cdentyes, that’s what I was saying13:16
bauzaswe couldn't support it because we had turned off something13:16
*** salv-orlando has quit IRC13:16
bauzasbut then, if we turn off something, then we delete the trait13:16
bauzasthat's MHO13:16
bauzasexactly like a CPU feature13:17
efriedbauzas Agree, as long as "we" is e.g. "the admin", not something that can be turned on via the spawn process.13:17
cdenthmm, I suspect it depends on what is doing the action, and whether changing it can be automated13:17
efriedcdent ++13:17
cdentjinx-ish13:17
*** salv-orl_ has quit IRC13:17
*** salv-orlando has joined #openstack-nova13:18
efriedI think we're all in agreement here.  Just needs to be spelled out carefully.13:18
* cdent nods13:20
openstackgerritJan Zerebecki proposed openstack/nova master: Only log not correcting allocation once per period  https://review.openstack.org/50826213:21
*** Dave has quit IRC13:22
*** jmlowe_ has quit IRC13:22
*** xinliang has quit IRC13:23
*** lennyb has quit IRC13:23
*** sree has joined #openstack-nova13:23
*** xyang1 has joined #openstack-nova13:24
alex_xuefried: hi, i'm here13:24
*** baoli has joined #openstack-nova13:25
efriedalex_xu I wanted to coordinate with your refactoring of AllocationCandidates.get_by_filters.13:25
alex_xuefried: yea13:25
efriedalex_xu As I started implementing the numbered grouping syntax...13:25
*** gcb has joined #openstack-nova13:25
alex_xuefried: ok...hope you didn't start too much...13:25
*** liverpooler has joined #openstack-nova13:26
*** smatzek has quit IRC13:26
efriedalex_xu https://review.openstack.org/514091 and https://review.openstack.org/51409213:26
efriedalex_xu First off, I'm completely changing the format of the 'filters' parameter.  That's described in that first patch.13:27
*** smatzek has joined #openstack-nova13:27
*** liverpoo1er has joined #openstack-nova13:27
*** Dave has joined #openstack-nova13:27
*** lennyb has joined #openstack-nova13:27
*** huanxie has quit IRC13:27
alex_xuefried: ok, that's cool13:27
efriedalex_xu Which I was going to implement, at least at first, by creating a separate method in AllocationCandidates - get_by_filters_grouped13:28
*** slaweq has quit IRC13:28
efriedBut of course, get_by_filters_grouped would be able to use a lot of the same functions you're refactoring out of get_by_filters, which is good.13:28
*** slaweq has joined #openstack-nova13:28
efriedalex_xu The place we need to coordinate initially, I think, is the parsing of the querystring to create the filters - that first patch.13:29
alex_xuefried: ok, I will read your patch13:29
efriedalex_xu Because at some point you were going to make changes to parse the 'required' query param to feed into your series.13:29
efriedalex_xu You haven't started that work yet, have you?13:30
alex_xuefried: I have, https://review.openstack.org/#/c/479776/ but didn't update recently13:30
*** liverpooler has quit IRC13:31
efriedalex_xu Okay, I'll read that one over.13:31
*** smatzek has quit IRC13:31
alex_xuefried: we probably need to put the schema and parse code of 'required' as common13:31
alex_xuefried: do you have any sample code for your get_by_filters_grouped13:31
alex_xuI guess that is the hard part13:31
efriedalex_xu Right, my thought was that we might as well do it all at once.  That is, make the API change from accepting just 'resources' to accepting 'resources[N]' + 'required[N]' in one microversion jump.13:32
*** slaweq has quit IRC13:33
*** tssurya has quit IRC13:33
alex_xuefried: since microversion is micro, we shouldn't put too much in single version. but that should be ok13:33
efriedalex_xu No, I haven't written any of that yet.  The easy part will be grabbing the candidates for a single numbered grouping.  Because (per the spec) that's always coming from a single RP - I don't need to mess with aggregates or trees.13:33
efriedalex_xu Slightly harder will be combining that with the results from the unnumbered group.  But even that shouldn't be *too* hard - we just need to match up the aggregates at that point.13:34
*** kylek3h has joined #openstack-nova13:35
efriedalex_xu But I haven't done any serious SQL in ~15y, and never done any sqlalchemy, so I'm definitely going to be looking for guidance/help/mentoring there.13:35
*** Eran_Kuris has quit IRC13:35
alex_xuefried: do you read the patch I just submitted https://review.openstack.org/#/q/topic:bp/add-trait-support-in-allocation-candidates+(status:open+OR+status:merged)?13:36
alex_xuI created few objects to instead of currenlty single huge method13:36
efriedalex_xu I saw those come through, but haven't looked them over yet.  But that's what prompted me to ping you.13:37
*** cdent has quit IRC13:37
alex_xuthe AggregatedProviderSummary obj will be the one to indicate a set of providers which are in the same aggregate13:37
*** cdent_ has joined #openstack-nova13:37
alex_xuso I'm think whether we need other object to build a tree for nested resource provider13:37
efriedalex_xu That already exists: ProviderTree13:38
alex_xuefried: ProviderTree is for nova, not for placement13:38
efriedoh, right.13:38
*** awaugama has joined #openstack-nova13:38
cdent_why do we need to represent the tree on the placement side? (not saying we don’t just want to clarify on why)13:38
efriedcdent_ Not strictly necessary, but sure does make it easier to conceptualize.13:39
cdent_“it”?13:39
efriedcdent_ For things like trait propagation13:39
*** mdnadeem has quit IRC13:39
cdent_I’m still not entirely clear on the reality of trait propagation13:39
cdent_especially in a world of a numbered resources13:40
mriedemdansmith: i think the bottom alternate hosts change is ready https://review.openstack.org/#/c/486215/13:40
mriedemyou and jay were +2 before13:40
alex_xuemm...any hint about trait propagation? my brain becomes empty, not sure due to the time or not...13:40
efriedcdent_ There was an example in the spec.  Possibly in review comments rather than in the actual text.  And it may not have been a great example, at that.13:41
efriedalex_xu cdent_ It may have been something like this:13:41
*** coreywright has quit IRC13:42
efriedalex_xu cdent_ I have multi-port SR-IOV NICs associated with NUMA nodes.  Each port is connected to a different physnet.13:42
dansmithmriedem: ack, I wanted to be happy with the usage of the object before we merged the one that added it, but I guess that bottom one is separate13:42
efriedalex_xu cdent_ So my model looks like: compute => numa node => PF13:43
*** yikun has quit IRC13:43
*** ragiman has joined #openstack-nova13:44
efriedalex_xu cdent_ The NUMA node has a trait indicating which node it is, and it holds the VCPU and MEMORY_MB inventory.  The PF has a trait indicating which physnet it's on, and it holds the VF inventory.13:44
*** salv-orl_ has joined #openstack-nova13:44
*** eharney has joined #openstack-nova13:44
*** salv-orlando has quit IRC13:45
efriedalex_xu cdent_ But the logic still needs to be able to know that the PF belongs to the NUMA node associated with its parent RP.13:45
alex_xuefried: the NUMA node needn't a ttrait to indicate, but nvm, let us continue your example13:45
*** tssurya has joined #openstack-nova13:46
alex_xuefried: yea, probably13:46
efriedPerhaps NUMA is a bad example because you wouldn't be making a request for a specific NUMA node.  This is more about affinity, which is for later.13:46
efriedLet me go see if I can dig up the example from the spec...13:46
*** jichen has quit IRC13:47
alex_xuefried: cdent_ I'm thinking we may need a obj just like this for shared rp https://review.openstack.org/#/c/514200/1/nova/objects/resource_provider.py@2552 to show the path for nested resource provider13:47
alex_xuso I mean we needn't build a tree13:47
alex_xuwhat we need a build a list of path for the condicates13:47
alex_xus/condicates/candidates13:48
alex_xuwhat we need is that building a list of pathes for the candidates13:48
efriedYou mean every possible "branch"?13:48
alex_xuefried: yea, branch13:48
*** takashin has joined #openstack-nova13:49
efriedIn the general case that could be a big list, but in reality I suppose it would probably be manageable.13:49
alex_xushrink the scope of all the branches by the sql, then filter the final branch in the code13:50
*** yangyapeng has quit IRC13:50
alex_xuif we want to done everything in the sql, that will be super complex sql just like current we have, and only Jay know how to write a such sql :)13:50
efriedalex_xu cdent_ Here's the example from the spec discussion: https://review.openstack.org/#/c/497713/6/specs/queens/approved/add-trait-support-in-allocation-candidates.rst@4213:50
*** yangyapeng has joined #openstack-nova13:51
*** hemna_ has joined #openstack-nova13:51
*** crushil has quit IRC13:52
*** vks1 has quit IRC13:52
*** trinaths has quit IRC13:53
*** jaypipes has joined #openstack-nova13:53
alex_xuefried: I see13:53
alex_xunot really propagation, but probably that is rule how we check each branch13:53
cdent_efried: yeah, I remember that. I think my issue isn’t so much with behavior as with terminology.13:54
efriedcdent_ We already spent a ton of emotional energy haggling over "propagate" vs "inherit".13:54
cdent_I agree that in the request example, you should get back a tree that includes the NUMA, and the compute node. that’s not quite the same as "hit"13:54
* cdent_ nods13:54
*** coreywright has joined #openstack-nova13:55
efriedcdent_ Well, in that context, "hit" means we actually got that candidate.  As opposed to NOT getting that candidate because the NUMA RP doesn't have the CPU trait itself.13:55
*** yangyapeng has quit IRC13:55
cdent_there is a successful allocation candidate because numa is in a tree that has the trait13:56
cdent_which maybe is exactly the same thing as you are saying13:56
*** hongbin has joined #openstack-nova13:56
efriedcdent_ Yes, except not "in a tree".13:57
cdent_well, it is13:57
efriedcdent_ Has to be at or above the resource.13:57
efriedYes, it's in the tree, but not *anywhere* in the tree.13:57
cdent_oh, yes “above"13:57
efriede.g. if the "cousin" numa node had the trait, that wouldn't be acceptable.13:57
*** jmlowe has joined #openstack-nova13:58
*** esberglu has joined #openstack-nova13:58
*** tojuvone has joined #openstack-nova13:58
efriedRewrite: if the "cousin" numa node had the trait (but the compute RP didn't), that wouldn't be acceptable.  (We would get the cousin, in fact.)13:58
cdent_I think the thing I’m getting hung up on is that in my mind the code is doing a “look up to see if I’m good enough”13:58
*** gcb has quit IRC13:58
cdent_but we keep talking about traits going down, and that just doesn’t say anything about the process13:58
*** gcb has joined #openstack-nova13:58
cdent_but in the end I think we agree, so probably best I just let it lie13:59
efriedWhere "good enough" includes both the notion of "has enough resources" and "has the right traits".  And we're just talking about the second thing right now.13:59
cdent_because my mental model for how placement works is probably, uh, unique13:59
efriedI agree the process is where there be tygers, but we're not supposed to worry about how the SQL's gonna work :)14:00
cdent_except that’s the part that’s the hardest to understand :(14:00
edleafeScheduler subteam meeting starting now in #openstack-meeting-alt14:00
efriedcdent_ Totally agree.14:01
jaypipesefried: not just the cousin. the brother/sister.14:02
efriedjaypipes Yes, right.14:02
efriedOr even second cousins once removed.14:02
stephenfinartom: What do you mean by the third comment here? https://review.openstack.org/#/c/345399/29/nova/tests/unit/console/securityproxy/test_rfb.py14:02
efriedBasically anything that ain't self or direct ancestor14:03
cdent_yeah, I think we all agree on that aspect14:03
*** cdent_ is now known as cdent14:05
*** sree_ has joined #openstack-nova14:05
*** dtantsur|brb is now known as dtantsur14:05
*** sree has quit IRC14:05
*** sree_ is now known as Guest5201414:05
*** crushil has joined #openstack-nova14:06
dtantsurefried, cdent, I'll send the doodle to the ML, will cc you, jaypipes and johnthetubaguy14:06
* dtantsur had errands in the city, will do now14:06
cdentthanks dtantsur14:06
efrieddtantsur Thanks!14:06
johnthetubaguyefried: should I re-read the scroll back?14:06
efriedjohnthetubaguy No, just the ML.14:06
*** mlavalle has joined #openstack-nova14:06
johnthetubaguyOK14:06
*** jaosorior has quit IRC14:08
*** ociuhandu has joined #openstack-nova14:09
*** smatzek has joined #openstack-nova14:09
*** smatzek has quit IRC14:10
*** vks1 has joined #openstack-nova14:10
*** smatzek has joined #openstack-nova14:10
*** ociuhandu has quit IRC14:12
*** slaweq has joined #openstack-nova14:13
artomstephenfin, dammit, your different upstream/downstream nicks confuse me14:16
artomstephenfin, to answer your question, \x04blah means, IIRC, reason-length of 4 bytes, then 'blah' is the reason length14:16
artomI was suggesting we test reason-lengths different to 4, since both test_fail and test_fail_server_message have the same content14:17
artom... with a reason-length of 414:17
*** namnh has joined #openstack-nova14:18
*** slaweq has quit IRC14:18
*** felipemonteiro_ has joined #openstack-nova14:18
artomOr, have a unit test just for _make_var_str, which is what transforms 'blah' into the '\x04blah' that you send over the wire14:18
artomAnd test it with at least 2 different reason-lengths14:19
*** felipemonteiro__ has joined #openstack-nova14:19
artomObviously not a massive deal14:19
artomJust seems more thorough :)14:19
*** gouthamr has joined #openstack-nova14:20
artomErr, in my second line, 'blah' is the reason, not reason length14:20
*** salv-orl_ has quit IRC14:20
efriedjohnthetubaguy In the scheduler meeting (#openstack-meeting-alt) there's about to be a discussion of whether you can remove a trait on a RP that has allocations - your interest is via https://review.openstack.org/#/c/504531/8/specs/approved/node-traits.rst@8614:20
*** salv-orlando has joined #openstack-nova14:20
*** david-lyle has joined #openstack-nova14:21
*** rodolof has quit IRC14:21
*** felipemonteiro_ has quit IRC14:23
*** gcb has quit IRC14:29
*** mlavalle has quit IRC14:29
*** mlavalle has joined #openstack-nova14:34
alex_xujaypipes: I try to refactor the alllocaion candidates in implement the traits support https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-trait-support-in-allocation-candidates, appreciate if you can give some feedback on the direction14:36
*** ragiman has quit IRC14:36
*** slaweq has joined #openstack-nova14:37
* sean-k-mooney thinks sorry to jaypipes for hardware defined software with stateful traits...14:37
*** slaweq has quit IRC14:37
*** tssurya has quit IRC14:37
*** slaweq has joined #openstack-nova14:37
*** jaypipes has quit IRC14:38
openstackgerritJan Zerebecki proposed openstack/nova master: Only log not correcting allocation once per period  https://review.openstack.org/50826214:39
*** jaypipes has joined #openstack-nova14:40
johnthetubaguyefried: did you cover that?14:40
efriedjohnthetubaguy Not yet.14:41
*** chyka has joined #openstack-nova14:41
*** baoli has quit IRC14:41
*** psachin has quit IRC14:42
*** baoli has joined #openstack-nova14:42
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Don't disregard cache mode for instance boot disks  https://review.openstack.org/51433914:43
*** peter-hamilton has joined #openstack-nova14:44
*** ragiman has joined #openstack-nova14:52
*** tssurya has joined #openstack-nova14:52
*** jaosorior has joined #openstack-nova14:57
*** yamahata has joined #openstack-nova14:58
*** takashin has left #openstack-nova15:00
cdentjaypipes: on the link state thing, we don’t want to kill an entire compute service for one pf do we?15:01
jaypipesalex_xu: yes, will try to review that whole series today. I'm doing new hire stuff and meetings today and tomorrow, though, so reviews will be slwo15:01
jaypipesslow15:01
alex_xucdent: jaypipes edleafe efried, the trait spec said the provider summary of the allocation candidates API response includes all the traits in that rp, not only the required. For the resources in the response, the current implementation is only include the requested resources, is that we expected? or it is bug15:02
jaypipessean-k-mooney: and yes, I definitely saw you mention "hardware-defined software" on the ML. :)15:02
alex_xujaypipes: thanks15:02
cdentalex_xu: sounds like bug15:02
jaypipesalex_xu: probably a bug15:02
efriedalex_xu In the provider summary or in the allocation_requests ?15:02
jaypipesjinx :)15:02
cdentthe provider summary should be “the provider"15:03
alex_xuefried: yea15:03
* cdent buys jaypipes a coke15:03
dtantsurcdent, jaypipes, johnthetubaguy, efried, https://doodle.com/poll/ts43k98kkvniv8uz (sorry, forgot to cc you)15:03
jaypipescdent: right, but there;'s a dict of resources in the provider summary showing remaining capacity for inventories that provider has.15:03
*** yangyapeng has joined #openstack-nova15:03
jaypipescdent: this was requested for weighers that want to sort based on remaining inventory capacity.15:03
alex_xuok...it is bug, then I need some change in my refactor patch15:03
*** Oku_OS is now known as Oku_OS-away15:04
efriedWhy is it a problem if the remaining capacity is included for classes that weren't claimed?15:04
openstackgerritMerged openstack/nova master: Move shred to privsep.  https://review.openstack.org/49553715:04
cdentjaypipes: i wasn’t meaning to be exclusive, rather that the entire provider should be reprsented (plus whatever else)15:04
*** yangyapeng has quit IRC15:04
openstackgerritMerged openstack/nova master: Move xend existence probes to privsep.  https://review.openstack.org/49553815:04
*** yangyapeng has joined #openstack-nova15:04
jaypipesdtantsur: done15:05
jaypipesefried: I don't think it is. I think it was probabl jjst an oversight15:05
jaypipesgod damnb this mac keyboard... :(15:05
efriedjaypipes Which part is an oversight?15:06
efriedjaypipes Including the extra RCs or excluding them?15:06
jaypipesefried: that the provider summary for a provider only includes resources that were requested and not all resources the provider had15:06
efriedjaypipes Which way are we saying it's *supposed* to be?15:06
efriedThat it should include *all* of them, I guess.  I agree with that.15:07
cdentefried: yes, all15:07
efried++15:08
edleafeefried: cdent: of what use do you see non-requested resources in the provider_summary?15:08
alex_xujinx means a person bring bad luck?15:08
jaypipesefried: yeah, sohuld include all the inventories, not just the ones for requested resources15:09
cdentalex_xu: that’s one meaning, but the meaning we are using is “you said something at the same time as me”15:09
efriedalex_xu Heh, not in this context.  It's a silly game kids play: when two people say the same thing at the same time, the first one to say "jinx" "wins"15:09
efriedalex_xu There's various penalties for "losing".  The one I grew up with was that the "loser" has to remain silent until you say their name.15:09
cdentin the real version of the game the loser can’t speak for some amount of time15:09
cdentjinx!15:09
cdentlet’s efried off the hook15:09
efriedalex_xu Another one I've heard of is that the "winner" gets to punch the "loser" until the loser recites some incantation.15:10
efried"Jinx poke, you owe me a coke, times onetwothreefourfive...."15:10
cdentedleafe: the provider summary, as I understand it, is supposed to list the resource providers in a complete. From what’s there we should be able to conceptualize what the resource provider providers, not just what we are using from it15:10
cdentwhether that is of use or not is unclear15:10
* efried feels alex_xu has received adequate occidental cultural indoctrination at this point.15:11
dansmithalex_xu: in case you're wondering, seeing it spelled out in writing like we just have makes it sound even dumber than it normally does15:11
cdents/, not/are, not/15:11
dansmithalex_xu: so if you're thinking "wtf is wrong with these guys" ... you're not alone15:11
cdentno need to be a dick about dansmith15:11
*** jaypipes has quit IRC15:11
dansmithcdent: s/about/about it/ right?15:12
alex_xuhaha :)15:12
cdentdansmith: if you like but the typo is in your favor, so maybe we keep it?15:12
dansmithheh15:12
*** jaypipes has joined #openstack-nova15:12
dansmithcdent: I'm only saying that I've never seen the rules of jinx written out before and I'm feeling kindof embarrassed for my entire generation now that I have :)15:13
efriedAgreed15:13
*** baoli has quit IRC15:13
cdentIt predates our generation yeah? I thought of it as kind of antique even when I was a child and I’m nearly the oldest here.15:13
* cdent glances sadly at old dying edleafe 15:13
efriedMy dad is the one who told me about the punching thing.15:13
edleafeyeah, it was old when *I* first heard about it15:14
*** jaypipes has quit IRC15:14
edleafewritten on stone tablets it seems even lamer15:14
* cdent puts on his lamer tshirt15:15
dansmithI have no idea, I assumed it was too dumb for generations prior, but I'm sure you're right15:16
*** jaypipes has joined #openstack-nova15:16
efriedThe internet (which is never wrong) hints that it may go back as far as 16th century Scottish tavern games.15:17
cdentIsn’t each subsequent generation immensely cooler than the last? My (adult) children assure me this is the case.15:17
*** suresh12 has joined #openstack-nova15:17
*** andreas_s has quit IRC15:18
*** jaypipes has quit IRC15:18
*** andreas_s has joined #openstack-nova15:18
dansmithheh15:18
edleafecdent: Your children are correct, but just in your family15:18
cdentthank you old man15:18
cdentScotland has a lot to answer for.15:19
*** salv-orlando has quit IRC15:19
*** salv-orlando has joined #openstack-nova15:20
* cdent goes home to continue trying reviewing many things15:20
mriedemdansmith: are you ok with this before i propose a backport https://review.openstack.org/#/c/513001/15:20
dansmithomg no15:20
dansmithwtf15:20
*** cdent has quit IRC15:20
dansmithugh15:21
efriedSo... not okay with it15:21
efriedJust reading between the lines here.15:21
mriedemi brought this up when it was initially being reviewed, pretty sure anyway - but there is no api schema restriction in the baremetal api about the resource class name15:21
*** tssurya has quit IRC15:21
dansmithI very specifically think this is completely broken15:22
dansmithI know and I fought hard to NOT do this ambiguous thing15:22
openstackgerritMatt Riedemann proposed openstack/nova master: Only add CUSTOM_ prefix if required  https://review.openstack.org/51300115:22
mriedempulled it out of the gate15:22
mriedemjohnthetubaguy: ^15:22
johnthetubaguyso VCPU becomes CUSTOM_VCPU, just CUSTOM_FOO stays CUSTOM_FOO15:24
*** baoli has joined #openstack-nova15:24
johnthetubaguyits a pain to setup the system with CUSTOM_ in only half the places15:24
*** Apoorva has joined #openstack-nova15:24
mriedemVCPU is a standared resource class15:24
johnthetubaguyyep, that's where it gets funky, that code keeps it as always CUSTOM_ at the start, it just avoids the double encoding15:25
mriedemif you set the ironic node.resource_class as vcpu, then we'd translate that to CUSTOM_VCPU yeah15:25
dansmithmriedem: thanks15:25
johnthetubaguythat's what I mean, I should have added a test for that really15:25
*** Apoorva has quit IRC15:26
dansmithI just commented on that patch,15:26
mriedemthe ironic docs are at least pretty clear about this https://docs.openstack.org/ironic/pike/install/configure-nova-flavors.html#scheduling-based-on-resource-classes15:26
dansmithbut I'm super against us adding this ambiguity15:26
dansmithanything else that wants to look at placement-related data for ironic nodes would have to implement the same "is it already right, or not" behavior when taking the ironic class and turning it into a placement class15:27
dansmithinstead of just "apply the following set of rules to get from ironic to placement"15:27
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication  https://review.openstack.org/34539715:27
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme  https://review.openstack.org/34539815:27
openstackgerritStephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation  https://review.openstack.org/34539915:27
openstackgerritStephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy  https://review.openstack.org/50054415:27
dansmithas documented in the ironic docs there15:27
johnthetubaguyI get that, just as an operator, I need to remember if its Nova or Ironic I am talking to when setting different names for things15:28
dansmithbut they're two totally different things15:28
dansmithironic's resource class predates our use of it,15:29
*** Apoorva has joined #openstack-nova15:29
dansmithand can be set to completely incompatible things15:29
mriedemyeah, note the baremetal.with-GPU example in the ironic docs,15:29
mriedemnova replaces special characters like the '.' with underscores15:29
mriedemso the CUSTOM_ prefix isn't the only thing that will be different15:30
johnthetubaguydansmith: the spec says it was added for placement: https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/node-resource-class.html15:30
johnthetubaguymy thinking was more simplistic really... can't I just set all the strings to the same thing?15:31
dansmithjohnthetubaguy: but like two releases before we actually had that stuff, and with zero restriction :)15:31
openstackgerritStephen Finucane proposed openstack/nova master: Move the idmapshift binary into privsep.  https://review.openstack.org/49554115:31
openstackgerritStephen Finucane proposed openstack/nova master: Move loopback setup and removal to privsep.  https://review.openstack.org/49566415:31
openstackgerritStephen Finucane proposed openstack/nova master: Move nbd commands to privsep.  https://review.openstack.org/50035115:31
openstackgerritStephen Finucane proposed openstack/nova master: Move kpartx calls to privsep.  https://review.openstack.org/50035415:31
openstackgerritStephen Finucane proposed openstack/nova master: Move blkid calls to privsep.  https://review.openstack.org/50039815:31
johnthetubaguydansmith: yep, agreed.15:31
dansmithjohnthetubaguy: yeah I know, but it's not equivalent15:31
*** vks1 has quit IRC15:31
*** jmlowe has quit IRC15:31
dansmithjohnthetubaguy: it'd be like us using the uuid, as CUSTOM_$UUID and then people wanting to set their uuid to CUSTOM_$UUID... they're not the same field15:32
johnthetubaguyit just seems to me like 80% of folks are setting the resource classes to something they can request in a flavor, they care little about the translation, even if they probably should15:33
johnthetubaguybut I guess its dangerous to assume they are the same thing, when you use them, because they may not be15:34
dansmithand integers are like floats, but usually without the decimal point?15:34
*** tssurya has joined #openstack-nova15:35
johnthetubaguyI was thinking about following up with a new microversion that forces all new resource classes to follow the placement patterns, etc.15:36
johnthetubaguybut that is kinda aweful in its own way15:36
dansmithyou have to migrate all their node names currently15:36
*** AlexeyAbashkin has quit IRC15:37
*** gyee has joined #openstack-nova15:37
*** vks1 has joined #openstack-nova15:37
dansmithand will you enforce CUSTOM_ as a prefix? you kinda have to, but then you have chicken and egg...15:37
dansmiththe ironic docs are exceedingly clear on this point15:38
*** edand has quit IRC15:38
edleafeWith hindsight, maybe prefixing standard classes with STD_ would have been simpler15:39
dansmithI don't think that changes this particular thing, but I agree it'd be nice to have consistent namespacing instead of one magic one15:39
openstackgerritpriyaduggirala proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py  https://review.openstack.org/50853315:39
*** jpena is now known as jpena|brb15:40
*** jmlowe has joined #openstack-nova15:40
dansmithanother argument for operator confusion would be:15:42
dansmithI named one class CUSTOM_GOLD and another SILVER15:42
dansmithboth show up in placement as CUSTOM_$thing, yet one tool can never find the silver ones for some reason15:42
dansmithor15:42
dansmithI have CUSTOM_SILVER and CSUTOM_GOLD15:42
dansmithand the gold one shows up in placement as CUSTOM_CSUTOM_GOLD for some reason15:43
*** andreas_s has quit IRC15:46
*** andreas_s has joined #openstack-nova15:46
*** ragiman has quit IRC15:48
*** rmart04 has joined #openstack-nova15:49
openstackgerritJohn Garbutt proposed openstack/nova master: Add test so we remember why CUSTOM_ prefix added  https://review.openstack.org/51300115:51
*** cdent has joined #openstack-nova15:52
johnthetubaguydansmith: I turned the change upside down ^15:52
*** Guest52014 has quit IRC15:53
*** andreas_s has quit IRC15:53
*** andreas_s has joined #openstack-nova15:53
*** sree has joined #openstack-nova15:53
dansmithjohnthetubaguy: +2 thanks15:53
* dansmith tries to calm his seething rage15:53
openstackgerritMurali Annamneni proposed openstack/nova master: Enables MySQL Cluster Support for Nova  https://review.openstack.org/44664315:55
*** sree has quit IRC15:57
*** udesale has joined #openstack-nova15:58
*** penick has joined #openstack-nova16:00
*** andreas_s has quit IRC16:03
*** andreas_s has joined #openstack-nova16:04
*** armax has joined #openstack-nova16:13
*** andreas_s has quit IRC16:13
*** lucasagomes is now known as lucas-afk16:14
openstackgerritEric Berglund proposed openstack/nova master: PowerVM Driver: config drive  https://review.openstack.org/40940416:14
openstackgerritBrianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620416:15
*** tbachman has joined #openstack-nova16:17
*** andreas_s has joined #openstack-nova16:18
*** itlinux has joined #openstack-nova16:18
openstackgerritMatt Riedemann proposed openstack/nova master: db: add flavors.description column  https://review.openstack.org/51437116:19
*** jpena|brb is now known as jpena16:19
*** penick has quit IRC16:19
*** sshwarts has quit IRC16:21
*** andreas_s has quit IRC16:22
*** yamahata has quit IRC16:25
openstackgerritStephen Finucane proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595416:27
openstackgerritStephen Finucane proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595416:29
*** lpetrut has quit IRC16:30
*** Guest40264 has quit IRC16:32
*** jaosorior has quit IRC16:35
openstackgerritMurali Annamneni proposed openstack/nova master: Enables MySQL Cluster Support for Nova  https://review.openstack.org/44664316:36
*** jaosorior has joined #openstack-nova16:36
*** jmccarthy has joined #openstack-nova16:39
*** penick has joined #openstack-nova16:39
*** eglynn has joined #openstack-nova16:40
*** jmlowe has quit IRC16:43
*** Tom____ has quit IRC16:44
*** crushil is now known as crushil_16:44
*** derekh has quit IRC16:45
mriedemhuh, it's odd that we allow flavors.name = None in the object but name is not nullable in the db16:46
*** rmart04 has quit IRC16:49
*** andreas_s has joined #openstack-nova16:49
*** mvk has quit IRC16:49
*** gszasz has quit IRC16:51
*** tssurya has quit IRC16:53
*** udesale has quit IRC16:53
*** andreas_s has quit IRC16:54
*** penick has quit IRC16:56
*** ralonsoh has quit IRC16:57
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field  https://review.openstack.org/44925716:57
*** suresh12 has quit IRC16:59
*** sree has joined #openstack-nova17:01
*** sree has quit IRC17:03
*** sree has joined #openstack-nova17:04
*** vks1 has quit IRC17:05
efriedmriedem How come https://review.openstack.org/#/c/510244/ isn't showing up in https://specs.openstack.org/openstack/nova-specs/specs/queens/ ?17:07
*** fragatina has joined #openstack-nova17:07
*** sree has quit IRC17:08
mriedembusted docs job?17:08
*** yamahata has joined #openstack-nova17:08
mriedemlooks like there is no publish job17:09
efriedIs this a post-zuulv3 thing?17:09
mriedemi believe so17:09
*** salv-orlando has quit IRC17:09
efriedOther specs are in that list, but I haven't dug in to see if any were ones that merged after the zuulv3 cutover17:10
*** salv-orlando has joined #openstack-nova17:10
efriedShould I follow up in -infra?  Or -docs?17:10
efriedor you got this?17:10
mriedemfind out where "openstack-specs-jobs" is defined17:11
mriedemlooks like it's in openstack-zuul-jobs17:11
*** slaweq has quit IRC17:11
mriedemthere is a post job "publish-openstack-specs"17:11
*** fragatina has quit IRC17:11
mriedemthat should publish the latest17:12
mriedemnot sure if that's broken or what17:12
*** slaweq has joined #openstack-nova17:12
*** andreas_s has joined #openstack-nova17:12
mriedemefried: you should follow up in -infra17:12
*** fragatina has joined #openstack-nova17:12
efriedmriedem ack17:12
*** fragatina has quit IRC17:13
*** AlexeyAbashkin has joined #openstack-nova17:13
*** fragatina has joined #openstack-nova17:13
*** fragatina has quit IRC17:13
*** fragatina has joined #openstack-nova17:13
*** salv-orlando has quit IRC17:14
*** jpena is now known as jpena|off17:16
*** slaweq has quit IRC17:16
*** andreas_s has quit IRC17:16
*** AlexeyAbashkin has quit IRC17:17
*** suresh12 has joined #openstack-nova17:17
*** lpetrut has joined #openstack-nova17:18
*** tesseract has quit IRC17:18
mriedemdansmith: want to push this through the specs repo to trigger the publish job? https://review.openstack.org/#/c/456756/17:19
*** eglynn has quit IRC17:22
*** suresh12 has quit IRC17:23
*** sambetts is now known as sambetts|afk17:23
*** sambetts has joined #openstack-nova17:24
*** jmlowe has joined #openstack-nova17:26
dansmithmriedem: done17:28
*** lpetrut has quit IRC17:29
*** andreas_s has joined #openstack-nova17:31
*** mvk has joined #openstack-nova17:32
*** andreas_s has quit IRC17:35
*** sree has joined #openstack-nova17:35
*** ijw has joined #openstack-nova17:36
*** suresh12 has joined #openstack-nova17:38
*** sree has quit IRC17:39
openstackgerritMerged openstack/nova-specs master: Add a new section: "Upgrade impact" to the template  https://review.openstack.org/45675617:40
*** jmccarthy has left #openstack-nova17:42
*** suresh12 has quit IRC17:44
*** Tom____ has joined #openstack-nova17:46
*** __zeus__ has joined #openstack-nova17:46
*** sridharg has quit IRC17:49
*** ijw has quit IRC17:50
*** kylek3h has quit IRC17:56
*** kylek3h has joined #openstack-nova17:56
*** andreas_s has joined #openstack-nova18:01
*** namnh has quit IRC18:03
*** ociuhandu has joined #openstack-nova18:04
*** dtantsur is now known as dtantsur|afk18:04
mriedemefried: ^ should have flushed a post job18:07
efriedmriedem Not yet https://specs.openstack.org/openstack/nova-specs/specs/queens/index.html18:07
efriedbeen watching.18:07
*** andreas_s has quit IRC18:10
*** baoli has quit IRC18:11
*** baoli has joined #openstack-nova18:12
*** baoli has quit IRC18:12
*** baoli has joined #openstack-nova18:18
*** crushil_ has quit IRC18:19
*** suresh12 has joined #openstack-nova18:21
*** suresh12 has quit IRC18:25
*** suresh12_ has joined #openstack-nova18:25
mriedemdansmith: remember this test? https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_sqlalchemy_migration.py#L37918:28
mriedemit's causing me some problems with the flavor.description stuff because it doesn't run the schema change that adds the description column, but it creates a flavor using the object, which has a flavor attribute now,18:28
mriedemso trying to think of non terrible ways to resolve that in the test18:29
dansmithmriedem: create the flavor with the model directly18:29
mriedemyeah was just thinking about bypassing the object and using nova.objects.Flavor._flavor_crate18:30
mriedem*create18:30
dansmithyeah18:30
*** avolkov has quit IRC18:31
*** suresh12_ has quit IRC18:31
*** ociuhandu has quit IRC18:32
*** suresh12 has joined #openstack-nova18:34
mriedemah that doesn't work, something to do with the sqla model18:36
*** rmcallis has joined #openstack-nova18:38
*** suresh12 has quit IRC18:39
mriedemi could manually add the flavors.description column in the test itself, which isn't fun but not sure what else can be done18:40
*** shvepsy has quit IRC18:45
*** __zeus__ has quit IRC18:45
*** shvepsy has joined #openstack-nova18:45
*** __zeus__ has joined #openstack-nova18:45
*** suresh12 has joined #openstack-nova18:47
dansmithwhat fails when using it directly?18:48
* cdent waves18:50
*** cdent has quit IRC18:50
*** jgriffith has quit IRC18:51
*** syqian__ has quit IRC18:51
*** jgriffith_ has joined #openstack-nova18:52
*** itlinux has quit IRC18:52
*** harlowja has joined #openstack-nova18:53
*** itlinux has joined #openstack-nova18:56
mriedemdansmith: same thing as via the object18:56
mriedemoslo_db.exception.DBError: (sqlite3.OperationalError) table flavors has no column named description [SQL: u'INSERT INTO flavors (created_at, updated_at, name, memory_mb, vcpus, root_gb, ephemeral_gb, flavorid, swap, rxtx_factor, vcpu_weight, disabled, is_public, description) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)'] [parameters: ('2017-10-23 18:35:19.692208', None, 'foo', 123, 1, 1, None, 'm1.foo', 0, 1.0, None, 0,18:56
mriedemone)]18:56
dansmithis it not nullable?18:57
mriedemit is18:57
mriedemin that test,18:57
mriedemwe haven't run the table schema change to add the column to the flavors table18:57
dansmithand you're not setting it on the model rght?18:57
dansmithright18:57
mriedemright18:57
dansmithhuh18:57
mriedemi think the model is defaulting it to None?18:57
dansmithI would have thought that would work18:57
dansmithso you could easily just set without the orm18:58
dansmithi.e. table.insert(...)18:58
mriedemyeah...18:58
mriedemwill try that18:58
*** namnh has joined #openstack-nova19:03
mriedemissue the 2nd: Flavor.save() isn't remotable, so preferences on either making that remotable or adding a new save_updates() remotable or something? Flavor.save() calls off to remotables for updating projects and extra_specs19:04
mriedemnot sure if we can add remotable for Flavor.save() now or not19:04
mriedemguessing that's bad form19:04
*** jgriffith_ has quit IRC19:06
*** suresh12 has quit IRC19:06
openstackgerritEric Fried proposed openstack/nova master: Include /resource_providers/uuid/allocations link  https://review.openstack.org/49982619:08
*** namnh has quit IRC19:08
dansmithmriedem: computes are the only things that really need things to be remotable, and they can't talk to conductors that could do any flavor saving19:09
*** jgriffith has joined #openstack-nova19:10
*** jgriffith has quit IRC19:11
*** jgriffith has joined #openstack-nova19:11
*** jgriffith has quit IRC19:11
*** slaweq has joined #openstack-nova19:12
*** slaweq_ has joined #openstack-nova19:13
*** slaweq_ has quit IRC19:14
*** Tom____ has quit IRC19:14
*** jgriffith has joined #openstack-nova19:15
mriedemtable.insert().execute(values) works btw - easier than the other hacks i was thinking of19:15
mriedemthanks19:15
dansmithcool19:16
*** suresh12 has joined #openstack-nova19:33
*** jgriffith has quit IRC19:37
*** jgriffith has joined #openstack-nova19:38
*** itlinux has quit IRC19:39
*** awaugama has quit IRC19:44
mriedemcan i use "彩虹和独角兽" as a description in tests?19:44
mikalyes19:45
mriedemi don't even know if we encode those first19:45
openstackgerritMichael Still proposed openstack/nova master: Move the idmapshift binary into privsep.  https://review.openstack.org/49554119:45
openstackgerritMichael Still proposed openstack/nova master: Move loopback setup and removal to privsep.  https://review.openstack.org/49566419:45
openstackgerritMichael Still proposed openstack/nova master: Move nbd commands to privsep.  https://review.openstack.org/50035119:45
openstackgerritMichael Still proposed openstack/nova master: Move kpartx calls to privsep.  https://review.openstack.org/50035419:45
openstackgerritMichael Still proposed openstack/nova master: Move blkid calls to privsep.  https://review.openstack.org/50039819:45
mikalsdague: I done fixed that release note for you19:46
*** rodolof has joined #openstack-nova19:47
dansmithmriedem: niiiice19:51
mriedemrainbows and unicorns are somehow more magical in mandarin19:52
*** sree has joined #openstack-nova19:55
*** crushil has joined #openstack-nova19:56
*** armax has quit IRC19:57
*** AlexeyAbashkin has joined #openstack-nova19:58
*** rtjure has joined #openstack-nova19:59
*** sree has quit IRC20:00
sdaguemikal: cool, I'll circle in a minute20:01
*** AlexeyAbashkin has quit IRC20:02
efriedAnyone know if we're supposed to handle queryparams with multiple values in the placement API?20:04
*** rtjure has quit IRC20:04
efriedE.g. ?resources=VCPU:1,MEMORY_MB:2048&resources=DISK_GB:520:04
efriededleafe dansmith jaypipes cdent ^ ?20:05
mikalsdague: yeah, no rush20:05
*** namnh has joined #openstack-nova20:05
efriedThe code doesn't appear to be set up to do so, but I could be reading it wrong fo sho.20:05
edleafeefried: my understanding is that each resource requires its own resource=20:06
efriededleafe If that's the case, I'm *definitely* reading the code wrong.  Let me grab some links...20:07
efriededleafe We grab the 'resources' qparam value here: https://github.com/openstack/nova/blob/a067f8c64654a058e04e41389019bd6ff5841cdc/nova/api/openstack/placement/handlers/allocation_candidate.py#L16620:09
*** namnh has quit IRC20:09
efriededleafe Then we start chewing it up here: https://github.com/openstack/nova/blob/a067f8c64654a058e04e41389019bd6ff5841cdc/nova/api/openstack/placement/util.py#L223-L23020:09
efriedThe latter chunk appears to be expecting a string, never a list20:10
efriedAnd if I'm understanding MultiDict correctly, you'll get a list if you do GET['resources'] and the qstring was like my example above.20:10
edleafeefried: yeah, I'm looking at normalize_resources_qs_param(), and it does look like you can do what you asked20:10
efriededleafe can or can't?20:11
edleafecan: multiple RCs per resource=. I don't see where multiple resource= is supported20:12
efriededleafe Right, in my example DISK_GB was specified in a separate resources=20:12
edleafeI would have to run a few tests with the combinations to say for sure.20:13
efriededleafe oh, interestingly, when you say MultiDict[key] you don't get a list.  You get the last one.20:13
*** rtjure has joined #openstack-nova20:13
efriededleafe You have to use getall to get the list.20:13
efriedSo: not supported.  Good to know.20:14
*** jmlowe has quit IRC20:14
edleafetoo bad cdent isn't here. I remember the back-and-forth we had, but not why we settled on the current approach. He might remember that.20:14
efriededleafe I will be sure to follow up with him on that at some point; but for now it's enough for me to know where we stand.20:16
*** Apoorva has quit IRC20:17
*** suresh12 has quit IRC20:18
*** rtjure has quit IRC20:18
*** Apoorva has joined #openstack-nova20:18
*** suresh12 has joined #openstack-nova20:21
*** liverpoo1er has quit IRC20:21
*** suresh12 has quit IRC20:21
*** suresh12 has joined #openstack-nova20:22
*** sahid has quit IRC20:24
*** rtjure has joined #openstack-nova20:25
*** smatzek has quit IRC20:27
*** markmc has quit IRC20:27
*** markmc has joined #openstack-nova20:28
*** rtjure has quit IRC20:29
*** peter-hamilton has quit IRC20:31
*** salv-orlando has joined #openstack-nova20:33
*** rtjure has joined #openstack-nova20:34
*** armax has joined #openstack-nova20:34
*** hemna_ has quit IRC20:36
*** salv-orlando has quit IRC20:37
*** salv-orlando has joined #openstack-nova20:37
*** eharney has quit IRC20:38
*** rtjure has quit IRC20:39
*** baoli has quit IRC20:44
*** lpetrut has joined #openstack-nova20:45
*** rtjure has joined #openstack-nova20:46
*** nicolasbock has quit IRC20:48
*** fragatina has quit IRC20:49
*** baoli has joined #openstack-nova20:50
*** baoli has quit IRC20:50
*** rtjure has quit IRC20:51
*** ijw has joined #openstack-nova20:54
*** hemna_ has joined #openstack-nova20:54
*** smatzek has joined #openstack-nova20:56
*** rtjure has joined #openstack-nova20:56
*** smatzek has quit IRC20:57
*** smatzek has joined #openstack-nova20:58
*** hemna_ has quit IRC20:59
*** jgriffith has quit IRC20:59
*** burt has quit IRC20:59
*** jgriffith has joined #openstack-nova21:00
*** baoli has joined #openstack-nova21:00
*** smatzek has quit IRC21:02
*** rtjure has quit IRC21:02
*** pchavva has quit IRC21:02
*** thorst has quit IRC21:02
*** smatzek has joined #openstack-nova21:02
*** andreas_s has joined #openstack-nova21:05
*** smatzek has quit IRC21:07
mriedemdansmith: just noticed something strange - not sure if this was intentional or not,21:07
mriedembut was writing a test where i create an instance,21:07
mriedemthen i set instance.flavor and assert the only changed field is the flavor21:08
mriedemself.assertEqual(['flavor'], list(instance.obj_what_changed()))21:08
mriedemthat failed with: testtools.matchers._impl.MismatchError: ['flavor'] != ['ec2_ids', 'flavor']21:08
*** rtjure has joined #openstack-nova21:08
mriedemit looks like on instance.create() we load ec2_ids and then reset the field, but because we aren't recursively resetting the EC2Ids object, it still shows up in the obj_what_changed() results21:09
dansmithah might be21:10
mriedemso on a subsequent instance.save() we'll call _save_ec2_ids but that's a no-op, so it's not like it hurts anything,21:10
mriedembut it's unnecessary processing21:10
*** ijw has quit IRC21:12
*** trozet has joined #openstack-nova21:12
*** rtjure has quit IRC21:13
*** suresh12 has quit IRC21:13
trozethi all. I was wondering if someone can help me with an issue.  Ocata + 5 ironic nodes.  I go to deploy instances (via tripleO) and see Starting with 0 host(s) get_filtered_objects /usr/lib/python2.7/site-packages/nova/filters.py:7021:13
trozetall ironic nodes are in hypervisor-list21:13
*** slaweq has quit IRC21:14
*** jmlowe has joined #openstack-nova21:15
*** slaweq has joined #openstack-nova21:15
*** suresh12 has joined #openstack-nova21:17
efriedtrozet I think you want to try the #openstack channel.21:18
*** rtjure has joined #openstack-nova21:18
*** andreas_s has quit IRC21:18
*** edmondsw has quit IRC21:20
*** slaweq has quit IRC21:20
*** ijw has joined #openstack-nova21:22
*** rtjure has quit IRC21:23
*** andreas_s has joined #openstack-nova21:23
*** suresh12 has quit IRC21:23
*** suresh12 has joined #openstack-nova21:23
*** xyang1 has quit IRC21:26
*** thorst has joined #openstack-nova21:27
*** ijw has quit IRC21:27
*** rtjure has joined #openstack-nova21:27
*** dave-mccowan has quit IRC21:28
*** sree has joined #openstack-nova21:29
*** thorst has quit IRC21:31
mriedemhmm, shouldn't any model update result in an auto-update to the updated_at field?21:31
mriedemgiven: updated_at = Column(DateTime, onupdate=lambda: timeutils.utcnow())21:31
*** hemna_ has joined #openstack-nova21:32
*** rtjure has quit IRC21:32
*** sree has quit IRC21:34
*** MasterOfBugs has joined #openstack-nova21:34
*** pramodrj07 has joined #openstack-nova21:34
mriedemha, nvm21:35
mriedemsomething something test assertions doing what you wanted them to do21:35
*** rtjure has joined #openstack-nova21:40
openstackgerritMatt Riedemann proposed openstack/nova master: Add Flavor.description attribute  https://review.openstack.org/51437121:40
*** andreas_s has quit IRC21:41
*** ijw has joined #openstack-nova21:42
*** slaweq has joined #openstack-nova21:45
*** takashin has joined #openstack-nova21:45
*** rtjure has quit IRC21:45
*** andreas_s has joined #openstack-nova21:46
*** mwynne has joined #openstack-nova21:47
mwynneHi guys. I'm trying to resize an instance but it just goes to an ACTIVE state and never resizes.21:47
mriedemmwynne: check compute logs for errors21:47
*** mriedem is now known as mriedem_away21:48
mwynnecompute logs on the compute node?21:48
mwynneIs there an easy way for me to tell which compute node the VM is on?21:48
*** ijw has quit IRC21:49
*** esberglu has quit IRC21:49
*** esberglu has joined #openstack-nova21:49
*** jmlowe has quit IRC21:50
*** jgriffith has quit IRC21:50
*** esberglu_ has joined #openstack-nova21:51
*** rtjure has joined #openstack-nova21:52
*** sambetts has quit IRC21:53
*** esberglu has quit IRC21:54
*** lpetrut has quit IRC21:55
*** esberglu_ has quit IRC21:56
melwitt mwynne: I think if you do a 'nova show' with admin credentials it will show what compute node it's on21:56
*** rtjure has quit IRC21:56
*** jgriffith has joined #openstack-nova21:58
*** __zeus__ has quit IRC22:01
*** esberglu has joined #openstack-nova22:01
*** rtjure has joined #openstack-nova22:02
*** rodolof has quit IRC22:02
*** andreas_s has quit IRC22:04
*** namnh has joined #openstack-nova22:07
*** rtjure has quit IRC22:07
*** jmlowe has joined #openstack-nova22:08
*** aloga has quit IRC22:08
mwynnemelwitt: hostId?22:08
*** ijw has joined #openstack-nova22:08
*** fragatina has joined #openstack-nova22:09
*** aloga has joined #openstack-nova22:09
*** andreas_s has joined #openstack-nova22:09
melwittmwynne: no, hypervisor_hostname22:10
*** namnh has quit IRC22:11
*** ijw has quit IRC22:11
*** claudiub|2 has quit IRC22:11
*** rtjure has joined #openstack-nova22:12
*** ijw has joined #openstack-nova22:13
*** andreas_s has quit IRC22:13
*** rtjure has quit IRC22:17
*** slaweq has quit IRC22:18
*** ijw has quit IRC22:19
*** aloga has quit IRC22:19
mwynnemelwitt: I don't see that field in a nova show22:19
mwynnemelwitt: I'm running newton. That's probably it?22:19
melwittmwynne: it only shows it if you use admin credentials22:19
*** rtjure has joined #openstack-nova22:21
*** ijw has joined #openstack-nova22:23
*** jmlowe has quit IRC22:24
*** jmlowe has joined #openstack-nova22:24
*** esberglu has quit IRC22:25
*** rtjure has quit IRC22:26
*** lyan has quit IRC22:27
*** mlavalle has quit IRC22:27
*** slaweq has joined #openstack-nova22:28
*** hemna_ has quit IRC22:29
*** aloga has joined #openstack-nova22:30
*** baoli has quit IRC22:30
*** rtjure has joined #openstack-nova22:31
*** salv-orlando has quit IRC22:34
*** salv-orlando has joined #openstack-nova22:34
*** rtjure has quit IRC22:36
*** andreas_s has joined #openstack-nova22:36
*** salv-orlando has quit IRC22:39
*** chyka has quit IRC22:40
*** aloga has quit IRC22:40
*** chyka has joined #openstack-nova22:40
*** rtjure has joined #openstack-nova22:42
*** ijw has quit IRC22:43
*** ijw has joined #openstack-nova22:44
*** andreas_s has quit IRC22:45
*** thorst has joined #openstack-nova22:46
openstackgerritEric Fried proposed openstack/nova master: Parse numbered & unnumbered resources & traits  https://review.openstack.org/51409122:46
openstackgerritEric Fried proposed openstack/nova master: WIP: Numbered groupings to GET /allocation_candidates  https://review.openstack.org/51409222:46
efriededleafe The 'Parse' one should be ready to go ^22:46
*** rtjure has quit IRC22:46
efried...assuming we can agree on the approach.22:47
*** edmondsw has joined #openstack-nova22:48
*** dave-mccowan has joined #openstack-nova22:49
*** thorst has quit IRC22:51
*** takashin has quit IRC22:51
*** takashin has joined #openstack-nova22:51
*** edmondsw has quit IRC22:52
efriedfungi mriedem_away FYI that publish job kicked in, https://specs.openstack.org/openstack/nova-specs/specs/queens/index.html appears to be fully populated now.  Thanks for the help.22:53
*** rtjure has joined #openstack-nova22:53
fungiefried: any time!22:54
fungiglad it's working finally22:54
*** ijw has quit IRC22:54
*** ijw has joined #openstack-nova22:55
efriedalex_xu We could slot https://review.openstack.org/514091 and L186 & L190 of https://review.openstack.org/#/c/514092/2/nova/api/openstack/placement/handlers/allocation_candidate.py in front of your series.  All of that is backward compatible, so you can have the microversion change all to yourself :)22:57
*** sdague has quit IRC22:58
*** Apoorva has quit IRC22:58
*** rtjure has quit IRC22:58
*** andreas_s has joined #openstack-nova22:59
*** ijw has quit IRC23:00
*** slaweq has quit IRC23:01
*** jgriffith has quit IRC23:01
*** ijw has joined #openstack-nova23:02
*** acormier has joined #openstack-nova23:04
*** sree has joined #openstack-nova23:04
*** Apoorva has joined #openstack-nova23:05
*** acormier has quit IRC23:07
*** acormier has joined #openstack-nova23:07
*** andreas_s has quit IRC23:08
*** jgriffith has joined #openstack-nova23:08
*** aloga has joined #openstack-nova23:08
*** sree has quit IRC23:08
*** slaweq has joined #openstack-nova23:10
*** vladikr has quit IRC23:12
*** vladikr has joined #openstack-nova23:12
*** felipemonteiro__ has quit IRC23:13
*** rtjure has joined #openstack-nova23:14
*** acormier has quit IRC23:16
*** acormier has joined #openstack-nova23:17
*** rtjure has quit IRC23:19
*** ijw has quit IRC23:21
*** acormier has quit IRC23:21
*** gmann_afk is now known as gmann23:22
*** rtjure has joined #openstack-nova23:24
*** dave-mccowan has quit IRC23:27
*** rtjure has quit IRC23:29
*** rtjure has joined #openstack-nova23:34
*** andreas_s has joined #openstack-nova23:36
*** rtjure has quit IRC23:39
*** hongbin has quit IRC23:39
*** andreas_s has quit IRC23:40
*** slaweq has quit IRC23:43
*** rtjure has joined #openstack-nova23:44
*** rtjure has quit IRC23:48
*** slaweq has joined #openstack-nova23:52
*** yassine has joined #openstack-nova23:53
*** rtjure has joined #openstack-nova23:53
*** yassine is now known as Guest8928323:54
openstackgerritMatt Riedemann proposed openstack/nova master: Add Flavor.description attribute  https://review.openstack.org/51437123:54
*** namnh has joined #openstack-nova23:56
*** tbachman has quit IRC23:56
*** rtjure has quit IRC23:59

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