| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-28 | |||
| 09:24:41 | sfinucan | gcb: Thanks for the ping - I'd missed that. +2 | |
| 09:25:50 | gcb | sfinucan, cool thanks | |
| 09:25:52 | gabor_antal | Hey, would someone please check https://review.openstack.org/#/c/448655/ ? I've hit this bug in another patch | |
| 09:26:09 | mdbooth | kaisers: Sucks about systemd-run :( I'm interested to know what problems you saw with the forking type, and what OS you're using. | |
| 09:28:02 | mdbooth | kaisers: IIUC, the behaviour of that unit type is that any failure of the top-level process will be reflected in the exit code of systemd-run, so if the process fails before it forks we'll know about it. | |
| 09:28:24 | Kevin_Zheng | gmann: thanks | |
| 09:28:28 | mdbooth | I guess mount.quobyte might fail after the fork, which would be disappointing. | |
| 09:29:54 | bauzas | alexey_weyl: hola | |
| 09:30:14 | alexey_weyl | bauzas: hi :) | |
| 09:30:23 | bauzas | alexey_weyl: not sure, lemme verify | |
| 09:30:46 | bauzas | AFAIR, it's not possible, but I don't remeber where it's verified | |
| 09:31:09 | alexey_weyl | bauzas: I have created such an example as I described for you on my system, but I am just trying to understand if those are 2 different AZs. | |
| 09:33:41 | bauzas | alexey_weyl: yup, it's not possible | |
| 09:34:08 | bauzas | alexey_weyl: because of nova.compute.api.is_safe_to_update_az | |
| 09:34:45 | bauzas | alexey_weyl: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4583-L4586 | |
| 09:34:52 | alexey_weyl | bauzas: please explain. Because as I said I have created on my devstack a host sggregate called HA1 with zone1 and another host aggregate HA2 with zone1 as well. | |
| 09:35:26 | bauzas | alexey_weyl: do you have hosts in the aggregates yet? | |
| 09:35:35 | alexey_weyl | bauzas: no | |
| 09:35:51 | bauzas | alexey_weyl: okay, so please add one host into HA1 | |
| 09:36:02 | bauzas | alexey_weyl: and then add a second host into HA1 | |
| 09:36:05 | bauzas | HA2* | |
| 09:36:20 | bauzas | then, it should be kaboom for adding the second host into HA2 | |
| 09:36:34 | alexey_weyl | bauzas: you are saying that if i'll add hosts to those host aggregates and AZs then it will through an exception? | |
| 09:36:40 | bauzas | yup | |
| 09:36:53 | alexey_weyl | bauzas: great, thanks very much!!! :) | |
| 09:37:03 | alexey_weyl | bauzas: very helpful! ) | |
| 09:37:06 | alexey_weyl | bauzas: :) | |
| 09:37:06 | bauzas | you can technically create two aggregates having same AZ metadata, but only one could have hosts in there | |
| 09:37:20 | bauzas | alexey_weyl: please verify it tho :) | |
| 09:37:21 | alexey_weyl | bauzas: Thanks :) | |
| 09:37:44 | alexey_weyl | bauzas: ahhhhhhh, that is quiet a problem for me. | |
| 09:37:58 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | |
| 09:38:08 | alexey_weyl | bauzas: because it is quite a procedure where I work to do such a thing. | |
| 09:39:04 | bauzas | alexey_weyl: well, adding a host into the second HA2 aggregate would return you an exception saying "One or more hosts already in availability zone(s)" | |
| 09:40:01 | gabor_antal | sfinucan: thanks for the review! :) | |
| 09:40:06 | alexey_weyl | bauzas: ok, great | |
| 09:42:00 | openstackgerrit | John Garbutt proposed openstack/nova master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/445782 | |
| 09:42:58 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add description to policies in lock_server.py https://review.openstack.org/448468 | |
| 09:48:05 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Switch Nova Quobyte volume driver to mount via systemd-run https://review.openstack.org/432344 | |
| 09:52:07 | rabel | hi everyone. someone here with quota-knowledge? ;) i don't understand quota classes, default quota, etc. so i'd like to ask some questions | |
| 09:53:06 | openstackgerrit | John Garbutt proposed openstack/nova master: Add description to policies in used_limits https://review.openstack.org/447710 | |
| 09:53:27 | rabel | first of all: what is a quota class? is that just a collection of quotas? so if i change the quotas for a project, that means there is a quota class assigned to that project in the background and i actually change the quotas in that class? | |
| 09:53:40 | rabel | is "default" also a quota class or something different? | |
| 09:54:23 | rabel | and what are default quotas for a project? since i thought the default quotas are those assigned to a new project. is this for creating new projects with --parent option? | |
| 09:59:47 | kaisers | mdbooth: Hi! I tested on centos 7.3 and in a xenial vm. The tests returned exit code 0 in all variants (except when i had a systemd-run parameter typo) and all tested sleep calls returned immediately | |
| 10:01:05 | kaisers | mdbooth: right now i'm looking into another option, mount.quobyte does send a systemd ready notification upon successful mount, possibly this is a simple solution. But currently i'm looking into retrieving possible error information in such a variant | |
| 10:01:29 | mdbooth | kashyap: On centos 7.3? | |
| 10:02:11 | mdbooth | kashyap: That was for kaisers :) | |
| 10:02:18 | mdbooth | kaisers: Confirmed here :( | |
| 10:02:28 | mdbooth | Behaviour is different on F25 | |
| 10:04:38 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: Explain why we bring up the lb in hybrd mode https://review.openstack.org/450692 | |
| 10:05:48 | openstackgerrit | Moshe Levi proposed openstack/os-vif master: Explain why we bring up the lb in hybrd mode https://review.openstack.org/447269 | |
| 10:08:45 | kaisers | mdbooth: yep, just re-verified: http://paste.openstack.org/show/604445/ | |
| 10:09:12 | kaisers | mdbooth: ok, I'll get back when i've looked into using systemd notify... | |
| 10:10:17 | mdbooth | kaisers: You could solve it with the post-verify method, but I think it would be much better if you didn't have to | |
| 10:23:18 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'move-nova-cmds-to-cliff' spec https://review.openstack.org/433603 | |
| 10:37:50 | mdbooth | kaisers: I suspect this is what we really want: https://www.reddit.com/r/linux/comments/4ypb2g/systemd_rolls_out_its_own_mount_tool/d6qicwi/ | |
| 10:37:53 | mdbooth | Bit new, though :( | |
| 10:38:01 | mdbooth | It's not even in Fedora | |
| 10:43:56 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: [WIP] PIC NUMA policy PoC https://review.openstack.org/390520 | |
| 11:01:23 | openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: XenAPI: support for VGPU https://review.openstack.org/450122 | |
| 11:29:30 | openstackgerrit | Sergey Nikitin proposed openstack/nova master: [WIP] PIC NUMA policy PoC https://review.openstack.org/390520 | |
| 11:52:12 | kaisers | mdbooth: yeah! This keeps on dangling fobidden ('cos too fresh) fruits in front of my face... :-D | |
| 12:04:08 | gabor_antal | bausas, sfinucan: would someone please readd WF+1, I had to rebase the patch: https://review.openstack.org/#/c/448655/ | |
| 12:04:19 | gabor_antal | oops, bauzas | |
| 12:04:57 | jaypipes | gabor_antal: done | |
| 12:05:17 | gabor_antal | thanks, Jay! that was quick :) | |
| 12:05:40 | jaypipes | gabor_antal: heh, well I was actually just reviewing that patch :) | |
| 12:10:50 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add policy-remove-scope-checks spec https://review.openstack.org/433037 | |
| 12:35:53 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Explicitly specify arguments of server_groups creation https://review.openstack.org/450763 | |
| 12:42:35 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add spec to use cinder's new attachment API https://review.openstack.org/373203 | |
| 12:43:13 | johnthetubaguy | mdbooth: would be awesome if you get chance to double check I didn't break anything with my fixes for mriedman's comments | |
| 12:53:12 | openstackgerrit | John Garbutt proposed openstack/nova master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/445782 | |
| 12:54:11 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Explicitly specify arguments of server_groups creation https://review.openstack.org/450763 | |
| 13:08:07 | kaisers | mdbooth: I'm changing this to rely on the sd_notify approach and i'm moving the systemd mount to a separate method as it becomes a completely separate mount/verify code path. This will take a bit, i'll ping you here once it's up for review, if that's ok... | |
| 13:08:57 | mdbooth | kaisers: Sounds good. Are you going to use the systemd-notify cmdline tool? | |
| 13:09:47 | kaisers | mdbooth: Nope, the mount.quobyte client is sending this, so the call will stay mostly the same | |
| 13:10:16 | mdbooth | kashyap: Does it send a notification on failure? | |
| 13:10:58 | mdbooth | kashyap: That was for kaisers (again) | |
| 13:11:14 | kashyap | Yeah, no worries, I know the tab-complete snafu all too well | |
| 13:11:16 | kaisers | mdbooth: no, only a success notification | |
| 13:11:33 | kaisers | mdbooth: but i'm adding a check for this | |
| 13:12:18 | mdbooth | Do you happen to know if there's anywhere we can run a functional test of this and inject a failure? | |
| 13:13:10 | kaisers | mdbooth: Should be doable by providing an invalid backend address | |
| 13:13:40 | kaisers | mdbooth: That'd provide a range of output and a timeout on the unit startup | |
| 13:13:41 | mdbooth | kaisers: I mean: do we have infra to run it? | |
| 13:13:51 | kaisers | mdbooth: our CI? | |
| 13:14:02 | mdbooth | Does your CI just run tempest? | |
| 13:14:11 | kaisers | mdbooth: currently yes | |
| 13:14:13 | mdbooth | Because this wouldn't be very tempesty | |
| 13:14:42 | mdbooth | This is a prime candidate for a functional test | |
| 13:14:45 | kaisers | mdbooth: hmm, i can add something like that. Is there doc about functional testing in CIs | |
| 13:14:46 | kaisers | ? | |
| 13:15:09 | mdbooth | kaisers: sdague might know ^^^ ? | |
| 13:16:32 | kaisers | <-- is adding a ticket for himself to add func testing to CI | |
| 13:25:37 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add additional-default-policy-roles spec https://review.openstack.org/427872 | |
| 13:36:21 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: serial console log (completed) https://review.openstack.org/450636 | |
| 13:41:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: remove ugly local import https://review.openstack.org/450787 | |
| 13:43:19 | gibi | johnthetubaguy: hi! thanks for the review https://review.openstack.org/#/c/418488/ ( | |
| 13:43:43 | gibi | johnthetubaguy: I've just noticed your comment in that | |