| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-05 | |||
| 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 | |
| 13:49:56 | mriedem | well, | |
| 13:50:10 | mriedem | there was also a general non-useful error if you configured the neutron creds in nova.conf incorrectly i think | |
| 13:50:16 | mriedem | there was an old bug for that from a long time ago | |
| 13:50:22 | 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:50:22 | mriedem | https://bugs.launchpad.net/nova/+bug/1656056 was from something i saw in gate runs | |
| 13:50:27 | mriedem | i did some investigation but didn't dupe it myself | |
| 13:50:35 | mriedem | i couldn't figure out what was going on at the time when i was triaging it | |
| 13:50:51 | dansmith | so the commit message makes it sound like something where we don't raise 401 if the token is an admin, even if we didn't get it from the nova.conf ? | |
| 13:50:57 | lbragstad | if the token is being expired during the operation i can take a poke at it - we should be able to fix that from a service user perspective if that helps | |
| 13:51:18 | rpodolyaka | dansmith: yeah, that's the case, IIRC | |
| 13:51:20 | mriedem | well, this is why we have the service user token spec | |
| 13:52:00 | rpodolyaka | I saw this on my env on requests done by ceilometer polling agent | |
| 13:52:01 | mriedem | https://blueprints.launchpad.net/nova/+spec/use-service-tokens-pike is for using a service token for long-running operations | |