Feature/support skip create image

[newbieJerry]

Overview of the Issue

Have a flag that allows user to skip the image creation, this can be useful for test builds and rapid validation.

Reproduction Steps

1. create a packer script with "skip_create_image":"true"
2. run the packer build

Plugin and Packer Version

1. Packer v1.8.3
2. packer-plugin-alicloud v1.0.5

Simplified Packer Buildfile

{
   "variables": {
     "access_key": "{{env `LTAI5tCzUL*********Z1iFQ`}}",
     "secret_key": "{{env `fwVAFD6UWC*********wgmilEhG6Td`}}"
   },
   "builders": [{
     "type":"alicloud-ecs",
     "access_key":"{{user `access_key`}}",
     "secret_key":"{{user `secret_key`}}",
     "region":"cn-beijing",
     "image_name":"packer_basic_2",
     "source_image":"ubuntu_20_04_x64_20G_alibase_20220824.vhd",
     "associate_public_ip_address":"true",
     "skip_create_image":"true",
     "ssh_username":"root",
     "instance_type":"ecs.n1.tiny",
     "internet_charge_type":"PayByTraffic",
     "io_optimized":"true"
   }],
   "provisioners": [{
     "type": "shell",
     "inline": ["echo foo"]
   }]
 }

Operating system and Environment details

macOS Monterey 12.6

Log Fragments and crash.log files

alicloud-ecs: Starting instance: i-2ze1xwqv0qz5fn3xpctk
==> alicloud-ecs: Using SSH communicator to connect: 39.105.44.173
==> alicloud-ecs: Waiting for SSH to become available...
==> alicloud-ecs: Connected to SSH!
==> alicloud-ecs: Provisioning with shell script: /var/folders/70/qctp4fb506l7yr9jxj2ctqx80000gp/T/packer-shell2504466809
alicloud-ecs: foo
==> alicloud-ecs: Stopping instance: i-2ze1xwqv0qz5fn3xpctk
==> alicloud-ecs: Waiting instance stopped: i-2ze1xwqv0qz5fn3xpctk
==> alicloud-ecs: Image build stopped because: you set skip_create_image true
alicloud-ecs: Detach keypair packer_6386b559-8cbe-66d6-8d95-e38bce34772c from instance: i-2ze1xwqv0qz5fn3xpctk
==> alicloud-ecs: Deleting EIP because of cancellation or error...
==> alicloud-ecs: Deleting instance because of cancellation or error...
==> alicloud-ecs: Deleting security group because of cancellation or error...

[hashicorp-cla]

All committers have signed the CLA.

[newbieJerry]

@nywilken Sorry for my novice behavior, I resubmit a PR to sign the CLA, and change the code according to your suggestion here, please review my code, thx 😄

[newbieJerry]

@nywilken, would you please review this code and get this PR merged?

[newbieJerry]

@lbajolet-hashicorp Could I trouble you to review this PR 😄？

[lbajolet-hashicorp]

Hi @newbieJerry, sorry for the long wait, reviewing this now.

Aside from my comments, the code looks good to me, when you've addressed them, we can merge this PR.

Pre-approving for merge later.

[lbajolet-hashicorp]

This looks redundant with the .idea below, is it voluntary?

[newbieJerry]

/.idea/ is deleted in the latest commit

[lbajolet-hashicorp]

I don't see this constant used anywhere, is this still relevant?

[newbieJerry]

Sorry, redundant constants are defined, which have been deleted in the latest commit.

[nywilken]

@newbieJerry @lbajolet-hashicorp thanks for pushing this forward. This looks good to me as well. I added a commit to address the conflict with the .gitignore file. But this is good to go.