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

Fix agentctl report command failing in StoneWork #1857

Merged
merged 1 commit into from
Jul 18, 2022

Conversation

samuel-dudik
Copy link
Collaborator

agentctl report command fails when run against StoneWork with this error message:

Retrieving agent kvscheduler NB configuration... Error... (this error will be part of the report zip file, see _failed-reports.txt):
dumping kvscheduler data failed due to:
Failed to get data for NB view and key prefix config/vpp/dns/v1/dnscache due to: Error response from daemon: [500] {
  "Error": "unknown key prefix"
}
, Failed to get data for NB view and key prefix config/vpp/ipfix/v2/flowprobe-feature/ due to: Error response from daemon: [500] {
  "Error": "unknown key prefix"
}
, Failed to get data for NB view and key prefix config/vpp/ipfix/v2/flowprobe-params due to: Error response from daemon: [500] {
  "Error": "unknown key prefix"
}
, Failed to get data for NB view and key prefix config/vpp/ipfix/v2/ipfix due to: Error response from daemon: [500] {
  "Error": "unknown key prefix"
}
, Failed to get data for NB view and key prefix config/vpp/wg/v1/peer/ due to: Error response from daemon: [500] {
  "Error": "unknown key prefix"
} 

agentctl is getting prefixes from all registered models and then searches for their corresponding descriptors. However, some models are registered even though they don't have registered their descriptor. This is normal and expected. That's why this PR tries to "suppress" the error messages. They still appear, but don't result in failure of the report command. The PR also changes some of the messages and the server status code.

Signed-off-by: Samuel Dudík samuel.dudik@pantheon.tech

Signed-off-by: Samuel Dudík <samuel.dudik@pantheon.tech>
@ondrej-fabry ondrej-fabry merged commit 202ac0c into ligato:master Jul 18, 2022
ondrej-fabry pushed a commit that referenced this pull request Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants