Monday, 2018-12-03

« Sunday, 2018-12-02 Index Tuesday, 2018-12-04 »
*** tbachman_ has joined #openstack-nova00:01
*** tbachman has quit IRC00:01
*** tbachman_ is now known as tbachman00:01
*** wolverineav has quit IRC00:04
*** eharney has quit IRC00:07
*** tbachman has quit IRC00:23
*** tetsuro has joined #openstack-nova00:28
*** wolverineav has joined #openstack-nova00:34
*** tetsuro has quit IRC00:38
*** takashin has joined #openstack-nova00:49
*** brinzhang has joined #openstack-nova01:15
*** tbachman has joined #openstack-nova01:19
*** yedongcan has joined #openstack-nova01:49
openstackgerritAlex Xu proposed openstack/nova master: Parse the numa topology from the new flavor when resize instance  https://review.openstack.org/62087402:05
openstackgerritAlex Xu proposed openstack/nova master: Parse the new PCI request from flavor extra specs when resize  https://review.openstack.org/62107702:05
*** hongbin has joined #openstack-nova02:13
*** mrsoul has joined #openstack-nova02:16
*** rambo_li has joined #openstack-nova02:21
openstackgerritCorey Bryant proposed openstack/nova stable/pike: Revert "Backport tox.ini to switch to stestr"  https://review.openstack.org/62122702:28
*** bhagyashris_ has joined #openstack-nova02:29
*** hshiina has joined #openstack-nova02:32
*** mhen has quit IRC02:38
*** mhen has joined #openstack-nova02:39
*** brinzhang has quit IRC02:40
*** brinzhang has joined #openstack-nova02:40
*** psachin has joined #openstack-nova02:42
*** hongbin has quit IRC02:46
*** imacdonn has joined #openstack-nova02:52
*** rambo_li has quit IRC02:58
*** wolverineav has quit IRC03:04
*** wolverineav has joined #openstack-nova03:04
*** ileixe has joined #openstack-nova03:12
*** hongbin has joined #openstack-nova03:21
*** wolverineav has quit IRC03:28
*** hongbin has quit IRC03:30
*** hamzy__ is now known as hamzy03:36
*** wolverineav has joined #openstack-nova03:55
openstackgerritYikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node  https://review.openstack.org/61349904:10
*** Nel1x has quit IRC04:39
*** ileixe has quit IRC04:47
*** jarodwl has quit IRC04:47
*** jarodwl has joined #openstack-nova04:48
*** ileixe has joined #openstack-nova04:48
*** ileixe has quit IRC04:49
*** ratailor has joined #openstack-nova04:51
*** ratailor has quit IRC04:52
*** ileixe has joined #openstack-nova04:53
*** brinzhang has quit IRC05:05
*** brinzhang has joined #openstack-nova05:06
*** threestrands has joined #openstack-nova05:21
*** threestrands has quit IRC05:21
*** wolverineav has quit IRC05:24
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7)  https://review.openstack.org/57199205:28
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8)  https://review.openstack.org/57199305:28
openstackgerritTakashi NATSUME proposed openstack/nova master: Add API ref guideline for body text  https://review.openstack.org/60562805:29
openstackgerritTakashi NATSUME proposed openstack/python-novaclient master: Fix flavor keyerror when nova boot vm  https://review.openstack.org/58214705:29
openstackgerritTakashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes  https://review.openstack.org/61912505:30
openstackgerritTakashi NATSUME proposed openstack/nova master: Add descriptions about microversions  https://review.openstack.org/61914405:30
openstackgerritTakashi NATSUME proposed openstack/nova stable/rocky: Add description of custom resource classes  https://review.openstack.org/61912205:30
*** ratailor has joined #openstack-nova05:30
*** abhishekk has joined #openstack-nova05:36
*** wolverineav has joined #openstack-nova05:43
*** igordc has joined #openstack-nova05:46
*** wolverineav has quit IRC05:48
*** ohorecny2 has quit IRC05:49
openstackgerritYongli He proposed openstack/nova master: Adds the server group info into show server detail API.  https://review.openstack.org/62147405:54
openstackgerritYongli He proposed openstack/nova master: Add numa topology and pci devices into server detail API  https://review.openstack.org/62147605:57
*** elbragstad has quit IRC06:03
*** whoami-rajat has joined #openstack-nova06:13
*** ileixe has quit IRC06:19
*** ircuser-1 has joined #openstack-nova06:29
*** wolverineav has joined #openstack-nova06:34
*** ralonsoh has joined #openstack-nova06:37
*** wolverineav has quit IRC06:55
*** wolverineav has joined #openstack-nova06:56
*** rcernin has quit IRC06:57
*** wolverineav has quit IRC07:01
*** ohorecny2 has joined #openstack-nova07:02
*** Luzi has joined #openstack-nova07:04
*** dpawlik has joined #openstack-nova07:15
*** diga has joined #openstack-nova07:15
*** dpawlik has quit IRC07:20
*** dpawlik_ has joined #openstack-nova07:20
*** adrianc has joined #openstack-nova07:24
*** pcaruana has joined #openstack-nova07:25
*** wolverineav has joined #openstack-nova07:27
*** jarodwl has quit IRC07:27
*** wolverineav has quit IRC07:28
*** wolverineav has joined #openstack-nova07:28
*** maciejjozefczyk has joined #openstack-nova07:32
*** ileixe has joined #openstack-nova07:36
openstackgerritTakashi NATSUME proposed openstack/nova master: Add descriptions of numbered resource classes and traits  https://review.openstack.org/62149407:37
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3)  https://review.openstack.org/57410407:38
openstackgerritTakashi NATSUME proposed openstack/nova stable/rocky: Remove unnecessary redirect  https://review.openstack.org/60740007:39
*** Dinesh_Bhor has joined #openstack-nova07:43
*** wolverineav has quit IRC07:43
openstackgerritYikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node  https://review.openstack.org/61349907:45
openstackgerritYikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration script  https://review.openstack.org/60999507:45
*** sahid has joined #openstack-nova07:49
*** Dinesh_Bhor has quit IRC07:55
*** jangutter has joined #openstack-nova08:02
*** igordc has quit IRC08:03
*** diga has quit IRC08:03
*** Dinesh_Bhor has joined #openstack-nova08:04
*** slaweq has joined #openstack-nova08:06
*** trident has quit IRC08:28
*** ttsiouts has joined #openstack-nova08:29
*** trident has joined #openstack-nova08:30
*** takashin has left #openstack-nova08:30
openstackgerritYongli He proposed openstack/nova master: Add numa topology and pci devices into server detail API  https://review.openstack.org/62147608:32
*** ShilpaSD has joined #openstack-nova08:32
openstackgerritZhenyu Zheng proposed openstack/nova master: Add method to allow fetch root_volume BDM by instance_uuid  https://review.openstack.org/61467208:33
openstackgerritZhenyu Zheng proposed openstack/nova master: Bump compute service to indicate attach/detach root volume is supported  https://review.openstack.org/61475008:34
openstackgerritYongli He proposed openstack/nova master: Adds the server group info into show server detail API.  https://review.openstack.org/62147408:35
*** ccamacho has joined #openstack-nova08:45
*** ttsiouts has quit IRC08:46
*** ttsiouts has joined #openstack-nova08:46
*** ttsiouts has quit IRC08:51
*** ttsiouts has joined #openstack-nova08:53
*** alexchadin has joined #openstack-nova08:58
openstackgerritYikun Jiang proposed openstack/nova master: Add ratio online data migration when load compute node  https://review.openstack.org/61349908:59
openstackgerritYikun Jiang proposed openstack/nova master: Add compute_node ratio online data migration script  https://review.openstack.org/60999508:59
*** xek has joined #openstack-nova08:59
*** moshele has joined #openstack-nova09:05
*** izza_ has joined #openstack-nova09:07
*** jarodwl has joined #openstack-nova09:07
izza_hi is anyone here already tried to deploy airship?09:08
*** k_mouza has joined #openstack-nova09:19
*** k_mouza has quit IRC09:35
*** k_mouza has joined #openstack-nova09:36
openstackgerritMaciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal  https://review.openstack.org/59160709:37
openstackgerritMichael Still proposed openstack/nova master: Fix sloppy initialization of the new disk ops semaphore.  https://review.openstack.org/62134609:40
openstackgerritMichael Still proposed openstack/nova master: Remove utils.execute() calls from xenapi.  https://review.openstack.org/61970009:40
openstackgerritMichael Still proposed openstack/nova master: Remove utils.execute() from libvirt remotefs calls.  https://review.openstack.org/61970109:40
openstackgerritMichael Still proposed openstack/nova master: Remove utils.execute() from quobyte libvirt storage driver.  https://review.openstack.org/61970209:40
openstackgerritMichael Still proposed openstack/nova master: Move nova.libvirt.utils away from using nova.utils.execute().  https://review.openstack.org/61970309:40
openstackgerritMichael Still proposed openstack/nova master: Imagebackend should call processutils.execute directly.  https://review.openstack.org/61970409:40
openstackgerritMichael Still proposed openstack/nova master: Remove final users of utils.execute() in libvirt.  https://review.openstack.org/61970509:40
openstackgerritMichael Still proposed openstack/nova master: Remove the final user of utils.execute() from virt.images  https://review.openstack.org/62000709:40
openstackgerritMichael Still proposed openstack/nova master: Remove utils.execute() from the hyperv driver.  https://review.openstack.org/62000809:40
openstackgerritMichael Still proposed openstack/nova master: Remove utils.execute() from virt.disk.api.  https://review.openstack.org/62000909:40
openstackgerritMichael Still proposed openstack/nova master: Move a generic bridge helper to a linux_net privsep file.  https://review.openstack.org/62001009:40
openstackgerritMichael Still proposed openstack/nova master: Move bridge creation to privsep.  https://review.openstack.org/62018009:40
openstackgerritMichael Still proposed openstack/nova master: Move some linux network helpers to use privsep.  https://review.openstack.org/62139809:40
openstackgerritMichael Still proposed openstack/nova master: Move simple execute call to processutils.  https://review.openstack.org/62152709:40
openstackgerritMichael Still proposed openstack/nova master: Move interface enabling to privsep.  https://review.openstack.org/62152809:40
openstackgerritMichael Still proposed openstack/nova master: Move setting mac addresses for network devices to privsep.  https://review.openstack.org/62152909:40
*** awalende has joined #openstack-nova09:44
izza_hi..can anyone help me here about error that we've encountered upon testing vgpu in devstack using tensorflow09:45
izza_encountering below error: The TensorFlow library was compiled to use SSE4.1 instructions, but these aren't available on your machine.09:46
*** Dinesh_Bhor has quit IRC09:46
*** erlon has joined #openstack-nova09:50
*** awalende_ has joined #openstack-nova09:53
*** awalende has quit IRC09:56
*** bhagyashris_ has quit IRC09:57
*** derekh has joined #openstack-nova09:57
bauzasizza_: good morning10:05
bauzasizza_: when you say "testing vgpus", you mean using https://docs.openstack.org/nova/latest/admin/virtual-gpu.html ?10:05
bauzasizza_: I guess you use the libvirt driver with KVM ?10:06
izza_able to detect the vgpu...the thing is when we tried to deploy tensorflow in devstack, we're encountering such error.. yes, it is already set as KVM10:08
bauzasizza_: I guess you probably don't have the right CPU model10:08
bauzasizza_: which CPU mode do you use for all your computes ?10:11
bauzascontext is https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.cpu_mode10:12
bauzasAFAIR, using "None" for your CPU mode in Nova makes you get a very old CPU model10:13
bauzasfor migration reasons10:13
*** Dinesh_Bhor has joined #openstack-nova10:14
*** k_mouza has quit IRC10:21
bauzasizza_: if you wanna get more on CPU models, see https://www.berrange.com/posts/2018/06/29/cpu-model-configuration-for-qemu-kvm-on-x86-hosts/10:23
*** tssurya has joined #openstack-nova10:26
openstackgerritBalazs Gibizer proposed openstack/nova master: DNM: troubleshoot cells v1 tests on gate  https://review.openstack.org/62153810:32
sean-k-mooneybauzas: using none for cpu mode should result in host model which will be the most recent cpu model that matches your host10:38
izza_so it should be none?10:40
sean-k-mooneyizza_: you should always set it explctly to a supported value or not at all.10:41
sean-k-mooneynone will indrectly result in host-model10:41
sean-k-mooneyoh actully maybe bauzas is correct https://github.com/openstack/nova/blob/master/nova/conf/libvirt.py#L490-L49310:43
*** k_mouza has joined #openstack-nova10:44
*** awalende has joined #openstack-nova10:45
*** k_mouza has quit IRC10:45
sean-k-mooneyit looks like we treat the string "none" differently form that value None10:46
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3796-L381510:46
*** k_mouza has joined #openstack-nova10:46
*** panda|pto is now known as panda10:47
openstackgerritLee Yarwood proposed openstack/nova master: WIP libvirt: Add workaround to remove instance dir during cleanup with rbd  https://review.openstack.org/61847810:47
sean-k-mooneyizza_: try setting it to host-model explcitly10:48
sean-k-mooneyizza_: bauzas may already have covered this but you might also need to use a different vGPU type to enable tensorflow to work with gpu accleration.10:49
sean-k-mooneyonly some models support opencl/cuda10:49
*** awalende_ has quit IRC10:49
*** Dinesh_Bhor has quit IRC10:50
bauzassean-k-mooney: izza_: the conf option help says that cpu_mode defaults to host-model if KVM10:50
bauzasso you should get the SSE4 instructions if your CPU is recent10:51
bauzasbut worth checking the guest XML10:51
izza_im checking it now10:51
bauzasizza_: please tell us the config options values for [libvirt]/cpu_mode and [libvirt]/cpu_model10:52
bauzasizza_: besides the virt type10:52
sean-k-mooneysse4 is pretty old. i would be surprised if the host did not supported it10:52
bauzasI agree10:52
bauzasthe only thing I could think of would be cpu_mode explicitely set to None10:53
izza_[libvirt]10:53
bauzaswhich would make the guest using a very old CPU model that doesn't support SSE110:53
bauzasSSE4*10:53
izza_live_migration_uri = qemu+ssh://stack@%s/system10:54
izza_cpu_mode = host-passthrough10:54
izza_cpu_model_extra_flags = sse4_110:54
izza_virt_type = kvm10:54
sean-k-mooneyizza_: host-passthrough will give you the same feature flags as the host. you dod not need the extra flags10:54
izza_noted on this..we will remove it..10:55
sean-k-mooneyizza_: you have sse in the hosts /proc/cpuinfo right?10:55
izza_but we tried to remove it and set is as passthrough without flags10:55
izza_but still unable to get the other flags10:55
sean-k-mooneyevery xeon since nehelem has had sse 4.2+10:56
izza_yes, sse4.1 is available in  /proc/cpuinfo10:56
*** priteau has joined #openstack-nova10:56
sean-k-mooneyizza_: and your setting cpu_mode = host-passthrough in the /etc/nova/nova-cpu.conf10:56
izza_it is set as "none" in /etc/nova/nova-cpu.conf10:58
sean-k-mooneyizza_: that the issue10:58
izza_what should i do?10:58
sean-k-mooney/etc/nova/nova-cpu.conf is used by the compute agent /etc/nova/nova.conf is used by the schduer/conductor10:58
izza_ok will try to change it10:59
izza_we will advise you for the result10:59
sean-k-mooney[[post-config|$NOVA_CPU_CONF]]11:00
sean-k-mooney[libvirt]11:00
sean-k-mooneycpu_mode = host-passthrough11:00
sean-k-mooneyvirt_type = kvm11:00
sean-k-mooneythat is how to set it in your local.conf by the way11:00
izza_ok noted on that11:00
izza_we'll try11:00
sean-k-mooneyyou can change it locally and then restart the nova services with sudo systemctl restart devstack@n-*11:01
bauzaswhat sean-k-mooney said11:01
sean-k-mooney* by locally i mean in /etc/nova/nova-cpu.conf11:01
bauzasizza_: your n-cpu service uses the wrong file11:01
izza_thanks guys11:01
*** adrianc has quit IRC11:02
sean-k-mooneybauzas: well a different file we use /etc/nova/nova-cpu.conf instead of /etc/nova/nova.conf intentionally11:04
izza_got it...already capture the cpu details after changing the said file11:04
bauzas'wrong' is maybe not the right word11:05
bauzaswhat I mean is that n-cpu uses another conf file by default11:05
bauzasbut you can change this11:05
sean-k-mooneyyep11:05
bauzasjust don't use systemctl in this case11:05
sean-k-mooneyya not?11:06
sean-k-mooney*why not?11:06
bauzasjust becauses it precludes the config file name11:06
bauzassometimes I prefer stopping the service and starting it by hand11:07
bauzasmaybe because I'm an old guy11:07
sean-k-mooneysystemctl is the correct way to do that in devstack for like 2-3 releases11:07
sean-k-mooneyinfact that devstack docs use n-cpu as the example  https://docs.openstack.org/devstack/latest/systemd.html#manipulating-units11:08
*** cdent has joined #openstack-nova11:08
jangutterbauzas: I'm still sometimes using 'ifconfig', where I should be using 'ip', and really, really, should be using 'nmcli'.11:14
bauzassean-k-mooney: you misunderstood me11:14
bauzassean-k-mooney: I'm not saying to not use systemctl (although there is still a devstack option for it :p )11:15
bauzassean-k-mooney: I'm saying that, most of the times, stopping the incriminated service and restarting it by hand is preferrable11:15
sean-k-mooneybauzas: noop the screen option was removed in rocky11:16
bauzasgtk11:16
*** ttsiouts has quit IRC11:16
sean-k-mooneyoh you mean sudo systemctl stop devstack@n-cpu.service the nova-coumpute ...11:16
*** ttsiouts has joined #openstack-nova11:17
sean-k-mooneyruning the service directly can be usefaul for debuging11:17
jangutterOne thing I like is that I can go 'systemctl status PID' and get all the startup (and a bit of logs) for a service.11:20
*** cdent has quit IRC11:20
jangutterIs there something similar for docker/containers?11:20
*** ttsiouts has quit IRC11:20
*** ttsiouts has joined #openstack-nova11:20
sean-k-mooneyjangutter: docker logs <container name or id>11:20
sean-k-mooneyor depeneding on your container it may log to the hosts journal so journalctl -u <service in contienr>11:21
*** jangutter_ has joined #openstack-nova11:22
*** jangutter_ has left #openstack-nova11:23
*** jangutter_ has quit IRC11:23
*** jangutter_ has joined #openstack-nova11:23
*** mhen has quit IRC11:23
*** mhen has joined #openstack-nova11:24
*** owalsh_ has quit IRC11:24
*** owalsh has joined #openstack-nova11:24
*** jangutter has quit IRC11:25
*** jangutter_ is now known as jangutter11:26
*** abhishekk has quit IRC11:27
ShilpaSDhi: All, please help me to understand, how this section, table format of notificaiotns,  is added here 'https://docs.openstack.org/nova/rocky/reference/notifications.html#existing-versioned-notifications'11:30
izza_hi11:31
izza_encounetered error upon execution of tensorflow session11:32
izza_>>> sess = tf.Session()11:32
izza_2018-12-03 11:30:11.110534: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA11:33
izza_raceback (most recent call last):11:33
izza_were using tensorflow 1.11.011:33
*** maciejjozefczyk has quit IRC11:33
* stephenfin has returned11:34
izza_we tried to upgrafe it to tensorflow 1.12.0 but same error11:34
*** adrianc has joined #openstack-nova11:34
*** maciejjozefczyk has joined #openstack-nova11:35
*** maciejjozefczyk has quit IRC11:35
*** brinzhang has quit IRC11:37
gibiShilpaSD: let me find the code that generates that table...11:39
*** tbachman has quit IRC11:39
*** adrianc has quit IRC11:39
*** maciejjozefczyk has joined #openstack-nova11:39
gibiShilpaSD: here is the code that generates the table https://github.com/openstack/nova/blob/master/doc/ext/versioned_notifications.py11:39
ShilpaSDgibi: thanks for that11:39
*** adrianc has joined #openstack-nova11:39
ShilpaSDgibi: how to configure this with notifications.rst?11:41
ShilpaSDthe linking?11:41
*** awalende_ has joined #openstack-nova11:41
*** ratailor has quit IRC11:41
ShilpaSDgibi: what this line is doing exactly? https://github.com/openstack/nova/blob/master/doc/source/conf.py#L4611:41
*** hamzy_ has joined #openstack-nova11:42
*** hamzy has quit IRC11:42
*** ttsiouts_ has joined #openstack-nova11:43
*** whoami-rajat has quit IRC11:43
gibiShilpaSD: the conf.py line:  ext.versioned_notifications instructs sphinx to load and run https://github.com/openstack/nova/blob/master/doc/ext/versioned_notifications.py#L5411:43
*** whoami-rajat has joined #openstack-nova11:44
*** awalende has quit IRC11:44
*** ttsiouts has quit IRC11:45
*** erlon has quit IRC11:46
ShilpaSDgibi:i if i have this value set this getting 'ERROR: InvocationError for command '/opt/stack/<CODE BASE>/.tox/docs/bin/python setup.py build_sphinx' (exited with code 1)' error ehile docs creation, what action i need to take?11:48
ShilpaSDthis is i have added >> extensions = ['sphinx.ext.autodoc', 'oslosphinx','ext.versioned_notifications']11:49
ShilpaSDis i am missing any other supporting extensions?11:50
gibiShilpaSD: I don't know. This is the original commit added the notification extension https://github.com/openstack/nova/commit/05adc8d006b482e0aed2fcc9dc4885924aca74d011:51
gibiShilpaSD: in that I did not have to add any extra extension11:51
gibiShilpaSD: btw, what do you want to achive? do you want to reuse ext.versioned_notifications outside of the nova project?11:52
gibiShilpaSD: if I insert some python error to ext/versioned_notifications.py then doc generation command prints a proper stack trace for me11:55
gibiShilpaSD: nova uses the following build command in tox: sphinx-build -W -b html -d doc/build/doctrees doc/source doc/build/html11:56
gibiShilpaSD: this part of the conf.py in nova could be intresting to you as well https://github.com/openstack/nova/blob/master/doc/source/conf.py#L2411:57
*** cdent has joined #openstack-nova11:58
*** awalende_ has quit IRC11:58
*** awalende has joined #openstack-nova11:59
*** erlon has joined #openstack-nova12:03
*** owalsh has quit IRC12:03
ShilpaSDgibi: Hi, i am in the process of introducing notifications in Masakari, i have completed all coding and api doc, not only part remained is this sample notifications table12:03
*** ttsiouts_ has quit IRC12:05
gibiShilpaSD: so you copied ext/versioned_notifications.py from nova to masakari and would like to use it in masakari12:05
ShilpaSDgibi: yes12:05
*** ttsiouts has joined #openstack-nova12:05
gibiShilpaSD: do you have a patch up on review that I can download and play with it locally?12:06
ShilpaSDgibi: sorry for that, as of now no patch i have uploaded12:07
ShilpaSDgibi: during tox, its giving error while 'Running Sphinx v1.8.2'12:08
openstackgerritBalazs Gibizer proposed openstack/nova master: Reject interface attach with QoS aware port  https://review.openstack.org/57007812:09
openstackgerritBalazs Gibizer proposed openstack/nova master: Reject networks with QoS policy  https://review.openstack.org/57007912:09
*** liuyulong has quit IRC12:10
openstackgerritBalazs Gibizer proposed openstack/nova master: Reject networks with QoS policy  https://review.openstack.org/57007912:10
*** ttsiouts has quit IRC12:10
openstackgerritChris Dent proposed openstack/nova master: Clean up header encoding handling in compute API  https://review.openstack.org/60696712:11
gibiShilpaSD: let me try to reproduce yor problem locally12:11
ShilpaSDgibi: thanks for this12:13
*** owalsh has joined #openstack-nova12:21
gibiShilpaSD: with this change https://review.openstack.org/621558 I can make sphinx load the extension in masakari12:22
gibiShilpaSD: of course it still fails as the code in versioned_notifications.py tries to import nova12:23
openstackgerritMerged openstack/nova stable/rocky: Remove unnecessary redirect  https://review.openstack.org/60740012:26
openstackgerritJan Gutter proposed openstack/os-vif master: Extend port profiles with datapath offload type  https://review.openstack.org/57208112:27
ShilpaSDgibi: sorry, its still giving me Extension error:, i have updated my code for 'https://review.openstack.org/#/c/621558/1/doc/source/conf.py@19'12:30
ShilpaSDfrom your patch12:30
ShilpaSDrest already i ahve taken from Nova12:30
gibiShilpaSD: I get the following from my patch12:31
gibiExtension error:12:31
gibiCould not import extension ext.versioned_notifications (exception: No module named 'nova')12:31
gibiERROR: InvocationError for command '/home/ebalgib/upstream/git/masakari/.tox/docs/bin/python setup.py build_sphinx' (exited with code 1)12:31
gibiShilpaSD: and as I said this is expected as I just blindly copied versioned_notifications.py from nova12:32
gibiand that python code imports nova12:32
ShilpaSDok, will check further, give me few time, will get back to you12:33
gibiShilpaSD: so the next step with that patch would be to adapt the code in versioned_notificaton.py to masakri12:33
gibiShilpaSD: sure12:33
*** awalende has quit IRC12:33
ShilpaSDgibi: sure12:34
*** ttsiouts has joined #openstack-nova12:34
ShilpaSDgibi: thanks, after adding 'https://review.openstack.org/#/c/621558/1/doc/source/conf.py@19', its giving me proper stack trace, and now i am on the way to resolve errors listed12:42
ShilpaSDearlier it gives, exception: cannot import name 'json_ref'12:42
gibiShilpaSD: cool12:43
ShilpaSDafter resolving its giving further error with detailed stack trace12:43
gibiShilpaSD: json_ref is also something that is inside nova12:43
ShilpaSDgibi: yes, that i have got it12:43
gibiOK12:43
ShilpaSDnow further issues with my code, will check and get back to you if any further hurdles12:44
*** alexchadin has quit IRC12:44
gibiShilpaSD: sure, you can ping me any time (I'm working mostly in EU timezone)12:44
*** awalende has joined #openstack-nova12:45
*** cdent has quit IRC12:45
ShilpaSDgibi: sure thnaks12:45
*** artom has joined #openstack-nova12:49
*** lpetrut has joined #openstack-nova12:52
*** erlon_ has joined #openstack-nova12:52
*** lpetrut has quit IRC12:55
*** lpetrut has joined #openstack-nova12:55
*** erlon has quit IRC12:56
*** cdent has joined #openstack-nova12:56
*** dave-mccowan has joined #openstack-nova12:57
*** tbachman has joined #openstack-nova12:59
*** alexchadin has joined #openstack-nova12:59
*** tbachman has quit IRC13:04
*** k_mouza has quit IRC13:05
*** tbachman has joined #openstack-nova13:09
sean-k-mooneysligtly off topic but does anyone know if any of the people from packet are on the nova irc?13:11
*** tbachman has quit IRC13:17
*** alexchadin has quit IRC13:29
*** k_mouza has joined #openstack-nova13:29
*** tbachman has joined #openstack-nova13:30
*** udesale has joined #openstack-nova13:30
*** dave-mccowan has quit IRC13:32
*** k_mouza_ has joined #openstack-nova13:35
*** k_mouza has quit IRC13:38
*** jroll has quit IRC13:38
*** cdent has quit IRC13:38
*** jroll has joined #openstack-nova13:38
*** takashin has joined #openstack-nova13:38
*** tetsuro has joined #openstack-nova13:39
*** tetsuro has quit IRC13:40
*** cfriesen has joined #openstack-nova13:42
*** tetsuro has joined #openstack-nova13:45
*** tetsuro has quit IRC13:46
*** yedongcan has left #openstack-nova13:47
*** jaosorior has joined #openstack-nova13:47
*** tetsuro has joined #openstack-nova13:48
*** whoami-rajat has quit IRC13:50
*** jdillaman has joined #openstack-nova13:52
*** fried_rice is now known as efried14:00
stephenfinsean-k-mooney: packet?14:01
*** cdent has joined #openstack-nova14:01
*** efried is now known as fried_rice14:01
*** fried_rice is now known as efried14:02
sean-k-mooneystephenfin: https://www.packet.com/14:02
*** tbachman has quit IRC14:02
stephenfinsean-k-mooney: You learn something new every day14:02
*** awaugama has joined #openstack-nova14:03
sean-k-mooneyi am planning on setting up a 2 node openstack over the christmas + nodepools/zuul but i was debating about using some burst capasity from them or another cloud provider also14:05
sean-k-mooneyi have the hardware locally to do ci but as you heard my rackmount server are not exactly quite14:06
*** mriedem has joined #openstack-nova14:07
sean-k-mooneyahyway i might not but just wanted to ask them a few questions like can i set a montly limit on pricing that kind of thing14:07
*** Nel1x has joined #openstack-nova14:08
adriancsean-k-mooney: Hi, regarding libvirt-neutron-sriov-livemigration, did you have a chance to look further at the code ? Also did you get a chance to work on the direct port support for POC ?14:14
adriancsean-k-mooney: i.e the detach/attach flow14:15
sean-k-mooneyadrianc: i hope to start working on the direct mode support this/ next week as my primary focus. i have been dealing with downstream bugs for the last week or more so other then deploying your code i have not made much progress14:18
sean-k-mooneyadrianc: i see you pushed a new version of the code yesterday14:19
adriancsean-k-mooney: ack, ill upload an updated PS today, dealing with some unit test issues14:19
sean-k-mooneyi should get a change to re review and redeploy with it tomorrow14:20
adriancsean-k-mooney: yes, addressing some of the issues that we talked abou, plus better clarifying the code14:20
sean-k-mooneyi will be respinnign the spec today/tommorow too by the way14:20
adriancsean-k-mooney: any major changes ? or mainly what was discussed in the review ?14:22
sean-k-mooneyill try and align the spec to the code you have submitted.14:22
mriedembauzas: while you're here, it would be good to get another core looking at the initial allocation ratio series, starting here https://review.openstack.org/#/c/620154/14:22
sean-k-mooneyadrianc: am bauzas had some cleanups/clarifing questions but no major changes14:22
bauzasmriedem: sure, will do14:23
sean-k-mooneyadrianc: ill be addressing his nits first but if you want to respond to any of his questions ill incoperate your answers tomorrow after i review the latest version fo the code14:23
adriancadrianc: great, so seems we are on the right track, ill look at his comments today and comment if needed14:24
*** psachin has quit IRC14:24
sean-k-mooneyadrianc: the main question resolves around how the claims will work and how we handel mixed verions of compute nodes14:25
*** jmlowe has quit IRC14:25
*** jmlowe has joined #openstack-nova14:26
*** SteelyDan is now known as dansmith14:28
sean-k-mooneyadrianc: i am hoping we can converge on the spec and approve it in the next week or so and then have the code ready for a runway not too long after. if we can land this before m2 that would be ideal but we will see how things go14:28
sean-k-mooneythere are several higher priority items ahead of it14:28
*** tbachman has joined #openstack-nova14:28
*** k_mouza has joined #openstack-nova14:31
*** k_mouza has quit IRC14:31
*** k_mouza has joined #openstack-nova14:32
*** jonher has joined #openstack-nova14:33
*** k_mouza_ has quit IRC14:34
*** k_mouza_ has joined #openstack-nova14:38
*** rodolof has joined #openstack-nova14:39
*** k_mouza has quit IRC14:41
*** burt has joined #openstack-nova14:43
*** k_mouza_ has quit IRC14:46
*** moshele has quit IRC14:46
*** leakypipes is now known as jaypipes14:48
*** k_mouza has joined #openstack-nova14:48
*** lbragstad has joined #openstack-nova14:49
*** sahid has quit IRC14:53
*** dave-mccowan has joined #openstack-nova14:55
*** awalende has quit IRC14:55
*** sahid has joined #openstack-nova14:55
jangutterjaypipes, sean-k-mooney: regarding the os-vif-generic-datapath-spec-of-doom, how would I go about "freezing the object versions prior to a major version bump"?14:56
sean-k-mooneyyou wouldnt we jsut would not bump them if that is what we agreed to do14:57
*** hshiina_ has joined #openstack-nova14:57
jangutterjaypipes, sean-k-mooney: OK, is there some way of submitting that as a doc patch to os-vif?14:57
jaypipessean-k-mooney: I've proposed just back-versioning everything to 1.0 on all those objects.14:57
jaypipessean-k-mooney: since nothing is sent over the wire anyway14:58
sean-k-mooneyjaypipes: so kuryr might be14:58
jaypipesugh14:58
sean-k-mooneythey should not be but they might be14:58
sean-k-mooneybut we will figure that out14:58
sean-k-mooneyim fine with flatneing to 1.0 or 2.014:58
*** lbragstad has quit IRC14:58
sean-k-mooneyjaypipes: did you see my mail by the way14:59
jaypipessean-k-mooney: yes but haven't gotten to it yet this morning14:59
sean-k-mooneyit kindof long you can skip most of it15:00
*** lbragstad has joined #openstack-nova15:00
jangutteris there a separate mailing list for os-vif (or is it mostly nova + neutron)...15:01
*** takashin has left #openstack-nova15:01
*** beekneemech is now known as bnemec15:01
sean-k-mooneyi will likely propsoe a patch to our docs later this week to update our usage docs to document some of expections we have more explictly15:01
sean-k-mooneyjangutter: i useally add [os-vif] and [nova][neutron]15:02
sean-k-mooneybut not really15:02
*** awalende has joined #openstack-nova15:02
*** awalende has quit IRC15:07
*** mlavalle has joined #openstack-nova15:12
*** adrianc has quit IRC15:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Recalculate request group - RP mapping during re-schedule  https://review.openstack.org/61952915:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding  https://review.openstack.org/56945915:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331715:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Reject interface attach with QoS aware port  https://review.openstack.org/57007815:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Reject networks with QoS policy  https://review.openstack.org/57007915:14
janguttersean-k-mooney, jaypipes: if I don't need to jump through hoops, I hope https://review.openstack.org/#/c/572081 is a good representative of 'Option 1'.15:18
*** adrianc has joined #openstack-nova15:18
mriedemdansmith: online data migration pattern check here https://review.openstack.org/#/c/613499/12/nova/objects/compute_node.py@211 - it works, but not sure if it's the "right way"15:21
openstackgerritBalazs Gibizer proposed openstack/nova master: Deprecate the unversioned notifications  https://review.openstack.org/60307915:21
dansmithmriedem: queued15:21
adriancAh i really need to setup an IRC bouncer, sean-k-mooney: understood, ive dealt with the mixed version in the code. we will see how it goes.15:21
*** Luzi has quit IRC15:21
sean-k-mooneyadrianc: i just set up znc on kubernets over the weekend. k8s is a pain on the ass but i now have a bouncer15:22
sean-k-mooneyit was so much more work then i had planned vs plain docker15:23
adrianci may go straight with docker then :)15:23
*** mchlumsky has joined #openstack-nova15:24
sean-k-mooneywell i had to set up helm to deploy nginx/ingress contoller + certmanger for lets encyrpet certs to then deploy znc with config via config maps + an init sidecar container15:25
sean-k-mooneyor i could have deployed 2 contienrs by hand.15:25
sean-k-mooneymy k8s cluster is a single node on an old laptop so its not like ita actully does anyting other then booting a docker container anyway15:26
* sean-k-mooney misses my home openstack deployment already.15:28
*** mchlumsky has quit IRC15:28
*** mchlumsky has joined #openstack-nova15:29
*** dpawlik_ has quit IRC15:31
*** spatel has joined #openstack-nova15:32
*** itlinux has quit IRC15:34
*** tetsuro has quit IRC15:46
adriancthinking about it further maybe ill spin up a VM on my development machine and run it there, quicker ramp-up time as i havent actually used docker but it may be a good experience on the other hand :)15:49
sean-k-mooneyyou can always just install znc to the host. people are still allowed to do that :)15:50
cdentheresy!15:51
*** jding1_ has joined #openstack-nova15:52
*** ondrejme has quit IRC15:53
*** Cardoe has quit IRC15:54
adrianclol15:54
*** sean-k-mooney has quit IRC15:55
*** rtjure has quit IRC15:55
*** lennyb_ has quit IRC15:55
*** Cardoe has joined #openstack-nova15:55
*** jhesketh has quit IRC15:55
cdentmriedem: on the nova-status tests: they rely on the placement database. Would you prefer: a) move the test into the functional hierarchy so only functional tests are using the "remote" placement database fixture, b) unwind the removal of a "local" placement database fixture (in nova/test.py, c) mock the shit out of that stuff, or d) not bother and leave the test removed?15:55
*** sean-k-mooney has joined #openstack-nova15:56
*** tbachman has quit IRC15:56
*** jhesketh has joined #openstack-nova15:57
*** lennyb has joined #openstack-nova15:57
*** jackding has quit IRC15:58
*** janki has joined #openstack-nova15:58
cdentefried, jaypipes, dansmith, edleafe ^16:00
cdente) use the placement database fixture under 'unit' is not an option, because only the functional tests install placement16:02
*** ttsiouts has quit IRC16:03
*** ttsiouts has joined #openstack-nova16:03
edleafeYou know I despise unit tests that test more than a unit16:03
edleafeIf it stays in unit, mock it. But moving to functional sounds like a better option16:04
*** jmlowe has quit IRC16:04
*** gyee has joined #openstack-nova16:04
cdentthanks edleafe, for now I'm going with move to functional as anything else is way more work than it is worth16:07
jaypipescdent: my vote would be same as Ed's. move it to functional...16:07
*** hshiina_ has quit IRC16:08
*** ttsiouts has quit IRC16:08
cdentthanks jaypipes16:08
gibi+1 for moving it to functional :)16:09
*** rtjure has joined #openstack-nova16:10
*** dpawlik has joined #openstack-nova16:11
*** Sundar has joined #openstack-nova16:13
mriedemsorry was in another channel,16:15
mriedemor e) i could rewrite those tests to use the placement rest api fixture (not the db fixture) in a patch below this one16:15
*** dklyle has joined #openstack-nova16:15
*** dpawlik has quit IRC16:16
cdentmriedem: the test, as currently written in master, cannot pass with the other changes in the patch, which remove the placement database, locally16:17
*** dtantsur is now known as dtantsur|afk16:17
cdentso it would definitely need to be a prioer to this one, if you're feeling inclined16:17
mriedembut the placement fixture (rest api) continues to work, right? but it gets the fixture from placement.16:18
cdentyes, but that fixture is only available to functional tests, not unit16:19
cdent(because only the functional tox jobs import placement master)16:19
cdentthis seemed a good safeguard to insure that unit tests are unit tests16:20
*** dpawlik has joined #openstack-nova16:20
cdentand is part of why this test ran into issues16:20
mriedemok in that case i guess (a) is best for now, unless we just drop the check altogether as i think we sort of flirted with last week16:21
cdentroger that, I'll carry on, thanks16:21
*** janki has quit IRC16:26
mriedemok, i'm going to dump some words into the ML16:26
* cdent holds breath16:31
*** itlinux has joined #openstack-nova16:33
*** itlinux has quit IRC16:33
*** itlinux has joined #openstack-nova16:33
*** lpetrut has quit IRC16:36
*** belmoreira has quit IRC16:37
*** belmoreira has joined #openstack-nova16:37
* mriedem feels 5 lbs lighter16:38
edleafeMust have been a big dump16:42
*** dpawlik has quit IRC16:42
dansmithI call BS on the 5lb number16:43
dansmithhttps://www.healthline.com/health/do-you-lose-weight-when-you-poop16:43
*** amodi has joined #openstack-nova16:45
artomI think BS (from bulls!) is definitely more than 5 lbs.16:45
dansmithprobably, but mriedem is decidedly smaller than a bull16:45
*** tbachman has joined #openstack-nova16:46
bauzasare we discussing how our body filters and weights stuff ?16:46
mriedemi'm going to take the high road here16:47
mriedemjust this once16:47
artomMy son blocked the toilet with his poop. Twice. He's not even 3 yet.16:48
*** itlinux_ has joined #openstack-nova16:50
*** dpawlik has joined #openstack-nova16:52
*** itlinux has quit IRC16:54
*** itlinux_ has quit IRC16:55
*** itlinux has joined #openstack-nova16:56
mriedemi've seen that from a kid once, at my gf's house in high school, her nephew birthed a loaf of bread i swear, it was amazing17:00
cdentso much for the high road17:00
mriedemyou can thank artom17:01
mriedemhe brings out the worst in me17:01
cdentyeah, he just gently laid that in there, and the house came down17:01
*** udesale has quit IRC17:02
dansmithartom: that's quite an, um, honor17:04
artomI did to this channel what my son did to the toilet17:08
artomI am... this channel's poop? Dammit >_<17:08
dansmithwe can't even see the high road from here17:08
artomYeah, but pot's legal in Canada now, so if I ever need to get to the high road, I'm all set17:10
artomWhich is why I can afford these shenanigans17:10
*** _pewp_ has quit IRC17:16
*** _pewp_ has joined #openstack-nova17:17
*** _hemna has quit IRC17:17
*** _hemna has joined #openstack-nova17:18
*** manjeets has joined #openstack-nova17:19
*** dpawlik has quit IRC17:19
openstackgerritJack Ding proposed openstack/nova master: [WIP] Preserve UEFI NVRAM variable store  https://review.openstack.org/62164617:22
dansmithartom: when are you thinking you'll have numa-aware-live-migration patches up/refreshed for looking?17:22
*** jding1_ has quit IRC17:22
*** jackding has joined #openstack-nova17:23
*** sahid has quit IRC17:26
*** ileixe has quit IRC17:32
openstackgerritMerged openstack/nova master: Fix sloppy initialization of the new disk ops semaphore.  https://review.openstack.org/62134617:32
jaypipessean-k-mooney: I think we need mdulko's response on your ML thread.17:32
artomdansmith, it's in my tasks for this downstream sprint17:34
artomSo I'm not sure how much I'll get done, but at least 117:34
*** adrianc has quit IRC17:35
*** dpawlik has joined #openstack-nova17:35
artomdansmith, I'm not really expecting it to land by our downstream deadlines. But if I can get it in upstream in Stein, and make it tech preview for OSP15, I'd consider that a win17:36
dansmithartom: okay, in the upstream channel I'm really only asking about things that matter to upstream :)17:36
*** moshele has joined #openstack-nova17:36
sean-k-mooneyjaypipes: is there a responce to the os-vif one or is it another one?17:37
dansmithjust wondering if there will be meat to review this year or not, as it doesn't seem like there's much up right now17:37
artomdansmith, I know, but was giving you context17:37
dansmithalright17:37
artomdansmith, maybe not a full 15 oz steak, but like, a chicken cutlet?17:38
*** adrianc has joined #openstack-nova17:38
jaypipessean-k-mooney: no, I mean you are asking for kuryr-kubernetes folks to respond to your questions in your ML thread and I'm saying I'll wait to hear from them until I weigh in.17:39
*** dpawlik has quit IRC17:39
sean-k-mooneyjaypipes: ah ok17:39
sean-k-mooneyya ill proably jump into there irc tommorw and flag it to them17:40
*** k_mouza_ has joined #openstack-nova17:44
*** erlon_ has quit IRC17:44
*** Sundar has quit IRC17:44
*** Sundar has joined #openstack-nova17:45
*** k_mouza has quit IRC17:48
*** k_mouza_ has quit IRC17:48
*** moshele has quit IRC17:51
mriedemso we're not trying hard to get numa-aware live migration done in stein upstream, got it17:54
*** amodi has quit IRC17:55
*** amodi has joined #openstack-nova17:56
efriedjaypipes: I redid the three patches starting at https://review.openstack.org/#/c/620711/ per your comments. Gate failures are spurious, rechecking. tssurya confirmed they've been running with this code in their env and it's performing as expected.17:58
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional regression test for bug 1806064  https://review.openstack.org/62166417:58
openstackbug 1806064 in OpenStack Compute (nova) rocky "Volume remains in attaching/reserved status, if the instance is deleted after TooManyInstances exception in nova-conductor" [Medium,Triaged] https://launchpad.net/bugs/180606417:58
sean-k-mooneymriedem: it will likely be hard to get it runway ready and reviewd before jan 10, it likely can be dont before upstream feature freeze but it likely will be non trival to review17:58
mriedems10: melwitt: ^17:58
cfriesenartom: we'll happily try to break any patches you come up with.  :)17:58
mriedemsean-k-mooney: feature freeze is not jan 1017:58
mriedemFF is march 717:59
sean-k-mooneyoh i taught it was mid febuary but i guess im mixing that up with non-client libs18:00
tssuryaefried: we will try to put up a blog post soon with the plots and perf-improvement18:01
efriedtssurya: That would be awesome, my boss was just asking me if such a thing was available.18:01
mriedemtssurya: are there plans to test that out with the ironic computes?18:01
mriedemso those can be upgraded?18:01
*** erlon has joined #openstack-nova18:01
mriedemyou had to downgrade ironic computes to queens right?18:02
tssuryamriedem: yea we are working on the ironic driver and for sure will have it upgraded asap18:02
tssuryamriedem: yea the prod is still in queens for ironic compute18:02
*** derekh has quit IRC18:02
mriedemefried: do i smell pbc results?18:02
mriedemer,18:03
artomcfriesen, please do18:03
mriedemcheckpoint results18:03
tssuryaefried: will let you know as soon as we post one :) we have a huge reduction in requests per min from 120K to 10K18:03
efriedmriedem: Yeah, we call it checkpoint these days.18:03
artommriedem, I *do* bring out the worst in you ;)18:03
openstackgerritChris Dent proposed openstack/nova master: Use external placement in functional tests  https://review.openstack.org/61794118:07
openstackgerritChris Dent proposed openstack/nova master: Delete the placement code  https://review.openstack.org/61821518:07
*** moshele has joined #openstack-nova18:07
*** k_mouza has joined #openstack-nova18:10
*** ralonsoh has quit IRC18:11
*** k_mouza has quit IRC18:15
*** moshele has quit IRC18:15
*** ttsiouts has joined #openstack-nova18:21
*** ohorecny2 has quit IRC18:24
*** udesale has joined #openstack-nova18:26
*** ttsiouts has quit IRC18:28
*** jmlowe has joined #openstack-nova18:36
*** jmlowe has quit IRC18:39
*** belmoreira has quit IRC18:40
jaypipesefried: tssurya also confirmed they're not using it for Ironic. :)18:40
jaypipesefried: or are you just referring to *some* of those patches and not the one that could be trouble for Ironic installs?18:41
efriedjaypipes: I fixed the one we were concerned about for ironic as well. But yeah, it would be nice if we could get results from CERN playing with it live.18:42
*** vabada has quit IRC18:43
*** tssurya has quit IRC18:47
*** adrianc has quit IRC18:54
*** wolverineav has joined #openstack-nova18:57
*** adrianc has joined #openstack-nova18:58
*** adrianc has quit IRC18:58
*** wolverineav has quit IRC19:01
*** wolverineav has joined #openstack-nova19:01
*** adrianc has joined #openstack-nova19:02
*** erlon has quit IRC19:03
*** munimeha1 has joined #openstack-nova19:04
*** adrianc has quit IRC19:06
*** jmlowe has joined #openstack-nova19:06
dansmithmriedem: questions in https://review.openstack.org/#/c/393930/419:12
dansmithwell, a question and a suggestive comment I guess19:14
*** adrianc_ has joined #openstack-nova19:15
*** igordc has joined #openstack-nova19:18
*** priteau has quit IRC19:20
jaypipesjangutter: I support sean-k-mooney's suggestions about renaming VIFBase to VIF.19:20
jaypipesjangutter: in the context of our planned changes to the 2.0 os-vif modeling.19:21
*** amotoki has quit IRC19:21
sean-k-mooneyjaypipes: at the end of the day it will have very little differnece from a code point of view but as a new person looking at it nameing is important and "base" implys meaning that we dont enforce which might confuse people19:22
jaypipes++19:22
sean-k-mooneyit was never something that annoyed me enough to "fix" but if we are cleaning up things we might as well clean that up too19:23
*** amotoki has joined #openstack-nova19:23
jaypipesright, my thoughts exactly.19:23
sean-k-mooneythat is a change that will break kuryr but again they should not have been inheriting form VIFBase anyway. we can use debtcollector to not break them untill i port over there vifs.19:25
sean-k-mooneyi wont get to it for a few weeks but i should get to it i january19:25
*** adrianc_ is now known as adrianc19:25
*** wolverineav has quit IRC19:27
*** wolverineav has joined #openstack-nova19:28
mriedemdansmith: replied19:28
jaypipessean-k-mooney: well, again, kuryr should be pinning to a *specific* os-vif library release, so we should be safe to make these changes, along as they don't up their pinned release. same for nova, frankly.19:29
*** adrianc_ has joined #openstack-nova19:31
sean-k-mooneyture but we have a kuryr ci job that will bitch since it runs against os-vif master19:32
*** wolverineav has quit IRC19:32
*** wolverineav has joined #openstack-nova19:32
sean-k-mooneysame as or nova job that test stuff before we merge it19:33
jaypipessean-k-mooney: that's fine (and works as expected, yeah?)19:33
*** spatel has quit IRC19:34
sean-k-mooneythey are there to catch non backwards compatible changes so yes it works as expected but i just ment unless we make the change in a backward compatible way the zuul job will fail19:35
*** adrianc_ has quit IRC19:35
*** adrianc_ has joined #openstack-nova19:36
*** spatel has joined #openstack-nova19:36
dansmithmriedem: yeah I forgot about the distinction of the before-virt and after-virt nature of the two places we stash that stuff19:36
sean-k-mooneye.g. if we renamed VIFBase to vif without the compaiblity code or removing the use of VIFbase in kuryer the kuryr-kubernetes-tempest-daemon-octavia would fail and stop use merging it in os-vif19:36
dansmithmriedem: note that pulling it from an api node via instance_extra is the same number of hits to the db as just doing an query for all the virtual_interfaces for the instance presumptively, since the api is hitting the db directly19:37
*** adrianc_ has quit IRC19:38
*** tssurya has joined #openstack-nova19:41
openstackgerritDavid Hill proposed openstack/nova master: Add secret=true to fixed_key configuration parameter  https://review.openstack.org/62168419:42
*** Sundar has quit IRC19:43
mriedemsince john hopkins isn't around anymore,19:47
mriedemi wonder who would care enough to unwind all of this deprecated code we have to remove it https://review.openstack.org/#/c/621684/1/nova/conf/key_manager.py19:47
mriedemnow that barbican is a required service or whatever the TC calls it19:48
mriedemcdent: what is the latest opinion on barbican being required and co-gating?19:48
mriedembase devstack still doesn't install barbican as far as i know19:48
* cdent falls in a hole19:48
mriedemalthough not sure we'd need barbican anyway since i think in the gate we'd just use a fake key manager from castellan19:48
cdentfungi is your guy on that stuff, as I recall, but also the new styleeee seems to be "depend on castellan instead"19:49
mriedemright b/c barbican is just a castellan backend19:49
mriedemi also see that nova registers but doesn't doc the castellan options in our config option docs build, yay19:50
cdentthe base services stuff, to my eye, stalled19:50
fungi_if_ you need users to provide data to the keystore though, recommend barbican rather than reimplementing bits of it badly just to avoid dependence on it19:50
mriedemi just want to burn code that's been deprecated in nova for years now19:51
mriedembut it was all added by john hopkins people that are no longer around19:51
cdenthttps://media.giphy.com/media/wKiZeZVXS5MQg/giphy.gif19:51
mriedemmoonrise kingdom?19:52
fungiaddams family ii19:52
sean-k-mooneyim not familar with that option when it refers to key manager is it refing to keyparis or to something else19:52
mriedemoh heh19:52
*** spatel has quit IRC19:52
cdentyeah19:52
fungisean-k-mooney: secret data store19:52
*** spatel has joined #openstack-nova19:53
fungioh, you mean in the nova key_manager config19:53
* fungi only just looked at the proposed change19:53
sean-k-mooneyya. oh like the secrets you would normally store in a tpm or secure enclave19:54
* cdent loves the word enclave19:54
* fungi blames apple for that wording19:54
sean-k-mooneycdent: makes me think of fallout or a furnace/oven19:55
mriedemi guess we do have the castellan opts in our [key_manager] group https://docs.openstack.org/nova/latest/configuration/config.html#key-manager19:55
mriedemwith barbican as the default backend19:55
* cdent retires19:55
cdentgood luck everyone19:55
*** cdent has quit IRC19:55
mriedemand we use that fixed_key in devstack yet https://github.com/openstack-dev/devstack/blob/78a564bb0304b6f930e1491e7e116a0a0f6d9ab6/stack.sh#L127119:56
mriedemhttps://review.openstack.org/#/c/439855/19:56
fungisean-k-mooney: yes, makes me wonder if apple devs were playing fallout ii at the time they came up with the term for their hsm19:57
*** wolverineav has quit IRC19:57
*** wolverineav has joined #openstack-nova19:58
mriedemah yes https://github.com/openstack-dev/devstack/blob/a61b4704871e7b0ae88a50932576e961eca615e4/lib/nova#L43019:58
fungimriedem: any reason we can't just add barbican to default devstack configuration? it doesn't have to be in the base services list19:58
mriedemanecdotal comments from when i asked sdague about that years ago was that barbican broke their api frequently19:59
fungii'm assuming the nova feature in question is not part of any required capability for interop19:59
mriedemso making that a required thing in the integrated gate was not something he wanted to do19:59
mriedemfungi: no, just would like to remove a bunch of old deprecated code in nova19:59
sean-k-mooneymaybe the iniset macros does someith im not aware of but should that not set it to the lital string nova.keymgr.conf_key_mgr.ConfKeyManager20:00
mriedemi also care very little about this issue atm - it's just something i think about every time i have to look at any of that code20:00
mriedemsean-k-mooney: it does20:00
fungiseems to me like telling people to deploy barbican if they want users to be able to upload key material is fine20:00
mriedemhttp://logs.openstack.org/77/615677/13/check/tempest-full/f0fd07c/controller/logs/etc/nova/nova_conf.txt.gz20:00
mriedemcinder is in the same boat here carrying the same debt20:01
mriedemhttp://logs.openstack.org/77/615677/13/check/tempest-full/f0fd07c/controller/logs/etc/cinder/cinder_conf.txt.gz20:01
sean-k-mooneyok was that intentional. looking at https://docs.openstack.org/nova/latest/configuration/config.html#key_manager.backend it looks wrong to me20:02
fungihttps://governance.openstack.org/tc/reference/base-services.html#current-list-of-base-services does specifically state "...any user interaction with secret material should be left to a dedicated API instead (preferably as provided by Barbican)."20:02
*** wolverineav has quit IRC20:03
openstackgerritAdrian Chiris proposed openstack/nova master: Add free for claimed, allocated devices  https://review.openstack.org/61612020:03
openstackgerritAdrian Chiris proposed openstack/nova master: Allow per-port modification of vnic_type and profile  https://review.openstack.org/60736520:03
openstackgerritAdrian Chiris proposed openstack/nova master: Add get_instance_pci_request_from_vif  https://review.openstack.org/61992920:03
openstackgerritAdrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support  https://review.openstack.org/62011520:03
fungithe only thing which has really slowed me from proposing a barbican-specific entry there similar to keystone's is identifying services exposing features which are better served by barbican's api (as opposed to just backend use via castellan)20:04
sean-k-mooneyhum it looks like https://github.com/openstack-dev/devstack/commit/a5b72b053efec9af8a57c59b35f96142e4f38433 set that value 4 year ago but that looks like its wrong to me sice both nova and nueton says the backend can be "valut" or "barbican"20:08
sean-k-mooneyactuly a little under 3 but same difference20:08
mriedemyou're missing this https://github.com/openstack/castellan/blob/6336a59b9b5f373efa653e6fcce8ea1c964fdbae/castellan/key_manager/migration.py#L3020:09
mriedemthere is a special hack in castellan for nova and cinder20:09
mriedemwhich have their own no-op in-tree key managers for testing20:09
mriedemConfKeyManager is not documented since it's not something we want people using in prod20:09
mriedemthe backend itself is open-ended https://github.com/openstack/castellan/blob/master/castellan/key_manager/__init__.py#L2520:10
*** igordc has quit IRC20:10
sean-k-mooney oh ok... it would be nice to atleast document that in hte nova code but maybe i missed it20:10
*** igordc has joined #openstack-nova20:10
openstackgerritMatt Riedemann proposed openstack/nova master: Create BDMs/tags in cell with instance when over-quota  https://review.openstack.org/62169220:15
*** david-lyle has joined #openstack-nova20:16
*** manjeets_ has joined #openstack-nova20:17
*** manjeets has quit IRC20:18
*** dklyle has quit IRC20:18
mnasercan i ask what's the state of https://review.openstack.org/#/c/554218 or https://review.openstack.org/#/c/554212 ? -- i have a use-case that might need pre-emptible instances so im curious about those20:18
mriedemthis ^ is a pretty straight-forward regression fix since pike20:18
sean-k-mooneymnaser: anyway i had taught we said we woudl try and complete those in denver but honestly i dont know20:23
*** jmlowe has quit IRC20:23
tssuryamnaser, sean-k-mooney: yea we had discussed it in denver ptg, at this point I think it needs more core reviews20:23
sean-k-mooneytssurya: and a couple of pep8 fixes :)20:24
mnasersean-k-mooney, tssurya: i can help in contributing the code needed and cleaning it up20:24
tssuryasean-k-mooney: heh right20:24
mnaserboth add a lot of flexibility in terms of being able to do more20:24
sean-k-mooneymnaser: yes when i was at intel i was also looking at other ways of using the pending state.20:25
* mnaser has wild ideas of having a pool of machines that run baremetal only and provisioning new hypervisors based on demand20:25
sean-k-mooneymnaser: e.g. useing it as  a maker to signal that i need to autoscale the cloud rather the reap instances20:25
sean-k-mooneymnaser: yep that was basicaly my idea too20:26
mnaserpretty much :)20:26
mnaseralso saving power20:26
tssuryamnaser: thanks for the interest, I think that would be great! my colleague had a couple of code patches also up for review https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/enable-rebuild-for-instances-in-cell0 and https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/introduce-pending-vm-state20:26
tssuryabut basically he is waiting for the specs to get merged, please feel free to chip in your ideas/ help is surely appreciated20:26
sean-k-mooneythe idea was if you have a mixed iroic/rsd and libvirt could you could provision new libvirt nodes in a seperate admin tenant using the resocses from teh ironic/rsd nodes20:27
mnasersean-k-mooney: and that means you can have baremetal and/or ironic in the same cloud and be able to share those resources across20:27
mnasertssurya: ill try to see whats going on with the pep8 at least20:27
mnasersounds silly but set it up for success :p20:27
mriedempersonally i haven't looked at the latter spec, and haven't been back to the former since i ripped it up in june20:28
mriedemand then no one was really talking about it anymore20:29
mriedemand we already have a bunch of other stuff going on20:29
mriedemso i wasn't going out of my way to look for more shit we can cram into this release while we still have other fires burning20:29
mriedembut that's just me20:29
sean-k-mooneymnaser: yep. we did not have time to explore it fully because of resoucign but we belived the pending state was generally useful beyond the proposed usecase20:29
openstackgerritMohammed Naser proposed openstack/nova-specs master: Enable rebuild for instances in cell0  https://review.openstack.org/55421820:29
mnaseri fixed the pep8 for all that matters20:29
*** udesale has quit IRC20:30
tssuryamnaser: thanks!20:30
mnasermriedem: do you see a more ideal/cleaner path for pre-emptible instances within nova?20:30
sean-k-mooneymnaser: mriedem i think johnthetubaguy  had this working out of tree last cycle i belive so it would be interesting to know what is out standing but ya i also know we have a lot of todos20:31
*** wolverineav has joined #openstack-nova20:32
sean-k-mooneyfrom a parity point of view vs aws it would be nice to have. anyway my pizza is just cooked so im going to say goodnight20:32
sean-k-mooneyo/20:32
mriedemmnaser: honestly i'm not at all up to date on what's being proposed anymore20:34
tssuryasean-k-mooney: there was a presentation at the summit on the same: https://www.openstack.org/summit/berlin-2018/summit-schedule/events/22438/science-demonstrations-preemptible-instances-at-cern-and-bare-metal-containers-for-hpc-at-ska20:34
mriedemsomething something build failure -> notification loop -> rebuild from cell020:34
mnasermriedem: yeah that idea seems to be around having an option where nova leaves thing in PENDING instead of ERROR when NoValidHost20:34
mnaserand then $some_service would do things and either request rebuild or reset state to error20:35
*** jmlowe has joined #openstack-nova20:39
*** jmlowe has quit IRC20:41
dansmithmnaser: yep that's what we discussed in dublin20:42
dansmithI too haven't circled back to it because progress seems to be very infrequent and it's hard to maintain context on it20:43
mnaseryeah i can imagine losing context over time with a lot of stuff going on can be hard :x20:43
mnaseri just mostly sync'd up through a bunch of reading and talks20:43
*** udesale has joined #openstack-nova20:58
*** amodi has quit IRC21:12
*** priteau has joined #openstack-nova21:19
*** manjeets_ is now known as manjeets21:20
*** udesale has quit IRC21:23
*** priteau has quit IRC21:41
*** igordc has quit IRC21:42
*** awaugama has quit IRC21:43
*** wolverineav has quit IRC21:56
*** rodolof has quit IRC22:00
*** jaosorior has quit IRC22:13
*** jaosorior has joined #openstack-nova22:16
mriedemdansmith: thanks, lots of good comments in there https://review.openstack.org/#/c/616037/22:17
*** rcernin has joined #openstack-nova22:18
*** pcaruana has quit IRC22:18
*** igordc has joined #openstack-nova22:20
*** udesale has joined #openstack-nova22:23
*** udesale has quit IRC22:25
*** wolverineav has joined #openstack-nova22:25
dansmithmriedem: cool, I'll try to get to the rest of it soonish22:25
*** udesale has joined #openstack-nova22:26
*** priteau has joined #openstack-nova22:27
*** priteau has quit IRC22:29
*** itlinux has quit IRC22:38
*** mriedem is now known as mriedem_away22:40
*** udesale has quit IRC22:46
*** k_mouza has joined #openstack-nova22:47
*** lbragstad has quit IRC22:51
*** lbragstad has joined #openstack-nova22:52
*** jaosorior has quit IRC22:53
openstackgerritSurya Seetharaman proposed openstack/nova master: Add DownCellFixture  https://review.openstack.org/61481022:58
openstackgerritSurya Seetharaman proposed openstack/nova master: WIP: API microversion bump for handling-down-cell  https://review.openstack.org/59165722:58
*** k_mouza has quit IRC22:59
*** spatel has quit IRC23:02
*** irclogbot_2 has joined #openstack-nova23:07
*** tssurya has quit IRC23:23
« Sunday, 2018-12-02 Index Tuesday, 2018-12-04 »

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