Skip to content
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

unit.cloud.clouds.test_ec2 test failing #52721

Closed
Ch3LL opened this issue Apr 26, 2019 · 2 comments

Comments

@Ch3LL
Copy link
Contributor

commented Apr 26, 2019

The test unit.cloud.clouds.test_ec2.EC2TestCase.test_get_password_data is failing on:

2019.2.1 failed salt-centos-7-py3, salt-centos-7-py3-pycryptodomex, salt-centos-7-py3-transport, salt-debian-9-py3, salt-fedora-28-py3, salt-fedora-29-py3, salt-opensuse-42-py3, salt-ubuntu-1604-py3, salt-ubuntu-1604-py3-pycryptodomex, salt-ubuntu-1604-py3-transport, salt-ubuntu-1804-py3, salt-windows-2016-py3

https://jenkinsci.saltstack.com/job/2019.2.1/view/Python3/job/salt-centos-7-py3/7/

Traceback (most recent call last):
  File "/usr/lib/python3.4/site-packages/mock/mock.py", line 1305, in patched
    return func(*args, **keywargs)
  File "/tmp/kitchen/testing/tests/unit/cloud/clouds/test_ec2.py", line 88, in test_get_password_data
    assert ret['password'] == 'testp4ss!'
KeyError: 'password'
@waynew

This comment has been minimized.

Copy link
Contributor

commented May 7, 2019

Still failing on develop across the platforms:

develop failed salt-debian-9-py3, salt-fedora-29-py3, salt-opensuse-42-py3, salt-ubuntu-1604-py3, salt-ubuntu-1604-py3-pycryptodomex, salt-ubuntu-1804-py3, salt-windows-2016-py2, salt-windows-2016-py3, salt-windows-2019-py2, salt-windows-2019-py3

'password'


Traceback (most recent call last):
  File "C:\Python35\lib\site-packages\mock\mock.py", line 1329, in patched
    return func(*args, **keywargs)
  File "c:\users\admini~1\appdata\local\temp\kitchen\testing\tests\unit\cloud\clouds\test_ec2.py", line 88, in test_get_password_data
    assert ret['password'] == 'testp4ss!'
KeyError: 'password'
		

Does this just need a merge forward?

@Akm0d

This comment has been minimized.

Copy link
Contributor

commented May 22, 2019

This continues to be an issue for develop but doesn't seem to block 2019.2.1

@Akm0d Akm0d closed this May 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.