-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Printing docs update #540
Printing docs update #540
Conversation
Changed title -- test commit
Mention that our printing system already does rasterization
Errors caught by make test fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, let me know when you are ready to merge this.
@fydai sounds good. If no one else has any comments or suggested changes, then I guess merge... |
looks like this: | ||
|
||
``` | ||
[Application] | ||
[Local machine] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we add the local machine CUPS to this chart as well?
through a filter that rasterizes any PDFs, are then forwarded onto printhost | ||
for further processing before finally being printed. | ||
|
||
The OCF's central print server is based around two components: CUPS, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You might want to reflow this paragraph. /shrug
@@ -115,8 +141,7 @@ afford the toner. | |||
|
|||
## Print accounting | |||
|
|||
The OCF uses a virtual CUPS printer backend called [Tea4CUPS][Tea4CUPS] to | |||
install a page accounting hook that runs before and after each job is actually | |||
The OCF uses a Tea4CUPS backend to install a page accounting hook that runs before and after each job is actually |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrap at 80 if possible
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also you should specify that this happens on printhost not locally here
Cool, cool, and thanks for the comments so far! Given all the recent comments and concerns about printing (see Slack -- the issues page is very out of date), I'm going to put this on the back burner for now, while we address the printing situation... |
I've updated the documentation to reflect the newer printing system in place. Let me know if there is something I should add or change!
Addresses #529
Issues page for the printing system itself: ocf/puppet#731