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

Sdd updates #1196

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Sdd updates #1196

wants to merge 2 commits into from

Conversation

AlexanderHodicke
Copy link
Member

No description provided.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 9, 2024

*Guide:*

*This chapter describes landing zone best practices and usually does not require any changes. If changes are required please refer to [Landing Zone GitHub](https://github.com/oracle-devrel/technology-engineering/tree/main/landing-zones). The full landing zone needs to be described in the Solution Design by the service provider.*
*This chapter describes naming convention best practices and usually does not require any changes. If changes are required please refer to [Landing Zone GitHub](https://github.com/oracle-devrel/technology-engineering/tree/main/landing-zones). The naming convention zone needs to be described in the Solution Design by the service provider.*
Copy link
Member

Choose a reason for hiding this comment

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

"naming convention zone" doesn't make sense.

Copy link
Member

Choose a reason for hiding this comment

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

Page 16: OCI Vulnerability Scanning Service should be replaced by OCI Cloud Guard Instance Security.

Copy link
Member

Choose a reason for hiding this comment

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

Page 16:

  • Delete "If changes are required please refer to Landing Zone GitHub"
  • "naming convention zone" does not make any sense.

Copy link
Member

Choose a reason for hiding this comment

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

Page 25:

  • Change "Enable VSS" to "Enable Cloud Guard Instance Security"
  • Change "Configure OCI IAM Domains" to "Configure Default OCI IAM Domain for administration and additional OCI IAM Domains for PaaS workloads"

Copy link
Member

Choose a reason for hiding this comment

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

Page 26: Should read "Two-Factor"

Copy link
Member

Choose a reason for hiding this comment

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

Page 9: OCI Vulnerability Scanning Service should be replaced by OCI Cloud Guard Instance Security.

Copy link
Member

Choose a reason for hiding this comment

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

Page 10:

  • Delete "If changes are required please refer to Landing Zone Github."
  • "naming convention zone" does not make any sense.

Copy link
Member

Choose a reason for hiding this comment

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

Page 14: Caption "OCI Architecture" is misleading. Be more specific.

Copy link
Member

Choose a reason for hiding this comment

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

Page 17:

  • Change "Enable VSS" to "Enable Cloud Guard Instance Security"
  • Change "Configure OCI IAM Domains" to "Configure Default OCI IAM Domain for administration and additional OCI IAM Domains for PaaS workloads"

Copy link
Member

Choose a reason for hiding this comment

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

Page 20: Caption "OCI Architecture" is misleading. Be more specific.

@oheimburger
Copy link
Member

@AlexanderHodicke please resolve the issues highlighted above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants