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

Cloudbees folder patch 1 #1086

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mthirdy
Copy link

@mthirdy mthirdy commented Jan 10, 2024

Compatibility with plugin cloudbees-folder'

Pull Request (PR) description

This PR provides the ability to manage elements of type/plugin 'cloudbees-folder'. In this case the name of the job could include slashes for subfolders. So the slashes in the job name have to be replaced by underscore for the temporary created file used for the jenkins api. Also the slashes have to replaced by '/jobs/' for the config.xml of a job while deleting the job via the jenkins api.

This Pull Request (PR) fixes the following issues

- the possible slashes in jobname are replaced by underscores to suppress the necessary of existing subfloders in the /tmp folder for temporary file creation
- the possible slashes in jobname are replaced by underscores so the correct file is deleted
- the possible slashes in jobname are replaced by '/jobs/' so the correct cloudbees-folder job config file is found and the job is deleted by jenkins api
@evgeni evgeni closed this Apr 23, 2024
@evgeni evgeni reopened this Apr 23, 2024
@evgeni evgeni closed this Apr 24, 2024
@evgeni evgeni reopened this Apr 24, 2024
@mthirdy
Copy link
Author

mthirdy commented Apr 24, 2024

@evgeni what did you do, so the checks have passed?

@evgeni
Copy link
Member

evgeni commented Apr 24, 2024

Fixed up the tests in master and then restarted your PR against that

- because of different jobdir for cloudbees-folder an already existing jobs was recreated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants