Artem Yakimenko

Results 29 issues of Artem Yakimenko

If fog file specifies just a test namespace, disk cannot be properly cleaned up: ``` test: google_project: my-proj google_json_key_location: /my-project-key.json ``` Since disk is not actually instantiated, it doesn't have...

If image family does not exist we trace back: ``` undefined method `self_link' for nil:NilClass /Users/temikus/.vagrant.d/gems/2.4.3/gems/vagrant-google-2.0.0/lib/vagrant-google/action/run_instance.rb:99:in `call' /opt/vagrant/embedded/gems/2.0.3/gems/vagrant-2.0.3/lib/vagrant/action/warden.rb:34:in `call' /Users/temikus/.vagrant.d/gems/2.4.3/gems/vagrant-google-2.0.0/lib/vagrant-google/action/warn_ssh_keys.rb:28:in `call' /opt/vagrant/embedded/gems/2.0.3/gems/vagrant-2.0.3/lib/vagrant/action/warden.rb:34:in `call' /Users/temikus/.vagrant.d/gems/2.4.3/gems/vagrant-google-2.0.0/lib/vagrant-google/action/warn_networks.rb:28:in `call' /opt/vagrant/embedded/gems/2.0.3/gems/vagrant-2.0.3/lib/vagrant/action/warden.rb:34:in `call' /opt/vagrant/embedded/gems/2.0.3/gems/vagrant-2.0.3/lib/vagrant/action/builtin/synced_folders.rb:87:in `call' /opt/vagrant/embedded/gems/2.0.3/gems/vagrant-2.0.3/lib/vagrant/action/warden.rb:34:in...

bug

Would be nice to have to achieve parity with vagrant-aws: https://github.com/mitchellh/vagrant-aws/pull/316

enhancement

In theory `fog` does that for us for free, however, we need to make sure that the key always contains the service account or check it so users won't get...

enhancement

See #127 - we need to either fail fast or provide a hint that the keys may not be set up correctly. Ideally, we would need to verify the metadata...

enhancement

In `run_instance` logic, disk cleanup should implement exponential backoff. Probably by inserting retriable somwhere over here: ``` def cleanup_disk(disk_name, env) zone = env[:machine].provider_config.zone autodelete_disk = env[:machine].provider_config.get_zone_config(zone).autodelete_disk if autodelete_disk disk =...

enhancement

We probably should add the default bunch: ``` "https://www.googleapis.com/auth/devstorage.read_only", "https://www.googleapis.com/auth/logging.write", "https://www.googleapis.com/auth/monitoring.write", "https://www.googleapis.com/auth/servicecontrol", "https://www.googleapis.com/auth/service.management.readonly", "https://www.googleapis.com/auth/trace.append" ``` , if `scopes` is empty. If it's set to an empty array, however, we should...

enhancement

Instance groups are not currently deleted after provisioning. This needs to happen on instance termination. A new handler should be added and put into the destroy action list.

enhancement

Currently we don't support passing credentials to ts-bridge directly. It's all done through `google-cloud-go` library support, so classic application default credentials. Design: - App would support additional 2 methods of...