SUBMARINE-576. Always return empty list when listing environments

[manirajv06]

What is this PR for?

https://issues.apache.org/jira/browse/SUBMARINE-576

What type of PR is it?

Feature

Todos

• - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/SUBMARINE-576

How should this be tested?

• First time? Setup Travis CI as described on https://submarine.apache.org/contribution/contributions.html#continuous-integration
• Strongly recommended: add automated unit tests for any new or changed behavior
• Outline any manual steps to test the PR here.

Screenshots (if appropriate)

Questions:

• Does the licenses files need update? Yes/No
• Is there breaking changes for older versions? Yes/No
• Does this needs documentation? Yes/No

[manirajv06]

@pingsutw Can you please review? Environment spec 'name' is not allowed for updates, hence modified corresponding test case as well. In addition, committed few more changes to #348 as well to reflect the same.

[pingsutw]

@manirajv06 Thanks for the contribution.
It looks good to me.
if I understand correctly, we store environment data in the cache currently.
Will we support to store data in DB in other JIRA ticket?

[manirajv06]

@pingsutw Data has been already stored in database (in "environment" table). In-memory cache support has been added mainly from performance perspective to save the db calls. Can you please commit the code as @lowc1012 might be expecting this change?

[xunliu]

LGTM, @manirajv06 Thank you contribution this feature.
Will merge if no more comments.

[manirajv06]

@liuxunorg PR #348 commits also has been merged into the master. We will need to remove those commits . Only this commit e7476c7 should have gone into the master. Can you please check? Am not sure whether this confusion would have started with the way I made commits from my local copy into my github fork.