enhance cognito deployment automation scripts

[surajkota]

Description of your changes:

• Enhance cognito automation script to include ingress and alb controller configuration
• Removed CNAME entry for *.default.example.com since it is not needed. From what I understand we only need kubeflow.platform.example.com and not even *.platform.example.com since all the apps are paths under the domain name. e.g. platform.example.com/pipelines
• Add badges to the README for customers to easily identify which Kubeflow version is under development. Open to other suggestions
  • Updated PR template to include license attribution
• Resolves: Should users be allowed to sign themselves up using Cognito hosted UI #122
  • made email as a option to use as username instead of having email only or username only
  • Disable self signup - only admins can create users
  • Make email a required attribute and add corresponding documentation
• updated e2e test readme
• fixed logging issues in e2e tests using the pytest.ini config. It was not printing logging logs and also some of the print statements
• fixed issue with ingress and load balancer cleanup in the cognito e2e test. deleting ingress is clean way to remove the ingress managed load balancer. Using cognito_resource_cleanup was leaving around some resources

Testing:

• e2e tests pass
• Manually tested cognito automation README

[ryansteakley]

Great work, just a few questions. Allowed only the admin to create accounts seems fine to me, seems like it also prevents the url from leaking and having someone sign up themselves thousands of times maliciously. However what is the reasoning for requiring the email field however?

[surajkota]

@ryansteakley good catch. I have updated the README with missing documentation from automation and related to email field.