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

chore: Deprecate maven.jkube.io annotation prefix from enrichers #1245

Conversation

rohanKanojia
Copy link
Member

@rohanKanojia rohanKanojia commented Jan 28, 2022

Description

maven.jkube.io seems to be a side effect of search-replace during
migration from FMP to JKube. Deprecate all annotation properties
starting with this prefix.

Introduce the use of jkube.eclipse.org prefix instead.

Signed-off-by: Rohan Kumar rohaan@redhat.com

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change
  • Chore (non-breaking change which doesn't affect codebase;
    test, version modification, documentation, etc.)

Checklist

@codecov
Copy link

codecov bot commented Jan 28, 2022

Codecov Report

Merging #1245 (beabba8) into master (be8514f) will increase coverage by 0.01%.
The diff coverage is 63.15%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1245      +/-   ##
============================================
+ Coverage     50.17%   50.18%   +0.01%     
- Complexity     3699     3705       +6     
============================================
  Files           457      457              
  Lines         20605    20618      +13     
  Branches       2805     2807       +2     
============================================
+ Hits          10338    10347       +9     
- Misses         9182     9184       +2     
- Partials       1085     1087       +2     
Impacted Files Coverage Δ
...jkube/enricher/generic/FileDataSecretEnricher.java 81.25% <57.14%> (-3.94%) ⬇️
...eclipse/jkube/enricher/generic/SecretEnricher.java 34.42% <57.14%> (+2.28%) ⬆️
...ipse/jkube/enricher/generic/ConfigMapEnricher.java 75.00% <75.00%> (+0.60%) ⬆️
...enricher/generic/DockerRegistrySecretEnricher.java 87.50% <100.00%> (+0.83%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be8514f...beabba8. Read the comment docs.

@rohanKanojia rohanKanojia force-pushed the pr/deprecate-maven-jkube-io-annotation branch from 0780a3b to 589a05e Compare January 28, 2022 16:58
@manusa manusa added this to the 1.6.0 milestone Jan 28, 2022
@manusa manusa added the bug Something isn't working label Jan 28, 2022
`maven.jkube.io` seems to be a side effect of search-replace during
migration from FMP to JKube. Deprecate all annotation properties
starting with this prefix.

Introduce use of `jkube.eclipse.org` prefix instead.

Signed-off-by: Rohan Kumar <rohaan@redhat.com>
@rohanKanojia rohanKanojia force-pushed the pr/deprecate-maven-jkube-io-annotation branch from 8c9765e to beabba8 Compare January 31, 2022 06:23
@manusa manusa requested a review from sunix January 31, 2022 06:25
@sonarcloud
Copy link

sonarcloud bot commented Jan 31, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell C 12 Code Smells

80.0% 80.0% Coverage
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants