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

custom resource: credentials_file #785

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

custom resource: credentials_file #785

wants to merge 13 commits into from

Conversation

damacus
Copy link
Member

@damacus damacus commented Mar 28, 2022

  • Add the credentials partial for reuse for all credentials resources
  • Move credentials_file to a custom resource

@damacus
- Move credentials_file to a custom resource
5331de8 
- Add the credentials partial for reuse for all credentials resources

Signed-off-by: Dan Webb <[email protected]>
@damacus damacus requested a review from a team as a code owner March 28, 2022 15:12
@damacus damacus added the Release: Minor Release to Chef Supermarket as a minor release when merged label Mar 28, 2022
damacus added 2 commits March 28, 2022 16:13
@damacus
Require Chef 16
505490f 
Signed-off-by: Dan Webb <[email protected]>
@damacus
Fix typo
20bee65 
Signed-off-by: Dan Webb <[email protected]>
@damacus damacus linked an issue Apr 7, 2022 that may be closed by this pull request
Convert LWRPs to custom resources #786
Open
16 tasks
damacus added 6 commits April 7, 2022 13:17
@damacus
Move resources to resources folder
c17d5f9 
Signed-off-by: Dan Webb <[email protected]>
@damacus
Move "slave" files
a129805 
Signed-off-by: Dan Webb <[email protected]>
@damacus
Add slave resources (these will be renamed to agent in a subsiquent pull
8f15d97 
request)

Signed-off-by: Dan Webb <[email protected]>
@damacus
Move more resources
bae07b5 
Remove specs that were either incomplete or very wrong

Signed-off-by: Dan Webb <[email protected]>
@damacus
Updates
1bd6ea2 
Signed-off-by: Dan Webb <[email protected]>
@damacus
LWRP --> Custom Resource
5c44869 
Signed-off-by: Dan Webb <[email protected]>
@ramereth ramereth added Release: Major Release to Chef Supermarket as a major change when merged and removed Release: Minor Release to Chef Supermarket as a minor release when merged labels Aug 7, 2022
@damacus damacus force-pushed the custom_resources/credentials_file branch from 91c6294 to 4f91046 Compare December 19, 2022 09:16
@github-actions
Copy link

😰 RSpec failure

8 examples, 1 failure
Example Description Message
./spec/recipes/master_spec.rb:42 jenkins::_master_war (default) system account is false creates the log directory undefined method `unified_mode' for main:Object

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xorima xorima xorima approved these changes

Assignees
No one assigned
Labels
Release: Major Release to Chef Supermarket as a major change when merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Convert LWRPs to custom resources
3 participants
@damacus @xorima @ramereth