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

docs: mention fix for rekor and apiserver host resolution failures #2380

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dheerajodha
Copy link
Member

While running the acceptance tests, you might encounter an issue with host resolution for the calls to the apiserver and rekor. The fix is to include an entry for these 2 within your host file.

This PR adds this info to our troubleshooting guide.

Copy link
Member

@lcarva lcarva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@lcarva lcarva enabled auto-merge March 13, 2025 13:10
Copy link

codecov bot commented Mar 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.87%. Comparing base (ae653ef) to head (2f190b0).
Report is 4 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2380      +/-   ##
==========================================
- Coverage   72.92%   67.87%   -5.06%     
==========================================
  Files          89      101      +12     
  Lines        7783     8380     +597     
==========================================
+ Hits         5676     5688      +12     
- Misses       2107     2692     +585     
Flag Coverage Δ
generative 67.87% <ø> (-5.06%) ⬇️
integration 67.87% <ø> (-5.06%) ⬇️
unit 67.87% <ø> (-5.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 13 files with indirect coverage changes

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lcarva
Copy link
Member

lcarva commented Mar 13, 2025

/ok-to-test

@lcarva
Copy link
Member

lcarva commented Mar 13, 2025

/lgtm

@dheerajodha
Copy link
Member Author

Build PLR failed while running the build-tekton-bundle task, the error:

Error: could not push image to registry as "quay.io/redhat-user-workloads/rhtap-contract-tenant/ec-main-ci/cli-main-ci:on-pr-d8c67f5b448c20438a8ef339817f4eed1caa8fff.bundle": GET https://quay.io:443/v2/redhat-user-workloads/rhtap-contract-tenant/ec-main-ci/cli-main-ci/blobs/uploads/: unexpected status code 405 Method Not Allowed: <!doctype html>
<html lang=en>
<title>405 Method Not Allowed</title>
<h1>Method Not Allowed</h1>
<p>The method is not allowed for the requested URL.</p>

maybe a random quay.io issue? retesting

/retest

@simonbaird
Copy link
Member

/retest

README.md Outdated
@@ -40,7 +40,7 @@ The `--debug` parameter enables debug logging. Setting `EC_DEBUG` environment
variable can be set to prevent deletion of temporary `ec-work-*` directories so
that the attestations, policy and data files can be examined.

When running acceptance tests you may experience issues with starting enough Docker containers to successfullyl complete testing. These issues may appear as repeated failures, such as seen below, and a failed acceptance test run:
1. When running acceptance tests you may experience issues with starting enough Docker containers to successfullyl complete testing. These issues may appear as repeated failures, such as seen below, and a failed acceptance test run:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
1. When running acceptance tests you may experience issues with starting enough Docker containers to successfullyl complete testing. These issues may appear as repeated failures, such as seen below, and a failed acceptance test run:
1. When running acceptance tests you may experience issues with starting enough Docker containers to successfully complete testing. These issues may appear as repeated failures, such as seen below, and a failed acceptance test run:

Copy link
Member

@simonbaird simonbaird left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks!

@dheerajodha dheerajodha force-pushed the mention-hostfile-update-for-acceptance branch from d8c67f5 to 2f190b0 Compare March 13, 2025 15:17
@lcarva lcarva disabled auto-merge March 13, 2025 16:13
@lcarva
Copy link
Member

lcarva commented Mar 13, 2025

Failure is due to an out of date Task, rpms-signature-scan. Merging this.

@joejstuart
Copy link
Member

if you rebase with main, it should pass now.

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.

4 participants