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

rddepman: bump trivy from 0.50.0 to 0.50.1 #277

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

mook-as
Copy link
Owner

@mook-as mook-as commented Mar 29, 2024

No description provided.

mook-as and others added 21 commits March 25, 2024 11:03
- On Windows, determine the window to capture using the window title,
  rather than the active window.  Sometimes we can end up with the wrong
  window being active, even with no human intervention (e.g. in CI).
- On Linux, manually walk the window tree to locate the window that is a
  child of the root window.  This ensures that we will capture any window
  frames correctly.  Also, switch to using GraphicsMagick instead of
  gnome-screenshot, because (at least in CI) we will not be in a full Gnome
  environment.
- Spawn commands as separate arguments rather than a single command line,
  to better handle things like spaces in file paths.
- Add some logging for the script, so that we can troubleshoot any errors
  in taking screenshots.
- Start Rancher Desktop with the test helper, rather than trying to run the
  command manually.  Note that we're not using the mock backend because we
  need to install extensions.

Signed-off-by: Mark Yen <mark.yen@suse.com>
Creating a Kubernetes client requires a valid kubeconfig (so that we can
figure out the parameters for it); this means that we need to defer
actually doing so until the point where the configuration is ready.  The
screenshot backend (used to fake the list of services) was incorrectly
doing so at the start before actually starting k3s; this lead to things
failing on fresh machines.

Signed-off-by: Mark Yen <mark.yen@suse.com>
If we capture stderr (and ignore its return value), we can see it in the
error that goes thrown when things go wrong.  This helps with debugging.

Signed-off-by: Mark Yen <mark.yen@suse.com>
This adds a CI job to take screenshots.  This avoid issues with having to
track down people around release across platforms, and what changes they
may have made to their local system.

Signed-off-by: Mark Yen <mark.yen@suse.com>
On Windows, for some reason `errors.Is(..., ENOENT)` doesn't work correctly
and fails to catch the error.

Signed-off-by: Mark Yen <mark.yen@suse.com>
Signed-off-by: Mark Yen <mark.yen@suse.com>
On Windows, it appears that running `rdctl` ended up draining standard
input; however, we needed that standard input to iterate over all snapshots
so we can delete them.  Work around the issue by explicitly redirecting
standard input from `/dev/null` so that the original standard input is left
alone.

Additionally, when deleting all snapshots, check that the snapshots have
actually been cleared afterwards.  This catches the above problem where we
do not delete all snapshots.

Signed-off-by: Mark Yen <mark.yen@suse.com>
Signed-off-by: Rancher Desktop Dependency Manager <donotuse@rancherdesktop.io>
@mook-as mook-as force-pushed the main branch 9 times, most recently from 3459556 to 01ece1d Compare April 5, 2024 00:35
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.

1 participant