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

attempt to fix deployment #863

Merged
merged 15 commits into from
Dec 7, 2023
Merged

attempt to fix deployment #863

merged 15 commits into from
Dec 7, 2023

Conversation

@@ -23,11 +23,6 @@ jobs:
fetch-depth: 0
submodules: true

- name: Get and verify tag value
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not needed here

@@ -122,11 +120,6 @@ jobs:
- name: Set up requirements
run: scripts/setup_runner_requirements.sh

- name: Get and verify tag value
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not needed here

submodules: true
fetch-depth: 0

- name: Verify release tag matches gProfiler version
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Kept here as verification before deploying

@@ -163,9 +165,6 @@ jobs:
name: gprofiler_aarch64
path: output/

- name: Get tag value
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wasn't really needed

docker image load < output/gprofiler_x86_64.img
docker tag gprofiler_x86_64 $X86_64_IMAGE
docker push $X86_64_IMAGE

# create manifests for the tag + for 'latest'
docker manifest create "$BASE_IMAGE" "$X86_64_IMAGE" "$AARCH64_IMAGE"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

docker buildx imagetools create is the modern, preferable way

@Jongy Jongy merged commit b49913f into master Dec 7, 2023
28 checks passed
@Jongy Jongy deleted the ci-tests branch December 7, 2023 09:56
@Jongy Jongy added the ci label Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant