Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improve the error message when source resolver is not resolving #1152

Merged
merged 8 commits into from
Mar 10, 2023

Conversation

yaelharel
Copy link
Contributor

No description provided.

Signed-off-by: Yael Harel <yharel@vmware.com>
@codecov-commenter
Copy link

codecov-commenter commented Feb 22, 2023

Codecov Report

Merging #1152 (595182f) into main (79126fe) will increase coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head 595182f differs from pull request most recent head e89de9a. Consider uploading reports for the commit e89de9a to get more accurate results

@@            Coverage Diff             @@
##             main    #1152      +/-   ##
==========================================
+ Coverage   70.25%   70.27%   +0.01%     
==========================================
  Files         124      124              
  Lines        6219     6223       +4     
==========================================
+ Hits         4369     4373       +4     
  Misses       1445     1445              
  Partials      405      405              
Impacted Files Coverage Δ
pkg/reconciler/image/reconcile_build.go 87.82% <100.00%> (+0.43%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Yael Harel added 2 commits March 7, 2023 15:09
Signed-off-by: Yael Harel <yharel@vmware.com>
Signed-off-by: Yael Harel <yharel@vmware.com>
Type: buildapi.ConditionSourceResolverReady,
Status: corev1.ConditionFalse,
Reason: buildapi.SourceResolverNotReady,
Message: "SourceResolver image-name-source is not ready: something is failing",
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@ncarlson ncarlson left a comment

Choose a reason for hiding this comment

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

Looks good!

@yaelharel yaelharel requested a review from ncarlson March 10, 2023 16:57
@yaelharel yaelharel merged commit 7857820 into main Mar 10, 2023
@yaelharel yaelharel deleted the source-resolver-status branch March 10, 2023 21:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants