Page MenuHomeDevCentral
Differential D4097

Rotate GRE tunnels log
Needs ReviewPublic
Actions

Authored by Duranzed on Wed, Apr 29, 16:36.
Tags
None
Referenced Files
F27213225: D4097.id10720.diff
Fri, May 1, 13:10
F27211872: D4097.id10722.diff
Fri, May 1, 12:00
F27209477: D4097.id10722.diff
Fri, May 1, 08:38
F27209476: D4097.id10720.diff
Fri, May 1, 08:38
F27208421: D4097.diff
Fri, May 1, 07:14
Unknown Object (File)
Thu, Apr 30, 02:14
Unknown Object (File)
Wed, Apr 29, 16:53
Unknown Object (File)
Wed, Apr 29, 16:37
Subscribers
None

Details

Reviewers
dereckson
yousra
Summary

This log is used by GRE failover script.

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Passed
Unit
No Test Coverage
Branch
tunnelLogs
Build Status
Buildable 6677
Build 6965: arc lint + arc unit

Event Timeline

Duranzed requested review of this revision.Wed, Apr 29, 16:36
Duranzed created this revision.
Harbormaster completed remote builds in B6677: Diff 10720.Wed, Apr 29, 16:36
dereckson requested changes to this revision.Wed, Apr 29, 17:46
dereckson added inline comments.
roles/core/network/gre_logging.sls
6

This idea of chmod in quotes is to allow the Ansible parser to parse as octal number, not as integer.

We use Salt, 644 is fine.

This revision now requires changes to proceed.Wed, Apr 29, 17:46
Duranzed updated this revision to Diff 10722.Wed, Apr 29, 18:34

syntax correction

Harbormaster completed remote builds in B6679: Diff 10722.Wed, Apr 29, 18:34
dereckson requested changes to this revision.Fri, May 1, 14:19
dereckson retitled this revision from log files for GRE creation tunnels to Rotate GRE tunnels log.
dereckson edited the summary of this revision. (Show Details)
dereckson added inline comments.
roles/core/network/gre_logging.sls
16

You put a C in newsyslog configuration file, so newsyslog will create that file, you can remove this block.

This revision now requires changes to proceed.Fri, May 1, 14:19
Duranzed updated this revision to Diff 10726.Fri, May 1, 15:05

Removed the redundant log file creation block, as newsyslog already creates the file via the C flag.

Harbormaster completed remote builds in B6683: Diff 10726.Fri, May 1, 15:05
Duranzed marked 2 inline comments as done.Fri, May 1, 15:06

Revision Contents
Changeset List

PathSize
roles/
core/
network/
files/
16 lines
17 lines
26 lines

Diff 10720

View Options

roles/core/network/files/newsyslog_gre.conf

Loading...
View Options

roles/core/network/files/syslog_gre.conf

Loading...
View Options

roles/core/network/gre_logging.sls

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator