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

fix(topic): only return error on http.StatusOK #1004

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

craicoverflow
Copy link
Contributor

The topic name validator should only be returning an error
when this topic name already exists.

Closes #1003

Verification Steps

  1. Run rhoas kafka create
  2. Enter a name in the interactive prompt.
  3. It should proceed past this stage.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation change
  • Other (please specify)

Checklist

  • Documentation added for the feature
  • CI and all relevant tests are passing
  • Code Review completed
  • Verified independently by reviewer

@craicoverflow craicoverflow requested a review from wtrocki August 31, 2021 15:54
@craicoverflow
Copy link
Contributor Author

@wtrocki I think we should backport this fix to a 0.27.1 - since we now already have 0.28.0-alpha1.

@wtrocki
Copy link
Collaborator

wtrocki commented Aug 31, 2021

👀

@wtrocki
Copy link
Collaborator

wtrocki commented Aug 31, 2021

Ignore alpha (I can remove it if needed). This should be released as hotfix. There were no significant changes on main so we might even release entire main as 0.27.1?

@craicoverflow
Copy link
Contributor Author

Weird stuff happening with the build failing on documentation check.

https://github.com/redhat-developer/app-services-cli/runs/3474949371

@wtrocki
Copy link
Collaborator

wtrocki commented Sep 1, 2021

Rebase will fix it

The topic name validator should only be returning an error
when this topic name already exists.
@craicoverflow craicoverflow force-pushed the topic-create-name-interactive branch from 8f8863b to d0ca314 Compare September 1, 2021 08:27
@wtrocki wtrocki merged commit da6f1fc into main Sep 1, 2021
@wtrocki
Copy link
Collaborator

wtrocki commented Sep 1, 2021

Verified.

@wtrocki wtrocki deleted the topic-create-name-interactive branch September 1, 2021 08:52
rkpattnaik780 pushed a commit that referenced this pull request Sep 20, 2021
The topic name validator should only be returning an error
when this topic name already exists.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kafka topics create in interactive mode is broken
2 participants