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.

Sign up for GitHub

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

Jump to bottom

unit.serializers.test_serializers.TestSerializers.test_serialize_complex_sls #52930

Closed
waynew opened this issue May 7, 2019 · 1 comment
Closed

unit.serializers.test_serializers.TestSerializers.test_serialize_complex_sls #52930

waynew opened this issue May 7, 2019 · 1 comment
Assignees
@Ch3LL
Labels
fixed-pls-verify fix is linked, bug author to confirm fix severity-critical top severity, seen by most users, serious issues Test-Failure Fix the CI failure ⛈ v2019.2.1 unsupported version
Milestone
Priority

Comments

@waynew
Copy link
Contributor

waynew commented May 7, 2019

2019.2.1 failed salt-fedora-28-py2, salt-fedora-29-py2

foo: 1 bar: 2 baz: true


Traceback (most recent call last):
  File "/tmp/kitchen/testing/tests/unit/serializers/test_serializers.py", line 82, in test_serialize_complex_sls
    assert serialized == '{foo: 1, bar: 2, baz: true}', serialized
AssertionError: foo: 1
bar: 2
baz: true
@waynew waynew added v2019.2.1 unsupported version Test-Failure Fix the CI failure ⛈ labels May 7, 2019
@waynew waynew added this to the Approved milestone May 7, 2019
@KChandrashekhar KChandrashekhar modified the milestones: Approved, Priority May 8, 2019
@KChandrashekhar KChandrashekhar added the severity-critical top severity, seen by most users, serious issues label May 8, 2019
@Ch3LL Ch3LL mentioned this issue May 9, 2019
Merged
@Ch3LL Ch3LL added the fixed-pls-verify fix is linked, bug author to confirm fix label May 9, 2019
@Ch3LL
Copy link
Contributor

Ch3LL commented May 9, 2019

Fixed by #52957

@Ch3LL Ch3LL closed this as completed May 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ch3LL Ch3LL

Labels
fixed-pls-verify fix is linked, bug author to confirm fix severity-critical top severity, seen by most users, serious issues Test-Failure Fix the CI failure ⛈ v2019.2.1 unsupported version
Projects
None yet
Milestone
Priority
Development

No branches or pull requests
3 participants
@waynew @Ch3LL @KChandrashekhar