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.

Sign up for GitHub

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

Jump to bottom

Deployment pipeline for enclave attestation service #8690

Merged
merged 8 commits into from May 3, 2024
Merged

Deployment pipeline for enclave attestation service #8690

merged 8 commits into from May 3, 2024

Conversation

itstauq
Copy link
Contributor

@itstauq itstauq commented Apr 10, 2024 •
edited

Description

Add steps in cd-syft.yml and cd-syft-dev.yml for building and pushing the enclave attestation service docker images to dockerhub

Closes OpenMined/Heartbeat#1176

Affected Dependencies

List any dependencies that are required for this change.

How has this been tested?

  • Describe the tests that you ran to verify your changes.
  • Provide instructions so we can reproduce.
  • List any relevant details for your test configuration.

Checklist

@itstauq itstauq changed the base branch from dev to tauquir/attestation-helmcharts April 10, 2024 00:24
@itstauq itstauq changed the title [WIP] Deployment pipeline for enclave attestation service Deployment pipeline for enclave attestation service Apr 23, 2024
@itstauq itstauq self-assigned this Apr 23, 2024
@itstauq itstauq requested a review from snwagh April 24, 2024 12:43
rasswanth-s
rasswanth-s approved these changes Apr 29, 2024
View reviewed changes
Copy link
Collaborator

@rasswanth-s rasswanth-s left a comment

Choose a reason for hiding this comment

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

Well Done @itstauq

Base automatically changed from tauquir/attestation-helmcharts to dev April 29, 2024 18:08
snwagh
snwagh approved these changes May 1, 2024
View reviewed changes
Copy link
Contributor

@snwagh snwagh left a comment

Choose a reason for hiding this comment

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

Haven't tested but don't want that to be a blocker.

snwagh
snwagh approved these changes May 2, 2024
View reviewed changes
Copy link
Contributor

@snwagh snwagh left a comment

Choose a reason for hiding this comment

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

LGTM

@snwagh snwagh enabled auto-merge May 2, 2024 12:14
@snwagh snwagh merged commit ee441d3 into dev May 3, 2024
35 checks passed
@snwagh snwagh deleted the tauquir/attestation-cd-pipeline branch May 3, 2024 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@snwagh snwagh snwagh approved these changes

@rasswanth-s rasswanth-s rasswanth-s approved these changes

@yashgorana yashgorana Awaiting requested review from yashgorana

Assignees

@itstauq itstauq

Labels
Purple Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@itstauq @snwagh @rasswanth-s