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

[BUG] [Package Tests] MacOS installers don't keep salt-pip extras directory in previous state #65073

Open
1 of 10 tasks
MKLeb opened this issue Aug 28, 2023 · 0 comments
Open
1 of 10 tasks

[BUG] [Package Tests] MacOS installers don't keep salt-pip extras directory in previous state #65073

MKLeb opened this issue Aug 28, 2023 · 0 comments
Assignees
@twangboy
Labels
Bug broken, incorrect, or confusing behavior Packaging Related to packaging of Salt, not Salt's support for package management.

Comments

@MKLeb
Copy link
Contributor

MKLeb commented Aug 28, 2023

Description
When upgrading or downgrading relenv macos packages, the extras-X.Y directory is reset to being empty.

Once fixed, the downgrade tests should be altered to test that this is working correctly in test_salt_downgrade.py.

Setup

  • on-prem machine
  • VM (Virtualbox, KVM, etc. please specify)
  • VM running on a cloud service, please be explicit and add details
    • Parallels
  • container (Kubernetes, Docker, containerd, etc. please specify)
  • or a combination, please be explicit
  • jails if it is FreeBSD
  • classic packaging
  • onedir packaging
  • used bootstrap to install

Steps to Reproduce the behavior

  • Install v3006.1
  • salt-pip install pep8
  • Notice that the installed pep8 package is in extras-X.Y
  • Install v3006.2
  • Notice that pep8 is no longer present in any fashion in extras-X.Y

Expected behavior
The extras directory should still have the installed pep8 package
@MKLeb MKLeb added the Bug broken, incorrect, or confusing behavior label Aug 28, 2023
@MKLeb MKLeb moved this to Not started in Test Coverage and Stability Aug 28, 2023
@twangboy twangboy mentioned this issue Aug 29, 2023
Merged
3 tasks
@OrangeDog OrangeDog added the Packaging Related to packaging of Salt, not Salt's support for package management. label Aug 29, 2023
@MKLeb MKLeb moved this from Not started to In Progress in Test Coverage and Stability Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@twangboy twangboy

Labels
Bug broken, incorrect, or confusing behavior Packaging Related to packaging of Salt, not Salt's support for package management.
Projects
Test Coverage and Stability
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
3 participants
@OrangeDog @twangboy @MKLeb