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

basho-patches #994

Open
martinsumner opened this issue Oct 28, 2019 · 3 comments
Open

basho-patches #994

martinsumner opened this issue Oct 28, 2019 · 3 comments
Assignees

Comments

@martinsumner
Copy link
Contributor

Some riak_test tests (e.g. ensemble_remove_node - https://github.com/basho/riak_test/blob/develop-2.9/tests/ensemble_basic2.erl) require a patch that survives restart using rt_intercept:add_and_save/2.

This add and save uses the basho_patches folder, which is required to be in the code path of the node. However this is no longer in the code path in develop-3.0.

The mkdir command in rebar.config

{mkdir, "lib/riak-patches"},
has been switched to riak-patches. However, although switching that back to basho-patches will create the path in make devrel, it will not add the path to the code_path (i.e. if you start a node and then call code_get_path() from riak remote_console it is not included).

Is this a relx issue?

@martincox might you be able to take a look?

martinsumner added a commit to basho/riak_test that referenced this issue Oct 28, 2019
Currently riak develop-3.0 does not support basho-patches in the code path.  Bodge this so we can still add and save a patch (maybe - or perhaps just adding the path this way won't have the intercept survive the reboot?).

basho/riak#994
@martincox
Copy link
Contributor

martincox commented Nov 5, 2019

@martinsumner it's a side effect of moving away from the runner in node_package. It's on my todo.

@martincox martincox self-assigned this Nov 5, 2019
@Bob-The-Marauder
Copy link
Contributor

Sorry, what's the official stance on this? I find code in 3.0.1 referring to basho-patches in riak-admin and riak-debug but the installed directory is only called lib/patches and I am not seeing a lib/riak-patches folder anywhere. If somebody can confirm what the official name is, I'm happy to update the above two files in /rel/files/bin/ to use the right name.

@martinsumner
Copy link
Contributor Author

#1089

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

No branches or pull requests

3 participants