feat: custom image name support for Docker builds

[dtsulik]

Which issue(s) does this change fix?

N/A

Why is this change necessary?

Currently the Docker build takes the logical resource ID from the template as the name for the image that gets built and tagged. We are able to customize the tag, but not the name. This forces the same restrictions on the image name allowed characters as the resource ID, meaning we cannot use dashes.

How does it address the issue?

Allows specifying custom image name.

What side effects does this change have?

None.

NOTE: make pr sometimes passes but sometimes fails with:

FAIL Required test coverage of 94% not reached. Total coverage: 94.00%

This seems to be rounding error, my lines 433-434 are covered. I can add some random tests to cover lines reported like 406 and 442 for example.

Update: added some easy tests covering exceptions.

Required test coverage of 94% reached. Total coverage: 94.00%

Mandatory Checklist

PRs will only be reviewed after checklist is complete

• Review the generative AI contribution guidelines
• Add input/output type hints to new functions/methods
• Write design document if needed (Do I need to write a design document?)
• Write/update unit tests
• Write/update integration tests
• Write/update functional tests if needed
• make pr passes
• make update-reproducible-reqs if dependencies were changed
• Write documentation

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[seshubaws]

Hello @dtsulik is this PR ready for review? Please take it out of draft mode if so

[dtsulik]

Hello @dtsulik is this PR ready for review? Please take it out of draft mode if so

Hi @seshubaws let me add a test covering this feature on weekend and I will move it forward.