globus / globus-compute

Globus Compute: High Performance Function Serving for Science

https://www.globus.org/compute

Apache License 2.0

146 stars 47 forks source link

Remove remote whitelist commands #1622

Closed chris-janidlo closed 1 month ago

chris-janidlo commented 1 month ago

Description

With the config-based allowed_functions feature, managing endpoint allow-lists via add_to_whitelist and delete_from_whitelist does not work as expected. Since allowed_functions is the recommended route, this simply removes those broken features.

[sc-34835]

Type of change

New feature (non-breaking change that adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Code maintenance/cleanup

chris-janidlo commented 1 month ago

thoughts on adding a couple of UTs?

Wasn't going to include any originally since they're pretty bare-bones functions, but I realized I forgot to update the doc string for Client.get_allowed_functions, so some tests snuck in alongside that