Address comments on !47334

[dougbu]

received a few comments in !47334 reviews to address in a follow-up PR:

1. change linux-helixbot-is-1000.sh to silently succeed if there's no existing user w/ UID 1000. it'll error after we merge !47334
2. remove duplicates between devtime_python_requirements.txt and runtime_python_requirements.txt. we don't use devtime_python_requirements.txt on its own
3. improve linux-helixbot-is-1000/validate.sh error message to make expected UID clear
4. correct a typo / misspelling in a dockerhelper.py comment

will also make a minor change in dockerhelper.py to do a more thorough job checking for an existing $HELIX_SCRIPTS_ROOT entry in $PYTHONPATH and update pipeline YAML to avoid potential errors and warnings

Release Note Category

• Feature changes/additions
• Bug fixes
• Internal Infrastructure Improvements

Release Note Description

touched up earlier work done in !47334. none of this should impact customers at all

[dougbu]

full list of changes go a bit further:

• change linux-helixbot-is-1000.sh to normally succeed if there's no existing user w/ UID 1000
  • still create helixbot user in this case for consistency
• remove devtime_python_requirements.txt / runtime_python_requirements.txt duplicates
  • go beyond psutil b/c more duplicates existed
  • reorder our python -m pip install commands to use devtime_python_requirements second
• update dockerhelper.py to use grep -E and set $PYTHONPATH unconditionally
  • match regular expressions correctly
  • $HELIX_SCRIPT_ROOT will frequently have newer helix-scripts/ version than venv
• make Validate_EndToEnd log file upload conditional in PR builds
  • avoid a potential build error when nothing is tested
• add $(buildArch) to Validate_EndToEnd log artifact name in CI builds
  • also checkout self in post-job validation
  • avoid potential build errors and warnings
• nits:
  • improve linux-helixbot-is-1000/validate.sh error message to make expected UID clear
  • correct a typo / misspelling in a dockerhelper.py comment
  • add azure-devops requirement to helix.base.yaml
  • move twine requirement to devtime_python_requirements.txt
    • no need for a separate installation step in our pipelines
  • sort requirements files
    • no real changes in runtime_python_requirements.txt
  • remove outdated virtualenv and wheel requirements
  • mark Validate as a validation job

[dougbu]

fixed in !47851

[dougbu]

additional fixes in !47826, !47827, and !47842 were done first due to CI build breaks