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

[BUG] salt-cloud fix doc on tmp folder info, and FR to disable use of uuid in tmp folder #58304

Open
amalaguti opened this issue Aug 27, 2020 · 1 comment
Labels
doc-correction docu page has an error or deadlink Documentation Relates to Salt documentation Feature new functionality including changes to functionality and code refactors, etc. Salt-Cloud severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around time-estimate-sprint
Projects
Milestone

Comments

@amalaguti
Copy link

Description
This would be a documentation fix plus feature request
Based on Cloud docs here https://docs.saltstack.com/en/latest/topics/cloud/troubleshooting.html#tmp-is-mounted-as-noexec
The tmp folder used to deploy minion on the instance should be /tmp/.saltcloud

But it's not correct, it's /tmp/.saltcloud-(random uuid)

tmp_dir = "{}-{}".format(tmp_dir.rstrip("/"), uuid.uuid4())

tmp_dir = "{0}-{1}".format(tmp_dir.rstrip("/"), uuid.uuid4())

Should the docs be updated?

As a feature request, would be nice if this was set using an option, by default enabled, but given the change to the user to not append the uuid.

Setup
Salt: 3001.1

Steps to Reproduce the behavior
Deploy an instance, in my case using AWS EC2 as provider.
use:
salt-cloud -p profile instance_name --keep-tmp

Expected behavior

  1. Update documentation to reflect the right information
  2. Offer the option to disable the use of uuid
@amalaguti amalaguti added the Feature new functionality including changes to functionality and code refactors, etc. label Aug 27, 2020
@Akm0d Akm0d added doc-correction docu page has an error or deadlink Documentation Relates to Salt documentation Salt-Cloud team-cloud and removed needs-triage labels Sep 18, 2020
@Akm0d Akm0d added this to the Approved milestone Sep 18, 2020
@sagetherage sagetherage added this to To do in Salt Docs via automation Apr 23, 2021
@barbaricyawps barbaricyawps added severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around time-estimate-sprint labels Mar 21, 2022
@barbaricyawps
Copy link
Contributor

Just adding my two cents that these should be two separate issues: one for the doc request and one for the feature request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-correction docu page has an error or deadlink Documentation Relates to Salt documentation Feature new functionality including changes to functionality and code refactors, etc. Salt-Cloud severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around time-estimate-sprint
Projects
Salt Docs
  
To do
Development

No branches or pull requests

4 participants