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

Use packaged msgpack #55354

Merged
merged 5 commits into from
Dec 18, 2019
Merged

Use packaged msgpack #55354

merged 5 commits into from
Dec 18, 2019

Commits on Dec 17, 2019

  1. Consistently use salt.utils.json and salt.utils.msgpack. 

    This commit introduces salt.utils.msgpack modifies all places in the
code that either use json or msgpack to use salt.utils.json or
salt.utils.msgpack respectively.

While this change itself does not have any effect, it is important to
allow for centrally dealing with objects that cannot be directly
serialied via json or msgpack.
    @smarsching @Akm0d
    smarsching authored and Akm0d committed Dec 17, 2019
    Configuration menu
    Copy the full SHA
    9c7d98e View commit details
    Browse the repository at this point in the history

  2. Use original msgpack module where needed. 

    When accessing msgpack.Unpacker, we cannot use salt.utils.msgpack, but
we do not have to either.
    @smarsching @Akm0d
    smarsching authored and Akm0d committed Dec 17, 2019
    Configuration menu
    Copy the full SHA
    3fe5ded View commit details
    Browse the repository at this point in the history

  3. Consistently use msgpack module, added tests

    @Akm0d
    Akm0d committed Dec 17, 2019
    Configuration menu
    Copy the full SHA
    89b01b5 View commit details
    Browse the repository at this point in the history

  4. Allow msgpack to be accessed easily by reference

    @Akm0d
    Akm0d committed Dec 17, 2019
    Configuration menu
    Copy the full SHA
    8c99fe9 View commit details
    Browse the repository at this point in the history

  5. isolated salt.utils.json changes to saltstack#55660

    @Akm0d
    Akm0d committed Dec 17, 2019
    Configuration menu
    Copy the full SHA
    6e3523d View commit details
    Browse the repository at this point in the history