Updates to apiserver swagger ui

[z103cb]

Why are these changes needed?

The swagger ui presented by the api-server was the stock pet store version, this PR rectifies this situation and displays the swagger for KubeRay API Server.

Related issue number

Closes #1343

Checks

• I've made sure the tests are passing.
• Testing Strategy
  • Unit tests
  • Manual tests
  • This PR is not tested :(

[anishasthana]

Have one immediate change. Will run through the instructions later today to confirm they work

[tedhtchang]

@z103cb For running local apiserver, should we have something like this in main.go so we can do KUBERAY_APISERVER_SWAGGER=local make run

	if os.Getenv("KUBERAY_APISERVER_SWAGGER") == "local" {
		p = path.Join("../proto/swagger/", p)
	} else {
		p = path.Join("/workspace/proto/swagger/", p)
	}

Then we would not need to manually change the file for local service.

[z103cb]

I was thinking about this issue, and I wasn't sure if I should include a fix in this PR. But since you've asked, you have convinced me that we should probably do it. I will push a commit along the lines you have suggested

[z103cb]

Take a look at the latest commit (b88d4bd)

[kevin85421]

@z103cb would you mind rebasing with the master branch? Thanks!

[tedhtchang]

LGTM. I verified the following:

1. Ran make run and opened http://localhost:8888/swagger-ui/ in browser.
2. Run make docker-image operator-image cluster load-image load-operator-image deploy deploy-operator and opened http://localhost:31888/swagger-ui/ in browser.
   • some hiccup. The deploy and deplpoy-operator would not work unless I --set image.repository=localhost/kuberay/apiserver and --set image.repository=localhost/kuberay/operator explicitly for the helm commands in the makefile. I will track in this issue.
3. Called a few APIs using swagger-ui and had no error
   • Create ComputeTemplate http://localhost:31888/swagger-ui/?urls.primaryName=ComputeTemplate%20Service#/ImageTemplateService/ImageTemplateService_CreateImageTemplate
     
   • Create a RayCluster
     
   • Create a RayJob
     

[z103cb]

The git history got way too crazy. Closing this PR in favour of: #1410