Skip to content

Conversation

@nwang92
Copy link
Contributor

@nwang92 nwang92 commented Mar 9, 2020

When tested with the splunk-ansible branch here, I get a success:

(venv) nwang@nwang-MBP-CCEA8:~/github/docker-splunk (develop) $ pytest -sv tests/test_docker_splunk.py::TestDockerSplunk::test_adhoc_1so_hec_idempotence --platform redhat-8
======== test session starts ========
platform darwin -- Python 2.7.16, pytest-4.6.9, py-1.8.1, pluggy-0.13.1 -- /Users/nwang/github/docker-splunk/venv/bin/python
cachedir: .pytest_cache
rootdir: /Users/nwang/github/docker-splunk/tests, inifile: pytest.ini
collected 1 item

tests/test_docker_splunk.py::TestDockerSplunk::test_adhoc_1so_hec_idempotence PASSED

======== warnings summary ========
venv/lib/python2.7/site-packages/_pytest/mark/structures.py:335
  /Users/nwang/github/docker-splunk/venv/lib/python2.7/site-packages/_pytest/mark/structures.py:335: PytestUnknownMarkWarning: Unknown pytest.mark.large - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/latest/mark.html
    PytestUnknownMarkWarning,

-- Docs: https://docs.pytest.org/en/latest/warnings.html
======== 1 passed, 1 warnings in 143.64 seconds ========

I'll also fix some of the other HEC tests that fail once the splunk-ansible PR is merged.

arctan5x
arctan5x previously approved these changes Mar 9, 2020
jmeixensperger
jmeixensperger previously approved these changes Mar 10, 2020
@nwang92 nwang92 dismissed stale reviews from jmeixensperger and arctan5x via d7edfcf March 11, 2020 00:56
@nwang92 nwang92 merged commit 231ffa9 into develop Mar 12, 2020
@nwang92 nwang92 deleted the bugfix/test-updates-for-hec branch March 12, 2020 17:53
alishamayor added a commit that referenced this pull request Mar 12, 2020
* Bugfix/uf uid gid (#320)

* Fixing uid/gid on UF images; adding tests

* Fixed tests for changes in splunk-ansible (#323)

* Adding tests for peered standalones (#326)

* update exec commands w/ splunk user (#329)

* Updating image for upgrade test (#330)

* Feature/adhoc search head (#327)

* Adding example scenario using free-standing search head

* Fixing scenario env var

* Adding new test for idempotent changes to HEC configuration (#328)

* Adding new test for idempotent changes to HEC configuration

* Fixing other HEC tests

* Updating to latest redhat ubi-minimal (#332)

* Updating changelog for release/8021 (#333)

Co-authored-by: Nelson Wang <nwang92@users.noreply.github.com>
Co-authored-by: mikedickey <mdickey@splunk.com>
Co-authored-by: Jack Meixensperger <jackm@splunk.com>
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.

6 participants