| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-04 | |||
| 07:49:29 | openstackgerrit | jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458 | |
| 07:52:26 | openstackgerrit | Alex Xu proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470 | |
| 07:53:57 | openstackgerrit | Alex Xu proposed openstack/nova master: Remove cloudpipe APIs https://review.openstack.org/459470 | |
| 08:13:21 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049 | |
| 08:19:48 | sfinucan | mdbooth: Could you take a look at https://review.openstack.org/#/q/topic:disk_info_processing today? It's been around a long time and it seems like something you should look at first | |
| 08:26:32 | openstackgerrit | shaofeng cheng proposed openstack/nova master: Replace newton to release_name in upgrade.rst https://review.openstack.org/457197 | |
| 08:48:12 | openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441 | |
| 08:49:51 | sfinucan | johnthetubaguy: Are we allowed to delete configuration without a deprecation cycle, even if they do nothing? https://review.openstack.org/#/c/459470/ | |
| 08:50:46 | johnthetubaguy | sfinucan: so cloudpipe only works with nova-network, and nova-network has been deprecated (on and off) for a long time | |
| 08:50:52 | johnthetubaguy | sfinucan: I would give that one a pass | |
| 08:51:51 | sfinucan | johnthetubaguy: Cool. Sounds fair | |
| 08:54:18 | openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441 | |
| 08:56:49 | johnthetubaguy | sfinucan: +1 deleting that already deprecated option that does nothing now | |
| 08:56:59 | abhi89 | johnthetubaguy: can you please review https://review.openstack.org/#/c/460305/ | |
| 08:57:28 | johnthetubaguy | abhi89: ah, I should take a look at that now | |
| 09:02:35 | johnthetubaguy | abhi89: I really like that regression test now, really nice proof that your change works | |
| 09:04:00 | abhi89 | johnthetubaguy: yes.. i have well tested the changes also | |
| 09:04:55 | johnthetubaguy | abhi89: I just love seeing the automated tests like that, really stops us breaking your fix by accident later | |
| 09:05:46 | abhi89 | johnthetubaguy: yes.. Thanks John | |
| 09:06:09 | johnthetubaguy | abhi89: thank you for sticking with it, great to get that fixed | |
| 09:10:13 | sfinucan | johnthetubaguy: Could you take a punt at https://review.openstack.org/#/c/431037/ and successors today? | |
| 09:11:18 | johnthetubaguy | sfinucan: yeah, I want to sit down with the quota code again at some point though | |
| 09:11:30 | openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Change the parameter of the exception error message https://review.openstack.org/461985 | |
| 09:14:40 | sfinucan | johnthetubaguy: ah, at some point then :) | |
| 09:24:01 | sfinucan | johnthetubaguy: Ta :) | |
| 09:58:51 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix default_availability_zone docs https://review.openstack.org/438887 | |
| 09:58:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group https://review.openstack.org/462469 | |
| 10:13:07 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | |
| 10:13:07 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625 | |
| 10:14:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tools/xenserver: Remove 'cleanup_sm_locks' https://review.openstack.org/416520 | |
| 10:15:17 | sfinucan | bauzas: Fancy taking a look at two availability zone changes? https://review.openstack.org/438887 https://review.openstack.org/462469 | |
| 11:16:28 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of instance error context manager in resize_instance https://review.openstack.org/462499 | |
| 11:16:54 | mdbooth | Hopefully pretty simple bugfix ^^^ | |
| 11:20:46 | sean-k-mooney | hi | |
| 11:21:02 | sean-k-mooney | is there a way to make devstack not use systemd still | |
| 11:21:23 | mdbooth | sean-k-mooney: That got pushed yesterday, I think. | |
| 11:21:36 | mdbooth | https://review.openstack.org/#/c/462163/3 | |
| 11:21:45 | sean-k-mooney | swapping to systemd or keeping the legacy way | |
| 11:22:02 | mdbooth | Keeps systemd option, but defaults to legacy | |
| 11:22:37 | sean-k-mooney | i pulled devstack today and it defaulted to systemd instead of screen | |
| 11:23:55 | sean-k-mooney | but ok ill look into it. i hate systemd... this might actully be the thing that makes me stop using devstack for my development | |
| 11:24:55 | sean-k-mooney | ah i have to set USE_SCREEN=True to get the old behavior | |
| 11:25:21 | dpawlik | Hello. Can someone of you tell me if I should add DocImpact to the commit message in this PS: https://review.openstack.org/#/c/459753/ ? | |
| 11:27:00 | sean-k-mooney | dpawlik: you will need to update the docs for the new image metadata item but i belive that would be in glance's docs not novas | |
| 11:28:39 | dpawlik | sean-k-mooney: hey. So better when I add here DocImpact and ask glance team to add it in its docs | |
| 11:30:01 | sean-k-mooney | dpawlik: https://wiki.openstack.org/wiki/Documentation/DocImpact | |
| 11:30:38 | dpawlik | sean-k-mooney: thx | |
| 11:44:23 | openstackgerrit | Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753 | |
| 11:45:07 | sean-k-mooney | sdague: hi, 2 quick quetstions regarding the new systemd workflow for devstack will the screen workflow be tested and maintined going forword? also will devstack now work across reboots again? | |
| 11:49:44 | sdague | sean-k-mooney: screen goes away in Queens | |
| 11:50:03 | sdague | http://lists.openstack.org/pipermail/openstack-dev/2017-May/116301.html | |
| 11:50:15 | sdague | I'm not sure what works across reboots means | |
| 11:50:37 | sdague | mdbooth: that's entirely possible... | |
| 11:50:51 | mdbooth | I don't think the journal itself drops messages | |
| 11:51:02 | sdague | mdbooth: we did disable it now, so we're out of that woods for the moment | |
| 11:51:09 | mdbooth | Yeah, makes sense | |
| 11:51:20 | mdbooth | One to investigate though, for sure | |
| 11:53:37 | sean-k-mooney | sdague: ok i might need to make changes to our devstack plugins/CIs. | |
| 11:54:11 | sean-k-mooney | i think they will work but they may not | |
| 11:54:32 | sdague | sean-k-mooney: yeh, that thread has some common issues people hit | |
| 11:56:21 | sean-k-mooney | sdague: i guess il have to give it a fair shot but honestly the fact that the process ran in screen sessions was one of the things i liked about devstack as it made it easy to wrap your head around when you were new to openstack | |
| 11:57:06 | kashyap | sean-k-mooney: It isn't any more? I haven't run a DevStack run in a couple of weeks | |
| 11:57:12 | sdague | sean-k-mooney: ok, an equally large number of people have stated that screen was the most confusing thing ever | |
| 11:57:24 | sdague | but, regardless, there are other reasons we need this change | |
| 11:57:35 | kashyap | I've seen the systemd / 'screen' related change whoosh by. /me looks closer... | |
| 11:57:50 | kashyap | Hmm, it's this patch: "make USE_SCREEN=False imply USE_SYSTEMD=True" | |
| 11:58:50 | sean-k-mooney | yes i swapped the flag back and it still failed to stack so my local failure wasnt related to the systemd change so im going to start with a clean environment again | |
| 11:59:14 | sean-k-mooney | kashyap: ^ | |
| 11:59:31 | kashyap | sean-k-mooney: What is the implication of that patch? Still reading this - https://review.openstack.org/#/c/460062/ | |
| 11:59:36 | sean-k-mooney | sdague: screen not being maintianed for one | |
| 12:00:17 | sean-k-mooney | kashyap: the openstack service deployed by devstack are now executed via systemd instead of in screen with there logs sent to the journal | |
| 12:01:11 | kashyap | sean-k-mooney: Hmm. I like systemd journal logs (for its nice 'query-ability'), but I miss firing up a quick `screen -r` to flip through all the services | |
| 12:02:23 | sean-k-mooney | kashyap: that and quickly running a service in a debugger or viewing the live logs though journalctl supports that too. | |
| 12:02:53 | kashyap | Hmm. What is the rationale for the "feature"? | |
| 12:03:02 | kashyap | I'd agree with mtreinish there, where he says: | |
| 12:03:08 | kashyap | "I think expanding this comment a bit to say the intent here would have been nice, since I got confused by this at first." | |
| 12:03:40 | sean-k-mooney | kashyap: make test closer to production | |
| 12:04:05 | sdague | kashyap: all sorts of stuff on the mailing list over the last month, I'd rather not just reiterate all that here | |
| 12:04:25 | kashyap | Then we can _really_ claim: "If it works on DevStack, it's all kosher" :P | |
| 12:04:39 | sean-k-mooney | kashyap: haha sure... | |
| 12:04:46 | kashyap | sdague: Yeah, sorry, I'd go catch up the thread. I did see the mail, but spent all of 2 seconds on it. | |
| 12:06:12 | sean-k-mooney | sdague: if devstack works across reboots again this would be cool sideffect but it might cause some people to swap to other tools for dev. | |
| 12:07:13 | openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova master: Use check_string_length from oslo_utils https://review.openstack.org/286390 | |
| 12:07:33 | kashyap | sean-k-mooney: But _does_ it work cope with reboots with this patch? If so, that'd be nice | |
| 12:07:35 | sean-k-mooney | sdague: anyway thanks for trying to keep it working for plugins. at least 1 of your plugins seeam to be working fine with the change ill check the otheres when i get time or delegate to the rest of my team :) | |
| 12:07:56 | sean-k-mooney | kashyap: dont know havent tried it | |
| 12:08:37 | sean-k-mooney | i miss rejoin stack for that abiltiy it was really useful for demoing stuff but it also had lots of issues if you didnt know how it worked | |
| 12:10:37 | openstackgerrit | Matthew Booth proposed openstack/nova master: WIP: Automatically revert resize which fails on destination https://review.openstack.org/462521 | |
| 12:15:26 | sdague | sean-k-mooney: you are going to need to explain what your reboot workflow is, because I only reboot my devstack environments to get rid of cruft. :) So I honestly am not sure what "works with reboots" means | |
| 12:17:06 | sean-k-mooney | sdague: before rejoin_stack.sh was removed you just had to start ovs,mysql,apache and rabbit then run rejoin_stack.sh and your devstack enviorment came back up | |
| 12:17:44 | sdague | sudo systemctl start devstack@* post boot might do that | |
| 12:17:44 | sean-k-mooney | sdague: this was useful for demos as you could prepera an enviorment and snapshot it in a vm and give it to someone to try | |
| 12:18:20 | sean-k-mooney | sdague: yep you are enabling the service by default i think so it might just work aslong as you dont run unstack.sh | |
| 12:18:28 | sean-k-mooney | sdague: or clean.sh i guess | |
| 12:18:30 | sdague | yep | |
| 12:18:52 | sean-k-mooney | anyway it might just be a nice sideffect to get that fucntionality back | |
| 12:19:28 | sean-k-mooney | i used to jsut run in offline mode instad but that ment you still had to recreated the vms/netwoks/flavour ... | |
| 12:19:41 | sean-k-mooney | thats not hard jsut extra steps | |
| 12:21:40 | alex_xu_ | johnthetubaguy: sdague sfinucan appreciate if you can help this two easy api-ref changes https://review.openstack.org/461998 https://review.openstack.org/458738 | |
| 12:22:20 | sdague | sean-k-mooney: yeh, let me know how it works. If we need some tweaks to make that use can solid for you, that would be cool | |