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

Upgrade Salt to 3003.1 #3436

Open
TeddyAndrieux opened this issue Jul 1, 2021 · 4 comments
Open

Upgrade Salt to 3003.1 #3436

TeddyAndrieux opened this issue Jul 1, 2021 · 4 comments
Labels
kind:dependencies Pull requests that update a dependency file state:blocked Something prevents this from being worked on state:question Further information is requested topic:salt Everything related to SaltStack in our product

Comments

@TeddyAndrieux
Copy link
Collaborator

Component:

'salt'

Why this is needed:

To follow new Salt versions

What should be done:

Upgrade Salt to the latest version (3003.1 when this ticket get opened)

Implementation proposal (strongly recommended):

This branch https://github.com/scality/metalk8s/compare/improvement/salt-3003


May need some more investigation,
An issue with Salt-API for the storage operator, I don't know if it's a bug in the new Salt version or if it's just something that changed in Salt-API (did not see anything related to this in Salt changelog).

When the storage operator tries to get the device name of a device the salt API return is wrong.

Manual tests output:

Salt API (existing device):

$ curl -Ski -b cookies.txt https://192.168.1.100:4507/ -H "Accept: application/x-yaml" -H "Content-type: application/json" -d '[{"client": "local", "tgt": "bootstrap", "fun": "metalk8s_volumes.device_name", "arg": ["/dev/loop1"]}]'
HTTP/1.1 200 OK
Content-Type: application/x-yaml
Server: CherryPy/5.6.0
Date: Wed, 30 Jun 2021 15:04:53 GMT
Allow: GET, HEAD, POST
Access-Control-Allow-Origin: *
Access-Control-Expose-Headers: GET, POST
Access-Control-Allow-Credentials: true
Cache-Control: private
Vary: Accept-Encoding
Content-Length: 27
Set-Cookie: session_id=23be7c9b8690d47c3cc695388c337fcadcae546e; expires=Thu, 01 Jul 2021 01:04:53 GMT; Path=/

return:
- bootstrap: false

Salt call (existing device)

$ salt-call metalk8s_volumes.device_name /dev/loop1
local:
    ----------
    result:
        loop1
    success:
        True

Salt call (device that does not exist)

$ salt-call metalk8s_volumes.device_name /dev/invalid-path
local:
    ----------
    result:
        device `/dev/invalid-path` not found
    success:
        False
@TeddyAndrieux TeddyAndrieux added state:question Further information is requested state:blocked Something prevents this from being worked on kind:dependencies Pull requests that update a dependency file topic:salt Everything related to SaltStack in our product labels Jul 1, 2021
@TeddyAndrieux
Copy link
Collaborator Author

Issue about salt-api get fixed in the branch just by mounting the salt cache directory in salt-api container

@TeddyAndrieux
Copy link
Collaborator Author

Still another issue about salt-ssh 3003 saltstack/salt#60620 to be investigated if it can be workarouned easily

@TeddyAndrieux
Copy link
Collaborator Author

TeddyAndrieux commented Oct 29, 2021

@TeddyAndrieux
Copy link
Collaborator Author

Issue saltstack/salt#60620 still there with salt-ssh 3005.1 (but seems fixed in 3006, which means we need to move to onedir deployment)

Branch with 3005.1 https://github.com/scality/metalk8s/compare/improvement/bump-salt-3005

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:dependencies Pull requests that update a dependency file state:blocked Something prevents this from being worked on state:question Further information is requested topic:salt Everything related to SaltStack in our product
Projects
None yet
Development

No branches or pull requests

1 participant