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
Add racktables machine reachability check #314
Conversation
01bf0e0
to
a66e346
Compare
a66e346
to
cf460fb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks OK.
cf460fb
to
c23e503
Compare
racktables/racktables.py
Outdated
req = self.s.get(join(self.url, "index.php"), params="&".join("%s=%s" % (k,v) for k,v in search_payload.items())) | ||
soup = BeautifulSoup(req.text, 'html.parser') | ||
result_table = soup.find("table", {"class": "cooltable"}) | ||
result_objs = result_table.find_all("tr", lambda tag: tag != None) #racktables does not use table-heads so we have to filter the header out (it has absolutely no attributes) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
result_objs = result_table.find_all("tr", lambda tag: tag != None) #racktables does not use table-heads so we have to filter the header out (it has absolutely no attributes) | |
result_objs = result_table.find_all("tr", lambda tag: tag != None) # Racktables does not use table-heads so we have to filter the header out (it has absolutely no attributes) |
Could you maybe indent the python scripts also by 4. (Or using tabs - I would totally approve that as well :P) |
c23e503
to
b1a6ef5
Compare
b1a6ef5
to
9fa9386
Compare
So all the changes you requested are in b1a6ef5 but the current version of both |
Related ticket: https://progress.opensuse.org/issues/158907