You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
I am running popeye with docker and I noticed that the same command using image v0.11.3 and v0.20.0 put the report file in different locations:
/tmp/popeye/report_file.html for v0.11.3
and
/tmp/popeye/<cluster_name or user>/<cluster_name or user>/my_report.txt for v0.20.0 and after (including v.0.21.3).
Setting a POPEYE_REPORT_DIR only replaces /tmp/popeye with the path from env variable.
To Reproduce
Steps to reproduce the behavior:
Run docker command with both versions:
docker run --rm -v $(pwd)/config:/home/nonroot/.kube/config -v /tmp:/tmp \
derailed/popeye:v0.21.3 -n bar --save --output-file my_report.txt
--result--
/tmp/popeye/<cluster_name or user>/<cluster_name or user>/my_report.txt
docker run --rm -v $(pwd)/config:/home/nonroot/.kube/config -v /tmp:/tmp \
derailed/popeye:v0.11.3 -n bar --save --output-file my_report.txt
--result--
/tmp/popeye/my_report.txt
Expected behavior
Is there a way to have a fixed path that I can set with POPEYE_REPORT_DIR and nothing extra to be added?
The text was updated successfully, but these errors were encountered:
You are correct. Since users may target different clusters and hence we want to dump the reports in cluster specific location.
That said we can relax this a bit and assume if the user overrides the default report location then this should be the final report resting place with no alterations.
I'll update the code to change the behavior ie if you set POPEYE_REPORT_DIR env var then the reports will land there.
Watch for this in the next drop.
Describe the bug
I am running popeye with docker and I noticed that the same command using image v0.11.3 and v0.20.0 put the report file in different locations:
/tmp/popeye/report_file.html for v0.11.3
and
/tmp/popeye/<cluster_name or user>/<cluster_name or user>/my_report.txt for v0.20.0 and after (including v.0.21.3).
Setting a POPEYE_REPORT_DIR only replaces /tmp/popeye with the path from env variable.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Is there a way to have a fixed path that I can set with POPEYE_REPORT_DIR and nothing extra to be added?
The text was updated successfully, but these errors were encountered: