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

Remove non-action loop proxies. #93

Merged
merged 2 commits into from Oct 22, 2020
Merged

Remove non-action loop proxies. #93

merged 2 commits into from Oct 22, 2020

Conversation

rabbah
Copy link
Member

@rabbah rabbah commented Oct 10, 2020

The non-action loop based proxies are not being maintained. They are replaced by their action loop counterparts.
This PR removes the relevant docker builds and code and updates the README as well.
I did not rename the action loop images in this PR.

Copy link
Member

@dgrove-oss dgrove-oss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM -- makes sense to leave changing the image name to latter. Would be nice to migrate to the "new" standard naming convention for some of these older images.

@rabbah
Copy link
Member Author

rabbah commented Oct 12, 2020

I've sent a note to the dev list for comments before proceeding with a merge in 72 hours.

@rabbah rabbah merged commit 8dd54c7 into apache:master Oct 22, 2020
@rabbah rabbah deleted the cleanup branch February 23, 2021 22:03
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.

None yet

2 participants