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

v0.54.0 results dir issue #1688

Closed
johnSchnake opened this issue May 3, 2022 · 2 comments
Closed

v0.54.0 results dir issue #1688

johnSchnake opened this issue May 3, 2022 · 2 comments
Labels
area/documentation Related to project documentation kind/bug Behavior isn't as expected or intended

Comments

@johnSchnake
Copy link
Contributor

Helping someone with a custom plugin and they were pinned to v0.54.0 which has known results_dir problems. It took a while to debug and I'd just like to make sure we document the known issue so its faster to debug next time.

SONOBUOY_RESULTS_DIR did not match RESULTS_DIR and was causing problems with finding the results.

@johnSchnake johnSchnake added kind/bug Behavior isn't as expected or intended area/documentation Related to project documentation labels May 3, 2022
@johnSchnake
Copy link
Contributor Author

After looking at the PRs and knowing what the issue is I'm not certain its worth much more effort than throwing this into a known issues listing at most. The fix is just to bump the Sonobuoy version; I think very few people would be stuck on this explicit version and forced to work around it. Considering that release is now approaching a year old, I dont know that it makes too much sense to even put it in the known issues though. Just closing for now.

If they do, they can find this suggestion:

  • specify the RESULTS_DIR env var manually on your plugin
  • continue to manually specify the results volume mount in your plugin
  • make sure the path for the results volume mount matches the resultsDir set in the sonobuoy config (e.g. sonobuoy gen config data, except the one fed into the sonobuoy run/gen you're trying to use)

@johnSchnake
Copy link
Contributor Author

I've ran into a situation which is close to this. An upgrade is depending on using the older CLI and moving to the new image. Currently this doesn't work.

If we update the resultsDir of the config things would be fine.
If the volumeMounts all matched the results dir, things would be fine.

It may make sense to try and solve this problem on the server-side in a way that doesn't put any onus on users to pass a different config value.

johnSchnake added a commit that referenced this issue May 14, 2022
Issue #1688 would cause issues even if the old CLI was targeting
a new server. I've had multiple people this week report issues with
this so I wanted to resolve it and push a new release.

Added a new test to ensure we dont regress on this.

Signed-off-by: John Schnake <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Related to project documentation kind/bug Behavior isn't as expected or intended
Projects
None yet
Development

No branches or pull requests

1 participant