Skip to content

Bash script for hotswapping leo jar on fiab#766

Merged
akarukappadath merged 8 commits intodevelopfrom
ak-hotswap
Feb 4, 2019
Merged

Bash script for hotswapping leo jar on fiab#766
akarukappadath merged 8 commits intodevelopfrom
ak-hotswap

Conversation

@akarukappadath
Copy link
Contributor

Wrote a bash script to perform hotswap of leo jar for a fiab. Run using

./automation/hotswap.sh fiab-your-fiab-name

from leonardo.

Have you read CONTRIBUTING.md lately? If not, do that first.

I, the developer opening this PR, do solemnly pinky swear that:

  • I've documented my API changes in Swagger

In all cases:

  • Get a thumbsworth of review and PO signoff if necessary
  • Verify all tests go green
  • Squash and merge; you can delete your branch after this
  • Test this change deployed correctly and works on dev environment after deployment

@@ -0,0 +1,17 @@
#!/usr/bin/env bash

Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you add some comments at the top describe what the script is for, what args to pass, etc?

Also maybe some code that prints the usage if the arg isn't specified.

These are just good practices for bash scripts.

Copy link
Collaborator

@rtitle rtitle left a comment

Choose a reason for hiding this comment

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

Looks good!

@akarukappadath
Copy link
Contributor Author

jenkins retest

@akarukappadath akarukappadath merged commit 536dccb into develop Feb 4, 2019
Qi77Qi pushed a commit that referenced this pull request Jun 25, 2019
* starting

* still trying

* hotswap.sh in automation

* changed permissions

* cleaned up

* comments and error
jchanbroad pushed a commit that referenced this pull request Jun 27, 2019
* starting

* still trying

* hotswap.sh in automation

* changed permissions

* cleaned up

* comments and error
@akarukappadath akarukappadath deleted the ak-hotswap branch July 8, 2019 20:27
jdcanas pushed a commit that referenced this pull request Jul 10, 2019
* starting

* still trying

* hotswap.sh in automation

* changed permissions

* cleaned up

* comments and error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants