| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-21 | |||
| 09:36:11 | kashyap | Huh, who said it was just for performance? | |
| 09:36:33 | bauzas | kashyap: don't expect operators to load contect on every relnote | |
| 09:36:36 | bauzas | context* | |
| 09:36:51 | bauzas | it's a massive amount of information they have to digest | |
| 09:37:07 | bauzas | and we suffered some pain in the past by overexpecting their knowledge | |
| 09:37:27 | kashyap | bauzas: I know very well about that. I take great care in not assuming people will have context. | |
| 09:37:56 | gibi | stephenfin: do you have any suggestion about the table issue in https://review.openstack.org/#/c/644810/1/specs/stein/approved/bandwidth-resource-provider.rst@624 | |
| 09:37:57 | bauzas | anyway, I don't want to spend more time on it, will spin a new revision | |
| 09:38:12 | bauzas | gibi: stephenfin is mostly eaten by meetings all week | |
| 09:38:31 | gibi | bauzas: thanks I will try to fight sphinx alone then :) | |
| 09:38:36 | bauzas | kashyap: let me just provide a new revision | |
| 09:38:47 | kashyap | bauzas: Hang on | |
| 09:38:53 | sean-k-mooney | kashyap could you update the release not with the explaning you gave to me about the 3 booleans that the cachemodes set to explain why its safe to ues writeback | |
| 09:39:00 | kashyap | bauzas: Just post your new text as a comment | |
| 09:39:34 | sean-k-mooney | kashyap: that is the context that ops will be missing. it was the context i was missing when i saw the commit first | |
| 09:39:59 | sean-k-mooney | ops wont see the commit message so all they have is the release note | |
| 09:40:35 | bauzas | sean-k-mooney: yeah I agree | |
| 09:41:01 | bauzas | sean-k-mooney: kashyap did a great work on documenting it in a docstring https://review.openstack.org/#/c/641981/13/nova/virt/libvirt/driver.py@410 | |
| 09:41:11 | bauzas | but we don't take the opportunity to explain it to ops | |
| 09:41:28 | bauzas | while it could be scary "heh, wtf, now I could loose data integrity?" | |
| 09:42:27 | sean-k-mooney | right well quickly looking at that it could be more or less copied into the release note | |
| 09:42:55 | sean-k-mooney | it might be a little verbose but that is not a bad thing | |
| 09:43:39 | sean-k-mooney | i think all of the context is already in the patch just need to duplicate some of it into the release note | |
| 09:55:18 | bauzas | mdbooth: kashyap: sean-k-mooney: so, I provided an example of what I'd like to see in the relnote | |
| 09:55:48 | bauzas | if operators don't want to expect modern OSes, it's their choices | |
| 09:55:51 | mdbooth | bauzas: I saw that. Honestly, though, this hasn't been an issue for at least a decade, maybe 2. | |
| 09:56:11 | bauzas | mdbooth: not really, you can enable caching on some OSes that don't flush, right? | |
| 09:56:13 | mdbooth | I think it just increases cognitive load on the reader for no practical benefit. | |
| 09:56:28 | bauzas | from my recall, W2K is one of those | |
| 09:56:28 | sean-k-mooney | mdbooth: there are still people running centos 5 | |
| 09:56:41 | mdbooth | sean-k-mooney: Centos 5 definitely flushes! | |
| 09:56:42 | sean-k-mooney | mdbooth: didnt it have issue with flushing | |
| 09:56:45 | sean-k-mooney | ok | |
| 09:57:16 | sean-k-mooney | that was the distro that i always worried about for this kind of thing | |
| 09:57:33 | mdbooth | If your OS doesn't flush data on request, you're going to find out about that real soon. | |
| 09:58:06 | sean-k-mooney | i know it does not partcally like virtio disks as i useulaly had to revert to useing other disk bus like ide to avoid kernel issues | |
| 09:58:32 | mdbooth | I remember having to be very careful shutting down my old 286-clone Tandy running DOS, though. | |
| 09:58:56 | bauzas | oh, I remember some PITAs I got with Win2K guests in the past with OpenStack Essex | |
| 09:59:03 | bauzas | I'm pretty sure they weren't flushing | |
| 09:59:23 | sean-k-mooney | im sure that at least 50% of all openstack workloads :P | |
| 09:59:26 | bauzas | but this could have been fixed in a SP or something else | |
| 09:59:35 | sean-k-mooney | dos gaming in the cloud | |
| 10:00:12 | bauzas | actually, that's a good question now we have containerizing OSes like Atomic | |
| 10:00:25 | bauzas | do we still flush on disk ? | |
| 10:00:40 | sean-k-mooney | with containers? | |
| 10:00:45 | sean-k-mooney | that shoudl not change | |
| 10:01:03 | mdbooth | bauzas: It's really an application issue: your application needs to flush. | |
| 10:01:23 | mdbooth | If the application flushes, the OS should also flush. | |
| 10:01:53 | mdbooth | The only thing writeback vs writethrough is going to fix is if your application flushes but the OS does not. | |
| 10:02:32 | mdbooth | If the application doesn't flush, the OS can't flush it anyway, as it may not have the data. | |
| 10:03:57 | bauzas | hum, reading https://lifehacker.com/do-i-really-need-to-eject-usb-drives-before-removing-th-5863810 | |
| 10:04:08 | bauzas | man, why is this so complicated ? | |
| 10:04:31 | bauzas | anyway, I agree, it's maybe just a strawman | |
| 10:04:43 | bauzas | but I still feel we need to explain it | |
| 10:05:10 | mdbooth | bauzas: USB drives don't need O_SYNC either if your application flushes and you wait for your application to finish before unplugging. | |
| 10:05:44 | mdbooth | Eject flushes the whole disk. | |
| 10:08:43 | bauzas | yeah, just reading http://msdn.microsoft.com/en-us/library/windows/desktop/aa364218%28v=vs.85%29.aspx | |
| 10:08:56 | bauzas | anyway, I'm balanced | |
| 10:09:21 | bauzas | see, we're talking for 30 mins about something needing more than just a glance | |
| 10:09:41 | bauzas | the fact that mdbooth and kashyap know the situation makes reasonable the change | |
| 10:09:58 | bauzas | but I just feel we need to capture this knowledge | |
| 10:10:08 | bauzas | and explain it thru the relnote | |
| 10:34:37 | stephenfin | gibi: Looks like you solved it :) | |
| 10:35:03 | gibi | stephenfin: yeah, I managed. | |
| 10:51:56 | sean-k-mooney | bauzas: for the record we can talk for 30 mins about just about any topic :) | |
| 11:41:39 | kashyap | bauzas: Just back; ah, I mistook your formatting comment | |
| 11:52:12 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fix links to neutron QoS minimum bandwidth doc https://review.openstack.org/645084 | |
| 12:01:03 | kashyap | sean-k-mooney: CentOS-5 is perfectly fine w.r.t. flushes. Again, as noted only *horribly* old OSes that don't matter (like RHEL-4 or even older versions of Windoze) | |
| 12:01:21 | kashyap | ... that had problems. And they're not relevant. | |
| 12:02:00 | sean-k-mooney | kashyap: ya | |
| 12:02:17 | kashyap | sean-k-mooney: /me reworks the rel note; and you're right - some of the info I could add there | |
| 12:02:30 | kashyap | But as Matt noted, I was being careful of not adding needless cognitive load to the reader. | |
| 12:02:49 | sean-k-mooney | i just have had data curption issue with centos 5 and centos 5 based vnf in the past and was not sure if that was due to flushing | |
| 12:05:20 | sean-k-mooney | that said it could equally have been down to the use of ext2/3? i cant rememebr but all i recal was we had to be very care to ensure we shutdown cleanly or we woudl have to do a rebuild with a clean image and/or do a rescue to fix the file system | |
| 12:15:10 | kashyap | sean-k-mooney: bauzas: Before I upload, does this read better? -- https://kashyapc.fedorapeople.org/PS14-writeback-cache-mode-for-guests-a7e4d2806c956164.yaml.txt | |
| 12:15:13 | kashyap | mdbooth: ^ | |
| 12:17:25 | kashyap | (Hit refresh, if you already clicked it :D) | |
| 12:17:49 | sean-k-mooney | haha i had | |
| 12:18:25 | kashyap | Sorry, removed a duplication. (/me is obsessed with words) | |
| 12:19:00 | sean-k-mooney | you also just fixed an issue i ws goint to raise | |
| 12:19:31 | sean-k-mooney | we are not ment to use phases like "we're" | |
| 12:19:33 | kashyap | Which one? | |
| 12:19:42 | kashyap | Yeah, I know. Contractions, etc. | |
| 12:20:14 | kashyap | I find it a bit irritating, though. I'd assume people (even non-native speakers) are competent enough to parse "we're" as "we are" | |
| 12:20:29 | kashyap | But I see where "I'd" can can confusion, as it can read: "I had", or "I would" :D | |
| 12:20:33 | sean-k-mooney | actully its not the contractrions | |
| 12:20:52 | kashyap | Oh? | |
| 12:21:25 | sean-k-mooney | we are ment to say nova in its the wrong perspective | |
| 12:23:18 | kashyap | tox -e releasenotes | |
| 12:23:19 | sean-k-mooney | https://docs.openstack.org/doc-contrib-guide/writing-style/general-writing-guidelines.html#write-in-second-person | |
| 12:23:23 | kashyap | Err, wrong window | |
| 12:24:19 | sean-k-mooney | "Use first person plural pronouns (we, our) judiciously." we are allowed to use "we" in some cases but its generally not teh perfered pharsing | |
| 12:24:22 | kashyap | Right, I normally avoid using "we", beceause it is so loaded | |
| 12:24:29 | kashyap | Thanks for the pointer, though :-) | |
| 12:25:09 | sean-k-mooney | kashyap: am so to your origninal question i would remove "And OSes do that." | |
| 12:25:54 | sean-k-mooney | i dont think its needed but otherswise that looks ok | |
| 12:26:03 | kashyap | sean-k-mooney: Not quite sure, otherwise it leaves the reader in the dark. They'll wonder: "will the OS do or not". But all OSes "that matter" do that | |
| 12:26:42 | sean-k-mooney | perhaps | |
| 12:27:11 | kashyap | I'll say "all modern OSes" | |
| 12:28:01 | sean-k-mooney | ya that read better how about "All modern OSes flush data as requried." | |