Skip to content

Improve error msg for "invalid DC name" webhook#1675

Open
frivoire wants to merge 1 commit intok8ssandra:mainfrom
frivoire:patch-1
Open

Improve error msg for "invalid DC name" webhook#1675
frivoire wants to merge 1 commit intok8ssandra:mainfrom
frivoire:patch-1

Conversation

@frivoire
Copy link
Copy Markdown

@frivoire frivoire commented Jan 21, 2026

What this PR does:
Just improve an error-message

Which issue(s) this PR fixes:
Fixes #1676

Checklist

  • Changes manually tested
  • Automated Tests added/updated ===> no need here I think
  • Documentation added/updated ===> no need here I think
  • CHANGELOG.md updated ===> I don't know which file to modify
  • CLA Signed: DataStax CLA ==> Could not resolve host: cla.datastax.com 😢

@frivoire
Copy link
Copy Markdown
Author

frivoire commented Jan 21, 2026

I created the issue and edited the PR's description to add it.

But still not ok for CI check 🤔

Error: No linked issues found. Please add the corresponding issues in the pull request description.

Strange, isn't it ?

@frivoire frivoire changed the title Improve error msg for invalid DC name Improve error msg for "invalid DC name" webhook Jan 21, 2026
@sonarqubecloud
Copy link
Copy Markdown

@adejanovski
Copy link
Copy Markdown
Contributor

I created the issue and edited the PR's description to add it.

But still not ok for CI check 🤔

Error: No linked issues found. Please add the corresponding issues in the pull request description.

Strange, isn't it ?

Hi, you need to specify Fixes in front of the issue number for GH to associate the PR with the issue. See here.

I made the change for you.

@frivoire
Copy link
Copy Markdown
Author

frivoire commented Jan 23, 2026

Hi, you need to specify Fixes in front of the issue number for GH to associate the PR with the issue. See here.
I made the change for you.

Ha yes, I understand now.
Thanks for the info & the fix (of the "fixes" 😆)

I'm switching the PR to "ready for review" now, since the CI is green.

@frivoire frivoire marked this pull request as ready for review January 23, 2026 11:59
@frivoire frivoire requested a review from a team as a code owner January 23, 2026 11:59
@frivoire
Copy link
Copy Markdown
Author

@adejanovski: do you think we can merge this 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.

Error msg for "invalid DC name" webhook should be more explicit

2 participants