Skip to content

[PHEE-999] Changed kafka port from 9092 to 9099#420

Open
abhinavm117 wants to merge 1 commit intoopenMF:masterfrom
abhinavm117:testing-1
Open

[PHEE-999] Changed kafka port from 9092 to 9099#420
abhinavm117 wants to merge 1 commit intoopenMF:masterfrom
abhinavm117:testing-1

Conversation

@abhinavm117
Copy link
Contributor

Description

  • PR title should have jira ticket enclosed in [].

    Format: [jira_ticket] description

    ex: [phee-123] PR title.
  • Add a link to the Jira ticket.
  • Describe the changes made and why they were made.

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Followed the PR title naming convention mentioned above.

  • Design related bullet points or design document link related to this PR added in the description above.

  • Updated corresponding Postman Collection or Api documentation for the changes in this PR.

  • Created/updated unit or integration tests for verifying the changes made.

  • Added required Swagger annotation and update API documentation with details of any API changes if applicable

  • Followed the naming conventions as given in https://docs.google.com/document/d/1Q4vaMSzrTxxh9TS0RILuNkSkYCxotuYk1Xe0CMIkkCU/edit?usp=sharing

@avikganguly01
Copy link
Contributor

avikganguly01 commented Apr 13, 2024

@abhinavm117 : Instead of making a pull request and then doing dev testing using CI, do the opposite. Test the case in isolation (local or cloud), and if it's working, then submit a PR. Often the reason why it can't be tested in cloud before submitting a draft PR to test via ephemeral CircleCI instance is flimsy so please continuously evaluate your arguments and submit your arguments in written format.

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