| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-05 | |||
| 09:06:26 | mdbooth | johnthetubaguy: ^^^ was a rebase after a merge conflict. Would you mind a fast re-approve? | |
| 09:09:03 | johnthetubaguy | sure | |
| 09:09:23 | mdbooth | Thanks | |
| 09:10:34 | mdbooth | Does anybody else use os-log-merger, btw? | |
| 09:26:15 | cdent | mdbooth: a) first I've heard of it but b) probably would have if I had known about it | |
| 09:26:40 | mdbooth | I've been using it for a bit, just about to start submitting patches | |
| 09:26:41 | cdent | there's a lot of opinion about logs: separate is better! merged is better! stop it you grapes, use journald! | |
| 09:26:47 | cdent | etc | |
| 09:27:08 | mdbooth | I'd previously written my own which performed the same function | |
| 09:27:30 | mdbooth | Merged is the only way to go in my experience :) | |
| 09:28:28 | kashyap | mdbooth: Yeah, first ajo called it "oslogmerger" | |
| 09:28:36 | kashyap | My brain used to read it as "Oslo GMerger" | |
| 09:28:56 | mdbooth | Unfortunately, os-log-merger unfortunately uses stupid sorting which makes it slow and a memory hog | |
| 09:29:11 | mdbooth | But it suffices and I don't have to maintain it myself | |
| 09:29:36 | mdbooth | The one I wrote in bash and awk was a lot faster, if a little obtuse :) | |
| 09:30:34 | kashyap | Ah, os-log-merger is part of OpenStack upstream repos | |
| 09:30:39 | mdbooth | Yep | |
| 09:31:22 | kashyap | mdbooth: Your `bash` / `awk` equivalent, all lives in one file I suppose? | |
| 09:31:47 | mdbooth | Yep | |
| 09:32:44 | mdbooth | http://paste.openstack.org/show/608943/ | |
| 09:34:07 | mdbooth | Basically it creates an awk process per input file which munges the inputs, then merge sorts the output of the multiple awk processes | |
| 09:34:21 | mdbooth | Like I said, it's very fast but a little obtuse :) | |
| 09:34:43 | mdbooth | Also not very flexible with multiple timestamp formats | |
| 09:36:02 | kashyap | What's obtuse in that line! :P - | |
| 09:36:03 | kashyap | if ($2 ~ /[0-9]{2}:[0-9]{2}:[0-9]{2}\.[0-9]{3}/) | |
| 09:36:11 | openstackgerrit | NidhiMittalHada proposed openstack/python-novaclient master: Nova volume-attach doesn't care for device name https://review.openstack.org/454048 | |
| 09:36:25 | mdbooth | kashyap: What? Nothing could be clearer! | |
| 09:37:11 | kashyap | I recall you've done Tcl, too | |
| 09:37:17 | mdbooth | Indeed | |
| 09:37:41 | mdbooth | And Java, C, and OCaml. | |
| 09:38:15 | mdbooth | Professionally, anyway. | |
| 09:38:52 | sfinucan | ...and Tcl, for that matter | |
| 09:38:57 | sfinucan | everything's a string, my ass | |
| 09:39:12 | kashyap | LOL | |
| 09:39:18 | mdbooth | sfinucan: Perl doesn't have a bug up it's arse about being a proper language :P | |
| 09:39:27 | mdbooth | It's right there in the name | |
| 09:39:42 | mdbooth | It's a disgusting bunch of hacks for sysadmins to use to get stuff done quickly with | |
| 09:39:57 | mdbooth | Same as Python, but python is delusional | |
| 09:41:01 | sfinucan | mdbooth: We had a test framework for OVS when I worked in Intel | |
| 09:41:55 | sfinucan | Interacted with some (outrageously expensive) IXIA traffic generators to run nightly performance benchmarking tests | |
| 09:42:06 | sfinucan | Bash. Thousands of lines of bash. | |
| 09:42:32 | mdbooth | Haha | |
| 09:42:57 | mdbooth | My rule of bash is that as soon as you need the manpage, you need to be planning a rewrite :) | |
| 09:42:59 | sfinucan | with some Tcl thrown in because IXIA knew how to build traffic generators but had zero idea about writing usable APIs | |
| 09:43:18 | sfinucan | :D very true | |
| 09:43:57 | sfinucan | I insist of 'set -e' at the top of any script | |
| 09:44:41 | sfinucan | *set -u, rather | |
| 09:45:08 | kashyap | "Treat unset variables and parameters other than the special parameters ‘@’ or ‘*’ as an error when performing parameter expansion. An error message will be written to the standard error, and a non-interactive shell will exit." | |
| 09:45:18 | sfinucan | -e is quit on any exit | |
| 09:45:21 | sfinucan | -u is that ^ :) | |
| 09:45:24 | kashyap | sfinucan: Yeah, was wondering about '-e', which is about exit rules involving pipelines | |
| 09:45:28 | sfinucan | *uit on any error | |
| 09:46:56 | sfinucan | Still better than what my college buddies who chose the hardware engineering track have to deal with | |
| 09:47:46 | sfinucan | Hardware testers with the cruftiest APIs you've ever seen (many using VB, oddly), and none of the skills to write wrappers and make things usable :( poor guys | |
| 09:49:03 | mdbooth | sfinucan: But it's Friday :) Also, I feel like I haven't moaned about Python in months. | |
| 09:50:08 | sfinucan | mdbooth: It is! And the suns out and people are cutting their lawns. No better day to embrace the way of Python ;) | |
| 09:50:31 | mdbooth | Hehe, from datetime: "An aware object is used to represent a specific moment in time that is not open to interpretation [1]." | |
| 09:50:40 | mdbooth | "[1]If, that is, we ignore the effects of Relativity" | |
| 09:54:16 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Delete existing allocations for move operations or retries https://review.openstack.org/460433 | |
| 09:54:17 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Allocate instance by the conductor https://review.openstack.org/460455 | |
| 09:54:17 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Don't delete allocation if instance being scheduled https://review.openstack.org/460177 | |
| 09:59:45 | mdbooth | cdent: I'd copyright it, but I expect it's not original | |
| 10:00:12 | cdent | seems likely to be plenty of independent developments | |
| 10:01:52 | johnthetubaguy | we may have just out nerded that author | |
| 10:02:00 | mdbooth | lol | |
| 10:02:03 | cdent | \o/ | |
| 10:02:24 | sfinucan | 🙈 | |
| 10:02:27 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | |
| 10:02:50 | openstackgerrit | suzhengwei proposed openstack/nova master: compute_model problem https://review.openstack.org/462897 | |
| 10:04:16 | openstackgerrit | suzhengwei proposed openstack/nova master: compute_model problem https://review.openstack.org/462897 | |
| 10:11:19 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Allocate instance by the conductor https://review.openstack.org/460455 | |
| 10:11:19 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Delete existing allocations for move operations or retries https://review.openstack.org/460433 | |
| 10:40:34 | bauzas | cdent: good point on the allocations being deleted when adding new ones | |
| 10:42:14 | cdent | bauzas: It's kind of non-obvious | |
| 10:42:27 | bauzas | cdent: I think that's the main problem | |
| 10:42:34 | bauzas | cdent: maybe we should be explicit ? | |
| 10:43:07 | bauzas | cdent: also, the problem is that if the scheduler doesn't provide a new destination, then we should still have the wrong allocation | |
| 10:43:22 | bauzas | cdent: so, probably something being discussed at the summit | |
| 10:43:22 | cdent | well it depends on how we want things to mean. The way in which it is possible to replace allocations is something that we added in the late rush at the end of newton | |
| 10:43:40 | cdent | I do agree that managing the information explicitly is usually a good idea | |
| 10:58:57 | openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Release the memory quota for video ram when deleting an instance. https://review.openstack.org/456050 | |
| 11:03:40 | openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Release the memory quota for video ram when deleting an instance. https://review.openstack.org/456050 | |
| 11:45:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: remove scality volume driver https://review.openstack.org/449364 | |
| 11:49:20 | ygl | hi all | |
| 11:49:40 | ygl | can anyone explain to me why boot from volume is faster than boot from image ? | |
| 12:07:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename qemu_enable_gluster_storage_driver https://review.openstack.org/447091 | |
| 13:23:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: [BugFix] Release the memory quota for video ram when deleting an instance. https://review.openstack.org/456050 | |
| 13:47:56 | openstack | Launchpad bug 1657774 in OpenStack Compute (nova) "duplicate for #1656056 Nova does not re-raise 401 Unauthorized received from Neutron for admin users" [Undecided,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | |
| 13:47:56 | dansmith | mriedem: you opened this bug: https://bugs.launchpad.net/nova/+bug/1656056 | |
| 13:48:06 | openstack | Launchpad bug 1657774 in OpenStack Compute (nova) "Nova does not re-raise 401 Unauthorized received from Neutron for admin users" [Undecided,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | |
| 13:48:06 | dansmith | mriedem: which was marked as a dupe of: https://bugs.launchpad.net/nova/+bug/1657774 | |
| 13:48:19 | dansmith | which has a pending change: https://review.openstack.org/#/c/422696 | |
| 13:48:32 | mriedem | and you have a customer that needs it fixed *today* | |
| 13:48:38 | dansmith | I'm not sure I grok what is going on | |
| 13:48:39 | dansmith | no | |
| 13:48:51 | mriedem | i thought sahid "fixed" this awhile ago | |
| 13:48:54 | dansmith | is the problem token expiry or something? | |
| 13:49:00 | dansmith | yeah, I remember that | |
| 13:49:08 | dansmith | that was before your bug was filed though | |
| 13:49:49 | dansmith | sahid's thing was token expiration during the operation I think | |