Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug 1970129: Add env variable OVS_SYS_LOG_LEVEL for ovn nodes to setup ovs syslog level #1142

Merged
merged 2 commits into from Jul 15, 2021
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 1 addition & 0 deletions README.md
Expand Up @@ -229,6 +229,7 @@ data:
ip-10-0-135-96.us-east-2.compute.internal: |
OVN_KUBE_LOG_LEVEL=5
OVN_LOG_LEVEL=dbg
OVS_LOG_LEVEL=dbg
Copy link
Contributor

Choose a reason for hiding this comment

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

should we call this OVS_SYS_LOG_LEVEL? That makes it more explicit, since you could set console and file logging too.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, explicit is always better

# to adjust master log levels, use _master
_master: |
OVN_KUBE_LOG_LEVEL=5
Expand Down
4 changes: 4 additions & 0 deletions bindata/network/ovn-kubernetes/ovnkube-node.yaml
Expand Up @@ -252,6 +252,8 @@ spec:
if [[ -n "${IPFIX_COLLECTORS}" ]] ; then
export_network_flows_flags="$export_network_flows_flags --ipfix-targets ${IPFIX_COLLECTORS}"
fi
ovs-appctl vlog/set "syslog:${OVS_LOG_LEVEL}"

exec /usr/bin/ovnkube --init-node "${K8S_NODE}" \
--nb-address "{{.OVN_NB_DB_LIST}}" \
--sb-address "{{.OVN_SB_DB_LIST}}" \
Expand Down Expand Up @@ -280,6 +282,8 @@ spec:
value: "{{.OVN_CONTROLLER_INACTIVITY_PROBE}}"
- name: OVN_KUBE_LOG_LEVEL
value: "4"
- name: OVS_LOG_LEVEL
value: info
{{ if .NetFlowCollectors }}
- name: NETFLOW_COLLECTORS
value: "{{.NetFlowCollectors}}"
Expand Down