Commit graph

351 commits

Author SHA1 Message Date
mattkirby
81c6e48528
Merge pull request #97 from puppetlabs/types-docs
(DIO-991) Add service command
2020-09-17 08:58:24 -07:00
mattkirby
e34964be00
Merge pull request #96 from puppetlabs/dio-911
( DIO-911) Include job_id in ABS --json output
2020-09-17 08:55:43 -07:00
9e61247cfe
Add service command
This change creates the `floaty service` command with two valid
arguemnts: types and examples

- `floaty service types` prints out a list of valid types for use in a
  service definition in ~/.vmfloaty.yml
- `floaty service examples` prints out example configuration files for
  both a single service and multiple services.
2020-09-16 12:55:42 -04:00
Samuel Beaulieu
4b7cbbf0e5 fix abs spec tests 2020-09-15 11:21:58 -05:00
mattkirby
512adb4af1
Merge pull request #94 from puppetlabs/fix-abs-vmpooler
ABS enables fallback to vmpooler for some scenarios
2020-09-11 13:27:02 -07:00
Samuel Beaulieu
c65b72d86b Document the ~/.vmfloaty content for ABS usage 2020-09-11 13:38:33 -05:00
Samuel Beaulieu
dbf6c54173 The main goal for this PR is to enable using all of the vmfloaty scenarios when
the service is set to ABS. Since ABS does not implement all the services, it
fallsback on vmpooler when needed (example increasing the lifetime value)

- Validating JSON can be parsed before parsing, as it used to throw uncaught errors
- self.list_active is returning hosts for both nspooler and vmpooler but was returning
job_ids for abs. Now standardized the returned values for use in the "modify" cli,
and created a separate list_active_job_ids for the cases where a job_id list is expected

- added a way to change the service in place for methods that do not make sense
for ABS, and instead fallback on using vmpooler in those cases:
1. summary
2. modify
3. snapshot
4. revert
5. disk
For those methods in the class itself, raising a NoMethodError, in case it is used directly

- query now returns the queue info. For information on VMs, users should use --service vmpooler
- pretty_print_hosts (used in list, and delete scenarios) will now print the job_id
ABS information and indent each VM within and print it's metadata. Useful for
knowing the running time and extending it.

- added a new utility method to get the config for the vmpooler service, used for the fallback
- added a passthrough for the vmpooler token to use when running ABS. This enables
vmpooler to track the VMs used by each token (user). Also aligns the list between
both ABS and vmpooler. Fixes the bit-bar issue where the VMs do not appear when created
via ABS
2020-09-11 13:38:24 -05:00
5a0640c515
Merge pull request #91 from nwolfe/maint/remove-warning
(maint) Remove warning about missing configuration file
2020-09-03 07:06:05 -04:00
Nate Wolfe
cb1ea8247b
WIP (DIO-911) Include job_id in ABS --json output
Prior to this commit, when using --service=abs and --json the job_id
value is not included in the JSON blob, which makes it difficult for
calling code to grab both the job_id and the hosts without text
processing the log messages printed to standard error.
2020-08-24 16:07:49 -07:00
Nate Wolfe
9a44cc480e
(maint) Add --user CLI option on delete and list subcommands
To support usage without a configuration file.
2020-08-24 12:11:06 -07:00
Nate Wolfe
8341a5f45d
(maint) Mention configuration file in CLI help text description
So users can be aware of the configuration file since it will no
longer be emitted as a warning during CLI invocation.
2020-08-21 15:45:10 -07:00
Nate Wolfe
04c73a970d
(maint) Remove warning about missing configuration file
Previously the CLI would emit a warning about missing configuration
file, but a configuration file does not appear to be a requirement, at
least based on the documentation and the fact that everything can be
specified directly on the command line.

This commit simply removes the warning.

