Skip to content

Conversation

@squakez
Copy link
Contributor

@squakez squakez commented Mar 17, 2025

Refactoring also to make this CR static as it makes no sense to run any reconciliation over the profile.

Closes #6091

Release Note

NONE

@squakez
Copy link
Contributor Author

squakez commented Mar 17, 2025

FYI @eshepelyuk - this should bring back the IntegrationProfile custom resource. However, we're making this resource static, as it does not make sense to make any reconciliation just to copy values from .spec to .status.

@github-actions
Copy link
Contributor

✔️ Unit test coverage report - coverage increased from 46.6% to 46.9% (+0.3%)

@github-actions
Copy link
Contributor

✔️ Unit test coverage report - coverage increased from 46.6% to 46.9% (+0.3%)

Refactoring also to make this CR static as it makes no sense to run any reconciliation over the profile.

Closes apache#6091
@github-actions
Copy link
Contributor

✔️ Unit test coverage report - coverage increased from 46.6% to 47% (+0.4%)

@squakez squakez marked this pull request as ready for review March 20, 2025 17:41
squakez added a commit to squakez/camel-k that referenced this pull request Mar 20, 2025
@squakez squakez merged commit 67e017a into apache:main Mar 21, 2025
10 checks passed
@squakez squakez deleted the chore/6091 branch March 21, 2025 08:30
squakez added a commit that referenced this pull request Mar 21, 2025
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.

Reconsider deprecation of IntegrationProfile

1 participant