Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Reduce memcache logging by default #1369

Merged
merged 1 commit into from
Sep 18, 2018
Merged

Reduce memcache logging by default #1369

merged 1 commit into from
Sep 18, 2018

Conversation

ellieayla
Copy link
Contributor

Unless debugging the image data being cached in memcached,
there is no value in recording all requests and responses between
fluxd and memcached into the logs. Reduce the logging, but
leave behind the comment in case someone needs it.

Unless debugging the image data being cached in memcached,
there is no value in recording all requests and responses between
fluxd and memcached into the logs. Reduce the logging, but
leave behind the comment in case someone needs it.
@ellieayla
Copy link
Contributor Author

I deployed this manifest without thinking, and paid about $248 last month
to store (replicated) memcached logs. That's a lot of beer. I have since
modified my deployment, but I hope to save someone else from having the
same initial experience with Flux.

Copy link
Member

@squaremo squaremo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An overdue change, thank you for this. Sorry about the beer :-S

@squaremo squaremo merged commit 5e2410f into fluxcd:master Sep 18, 2018
squaremo added a commit that referenced this pull request Sep 26, 2018
Reduce memcache logging by default
squaremo added a commit that referenced this pull request Oct 4, 2018
This pulls the release branch 1.7.x back into master, so that master
gets the changelog entry for the 1.7.1 release.

Below are the cherry-picked changes that were on the release/1.7.x
branch at the time of this merge.

* origin/release/1.7.x:
  Update changelog for v1.7.1
  Merge pull request #1389 from dholbach/fix-827
  Merge pull request #1372 from weaveworks/issue/1371-podcontroller-encapsulation
  Merge pull request #1369 from alanjcastonguay/patch-1
@ellieayla ellieayla deleted the patch-1 branch January 29, 2019 02:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants