search

cloudfoundry / docs-loggregator

Apache License 2.0
3 stars 36 forks source link

Separate Log Cache in diagrams #58

Closed ctlong closed 2 years ago

cf-gitbot commented 2 years ago

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

ctlong commented 2 years ago

We think that we only edited the graffle diagrams related to the two pngs that needed to be updated. However, due to our lack of familiarity with OmniGraffle (apparently the go-to tool to edit .graffle diagrams), it's possible that we changed other things too. This is also hard to compare because the newer version of OmniGraffle may export in ways that are dissimilar to the version used to generate the pngs 16 months ago.

@mjgutermuth if you want, we can upload all the other pngs generated from exporting images/architecture/loggregator.graffle in order to compare them side-by-side with the previous pngs, and to make all the diagrams consistent with this version of OmniGraffle. Just let us know.

mjgutermuth commented 2 years ago

Thank you @ctlong! I think this is fine. Ideally, we'll move away from OmniGraffle at some point (I'm not sure if the docs team even has a license for it anymore) and use something more appropriate for open source. For now, I think this is good!

ctlong commented 2 years ago

Moving away from it would be great @mjgutermuth! We happened to have a team member with a license, but otherwise it may have been a blocker for us.