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

[DO-NOT-MERGE] Release kafka_consumer 3.1.4 beta #15795

Closed
wants to merge 5 commits into from

Conversation

yzhan289
Copy link
Contributor

@yzhan289 yzhan289 commented Sep 8, 2023

What does this PR do?

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Changelog entries must be created for modifications to shipped code
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

* Remove unused TLS context

* Update test_tls_config_legacy

* Fix test for legacy tls_validate_hostname config

* Fix style
* Set tls_verify to string rather than boolean

* Remove unused TLS context

* Add Changelog entry

* Update test_tls_config_legacy

* Fix test for legacy tls_validate_hostname config

* Fix style

* Fix style

* Move script.py to separate PR
@ghost ghost added the release label Sep 8, 2023
@github-actions
Copy link

github-actions bot commented Sep 8, 2023

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

❗ No coverage uploaded for pull request base (7.47.x@ca73a51). Click here to learn what that means.
The diff coverage is n/a.

Flag Coverage Δ
kafka_consumer 93.48% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

@github-actions
Copy link

github-actions bot commented Sep 8, 2023

Test Results

    8 files      8 suites   6m 24s ⏱️
  63 tests   62 ✔️ 0 💤 1
256 runs  251 ✔️ 4 💤 1

For more details on these failures, see this check.

Results for commit b4a4e07.

@yzhan289 yzhan289 closed this Sep 11, 2023
@yzhan289 yzhan289 deleted the az/beta-kafka-consumer-tls-verify branch September 11, 2023 12:18
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.

1 participant