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

How to understand the output path of auto config file? #1406

Open
Martin-Jia opened this issue Mar 19, 2025 · 1 comment
Open

How to understand the output path of auto config file? #1406

Martin-Jia opened this issue Mar 19, 2025 · 1 comment

Comments

@Martin-Jia
Copy link

Martin-Jia commented Mar 19, 2025

Hi, just wanted to understand this part of code.

func (m *Manager) PersistMultusConfig(config string) (string, error) {

According to the comment, the output path should be <multus auto config dir>/00-multus.conf, but the target file path is actually using <cni config dir>/00-multus.conf which comes from a different configuration parameter that can be different from the auto config dir.
multusConfigFilePath: filepath.Join(config.CniConfigDir, multusConfigFileName),

Then how to understand this discrepancy? It's actually writing to the cni config dir or the cni config dir is actually overridden by the auto config dir somewhere?

@dougbtv
Copy link
Member

dougbtv commented Apr 10, 2025

Good question -- it's worth digging into to see if one parameter wins over the other, which maybe you're showing there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants