Task/WP-832: switching to TMS for credentials (do not merge)

[chandra-tacc]

Note

Redo of PR: #1051

Overview

Core Portal changes for TMS Keys migration process.
Adjusted Scenarios:

1. Onboarding:

   • System supporting TMS - credentials are created with createTmsKeys flag set.
   • System not supporting TMS - credentials are created with portal generated public and private key.

2. Data files

   • when listing files fails, pushkeySystem is sent back to the client.
   • for tms based systems, a credentials is created preemptively to check if that fixes listing.
   • UI will not show push modal for tms key based system. Instead it shows an error status asking to contact support because this is an api error.

3. App retrieval

   • when listing files fails, pushkeySystem is sent back to the client.
   • for tms based systems, a credentials is created preemptively to check if that fixes listing.
   • UI will not show push modal for tms key based system. Instead it shows an error status asking to contact support because this is an api error.

Related

• WP-832

Changes

1. Add additonal createUserCredentials to use createTmsKeys set to true
2. Stop generating and using private and public keys
3. On file listing failures, create credential.

Testing

Look for staging tenant DEV.CEP.STAGING in https://stache.utexas.edu/entry/bedc97190d3a907cb44488785440595c
and setup cep.test and login
See scenarios in Overview section for testing

1. Remove credentials.

Removing credential to test onboarding

from django.contrib.auth import get_user_model
user = get_user_model().objects.get(username='')
tapis = user.tapis_oauth.client
tapis.systems.removeUserCredential(systemId='cloud.data', userName='')

2. reset onboarding for system setup. onboarding should work
3. Remove credentials again
4. go to data files and check that credentials should be created again. No push modal is shown, but the listing fails due to other reasons(There was a problem accessing this file system.).
5. Remove credentials again
6. go to app and see that the credentials work but wma-exec-01 host related message shows up.(which is expected)

[codecov]

Codecov Report

Attention: Patch coverage is 47.45763% with 31 lines in your changes missing coverage. Please review.

Project coverage is 70.36%. Comparing base (c210828) to head (dc0bbb6).

Files with missing lines	Patch %	Lines
server/portal/apps/workspace/api/utils.py	37.50%	10 Missing ⚠️
...r/portal/apps/onboarding/steps/system_access_v3.py	30.76%	9 Missing ⚠️
...onents/DataFiles/DataFilesTable/DataFilesTable.jsx	60.00%	8 Missing ⚠️
server/portal/apps/workspace/api/views.py	50.00%	3 Missing ⚠️
server/portal/apps/accounts/api/views/systems.py	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1070      +/-   ##
==========================================
+ Coverage   70.34%   70.36%   +0.01%     
==========================================
  Files         538      538              
  Lines       33328    33318      -10     
  Branches     2953     2953              
==========================================
- Hits        23446    23445       -1     
+ Misses       9684     9675       -9     
  Partials      198      198              

Flag	Coverage Δ
javascript	72.53% <63.63%> (-0.01%)	⬇️
unittests	61.10% <37.83%> (+0.09%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...nt/src/components/Applications/AppForm/AppForm.jsx	87.64% <100.00%> (-0.04%)	⬇️
...orkspace_operations/shared_workspace_operations.py	94.69% <ø> (ø)
server/portal/apps/accounts/api/views/systems.py	60.86% <50.00%> (ø)
server/portal/apps/workspace/api/views.py	57.24% <50.00%> (+2.21%)	⬆️
...onents/DataFiles/DataFilesTable/DataFilesTable.jsx	85.97% <60.00%> (-0.04%)	⬇️
...r/portal/apps/onboarding/steps/system_access_v3.py	29.50% <30.76%> (+0.52%)	⬆️
server/portal/apps/workspace/api/utils.py	51.72% <37.50%> (-17.51%)	⬇️

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.