-
-
Notifications
You must be signed in to change notification settings - Fork 637
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
windows master support #668
base: main
Are you sure you want to change the base?
Conversation
550a495
to
dac3150
Compare
The basic existing tests are working for me now. Given a run_list of:
all tests pass in I started working on updating additional tests, but they don't all appear to be up-to-date/working (e.g. I don't want to prematurely check off the testing tasks, but FWIW the "new" functionality of install via MSI is covered and pretty much everything else leverages the web ui or local java execution (which is already tested). Open to suggestions on next steps. |
Signed-off-by: troyready <[email protected]>
dac3150
to
04d18a1
Compare
any momentum on getting this PR pulled in? |
@troyready can you rebase this on current master and verify all checks still pass. Then we can possibly get someone to review it. |
I don't have the bandwidth to finish this now, sorry. The changes should be pretty straightforward for someone to pick up, only real functional change I think is figuring out the new conditional check for inspec (I think |
Need this please merge |
@troyready / @dnorenavega-EDR can we get this rebased so we can look to have it merged please? |
Description
Adds support for Jenkins master on Windows
Issues Resolved
#437 , #547
Check List