Description of problem: The Internet carries a variety of services, some requiring timely delivery of traffic, and others not. The packages providing the majority of server traffic (http, smtp, imap, ftp, etc) should mark their traffic appropriately so that other time-sensitive protocols may be differentiated. RFC-4594 ("Guidelines for DiffServ Service Classes") Section 4.8 designates Email as a "High-Throughput Data Service Class", requiring markings AF11, AF12, or AF13. Currently, Cyrus provides no mechanism to mark traffic. Version-Release number of selected component (if applicable): 8.14.4 How reproducible: Source inspection or using tcpdump on an IMAP session. Steps to Reproduce: 1. 2. 3. Actual results: Packets are send with ip_tos as 0x00 Expected results: Should be 0x28, 0x30, or 0x38. Additional info: This fix as been submitted upstream for consideration, and the response was, "The patch that you already submitted looks fine already." (from Ken on 01/11/2010) but it might be a while before it actually appears in a published release. A quick look at Cyrus's CVS tree shows it hasn't gone in yet. This functionality isn't enabled until the following change is made to /etc/imapd.conf: +qosmarking: af12
Errr... wrong version. Should be 2.3.16
Hi Philip, I appreciate your work, I really do, but I'm not going to add any feature patch to cyrus-imapd. We have too much patches now and they are causing a lot of troubles. So this patch has to go upstream first. As you've cited from that mail, it seems upstream will add your patch, but it can take some more time. I'd recommend to add this patch to their bugzilla so they don't forget about it. After you add it to bugzilla (if you did not do it already) send me a link and I'll add my comments there so they can see more interest and they will add the patch in the repository sooner (hopefully).
The bug is linked above (see also): https://bugzilla.andrew.cmu.edu/show_bug.cgi?id=3184
I have images for f13 and f12 if anyone wants to try these and post their results. http://koji.fedoraproject.org/koji/taskinfo?taskID=2113556 http://koji.fedoraproject.org/koji/taskinfo?taskID=2113606 Note that the line: # qosmarking af13 will need to be uncommented in /etc/imapd.conf
thanks, just note: you should put your builds somewhere, because koji deletes scratch builds quite soon
Moving rpm's to http://philipp.fedorapeople.org/
upstream included that patch in the cvs, so I've added it in Fedora 12+ packages
Philip, please do not remove your rpm builds from fedorapeople for a while, because I've used one of them for bug report #583982 thanks
cyrus-imapd-2.3.16-3.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/cyrus-imapd-2.3.16-3.fc12
cyrus-imapd-2.3.16-3.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/cyrus-imapd-2.3.16-3.fc13
(In reply to comment #8) > Philip, > > please do not remove your rpm builds from fedorapeople for a while, because > I've used one of them for bug report #583982 > > thanks Understood.
cyrus-imapd-2.3.16-3.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update cyrus-imapd'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/cyrus-imapd-2.3.16-3.fc12
cyrus-imapd-2.3.16-3.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update cyrus-imapd'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/cyrus-imapd-2.3.16-3.fc13
cyrus-imapd-2.3.16-3.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
Note: there is a missing ':' in the comment on the "qosmarking" line. I refreshed the .rpm's I have on: http://philipp.fedorapeople.org/ after noticing this, but wasn't sure it was worth updating the push for it...
fixed in the cvs
cyrus-imapd-2.3.16-3.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.