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.

Sign up for GitHub

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

Jump to bottom

Clustering trust_domain is broken due to api changes in tm.sys.application #521

Closed
pjbreaux opened this issue Jul 3, 2016 · 1 comment · Fixed by #522 or #530
Closed

Clustering trust_domain is broken due to api changes in tm.sys.application #521

pjbreaux opened this issue Jul 3, 2016 · 1 comment · Fixed by #522 or #530
Assignees
@pjbreaux
Labels
bug critical enhancement

Comments

@pjbreaux
Copy link
Contributor

pjbreaux commented Jul 3, 2016

The clustering module trust_domain.py is broken because the api changed for tm.sys.applications to tm.sys.application. I will fix the issue and submit the fix, and we'll have to spin another release to remedy this.
@pjbreaux pjbreaux self-assigned this Jul 3, 2016
@pjbreaux
Copy link
Contributor Author

pjbreaux commented Jul 3, 2016

https://github.com/F5Networks/f5-common-python/blob/development/f5/multi_device/trust_domain.py#L215

pjbreaux pushed a commit to pjbreaux/f5-common-python-1 that referenced this issue Jul 4, 2016
Clustering trust_domain is broken due to api changes in
74930af 
tm.sys.application

Issues:
Fixes F5Networks#521

Problem:
The clustering module trust_domain.py is broken because the api changed
for tm.sys.applications to tm.sys.application. I will fix the issue and
submit the fix, and we'll have to spin another release to remedy this.

Analysis:
Modified the references to applications in trust_domain.py and reran the
clustering tests.

Tests:
Clustering tests passed locally.
@pjbreaux pjbreaux mentioned this issue Jul 4, 2016
Merged
pjbreaux pushed a commit to pjbreaux/f5-common-python-1 that referenced this issue Jul 4, 2016
Clustering trust_domain is broken due to api changes in
10b4aca 
tm.sys.application

Issues:
Fixes F5Networks#521

Problem:
The clustering module trust_domain.py is broken because the api changed
for tm.sys.applications to tm.sys.application. I will fix the issue and
submit the fix, and we'll have to spin another release to remedy this.

Analysis:
Modified the references to applications in trust_domain.py and reran the
clustering tests.

Tests:
Clustering tests passed locally.
pjbreaux pushed a commit to pjbreaux/f5-common-python-1 that referenced this issue Jul 5, 2016
Clustering trust_domain is broken due to api changes in
249c745 
tm.sys.application

Issues:
Fixes F5Networks#521

Problem:
The clustering module trust_domain.py is broken because the api changed
for tm.sys.applications to tm.sys.application. I will fix the issue and
submit the fix, and we'll have to spin another release to remedy this.

Analysis:
Modified the references to applications in trust_domain.py and reran the
clustering tests.

Tests:
Clustering tests passed locally.
@pjbreaux pjbreaux mentioned this issue Jul 6, 2016
Merged
caphrim007 pushed a commit to caphrim007/f5-common-python that referenced this issue Aug 2, 2016
@caphrim007
Clustering trust_domain is broken due to api changes in
4359505 
tm.sys.application

Issues:
Fixes F5Networks#521

Problem:
The clustering module trust_domain.py is broken because the api changed
for tm.sys.applications to tm.sys.application. I will fix the issue and
submit the fix, and we'll have to spin another release to remedy this.

Analysis:
Modified the references to applications in trust_domain.py and reran the
clustering tests.

Tests:
Clustering tests passed locally.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pjbreaux pjbreaux

Labels
bug critical enhancement
Projects
None yet
Milestone
No milestone
1 participant
@pjbreaux