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(cleanup): small clean ups in CI #299

Merged
merged 4 commits into from
Nov 20, 2023
Merged

Conversation

mwangggg
Copy link
Member

@mwangggg mwangggg commented Nov 18, 2023

related to: #283

  • adding permissions for release-drafter.yml
  • adding permissions for status checks + comment
  • checking out correct ref in build-core in ci.yml -> fixes build(deps): downgrade spotbugs #296 (comment)
  • consistency with YAML file extensions

Copy link

ARCH IMAGE
amd64 ghcr.io/cryostatio/cryostat-core:pr-299-a05cccbafdccc0c18e755ca149de535e810d591e-linux-amd64
arm64 ghcr.io/cryostatio/cryostat-core:pr-299-a05cccbafdccc0c18e755ca149de535e810d591e-linux-arm64

To run smoketest:

# amd64
CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-core:pr-299-a05cccbafdccc0c18e755ca149de535e810d591e-linux-amd64 sh smoketest.sh

# or arm64
CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-core:pr-299-a05cccbafdccc0c18e755ca149de535e810d591e-linux-arm64 sh smoketest.sh

@andrewazores andrewazores changed the title chore(ci): small clean ups in CI ci(cleanup): small clean ups in CI Nov 20, 2023
Copy link

ARCH IMAGE
amd64 ghcr.io/cryostatio/cryostat-core:pr-299-a05cccbafdccc0c18e755ca149de535e810d591e-linux-amd64
arm64 ghcr.io/cryostatio/cryostat-core:pr-299-a05cccbafdccc0c18e755ca149de535e810d591e-linux-arm64

To run smoketest:

# amd64
CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-core:pr-299-a05cccbafdccc0c18e755ca149de535e810d591e-linux-amd64 sh smoketest.sh

# or arm64
CRYOSTAT_IMAGE=ghcr.io/cryostatio/cryostat-core:pr-299-a05cccbafdccc0c18e755ca149de535e810d591e-linux-arm64 sh smoketest.sh

@andrewazores andrewazores merged commit a9cde35 into cryostatio:main Nov 20, 2023
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants