Skip to content

Commit

Permalink
Merge pull request #413 from fasaxc/handle-status-report
Browse files Browse the repository at this point in the history
Drop status reports in KDD to avoid spammy warning logs in felix.
  • Loading branch information
fasaxc authored May 4, 2017
2 parents 8c34cdc + df62eb4 commit fbc28ba
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions lib/backend/k8s/k8s.go
Original file line number Diff line number Diff line change
Expand Up @@ -335,6 +335,12 @@ func (c *KubeClient) Apply(d *model.KVPair) (*model.KVPair, error) {
return c.ipPoolClient.Apply(d)
case model.NodeKey:
return c.nodeClient.Apply(d)
case model.ActiveStatusReportKey, model.LastStatusReportKey,
model.HostEndpointStatusKey, model.WorkloadEndpointStatusKey:
// Felix periodically reports status to the datastore. This isn't supported
// right now, but we handle it anyway to avoid spamming warning logs.
log.WithField("key", d.Key).Debug("Dropping status report (not supported)")
return d, nil
default:
log.Warn("Attempt to 'Apply' using kubernetes backend is not supported.")
return nil, errors.ErrorOperationNotSupported{
Expand Down

0 comments on commit fbc28ba

Please sign in to comment.