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

Infrastructure as code #111

Merged
merged 13 commits into from
Jun 21, 2021
Merged

Infrastructure as code #111

merged 13 commits into from
Jun 21, 2021

Conversation

ArjaanBuijk
Copy link
Contributor

@ArjaanBuijk ArjaanBuijk commented Mar 19, 2021

Create a CD pipeline, deploying & smoke testing the artifacts created by the CI pipeline, using AWS resources (S3, ECR, EKS).

@ArjaanBuijk ArjaanBuijk self-assigned this Mar 19, 2021
@ArjaanBuijk ArjaanBuijk added the enhancement New feature or request label Mar 19, 2021
Base automatically changed from master to main March 24, 2021 09:01
@ArjaanBuijk ArjaanBuijk force-pushed the infrastructure-as-code branch 4 times, most recently from ae24b0f to 27297ad Compare March 24, 2021 20:15
@ArjaanBuijk ArjaanBuijk force-pushed the infrastructure-as-code branch 2 times, most recently from dacf207 to 4dff153 Compare May 27, 2021 14:15
Reorganize ci pipeline


Reorganize ci pipeline


Reorganize ci pipeline


Reorganize ci pipeline


Cache pip


Reorganize ci pipeline


cleanup


cleanup


cleanup


cleanup


cleanup


All in one cicd


All in one cicd


All in on cicd


All in on cicd


aws-sit-create-infrastructure


Trigger workflow


Fix format


Trigger workflow


infra


infra


trigger


infra


trigger


trigger


trigger


trigger


infra


infra


trigger


patch


trigger


infra


trigger


patch


trigger


patch


trigger


executive-summary


executive-summary


trigger


executive summary


trigger


parameters


trigger


parameters


trigger


trigger actions


cicd


trigger


cicd


trigger


cicd


t


cicd


t


c


t


c


ta


c


t


c


t1


c


t3


c


tr3


c


T3


c


t3


t1


c


T1


c


tr


c


tr


c


c


c


t


c


t


c


c


p


t


c


t


c


t


c


t


c


t


c


t


c


t


c


t


c


t


c


c


c


t


c


t


c


t


c


t


c


t


action_server (to AWS ECR)


rasa_model (to S3)


c


train model


aws-eks-namespace-test


t


c


t


c


t


c


t


c


t


c


t


c


t


c


t


c


t


c


t


c


c


c


t


c


t


c


t


c


t


c


t


c


t


c


t


c


t


c


t


c


c


t


c


c


t


c


t


c


t


c


t


c


t


demo


Sharing session demo, up to deploy_TEST


Rasa Enterprise Deploy placeholders


Install rasa enterprise


With action server


Deploy


t


Deploy


t


Deploy


t


Deploy rasa model


t


Rasa Enterprise Health


t


Deploy model


try deploy model


deploy after wait for endpoint


Run smoketest


T


Fix workflow


t


Fix


fix spacy


Fix smoketest


t


Upgrade eksctl


tr


Increase initialProbeDelay for rasa livenesscheck


tr1


Fix smoketest


tr2


Add sleep after tagging production


tr3


tr4


tr5


Update README


Test full pipeline w. cluster in place


tr1


Production cluster


production deployment


prod


pr1


pe1


prr1


Attempt 6


Attempt 7


sigh


sigh2


sigh 3


Use only steps based skipping

Doing it on job basis if very flaky and upredictable
fix if


Final
@ArjaanBuijk ArjaanBuijk marked this pull request as ready for review June 2, 2021 20:02
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@b-quachtran
Copy link
Contributor

Ah @ArjaanBuijk, looks like Github is hiding 23 of my added comments. Could you go through the hidden comments as well?

@ArjaanBuijk
Copy link
Contributor Author

Ah @ArjaanBuijk, looks like Github is hiding 23 of my added comments. Could you go through the hidden comments as well?

@b-quachtran ,
Thanks for pointing that out. I now reviewed & fixed/commented-on all your conversations.
Please check them out again.

Copy link
Contributor

@b-quachtran b-quachtran left a comment

Choose a reason for hiding this comment

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

Awesome! Looks good to me, only thing left outstanding is the nlu.yml conflict that needs to be resolved.

Help section in Makefile


Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Update README.md

Co-authored-by: Ben Quachtran <[email protected]>
Resolve 2 more items
@ArjaanBuijk ArjaanBuijk merged commit c7d0f6f into main Jun 21, 2021
@ArjaanBuijk ArjaanBuijk deleted the infrastructure-as-code branch June 21, 2021 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants