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

retrieve the Monitor object in the GSLB._build() method #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leu2z
Copy link

@leu2z leu2z commented Dec 16, 2016

The monitor attribute is always None when getting GSLB objects, for example when retrieved with Zone.get_all_gslb().
This patch allows the Monitor object to be instantiated inside the GSLB _build() method.

The monitor attribute is always None when getting GSLB objects, for example when retrieved with Zone.get_all_gslb().
This patch allows the Monitor object to be instantiated inside the GSLB _build() method, perhaps could be less spaghetti coded.
@derekargueta
Copy link

@leu2z if there's interest in finishing this, the build is failing on flake8 linter errors

dyn/tm/services/gslb.py:817:80: E501 line too long (91 > 79 characters)
dyn/tm/services/gslb.py:819:19: E111 indentation is not a multiple of four

cc @Northover @Marchowes

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.

2 participants