updated execution environment id field to name

[hamersaw]

Tracking issue

NA

Why are the changes needed?

Flyte uniformly uses the term id to represent a unique identifer and the term name to represent a human-readable name. In this protobuf spec, the unique environment ID will be an aggregation of project, domain, name, and version.

What changes were proposed in this pull request?

^^^

How was this patch tested?

Locally.

Setup process

Screenshots

Check all the applicable boxes

• I updated the documentation accordingly.
• All new and existing tests passed.
• All commits are signed-off.

Related PRs

NA

Docs link

NA

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.99%. Comparing base (5cc7f58) to head (f7cc87d).
Report is 117 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #5514   +/-   ##
=======================================
  Coverage   60.99%   60.99%           
=======================================
  Files         794      794           
  Lines       51475    51475           
=======================================
  Hits        31398    31398           
  Misses      17185    17185           
  Partials     2892     2892           

Flag	Coverage Δ
unittests-datacatalog	69.31% <ø> (ø)
unittests-flyteadmin	58.73% <ø> (ø)
unittests-flytecopilot	17.79% <ø> (ø)
unittests-flytectl	68.03% <ø> (ø)
unittests-flyteidl	79.04% <ø> (ø)
unittests-flyteplugins	61.85% <ø> (ø)
unittests-flytepropeller	57.30% <ø> (ø)
unittests-flytestdlib	65.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.