Skip to content

[data](deps): Bump responses from 0.13.4 to 0.21.0 in /python/requirements/data_processing - #15

Closed
dependabot[bot] wants to merge 1 commit into
masterfrom
dependabot/pip/python/requirements/data_processing/responses-0.21.0
Closed

[data](deps): Bump responses from 0.13.4 to 0.21.0 in /python/requirements/data_processing#15
dependabot[bot] wants to merge 1 commit into
masterfrom
dependabot/pip/python/requirements/data_processing/responses-0.21.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github May 28, 2022

Copy link
Copy Markdown

Bumps responses from 0.13.4 to 0.21.0.

Release notes

Sourced from responses's releases.

0.21.0

  • Add threading.Lock() to allow responses working with threading module.
  • Add urllib3 Retry mechanism. See #135
  • Removed internal _cookies_from_headers function
  • Now add, upsert, replace methods return registered response. remove method returns list of removed responses.
  • Added null value support in urlencoded_params_matcher via allow_blank keyword argument
  • Added strict version of decorator. Now you can apply @responses.activate(assert_all_requests_are_fired=True) to your function to validate that all requests were executed in the wrapped function. See #183

0.20.0

  • Deprecate responses.assert_all_requests_are_fired, responses.passthru_prefixes, responses.target since they are not actual properties of the class instance. Use responses.mock.assert_all_requests_are_fired, responses.mock.passthru_prefixes, responses.mock.target instead.
  • Fixed the issue when reset() method was called in not stopped mock. See #511

0.19.0

  • Added a registry that provides more strict ordering based on the invocation index. See responses.registries.OrderedRegistry.
  • Added shortcuts for each request method: delete, get, head, options, patch, post, put. For example, to add response for POST request you can use responses.post() instead of responses.add(responses.POST).
  • Prevent responses.activate decorator to leak, if wrapped function called from within another wrapped function. Also, allow calling of above mentioned chain. See #481 for more details.
  • Expose get_registry() method of RequestsMock object. Replaces internal _get_registry().
  • query_param_matcher can now accept dictionaries with int and float values.
  • Add support for the loose version of query_param_matcher via named argument strict_match.
  • Added support for async/await functions.
  • response_callback is no longer executed on exceptions raised by failed Responses
  • Change logic of _get_url_and_path to comply with RFC 3986. Now URL match occurs by matching schema, authority and path, where path is terminated by the first question mark ("?") or number sign ("#") character, or by the end of the URI.
  • An error is now raised when both content_type and headers[content-type] are provided as parameters.
  • When a request isn't matched the passthru prefixes are now included in error messages.

0.18.0

  • Dropped support of Python 2.7, 3.5, 3.6
  • Fixed issue with type annotation for responses.activate decorator. See #468
  • Removed internal _is_string and _ensure_str functions
  • Removed internal _quote from test_responses.py
  • Removed internal _matches attribute of RequestsMock object.
  • Generated decorator wrapper now uses stdlib features instead of strings and exec
  • Fix issue when Deprecation Warning was raised with default arguments in responses.add_callback due to match_querystring. See #464

0.17.0

  • This release is the last to support Python 2.7.
  • Fixed issue when response.iter_content when chunk_size=None entered infinite loop
  • Fixed issue when passthru_prefixes persisted across tests.

... (truncated)

Changelog

Sourced from responses's changelog.

0.21.0

  • Add threading.Lock() to allow responses working with threading module.
  • Add urllib3 Retry mechanism. See #135
  • Removed internal _cookies_from_headers function
  • Now add, upsert, replace methods return registered response. remove method returns list of removed responses.
  • Added null value support in urlencoded_params_matcher via allow_blank keyword argument
  • Added strict version of decorator. Now you can apply @responses.activate(assert_all_requests_are_fired=True) to your function to validate that all requests were executed in the wrapped function. See #183

0.20.0

  • Deprecate responses.assert_all_requests_are_fired, responses.passthru_prefixes, responses.target since they are not actual properties of the class instance. Use responses.mock.assert_all_requests_are_fired, responses.mock.passthru_prefixes, responses.mock.target instead.
  • Fixed the issue when reset() method was called in not stopped mock. See #511

0.19.0

  • Added a registry that provides more strict ordering based on the invocation index. See responses.registries.OrderedRegistry.
  • Added shortcuts for each request method: delete, get, head, options, patch, post, put. For example, to add response for POST request you can use responses.post() instead of responses.add(responses.POST).
  • Prevent responses.activate decorator to leak, if wrapped function called from within another wrapped function. Also, allow calling of above mentioned chain. See #481 for more details.
  • Expose get_registry() method of RequestsMock object. Replaces internal _get_registry().
  • query_param_matcher can now accept dictionaries with int and float values.
  • Add support for the loose version of query_param_matcher via named argument strict_match.
  • Added support for async/await functions.
  • response_callback is no longer executed on exceptions raised by failed Responses
  • Change logic of _get_url_and_path to comply with RFC 3986. Now URL match occurs by matching schema, authority and path, where path is terminated by the first question mark ("?") or number sign ("#") character, or by the end of the URI.
  • An error is now raised when both content_type and headers[content-type] are provided as parameters.
  • When a request isn't matched the passthru prefixes are now included in error messages.

