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

Added windows zip file generation #3817

Merged
merged 5 commits into from
Jul 31, 2022

Conversation

adhithyasrinivasan
Copy link
Contributor

Which problem is this PR solving?

Short description of the changes

  • Added a function to generate zip file instead of tar file for windows machines.

Signed-off-by: adhithyasrinivasan [adhithyasrinivasan@gmail.com)

@adhithyasrinivasan adhithyasrinivasan requested a review from a team as a code owner July 17, 2022 17:36
@adhithyasrinivasan adhithyasrinivasan changed the title WIP - Added windows zip file generation Added windows zip file generation Jul 17, 2022
@codecov
Copy link

codecov bot commented Jul 17, 2022

Codecov Report

Merging #3817 (4ebaa26) into main (32742d1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3817   +/-   ##
=======================================
  Coverage   97.59%   97.59%           
=======================================
  Files         291      291           
  Lines       16939    16939           
=======================================
  Hits        16532    16532           
  Misses        321      321           
  Partials       86       86           

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 32742d1...4ebaa26. Read the comment docs.

dependabot bot and others added 3 commits July 31, 2022 13:36
…g#3814)

Bumps [github.com/prometheus/common](https://github.com/prometheus/common) from 0.36.0 to 0.37.0.
- [Release notes](https://github.com/prometheus/common/releases)
- [Commits](prometheus/common@v0.36.0...v0.37.0)

---
updated-dependencies:
- dependency-name: github.com/prometheus/common
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Adhithya Srinivasan <adhithya.srinivasan@walmart.com>
Signed-off-by: Adhithya Srinivasan <adhithya.srinivasan@walmart.com>
Signed-off-by: Adhithya Srinivasan <adhithya.srinivasan@walmart.com>
Copy link
Member

@yurishkuro yurishkuro left a comment

Choose a reason for hiding this comment

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

Thanks

@yurishkuro yurishkuro enabled auto-merge (squash) July 31, 2022 16:13
@yurishkuro yurishkuro merged commit faf4bc9 into jaegertracing:main Jul 31, 2022
Ashmita152 added a commit to Ashmita152/jaeger that referenced this pull request Aug 5, 2022
Earlier we used to have tar.gz for windows release. jaegertracing#3817 changed the windows release from tar.gz to zip format. This PR also restores that i.e. now we have both tar.gz and zip for windows release.
Ashmita152 added a commit to Ashmita152/jaeger that referenced this pull request Aug 5, 2022
Earlier we used to have tar.gz for windows release. jaegertracing#3817 changed the windows release from tar.gz to zip format. This PR also restores that i.e. now we have both tar.gz and zip for windows release.

Signed-off-by: Ashmita152 <ashmita.bohara@logz.io>
yurishkuro pushed a commit that referenced this pull request Aug 5, 2022
Earlier we used to have tar.gz for windows release. #3817 changed the windows release from tar.gz to zip format. This PR also restores that i.e. now we have both tar.gz and zip for windows release.

Signed-off-by: Ashmita152 <ashmita.bohara@logz.io>
albertteoh pushed a commit to albertteoh/jaeger that referenced this pull request Aug 8, 2022
albertteoh pushed a commit to albertteoh/jaeger that referenced this pull request Aug 8, 2022
Earlier we used to have tar.gz for windows release. jaegertracing#3817 changed the windows release from tar.gz to zip format. This PR also restores that i.e. now we have both tar.gz and zip for windows release.

Signed-off-by: Ashmita152 <ashmita.bohara@logz.io>
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.

2 participants