Users will no longer get a warning when doing things like `floaty
--version`, `floaty help`, and any other subcommand.
2020-08-21 14:38:04 -07:00
mattkirby
b9bb08e3c0
Merge pull request #89 from puppetlabs/release_0111
Update version for 0.11.1 release
2020-08-20 14:07:45 -07:00
kirby@puppetlabs.com
3f10e987b4 Update version for 0.11.1 release 2020-08-20 14:06:08 -07:00
mattkirby
d3d6ebeb1a
Merge pull request #86 from McdonaldSeanp/json_output
Json output for delete/list + better ABS error handling
2020-08-20 14:02:53 -07:00
mattkirby
13fee7ef13
Merge pull request #87 from jarretlavallee/fix/master/nspooler
Fix the argument list for nonstandardpooler
2020-08-20 14:01:38 -07:00
mattkirby
5436f54258
Merge pull request #88 from cthorn42/maint/master/no_config_file_non_vmpooler_service_fix
(maint) Fix using ABS service without a .vmfloaty.yml file
2020-08-20 14:00:48 -07:00
Christopher Thorn
701534872d (maint) Fix using ABS service without a .vmfloaty.yml file
If you do not use a config file, and define everything on the command line,
previously the service always defaulted to vmpooler. Even if you requested ABS.
This PR fixes that, now we'll see if the config has no service defined and we
defined on the command line a service, we'll use that service.
2020-08-20 11:37:20 -07:00
Jarret Lavallee
e684933525 (maint) Fix the argument list for nonstandardpooler
Prior to this commit, 7 arguments would be sent into the `retrieve`
method in nonstandardpooler. There were only 6 arguments in the method
definition so an error would be thrown. This commit adds the `ondemand`
argument to the `retrieve` method, but does not utilize it.
2020-08-13 12:37:09 -06:00
Sean P. McDonald
1eaa80706c
(maint) Fail in ABS when queue requests don't return 200/202
When ABS returns an http status code other than 200 or 202, the ABS code
should fail
2020-08-12 16:14:02 -05:00
Sean P. McDonald
d22e2b4194
(new-feature) Add json option to list/delete command
This commit adds an option `--json` to the list and delete commands. When the
json flag is used the commands will print output as formatted JSON rather than
pretty printed strings.
2020-08-12 16:13:58 -05:00
b944760c93
Merge pull request #85 from puppetlabs/release_0110
Bump version for 0.11.0 release
2020-08-12 14:54:57 -04:00
kirby@puppetlabs.com
b44b9c6e98 Bump version for 0.11.0 release 2020-08-12 11:47:58 -07:00
mattkirby
3b91d6f582
Merge pull request #84 from austb/print-debug-to-stderr
Print all non-success output to STDERR
2020-08-12 11:34:11 -07:00
Austin Blatt
35faeab6be Move logger instance to class var in FloatyLogger
This moves the instance of the logger class to a class variable in the
`FloatyLogger` class and provides three class methods to log with in the
rest of the project `FloatyLogger.info`, `FloatyLogger.warn`, and
`FloatyLogger.error`.
2020-08-12 09:57:18 -07:00
barriserloth
8ec90007ca Use Logger for non-result output
This adds the Logger class for log output, all of which goes to STDERR,
and uses puts to send only the command result to STDOUT.
2020-08-11 12:41:22 -07:00
Austin Blatt
4755343df2 Print all non-success output to STDERR 2020-08-10 16:20:25 -07:00
mattkirby
d843066a56
Merge pull request #83 from puppetlabs/maint_update_travis
Update travis.yml
2020-08-06 12:22:59 -07:00
Belen Bustamante
309929f8d8 Update travis.yml 2020-08-06 11:50:43 -07:00
John O'Connor
13b6bc31f3
Merge pull request #82 from puppetlabs/version_010
Bump version to 0.10.0 for release
2020-08-05 11:42:39 +01:00
kirby@puppetlabs.com
a245316252 Bump version to 0.10.0 for release 2020-08-04 15:16:19 -07:00
mattkirby
d1fc70f96f
Merge pull request #67 from mattkirby/pooler_158
Add support for vmpooler on demand provisioning
2020-08-04 15:03:46 -07:00
kirby@puppetlabs.com
7b0a7679da Add Dockerfile to make it easier to test changes to vmfloaty by using Docker 2020-08-04 11:24:11 -07:00
kirby@puppetlabs.com
05a133fc72 Add support for ondemand provisioning to vmfloaty
This commit adds support for provisioning instances on demand with
vmpooler. Additionally, this change adds a capability to detect on
demand pools available in ABS. Without this change vmfloaty does not
support provisioning instances via the vmpooler ondemand endpoints.
2020-08-04 11:24:11 -07:00
482d4328d1
Merge pull request #73 from puppetlabs/dependabot/bundler/commander-gte-4.4.3-and-lt-4.6.0
Update commander requirement from ~> 4.4.3 to >= 4.4.3, < 4.6.0
2020-07-07 10:48:49 -04:00
b873e4a099
Merge pull request #78 from puppetlabs/dependabot/add-v2-config-file
Update Dependabot config file
2020-07-07 10:47:16 -04:00
dependabot-preview[bot]
980793c979
Update Dependabot config file 2020-07-07 14:43:17 +00:00
dependabot-preview[bot]
d0f7452998
Update commander requirement from ~> 4.4.3 to >= 4.4.3, < 4.6.0
Updates the requirements on [commander](https://github.com/commander-rb/commander) to permit the latest version.
- [Release notes](https://github.com/commander-rb/commander/releases)
- [Changelog](https://github.com/commander-rb/commander/blob/master/History.rdoc)
- [Commits](https://github.com/commander-rb/commander/compare/v4.4.3...v4.5.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
2020-07-07 14:35:22 +00:00
f357607187
Merge pull request #75 from puppetlabs/dependabot/bundler/rspec-tw-3.9.0
Update rspec requirement from ~> 3.5.0 to ~> 3.9.0
2020-07-07 10:34:09 -04:00
dependabot-preview[bot]
458ddf192d
Update rspec requirement from ~> 3.5.0 to ~> 3.9.0
Updates the requirements on [rspec](https://github.com/rspec/rspec) to permit the latest version.
- [Release notes](https://github.com/rspec/rspec/releases)
- [Commits](https://github.com/rspec/rspec/compare/v3.5.0...v3.9.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
2020-07-07 14:21:10 +00:00
2ff562cb40
Merge pull request #70 from puppetlabs/tools
Add Dependabot and Coveralls
2020-07-07 10:18:57 -04:00
a6f53c0310
Add Dependabot and Coveralls
This adds Dependabot to keep things updated and Coveralls so we can see
how code coverage changes over time. Coveralls / SimpleCov are
configured so that both the SimpleCov html report and the Coveralls
report are generated. This facilitates easily seeing coverage locally in
addtion to via CI.
2020-07-07 10:17:10 -04:00
498a9ac94f
Merge pull request #71 from puppetlabs/codeowners-fix
Fix formatting of CODEOWNERS
2020-07-07 10:14:21 -04:00
7a969d121e
Fix formatting of CODEOWNERS 2020-07-07 10:12:23 -04:00
Brandon High
2de284f2c0
Merge pull request #69 from puppetlabs/docs-update
Update docs
2020-07-02 11:13:00 -07:00
28fb9290d4
Update docs
This chane updates docs to reflect that ownership has moved to the DIO
team at Puppet. It also includes some formatting changes.
2020-07-02 12:52:32 -04:00
Brandon High
b7a210aa15
Merge pull request #68 from puppetlabs/fixup/edit-maintainers
Remove old maintainer note
2020-07-01 14:58:39 -07:00
Brian Cain
e942e7534d
Remove rubocop 2020-07-01 14:55:32 -07:00
Brian Cain
d4bd4f5163
Comment out rubocop rules that aren't helpful 2020-07-01 14:52:39 -07:00
Brian Cain
4ff9b9d0d5
Remove old maintainer note 2020-07-01 14:41:57 -07:00