test auth_file_propagation should be removed

Bug #1916262 reported by Joshua Genet
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Charmed Kubernetes Testing
Triaged
Medium
Unassigned

Bug Description

As seen here: https://solutions.qa.canonical.com/testruns/testRun/b9c16068-5ce4-408a-a5a8-47d1424964e4

Right now when this test fails, it is hard to tell what went wrong. The only way that I can see to triage failures would be to look at the /root/cdk/basic_auth.csv on each of the kubernetes-master nodes. It would be useful if the test was a bit more verbose.

---

Here's the output and how we are running it:

============================= test session starts ==============================
platform linux -- Python 3.8.5, pytest-6.1.2, py-1.9.0, pluggy-0.13.1 -- /home/ubuntu/k8s-validation/.tox/py3/bin/python
cachedir: .tox/py3/.pytest_cache
metadata: {'Python': '3.8.5', 'Platform': 'Linux-5.4.0-1035-aws-x86_64-with-glibc2.29', 'Packages': {'pytest': '6.1.2', 'py': '1.9.0', 'pluggy': '0.13.1'}, 'Plugins': {'html': '2.1.1', 'mock': '3.3.1', 'asyncio': '0.14.0', 'metadata': '1.10.0', 'flaky': '3.7.0'}}
rootdir: /home/ubuntu/k8s-validation, configfile: pytest.ini
plugins: html-2.1.1, mock-3.3.1, asyncio-0.14.0, metadata-1.10.0, flaky-3.7.0
collecting ... collected 1 item

jobs/integration/validation.py::test_auth_file_propagation
-------------------------------- live log setup --------------------------------
unknown facade CAASModelOperator
unexpected facade CAASModelOperator found, unable to decipher version to use
unknown delta type: id

2021-02-13-02:12:11 root ERROR [localhost] STDERR follows:

Traceback (most recent call last):
  File "/usr/local/bin/fce", line 11, in <module>
    load_entry_point('foundationcloudengine', 'console_scripts', 'fce')()
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/main.py", line 170, in entry_point
    sys.exit(main(sys.argv[1:]))
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/main.py", line 161, in main
    opts.func(opts)
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/build.py", line 112, in build_main
    build_and_validate_if_needed(
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/build.py", line 86, in build_and_validate_if_needed
    layer.validate()
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/layers/baselayer.py", line 200, in validate
    validator.run()
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/layers/baselayer.py", line 389, in run
    self.run_inner()
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/layers/kubernetes.py", line 208, in run_inner
    self.run_tests(controller, model)
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/layers/kubernetes.py", line 237, in run_tests
    local(cmd, output_mode="live", env=my_env)
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/remotehelpers.py", line 227, in local
    return run_cmd(cmd, target_machine, **kwargs)
  File "/home/ubuntu/cpe/foundation/foundationcloudengine/foundationcloudengine/remotehelpers.py", line 165, in run_cmd
    raise subprocess.CalledProcessError(ps.returncode, cmd, output=out, stderr=err)
subprocess.CalledProcessError: Command '['timeout', '1800', 'tox', '--workdir', '.tox', '-c', '/home/ubuntu/k8s-validation/tox.ini', '-e', 'py3', '--', 'pytest', '-v', '-s', '--junit-xml=/home/ubuntu/project/generated/kubernetes/k8s-suite/test_auth_file_propagation-junit.xml', '--controller=foundations-aws', '--model=kubernetes', '/home/ubuntu/k8s-validation/jobs/integration/validation.py::test_auth_file_propagation']' returned non-zero exit status 124.

Revision history for this message
George Kraft (cynerva) wrote :

Honestly, this test seems kinda silly now given that Kubernetes no longer supports Basic Auth. The kubernetes-master charm still dutifully propagates the basic_auth.csv file, but for no real reason.

I think we should just remove this test.

Changed in charmed-kubernetes-testing:
importance: Undecided → High
status: New → Triaged
Revision history for this message
Joshua Genet (genet022) wrote :

Oh snap yeah, removed since 1.19.
I'm all for not running this. I'll remove it on our end.
Thanks!

George Kraft (cynerva)
Changed in charmed-kubernetes-testing:
importance: High → Medium
summary: - test auth_file_propagation needs better logging
+ test auth_file_propagation should be removed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.