[Provisioner] remove stale port code in new provisioner

[cblmemo]

This PR removes some stale code in the current new provisioner implementation.

Tested (run the relevant ones):

• Code formatting: bash format.sh
• Any manual or new tests for this PR (please specify below)
  • sky launch --cloud aws --cpus 4 --ports 23783 'python -m http.server 23783' and curl the endpoint
• All smoke tests: pytest tests/test_smoke.py
• Relevant individual smoke tests: pytest tests/test_smoke.py::test_fill_in_the_name
• Backward compatibility tests: bash tests/backward_comaptibility_tests.sh

[Michaelvll]

Good catch @cblmemo! Will there be any backward compatibility related issue if a user already created a cluster on AWS with ports opened?

[cblmemo]

Good catch @cblmemo! Will there be any backward compatibility related issue if a user already created a cluster on AWS with ports opened?

No, because

• We clean up ports by just deleting the whole SG in AWS, no matter what ports are specified;
• At the time the new provisioner PR is merged, there is no ports section in the ray yaml, so config.provider_config.get('ports', []) will always return [] in the deleted code inside this PR.