Skip to content

Conversation

leiicamundi
Copy link
Contributor

@leiicamundi leiicamundi commented Mar 4, 2025

Description

InfraEx is consolidating all the reference architectures in a single repository: https://github.com/camunda/camunda-deployment-references

This PR address relocation of the content from the old repos to the new one.
As some links were already using the new repo, this PR should be merged when a documentation release is planned as the PRs below will be break the existing live doc when merged.

To be replaced after each PR is merged

SM changes:

TODO

  • Align with @camunda/tech-writers to release a doc release when this PR is merged, otherwise existing doc will break

When should this change go live?

  • This is a bug fix, security concern, or something that needs urgent release support. (add bug or support label)
  • This is already available but undocumented and should be released within a week. (add available & undocumented label)
  • This is on a specific schedule and the assignee will coordinate a release with the DevEx team. (create draft PR and/or add hold label)
  • This is part of a scheduled alpha or minor. (add alpha or minor label)
  • There is no urgency with this change (add low prio label)

PR Checklist

  • My changes are for an upcoming minor release and:
    • are in the /docs directory (version 8.8).
    • are in the /versioned_docs/version-8.7/ directory (version 8.7).
  • My changes are for an already released minor and are in a /versioned_docs directory.

@leiicamundi leiicamundi added the component:self-managed Docs and issues related to Camunda Platform 8 Self-Managed label Mar 4, 2025
@leiicamundi leiicamundi self-assigned this Mar 4, 2025
Copy link
Contributor

github-actions bot commented Mar 4, 2025

👋 🤖 🤔 Hello, @leiicamundi! Did you make your changes in all the right places?

These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.6/.

  • docs/self-managed/setup/deploy/amazon/aws-ec2.md
  • docs/self-managed/setup/deploy/amazon/openshift/terraform-setup-dual-region.md
  • docs/self-managed/setup/deploy/openshift/dual-region.md
These files were changed only in docs/. You might want to duplicate these changes in versioned_docs/version-8.7/.
  • docs/self-managed/setup/deploy/amazon/aws-ec2.md
These files were changed only in versioned_docs/version-8.7/. You might want to duplicate these changes in docs/.
  • versioned_docs/version-8.7/self-managed/zeebe-deployment/configuration/broker.md
  • versioned_docs/version-8.7/self-managed/zeebe-deployment/configuration/gateway.md
These files were changed only in versioned_docs/version-8.7/. You might want to duplicate these changes in versioned_docs/version-8.6/.
  • versioned_docs/version-8.7/self-managed/setup/deploy/amazon/openshift/terraform-setup-dual-region.md
  • versioned_docs/version-8.7/self-managed/setup/deploy/openshift/dual-region.md
These files were changed only in versioned_docs/version-8.6/. You might want to duplicate these changes in docs/.
  • versioned_docs/version-8.6/self-managed/zeebe-deployment/configuration/broker.md
  • versioned_docs/version-8.6/self-managed/zeebe-deployment/configuration/gateway.md

You may have done this intentionally, but we wanted to point it out in case you didn't. You can read more about the versioning within our docs in our documentation guidelines.

@leiicamundi leiicamundi requested a review from Langleu March 12, 2025 09:08
@leiicamundi leiicamundi added the deploy Stand up a temporary docs site with this PR label Mar 12, 2025
Copy link
Member

@Langleu Langleu left a comment

Choose a reason for hiding this comment

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

Leaving my initial feedback on this.
Mostly links being 404.

Copy link
Member

Choose a reason for hiding this comment

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

leaving this as a single comment for the file to switch it to main for the next version.

@Langleu
Copy link
Member

Langleu commented Mar 12, 2025

had a quick look over the rendered docs and the bash references seem to all be in place properly except for the ones not working with 404.

@leiicamundi leiicamundi requested review from a team and Langleu March 12, 2025 15:24
@akeller akeller moved this to 👀 In Review in Documentation Team Mar 12, 2025
@github-actions github-actions bot temporarily deployed to camunda-docs March 12, 2025 15:56 Destroyed
Copy link
Contributor

@conceptualshark conceptualshark left a comment

Choose a reason for hiding this comment

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

Running through the links and formatting, everything seems to have been resolved/works well for me, so I think this PR will be ready to merge when you are. Would you like a hold label on it to ensure it isn't merged in advance?

Happy to approve and coordinate a specific release for you if you need to time it with the other repo, just let me know!

Langleu
Langleu previously approved these changes Mar 13, 2025
Copy link
Member

@Langleu Langleu left a comment

Choose a reason for hiding this comment

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

Just one small comment which isn't hindering a merge or so.
Had a quick check with lychee --include-verbatim and seems like all 404 were resolved 🚀.

@leiicamundi leiicamundi marked this pull request as ready for review March 13, 2025 14:56
@conceptualshark conceptualshark enabled auto-merge (squash) March 13, 2025 16:08
@github-actions github-actions bot temporarily deployed to camunda-docs March 13, 2025 16:14 Destroyed
@conceptualshark conceptualshark merged commit e4263b3 into main Mar 13, 2025
8 checks passed
@conceptualshark conceptualshark deleted the feature/integrate-reference-arch-changes branch March 13, 2025 16:15
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in Documentation Team Mar 13, 2025
Copy link
Contributor

The preview environment relating to the commit df69c04 has successfully been deployed. You can access it at https://preview.docs.camunda.cloud/pr-5142/index.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:self-managed Docs and issues related to Camunda Platform 8 Self-Managed deploy Stand up a temporary docs site with this PR

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants