Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[release-0.11] Cherry pick docs fixes #1565

Merged
merged 4 commits into from
May 22, 2020

Conversation

hasheddan
Copy link
Member

Description of your changes

Cherry picks docs fixes from the following PRs:

#1560 (@zzxwill)
#1562 (@zzxwill)
#1564 (@Zee-Khan)

How has this code been tested?

[skip ci]

Checklist

I have:

  • Run make reviewable to ensure this PR is ready for review.
  • Ensured this PR contains a neat, self documenting set of commits.
  • Updated any relevant documentation and examples.
  • Reported all new error conditions into the log or as an event, as
    appropriate.

For more about what we believe makes a pull request complete, see our
definition of done.

zzxwill and others added 4 commits May 22, 2020 10:36
region is required resource type for kind Provider
of Alibaba Cloud or it will raise error message:
error: the server doesn't have a resource type "region"

Signed-off-by: zzxwill <zzxwill@gmail.com>
(cherry picked from commit e11286b)
Followed provision infrasture to provision an RDSInstance CRD,
it failed with the issue connect failed: cannot get provider
secret: Secret "alibaba-creds" not found. By creating the secret
in namespace `crossplane-system` will fix the issue

Signed-off-by: zzxwill <zzxwill@gmail.com>
(cherry picked from commit f3e19a7)
Changed Azure postgres version from 5.7 to 9.6

Signed-off-by: Zohaib Khan <zohaib.asghar.khan@gmail.com>
Signed-off-by: zohaib.asghar.khan@gmail.com <zohaib.khan@swisscom.com>
(cherry picked from commit 3eb55b0)
Changed Azure postgres version from 5.7 to 9.6

Signed-off-by: Zohaib Khan <zohaib.asghar.khan@gmail.com>
Signed-off-by: zohaib.asghar.khan@gmail.com <zohaib.khan@swisscom.com>
(cherry picked from commit cb8bbaf)
@hasheddan hasheddan requested a review from jbw976 May 22, 2020 15:45
@hasheddan
Copy link
Member Author

@jbw976 it looks like DCO had some trouble trying to parse the sign-off despite it working on the original PR. I can amend these commits or we can just set DCO to passing. Do you have a preference? I think I lean towards the latter so that these remain strict cherry picks.

@jbw976
Copy link
Member

jbw976 commented May 22, 2020

That sounds good @hasheddan, I examined the failing DCO parsing and the commit looks good to me with a human readable sign-off, so I have set the DCO check on this PR to pass now.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants