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

ci: show latest created packages in S3 Repository #1977

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rubenruizdegauna
Copy link
Contributor

@rubenruizdegauna rubenruizdegauna commented Dec 24, 2024

GH Actions to:

  • list the latest N deb|rpm packages in S3 Repository
  • list N files created before and after date-time in S3 Repository

Example jobs:
https://github.com/newrelic/infrastructure-agent/actions/runs/12482710904/job/34837305390
https://github.com/newrelic/infrastructure-agent/actions/runs/12482710908/job/34837305292

@rubenruizdegauna rubenruizdegauna requested a review from a team as a code owner December 24, 2024 14:07
@rubenruizdegauna rubenruizdegauna force-pushed the NR-229418_ci_show_latest_pkgs branch from d25760a to c1ecf7f Compare December 24, 2024 14:13
@rubenruizdegauna rubenruizdegauna changed the title Nr 229418 ci show latest pkgs ci: show latest created packages in S3 Repository Dec 24, 2024
@rubenruizdegauna rubenruizdegauna force-pushed the NR-229418_ci_show_latest_pkgs branch from c1ecf7f to ba31b37 Compare December 24, 2024 14:15
@coveralls
Copy link

Pull Request Test Coverage Report for Build 12482806931

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 9 unchanged lines in 4 files lost coverage.
  • Overall coverage decreased (-0.02%) to 57.306%

Files with Coverage Reduction New Missed Lines %
internal/agent/event_sender_vortex.go 1 73.78%
pkg/metrics/sampler/sampler_routine.go 2 95.35%
internal/integrations/v4/testhelp/testemit/testemit.go 2 90.63%
pkg/integrations/v4/logs/cfg_watcher.go 4 67.5%
Totals Coverage Status
Change from base Build 12416780002: -0.02%
Covered Lines: 15354
Relevant Lines: 26793

💛 - Coveralls

Copy link
Contributor

@rajrohanyadav rajrohanyadav left a comment

Choose a reason for hiding this comment

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

LGTM

workflow_dispatch:
inputs:
count:
description: 'Amount of files to show'
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Would it be clearer to use Number of files to show?

default: '500'
required: true
date_time:
description: 'Date and time to show files added before and after (YYYY-MM-DD HH:MM:SS)'
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Would it be required to know that it is in UTC timezone? Maybe the description can mention that.


- name: List files created before and after datetime in S3
run: |
TZ="UTC" aws s3 ls s3://nr-downloads-main/infrastructure_agent/ --recursive | sort | grep --color -m 1 -C {{ github.event.inputs.count }} "^${{ github.event.inputs.date_time }}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Q: Is it only required for Prod?

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.

3 participants