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

Add diagram for eks deployment #552

Merged
merged 11 commits into from
Dec 21, 2023
Merged

Conversation

feenr
Copy link
Member

@feenr feenr commented Apr 7, 2023

I added a diagram for a recomended approach to deploying the Pega Helm charts on AWS, with some consideration given to high availability.

@feenr feenr requested review from taz-pega-work, kingakowalska1 and a team as code owners April 7, 2023 15:26
Copy link
Contributor

@taz-pega-work taz-pega-work left a comment

Choose a reason for hiding this comment

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

Can you add a Hazelcast service into the mypega namespace? you add the hazelcast Docker image to the pega values.yaml so when you deploy that namespace, it brings up Hazelcast-dedicated pod

@feenr feenr requested a review from taz-pega-work April 7, 2023 18:09
Copy link
Contributor

@taz-pega-work taz-pega-work left a comment

Choose a reason for hiding this comment

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

that looks great. thank you!

@petejo
Copy link

petejo commented Apr 7, 2023

We should probably identify the versions of Pega this diagram is applicable for.

Copy link
Contributor

@taz-pega-work taz-pega-work left a comment

Choose a reason for hiding this comment

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

looks great! thanks for adding the addons tier

Copy link
Contributor

@kingakowalska1 kingakowalska1 left a comment

Choose a reason for hiding this comment

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

Looks great!

@MadhuriArugula
Copy link
Collaborator

@feenr Are we good to merge this PR?

@feenr feenr merged commit 7442028 into pegasystems:master Dec 21, 2023
6 checks passed
@feenr
Copy link
Member Author

feenr commented Dec 21, 2023

@MadhuriArugula sorry for ignoring your previous comment. Branch is merged now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants