-
Notifications
You must be signed in to change notification settings - Fork 23
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
fscloud profile and example #306
Conversation
6eac619
to
9f5a833
Compare
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
6239bb4
to
69abdd7
Compare
Co-authored-by: Renovate Bot <renovate@whitesourcesoftware.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see comments
examples/fscloud/version.tf
Outdated
# Pin to the lowest provider version of the range defined in the main module to ensure lowest version still works | ||
ibm = { | ||
source = "ibm-cloud/ibm" | ||
version = ">= 1.51.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As per the comment, pin the lowest version in the example - aka 1.51.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this one not done - examples should pin to lowest version in the range
…form-ibm-cos into latest_fscloud
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@daniel-butler-irl thanks for updates - still a couple of comments pending
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
Signed-off-by: Daniel Butler <dannyeb@gmail.com>
🎉 This PR is included in version 5.11.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Description
Adding FS cloud profile and tests
Types of changes in this PR
No release required
Release required
x.x.X
): Change that fixes an issue and is compatible with earlier versions)x.X.x
): Change that adds functionality and is compatible with earlier versions)X.x.x
): Change that is likely incompatible with previous versions)Release notes content
Replace this text with information that users need to know about the bug fixes, features, and breaking changes. This information helps the merger write the commit message that is published in the release notes for the module.
Checklist for reviewers
Merge actions for mergers
Merge by using "Squash and merge".
Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).