0.18.0

  • Dropped support of Python 2.7, 3.5, 3.6
  • Fixed issue with type annotation for responses.activate decorator. See #468
  • Removed internal _is_string and _ensure_str functions

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot @github

dependabot Bot commented on behalf of github May 28, 2022

Copy link
Copy Markdown
Author

Dependabot tried to add @scv119 and @clarkzinzow as reviewers to this PR, but received the following error from GitHub:

POST https://api.github.com/repos/nikitavemuri/ray/pulls/15/requested_reviewers: 422 - Reviews may only be requested from collaborators. One or more of the users or teams you specified is not a collaborator of the nikitavemuri/ray repository. // See: https://docs.github.com/rest/reference/pulls#request-reviewers-for-a-pull-request

@dependabot dependabot Bot added the dependencies Pull requests that update a dependency file label May 28, 2022
Bumps [responses](https://github.com/getsentry/responses) from 0.13.4 to 0.21.0.
- [Release notes](https://github.com/getsentry/responses/releases)
- [Changelog](https://github.com/getsentry/responses/blob/master/CHANGES)
- [Commits](getsentry/responses@0.13.4...0.21.0)

---
updated-dependencies:
- dependency-name: responses
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot
dependabot Bot force-pushed the dependabot/pip/python/requirements/data_processing/responses-0.21.0 branch from cae34e7 to 87bacea Compare July 14, 2022 18:01
nikitavemuri pushed a commit that referenced this pull request Jul 28, 2022
We encountered SIGSEGV when running Python test `python/ray/tests/test_failure_2.py::test_list_named_actors_timeout`. The stack is:

```
#0  0x00007fffed30f393 in std::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string(std::string const&) ()
   from /lib64/libstdc++.so.6
#1  0x00007fffee707649 in ray::RayLog::GetLoggerName() () from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#2  0x00007fffee70aa90 in ray::SpdLogMessage::Flush() () from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#3  0x00007fffee70af28 in ray::RayLog::~RayLog() () from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#4  0x00007fffee2b570d in ray::asio::testing::(anonymous namespace)::DelayManager::Init() [clone .constprop.0] ()
   from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#5  0x00007fffedd0d95a in _GLOBAL__sub_I_asio_chaos.cc () from /home/admin/dev/Arc/merge/ray/python/ray/_raylet.so
#6  0x00007ffff7fe282a in call_init.part () from /lib64/ld-linux-x86-64.so.2
#7  0x00007ffff7fe2931 in _dl_init () from /lib64/ld-linux-x86-64.so.2
#8  0x00007ffff7fe674c in dl_open_worker () from /lib64/ld-linux-x86-64.so.2
#9  0x00007ffff7b82e79 in _dl_catch_exception () from /lib64/libc.so.6
#10 0x00007ffff7fe5ffe in _dl_open () from /lib64/ld-linux-x86-64.so.2
#11 0x00007ffff7d5f39c in dlopen_doit () from /lib64/libdl.so.2
#12 0x00007ffff7b82e79 in _dl_catch_exception () from /lib64/libc.so.6
#13 0x00007ffff7b82f13 in _dl_catch_error () from /lib64/libc.so.6
#14 0x00007ffff7d5fb09 in _dlerror_run () from /lib64/libdl.so.2
#15 0x00007ffff7d5f42a in dlopen@@GLIBC_2.2.5 () from /lib64/libdl.so.2
#16 0x00007fffef04d330 in py_dl_open (self=<optimized out>, args=<optimized out>)
    at /tmp/python-build.20220507135524.257789/Python-3.7.11/Modules/_ctypes/callproc.c:1369
```

The root cause is that when loading `_raylet.so`, `static DelayManager _delay_manager` is initialized and `RAY_LOG(ERROR) << "RAY_testing_asio_delay_us is set to " << delay_env;` is executed. However, the static variables declared in `logging.cc` are not initialized yet (in this case, `std::string RayLog::logger_name_ = "ray_log_sink"`).

It's better not to rely on the initialization order of static variables in different compilation units because it's not guaranteed. I propose to change all `RAY_LOG`s to `std::cerr` in `DelayManager::Init()`.

The crash happens in Ant's internal codebase. Not sure why this test case passes in the community version though.

BTW, I've tried different approaches:

1. Using a static local variable in `get_delay_us` and remove the global variable. This doesn't work because `init()` needs to access the variable as well.
2. Defining the global variable as type `std::unique_ptr<DelayManager>` and initialize it in `get_delay_us`. This works but it requires a lock to be thread-safe.
@dependabot @github

dependabot Bot commented on behalf of github Oct 15, 2022

Copy link
Copy Markdown
Author

Superseded by #51.

@dependabot dependabot Bot closed this Oct 15, 2022
@dependabot
dependabot Bot deleted the dependabot/pip/python/requirements/data_processing/responses-0.21.0 branch October 15, 2022 07:14
nikitavemuri pushed a commit that referenced this pull request Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants