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

Refactor cicd #4043

Merged
merged 31 commits into from
Oct 31, 2022
Merged

Refactor cicd #4043

merged 31 commits into from
Oct 31, 2022

Conversation

bogdan-1337
Copy link
Collaborator

Description

  • Adding github workflow for build and deploy CI/CD process. Moved Dapp dockerfile from colony-docker-images repo into this repo to centralise the process.

New stuff ✨

Github workflow
Dockerfile

Changes 🏗

  • Changes go here

Deletions ⚰️

  • Deletions go here

TODO

  • Add TODOs

(Resolves | Contributes to) #31415

@bogdan-1337
Add github workflow and dockerfile
9e24b23
@bogdan-1337
Add ls to dockerfile to troubleshoot npm error
20b5c30
@bogdan-1337
Remove colony images repo checkout
17903cb
@bogdan-1337
Trying docker build github plugin
d15be32
@bogdan-1337
Update dockertag
92e04bf
@bogdan-1337
Update docker build context
b9efe61
@bogdan-1337
Fix github_workspace variable syntax
d088be1
@bogdan-1337
Fix github_workspace variable syntax typo
cbc4f08
@bogdan-1337
Update github_workspace variable name
532de50
@bogdan-1337
Add extra step to troubleshoot workspace path
ed8468c
@bogdan-1337
Add extra troubleshooting commands
8d80383
@bogdan-1337
Update workflow and ls command
939a141
@bogdan-1337
Update workflow to include checkout
aa1cab9
@bogdan-1337
Update dockerfile to copy over code
589c61c
@bogdan-1337
ls removed from dockerfile
7a2b893
@bogdan-1337
Add build script and modify workflow to use script
9c11291
@bogdan-1337
Update script path
40b5745
@bogdan-1337
Update workflow trigger and script
fbc57aa
@bogdan-1337
Take out circleci pipeline
3ae2a1c
@bogdan-1337
Update script and workflow env var
8c30d3f
@bogdan-1337
Add deploy step to workflow
18fcc09
@bogdan-1337
Update deployment name in deploy step
4baf0d7
@bogdan-1337
Update deployment repo name
f5ae099
@bogdan-1337
Update set image command with correct container name
33b470a
@bogdan-1337
Update workflow with discord notification
00ab9ff
@bogdan-1337
Add conditional for image tag
3e28b47
@bogdan-1337
Update account id for ecr repo and remove region as secret
9250491
@bogdan-1337
Remove build script
423235e
@bogdan-1337
Uncomment ci pipeline
9668391
@CLAassistant
Copy link

CLAassistant commented Oct 28, 2022 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@bogdan-1337 bogdan-1337 requested review from rdig and area October 28, 2022 13:56
rdig
rdig approved these changes Oct 31, 2022
View reviewed changes
Copy link
Member

@rdig rdig left a comment

Choose a reason for hiding this comment

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

Seems fine from my (limited) perspective

Since you're already an admin, feel free to bypass the two reviews requirement and merge it directly (after you rebase it on the lastest changes, obviously)

.github/workflows/buildDeploy.yml Outdated Show resolved Hide resolved
@bogdan-1337 bogdan-1337 force-pushed the refactor-cicd branch 2 times, most recently from 82bdbb7 to 8f2cded Compare October 31, 2022 22:01
@bogdan-1337 bogdan-1337 merged commit 5f7ea12 into master Oct 31, 2022
@bogdan-1337 bogdan-1337 deleted the refactor-cicd branch October 31, 2022 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rdig rdig rdig approved these changes

@area area Awaiting requested review from area

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bogdan-1337 @CLAassistant @rdig