Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

rename rpm/deb artifact name #371

Merged
merged 1 commit into from
Jan 26, 2021

Conversation

ylwu-amzn
Copy link
Contributor

Issue #, if available:

Description of changes:

Rename artifact name for rpm and deb

Test

./gradlew buildPackages

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov
Copy link

codecov bot commented Jan 26, 2021

Codecov Report

Merging #371 (c4097f5) into master (8230412) will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #371      +/-   ##
============================================
+ Coverage     78.88%   78.93%   +0.04%     
  Complexity     2653     2653              
============================================
  Files           247      247              
  Lines         11719    11719              
  Branches       1009     1009              
============================================
+ Hits           9245     9250       +5     
+ Misses         1996     1992       -4     
+ Partials        478      477       -1     
Flag Coverage Δ Complexity Δ
cli 79.27% <ø> (ø) 0.00 <ø> (ø)
plugin 78.90% <ø> (+0.04%) 0.00 <ø> (ø)

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

Impacted Files Coverage Δ Complexity Δ
...pendistroforelasticsearch/ad/cluster/HashRing.java 78.94% <0.00%> (-5.27%) 10.00% <0.00%> (-1.00%)
...troforelasticsearch/ad/task/ADBatchTaskRunner.java 87.82% <0.00%> (-0.87%) 63.00% <0.00%> (-1.00%)
...asticsearch/ad/cluster/ADClusterEventListener.java 94.00% <0.00%> (+2.00%) 15.00% <0.00%> (+1.00%)
...stroforelasticsearch/ad/AnomalyDetectorRunner.java 46.25% <0.00%> (+6.25%) 7.00% <0.00%> (+1.00%)
...port/SearchAnomalyDetectorInfoTransportAction.java 63.63% <0.00%> (+9.09%) 4.00% <0.00%> (ø%)

Copy link
Contributor

@yizheliu-amazon yizheliu-amazon left a comment

Choose a reason for hiding this comment

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

just curious: is last step delete needed? I guess the original file is renamed so that it won't exist any more, not sure what's the behavior in gradle rename.

@ylwu-amzn ylwu-amzn merged commit f40e9b8 into opendistro-for-elasticsearch:master Jan 26, 2021
@ohltyler ohltyler added the infra label Feb 2, 2021
ylwu-amzn added a commit that referenced this pull request Feb 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants