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

fixing issues with pipeline_dependencies and status #124

Conversation

acornett21
Copy link
Contributor

With the introduction of sub-reconcilers for the dependencies and status, it seem we never fully tested this, and when a CR is applied the application error'd out.

Changes

  • Remove hardcoded path during deletion.
  • Using tekton.Pipeline and tekton.Task instead of client.Object since we cannot marshal to this object, and this follow the patter elsewhere in the application.
  • Creating arrays with a length of zero and a capacity of ten, otherwise length check of array > 0 is always true and reconcile never completes successfully, even if there are no errors.
  • Joining the filename with the entity.Name() since this value is just the filename, and not the full path plus filename.

Testing

  • Locally against a cluster
  • Created a new controller, bundle and catalog images and deployed to a cluster

The UI now shows status properly and I see no exceptions thrown. However, due to how these sub-reconcilers currently work, there are alot of exceptions that are trapped and never bubble up to a log statement. Which is something we should look at fixing later.

image

Signed-off-by: Adam D. Cornett adc@redhat.com

… reconciles a CR and updates Conditions properly

Signed-off-by: Adam D. Cornett <adc@redhat.com>
@acornett21 acornett21 requested review from jomkz and removed request for samira-barouti April 25, 2022 23:03
Copy link
Contributor

@bcrochet bcrochet left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 26, 2022
@bcrochet bcrochet merged commit fce723b into redhat-openshift-ecosystem:main Apr 26, 2022
@acornett21 acornett21 deleted the fix_reconcile_and_conditions branch January 2, 2024 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants