Bz1699188

[lhellebr]

Close-loop BZ 1699188. Create user, assign him a role with permission filter, gather facts on a host, get facts from this host through API as that user.
Depends on: SatelliteQE/nailgun#936

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[synkd]

Looks good overall. ACK pending addressing @shweta83's suggestions.

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

$ pytest tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
===================================================== test session starts ======================================================
platform linux -- Python 3.9.16, pytest-7.3.1, pluggy-1.0.0 -- /home/lhellebr/git/robottelo/venv/bin/python
Mandatory Requirements Available: pytest-reportportal==5.1.9 broker[docker]==0.3.2 pytest-xdist==3.3.1 requests==2.31.0 wrapanapi==3.5.15 cryptography==41.0.1 pytest==7.3.2
Optional Requirements Available: sphinx-autoapi==2.1.1 sphinx==7.0.1 redis==4.5.5 pre-commit==3.3.2
cachedir: .pytest_cache
shared_function enabled - OFF - scope:  - storage: file
rootdir: /home/lhellebr/git/robottelo
configfile: pytest.ini
plugins: cov-3.0.0, services-2.2.1, xdist-3.2.1, reportportal-5.1.7, mock-3.10.0, ibutsu-2.2.4
collected 3 items                                                                                                              

tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter[rhel6] PASSED                                    [ 33%]
tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter[rhel7] PASSED                                    [ 66%]
tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter[rhel8] PASSED                                    [100%]

======================================================= warnings summary =======================================================
tests/foreman/api/test_ansible.py: 201 warnings
  /home/lhellebr/git/robottelo/venv/lib/python3.9/site-packages/urllib3/connectionpool.py:1045: InsecureRequestWarning: Unverified HTTPS request is being made to host '<AT-FQDN>'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
    warnings.warn(

tests/foreman/api/test_ansible.py: 56 warnings
  /home/lhellebr/git/robottelo/venv/lib/python3.9/site-packages/urllib3/connectionpool.py:1045: InsecureRequestWarning: Unverified HTTPS request is being made to host '<FQDN>'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
    warnings.warn(

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
========================================= 3 passed, 257 warnings in 1615.55s (0:26:55) =========================================

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

I suspect there is a race condition causing ISE when attempting to delete a host while its facts haven't yet been imported. It happens only for RHEL8, perhaps because it has more facts. It doesn't happen locally but it does in PRT. I'll switch this to Draft so I can use PRT to come up with a workaround.

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

PRT failures caused by AT migration, waiting for it to work.

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[ogajduse]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter
nailgun: 936

[lhellebr]

Can we celebrate this PR's one month anniversary by merging it?

[omkarkhatavkar]

@rplevka can you please confirm the questions you have addressed?

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter

[omkarkhatavkar]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter

[Gauravtalreja1]

@lhellebr Can you look into failing PRT results?

[lhellebr]

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::test_positive_read_facts_with_filter

[ogajduse]

PRT passed, ACK