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

New collection named core following appropriate standards #32

Closed
wants to merge 12 commits into from
Closed

New collection named core following appropriate standards #32

wants to merge 12 commits into from

Conversation

fgiorgetti
Copy link
Member

  • Collection is now named core instead of network
  • All variables are now following Ansible Lint recommendations (no rule skipped)
  • All roles have an appropriate argument_specs.yml file
  • Documentation looks complete
  • Basic Hello World end-to-end test added (Pending CI integration)

@fgiorgetti
Added initial support for task vars
8f1b1ce 
* Some roles and modules were not expecting variables as part of task_vars
* Modules now have mechanisms to read arguments from both host and task
@fgiorgetti
Updated docs for hello world example
4948607
@fgiorgetti
Updated CLI version to 1.5.3
0f78fe2
@fgiorgetti
Ensure that skupper_link works with statically provided token
1206f81
@fgiorgetti
Draft: Renamed all variables to comply with Ansible lint
a9f3884 
* This is a breaking change (existing playbooks won't work)
* Collection name might change to avoid problems to those already using the collection
@fgiorgetti
Renamed collection
93e87bb 
* New collection is now named core
* Renamed skupper role to network_config
* Added e2e test
@fgiorgetti
Added argument specs for all roles expect services
3403624
@fgiorgetti
All argument spec files done and docs look good
722b020
@fgiorgetti
All argument spec files done and docs look good
1f32596
@fgiorgetti
Removed collection inner docs dir
e6a053c
@fgiorgetti
Fixed services_load unit test
9faad77
@fgiorgetti
Renamed collection to skupper
557ce4f
@fgiorgetti
Copy link
Member Author

Opening a better one shortly.

@fgiorgetti fgiorgetti closed this Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fgiorgetti