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_sls_aggregate #52927

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

unit.serializers.test_serializers.TestSerializers.test_sls_aggregate #52927

waynew opened this issue May 7, 2019 · 1 comment
Assignees
@garethgreenaway
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

{a: lol, foo: [hello], bar: [1, 2, 3], baz: {a: 42, b: 666, c: the beast}}


Traceback (most recent call last):
  File "/tmp/kitchen/testing/tests/unit/serializers/test_serializers.py", line 187, in test_sls_aggregate
    """).strip() == yamlex.serialize(sls_obj), sls_obj
AssertionError: {a: lol, foo: [hello], bar: [1, 2, 3], baz: {a: 42, b: 666, c: the beast}}
@waynew waynew added Test-Failure Fix the CI failure ⛈ develop Pointing to develop branch; needs rebase labels May 7, 2019
@waynew waynew added this to the Approved milestone May 7, 2019
@waynew waynew mentioned this issue May 7, 2019
Closed
@KChandrashekhar KChandrashekhar added v2019.2.1 unsupported version severity-critical top severity, seen by most users, serious issues and removed develop Pointing to develop branch; needs rebase labels May 7, 2019
@KChandrashekhar KChandrashekhar modified the milestones: Approved, Priority May 8, 2019
@Ch3LL Ch3LL mentioned this issue May 9, 2019
Merged
@Ch3LL
Copy link
Contributor

Ch3LL commented May 9, 2019

this should be fixed by #52957

@Ch3LL Ch3LL added the fixed-pls-verify fix is linked, bug author to confirm fix label May 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@garethgreenaway garethgreenaway

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
4 participants
@waynew @garethgreenaway @Ch3LL @KChandrashekhar