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 IsMissingNamespaceErr function #905

Merged
merged 2 commits into from
May 6, 2024

Conversation

tpantelis
Copy link
Contributor

Checks if the error indicates the namespace for a resource is missing when trying to create it. In this case the error is NotFound and the status details reflects the "namespaces" Kind. For example:

{
  "ErrStatus": {
    "status": "Failure",
    "message": "namespaces \"foo\" not found",
    "reason": "NotFound",
    "details": {
      "name": "foo",
      "kind": "namespaces"
    },
    "code": 404
  }
}

Checks if the error indicates the namespace for a resource is missing
when trying to create it. In this case the error is NotFound and the
status details reflects the "namespaces" Kind. For example:

{
  "ErrStatus": {
    "status": "Failure",
    "message": "namespaces \"foo\" not found",
    "reason": "NotFound",
    "details": {
      "name": "foo",
      "kind": "namespaces"
    },
    "code": 404
  }
}

Signed-off-by: Tom Pantelis <[email protected]>
@tpantelis tpantelis self-assigned this May 4, 2024
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr905/tpantelis/missing_ns_err

...as is done in other projects.

Signed-off-by: Tom Pantelis <[email protected]>
@tpantelis tpantelis requested a review from dfarrell07 May 6, 2024 12:31
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr905/tpantelis/missing_ns_err]

@tpantelis tpantelis reopened this May 6, 2024
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr905/tpantelis/missing_ns_err

@tpantelis tpantelis enabled auto-merge (rebase) May 6, 2024 13:33
@tpantelis tpantelis merged commit 51aed88 into submariner-io:devel May 6, 2024
30 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr905/tpantelis/missing_ns_err]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants