Rotate GRE tunnels log
AcceptedPublic
Actions

Authored by Duranzed on Wed, Apr 29, 16:36.

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 6683
Build 6971: 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

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
15	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

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

dereckson accepted this revision.Fri, May 1, 17:59

This revision is now accepted and ready to land.Fri, May 1, 17:59

Revision Contents
Changeset List

Path

Size

roles/

core/

network/

gre_logging.sls

7 lines

Diff 10726

View Options

roles/core/network/gre_logging.sls

Rotate GRE tunnels logAcceptedPublicActions