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

(CONT-217) Correct Kubernetes etcd_data_dir spec tests #582

Merged
merged 1 commit into from
Oct 19, 2022

Conversation

GSPatton
Copy link
Contributor

Removed incorrect functionality and test cases.
Added spec test to ensure that ETCD_DATA_DIR exists in the config file generated and ensures the default value of etcd_data_dir is its value
Additional tests will come with CONT-215

@puppet-community-rangefinder
Copy link

kubernetes::config::kubeadm is a class

that may have no external impact to Forge modules.

This module is declared in 0 of 579 indexed public Puppetfiles.


These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

@GSPatton GSPatton force-pushed the correct-etcd_data_dir-spec-tests branch from 1bff7fe to 38e28a1 Compare October 18, 2022 16:14
@GSPatton
Copy link
Contributor Author

spec test and integration test failures are the same failures seen in previous tests so are not related to this PR

@GSPatton GSPatton force-pushed the correct-etcd_data_dir-spec-tests branch from 38e28a1 to 418c681 Compare October 19, 2022 09:16
Removed incorrect functionality and test cases.
Added spec test to ensure that ETCD_DATA_DIR exists in the config file generated and ensures the default value of etcd_data_dir is its value
Added spec tests to ensure the value of ETCD_DATA_DIR in the config file updates successfully when given various values of etcd_data_dir
Fixed lint and Rubocop issues
@GSPatton GSPatton force-pushed the correct-etcd_data_dir-spec-tests branch from 418c681 to 6db7e2a Compare October 19, 2022 10:16
@chelnak chelnak merged commit fdc7e87 into main Oct 19, 2022
@chelnak chelnak deleted the correct-etcd_data_dir-spec-tests branch October 19, 2022 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants