the main thing thats missing is ds_cluster is hardcoded to ds_cluster
and i've only tested against an esxi box
will see if i can add ds_cluster tonight as well
there is a performance hit on ESXi, but its still only around 11s
Jc2k
@Jc2k
might have ds_cluster added but might need to take you up on vCenter creds to finish testing
@pryorda going to bed now but if you could try ^ and let me know how you get on (just pushed a bit more)
Daniel Pryor
@pryorda
Ill work on trying it out today, sorry for the delay been working on "work"
Daniel Pryor
@pryorda
@Jc2k fix_folders works for nested directories. I'm going to create a beta build it it will let me.
Daniel Pryor
@pryorda
@Jc2k let me know what you think about missing labels.
Jc2k
@Jc2k
heya
sorry its been a while i've forgotten what i did
vague feeling the missing labels was maybe intentional because i didn't fully finish it yet
Jc2k
@Jc2k
@pryorda left some requests on the PR
Daniel Pryor
@pryorda
Ill take a peak
added the logs statements
running it now
Jc2k
@Jc2k
Coo
Office -> Home but will try to touch base later, I have some other PR farming to do as well hopefully
Daniel Pryor
@pryorda
heh okay
New output added
_
Daniel Pryor
@pryorda
@Jc2k let me know if you need anything else from me.
Martin Kremers
@kremers
Hi there :)
Jc2k
@Jc2k
Afternoon =)
Martin Kremers
@kremers
I've tested 9f75cdc against the VSphere environment, works as expected.
Jc2k
@Jc2k
Nice one
Martin Kremers
@kremers
Would be nice to add pypy3 to the build matrix. Haven't tested it yet
Jc2k
@Jc2k
if it works i'd be very happy about that. their 3.6 support is still beta i think, so we might want to have it be allowed to fail rather than a hard fail?
Martin Kremers
@kremers
Regarding the stages in travisci i've rather no experience. If you can provide a follow up PR that would be great
if it works i'd be very happy about that. their 3.6 support is still beta i think, so we might want to have it be allowed to fail rather than a hard fail?
Yes! Allow to fail is a good idea IMO
Jc2k
@Jc2k
can do a follow up pr for stages. will mean that the semantic-release only happens if all mandatory steps pass
Martin Kremers
@kremers
Awesome :)
Jc2k
@Jc2k
your PR made me realise that we can release with failing travis steps atm D:
:(
Martin Kremers
@kremers
The whole CI/CD part is hard to get right. I was quite happy to see lint checks and tests for the project
Jc2k
@Jc2k
yeah we had a rough few releases when we added the extra parallelism. took 20m scrapes down to 20s for some environments, made quite a few others not work at all :/
always more to do, though
Martin Kremers
@kremers
The current project has a really fancy folder structure. Datastores and Hosts have not worked at all in my case
Jc2k
@Jc2k
my current project doesn't even have vSphere so it has been quite hard not to regress that for me :)
Martin Kremers
@kremers
:P
IMO the PR is feature complete. I would create others following with extended testing
The debug logging is pretty nice. If there is something not discovered it will throw a ? as Object type in the printed tree. This can enhance tracing of such problems a lot I hope.
I think I am happy with this PR if @pryorda is, assuming it passes testing in his environment as well. On the condition that there is a follow up PR for more tests.
Martin Kremers
@kremers
Fully agree. I want to at least test-implement the VCenter folder structure here
How does the release management work? Any idea how versions are bumped?