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

JBIDE-16407 Breakinbg connection between openshift and legacy framework... #355

Closed
wants to merge 1 commit into from

Conversation

robstryker
Copy link
Member

... for servers.

updating dependency version

removing legacy classes

small cleanup of patch

…rk for servers.

updating dependency version

removing legacy classes

small cleanup of patch
@robstryker
Copy link
Member Author

Please test this patch before accepting it. I know my testing abilities on openshift are not where they need to be so it'd be good to get another tester to verify.

@maxandersen
Copy link
Member

is this unrelated to the refactoring of server that is not yet in master ?

@robstryker
Copy link
Member Author

This is related, and the goal is to decouple logic that has no reason at all to coupled. In this way, openshift no longer depends on these classes at all, and puts its publishing logic 100% in the hands of openshift plugin instead of delegating around through a framework.

So I would say its related, but not dependent. It's also something I suggested be cleaned up months ago, as well as the unused classes for a 'binary' behavior which does not exist.

@robstryker
Copy link
Member Author

So I would say its related, but not dependent.

To be clear, openshift would need to commit this change if my work ever makes it into master. But, even if my work never makes it into master, openshift would benefit from a cleaner workflow by applying this patch.

@maxandersen
Copy link
Member

Okey - then i'm just confused sicne the reasons on JBIDE-16407 seems to be counter to the changes planned for openshift. Maybe take this to the jira where other reasons a listed ?

@adietish
Copy link
Member

this was merged in #358 together with further changes (ex. disabling start/stop/restart)

@adietish adietish closed this Feb 12, 2014
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

3 participants