Skip to content

Conversation

@LeiQL
Copy link
Owner

@LeiQL LeiQL commented May 23, 2024

  • Create container resources for materialize scenario, same for scenarios using API calls to create version resources.

  • Require metadata.name formatted "containerName:version" in the properties of catalog to ensure managers know what container resource to be created. And this materialized resource using "containerName-version" as metadata name.

  • In mutation webhook, handle owner reference creation if container resource exists

  • In validation webhook, check object name should consist of -

  • In validation webhook, check if spec.RootResource exists and ownerReference should not be empty list.

  • Materialize: create container resources for versioned objects, same for scenarios using API calls to create versioned objects.

  • Materialize: require materialized obj metadata.name formatted "containerName:version" in the properties of a catalog to ensure managers know what container resource to be created. And this materialized resource using "containerName-version" as metadata name. For the catalog spec sample below , webappconfig as container resource name and webappconfig-v1 as materialized resource name.

properties:
    metadata: 
      name: webappconfig:v1
  • Webhook: check and handle owner reference creation if container resource exists in mutation webhook
  • Webhook: check object name should consist of <rootResource>-<version> in validation webhook,
  • Webhook: check if spec.RootResource exists and ownerReference should not be empty list in validation webhook,

@LeiQL LeiQL requested a review from coderdjw as a code owner May 23, 2024 12:00
@LeiQL LeiQL changed the title test Create container obj May 24, 2024
@LeiQL LeiQL force-pushed the users/leqia/rootresource branch 4 times, most recently from b602a09 to 5aba901 Compare June 6, 2024 03:42
@LeiQL LeiQL force-pushed the users/leqia/rootresource branch 2 times, most recently from c6d1c1c to 2484215 Compare June 6, 2024 14:38
@LeiQL LeiQL force-pushed the users/leqia/rootresource branch from 2484215 to f05ef66 Compare June 6, 2024 15:19
@LeiQL LeiQL force-pushed the users/leqia/rootresource branch from 5760cf9 to 2026d18 Compare June 7, 2024 03:32
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.

2 participants