Skip to content

doc: quickstart for Ansible projects #3555

Merged
camilamacedo86 merged 2 commits intooperator-framework:masterfrom
camilamacedo86:quick-ansible
Jul 27, 2020
Merged

doc: quickstart for Ansible projects #3555
camilamacedo86 merged 2 commits intooperator-framework:masterfrom
camilamacedo86:quick-ansible

Conversation

@camilamacedo86
Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 commented Jul 26, 2020 •
edited
Loading

Description of the change:

  • The quickstart was moved to tutorial
  • Added new quickstart less verbose and with the concise steps.
  • Update the links and the order to show the docs

See: https://deploy-preview-3555--operator-sdk.netlify.app/docs/building-operators/ansible/quickstart/

Motivation for the change:

#3327
Closes: #2774

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 26, 2020
@camilamacedo86 camilamacedo86 requested a review from jmrodri July 26, 2020 23:20
@joelanford joelanford mentioned this pull request Jul 26, 2020
Closed
92 tasks
@camilamacedo86 camilamacedo86 changed the title wip: doc: add ansible quickstart wip: doc: quickstart for Helm projects Jul 26, 2020
@camilamacedo86 camilamacedo86 changed the title wip: doc: quickstart for Helm projects wip: doc: quickstart for Ansible projects Jul 26, 2020
@camilamacedo86 camilamacedo86 changed the title wip: doc: quickstart for Ansible projects doc: quickstart for Ansible projects Jul 26, 2020
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 26, 2020
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

camilamacedo86 commented Jul 26, 2020 •
edited
Loading

Hi @fabianvf @asmacdo @jmrodri

I did it equals the others. Also, IMO I cannot test all steps because of #3556 but I think we can move forward here and after perform an follow up if required. wdyt?

@camilamacedo86 camilamacedo86 mentioned this pull request Jul 27, 2020
Closed
@camilamacedo86 camilamacedo86 added this to the v1.0.0 milestone Jul 27, 2020
@camilamacedo86 camilamacedo86 added language/ansible Issue is related to an Ansible operator project kind/documentation Categorizes issue or PR as related to documentation. labels Jul 27, 2020
asmacdo
asmacdo suggested changes Jul 27, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@asmacdo asmacdo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small change.

Comment thread website/content/en/docs/building-operators/ansible/quickstart.md
Comment thread website/content/en/docs/building-operators/ansible/tutorial.md
jmrodri
jmrodri approved these changes Jul 27, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 27, 2020
@camilamacedo86 camilamacedo86 merged commit b468973 into operator-framework:master Jul 27, 2020
@camilamacedo86 camilamacedo86 deleted the quick-ansible branch July 27, 2020 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmrodri jmrodri jmrodri approved these changes

@fabianvf fabianvf Awaiting requested review from fabianvf

+1 more reviewer

@asmacdo asmacdo asmacdo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jmrodri jmrodri

Labels

kind/documentation Categorizes issue or PR as related to documentation. language/ansible Issue is related to an Ansible operator project lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

v1.0.0

Development

Successfully merging this pull request may close these issues.

Improve Ansible User Guide documentation for Building/Running the Operator

4 participants

@camilamacedo86 @jmrodri @asmacdo @openshift-ci-robot