Skip to content
This repository was archived by the owner on Sep 26, 2025. It is now read-only.

Conversation

@rockb1017
Copy link
Contributor

Proposed changes

Circle CI release stage to automatically upload gem to rubygems.org.
For bash scripts to get fluentd HEC plugin version from file in the repo rather than from CircleCI env.

Types of changes

What types of changes does your code introduce?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • [ x ] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING doc
  • I have read the CLA
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

@rockb1017 rockb1017 self-assigned this Oct 9, 2019
VERSION_COMMIT=$(git log -1 --format=format:%H VERSION)
if [ $VERSION_COMMIT = $LATEST_COMMIT ];
then
VERSION=`cat VERSION`
Copy link
Contributor

Choose a reason for hiding this comment

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

if VERSION is empty/null/not set abort

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Implemented where VERSION is empty. it will error if there is no VERSION file from line 6.

Gemfile.lock Outdated
remote: .
specs:
fluent-plugin-kubernetes-metrics (1.1.0)
fluent-plugin-kubernetes-metrics (1.1.1)
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to update the Gemfile.lock if there are no updates in dependencies?

Copy link
Contributor Author

@rockb1017 rockb1017 Oct 10, 2019

Choose a reason for hiding this comment

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

changes to Gemfile.lock has been removed from this PR

@chaitanyaphalak chaitanyaphalak merged commit 2276e08 into develop Oct 11, 2019
@chaitanyaphalak chaitanyaphalak deleted the release_process branch October 11, 2019 14:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants