-
Notifications
You must be signed in to change notification settings - Fork 21
Security and Maintenance
Please refer to Terra UI Maintenance regarding the maintenance rotation and the related responsibilities.
Every three months, the service account key used by CircleCI to deploy Terra UI to prod
needs to be rotated for security purposes, by the maintaining team. The steps below describe the process:
- Having logged in with your
@firecloud.org
account, go to Google Cloud Console's CircleCI Service Account Keys page of theIAM & Admin
section for projectBVDP Saturn Prod
. If you receive a permissions error, ask on#terra-ui
Slack channel to be grantedEditor
role on Google projectBVDP Saturn Prod
. - Click on the
ADD KEY
button. - Select
Create new key
. - Make sure
JSON
is selected as key type, and click onCREATE
. - Make sure the
.json
file was downloaded onto your computer.
- Create a new browser tab, and go to CircleCI's Environment Variables page. If you can't view this page, ask on
#terra-ui
Slack channel to be added to GitHub team SaturnDeployers. - Delete the environment variable named
GCP_SA_KEY
by clicking onX
to the right of it. - Click on the button
Add Environment Variable
and fill outName
andValue
on the dialog that pops up.-
Name
should beGCP_SA_KEY
-
Value
is supposed to be the text content of the.json
file you had downloaded.
-
- Manually redeploy to
prod
to verify the new SA key works by following the instructionshere. - Observe on the saturn-ui-prod-deploy CircleCI page that the job has succeeded.
- Go to https://app.terra.bio/#workspaces; click on the hamburger menu on the upper left; click on the link to the right of
Built on:
on the buttom; verify that the latest GitHub commit is on the new tab that pops up.
- Now you can disable the old key(s). It's not possible to disable a SA key on the GCP console. You will need to use
gcloud
on your local system (e.g.Terminal
on MacOS). After making sure you're authenticated with your@firecloud.org
account on your local system, rungcloud iam service-accounts keys disable <service_account_key_name> [email protected] --project bvdp-saturn-prod
. Substitute into that command, the key obtained fromKey
column on the GCP console page for SA keys. More details on disabling service account keys can be found here. If you getERROR: (gcloud.iam.service-accounts.keys) Invalid choice: 'disable'.
, try upgrading your SDK viagcloud components update
. Verify on the GCP console page that the key'sStatus
is nowDisabled
(as opposed toActive
). - To be sure, rerun the prod deploy job and verify that everything still works.
- Finally, delete the old key using the trash can icon in the UI. This is an important step to make sure that the key being retired can’t accidentally be re-enabled.
- Set a Slack reminder on
#terra-ui
channel for the next maintaining team by typing something like/remind :alert_red_light: Reminder to rotate the service account key for prod deploys by following these (https://github.com/DataBiosphere/terra-ui/wiki/Security-and-Maintenance#service-account-key-rotation) instructions in 85 days
.
- Add the new
.json
key file (that you downloaded at the beginning) toVault
(e.g.vault write secret/dsp/service-accts/bvdp-saturn-prod-sa-2021-12-14.json @/Users/<your username>/Downloads/<new sa file name>.json
).
In order to see the Dependabot alerts, your GitHub handle must be added to the GitHub team TerraUISecurityAlertAccess. You may also want to check your personal notifications for any filtering.
A lot of the Dependabot alerts can be safely dismissed. These are all valid reasons for dismissing the alerts:
- The alert is for a library version that is only used by integration tests, development tools or a library that is only used in the packaging of our code (for example, packaging CSS), and thus not exposed to users. To figure this out, use
yarn why -R LIBRARY_NAME
. For instance, if you see a root node name in theyarn why
output that readsterra-integration-tests@workspace:integration-tests
, we know that the library is used byintegration-tests
.yarn why -R --json LIBRARY_NAME | jq '.value'
can also be helpful in seeing the root packages at a glance without the entire tree. - The alert is incorrect.
- The alert is based on
integration-tests/yarn.lock
(as opposed toyarn.lock
at the root of the repo) and the package is not listed as a direct dependency onintegration-tests/package.json
. Dependabot has a bug where it may continue alerting based on files that no longer exists in a repo. This has happened for this repo withintegration-tests/yarn.lock
that had been removed. - By examining the
yarn.lock
file and usingyarn why LIBRARY_NAME
it is clear that we actually don't use the library version that the alert is warning about.
- The alert is based on
If the library version does need to be upgraded because it is exposed to users in Terra's UI, these are the recommended steps:
- If the library is one of our direct dependencies, update the version in
package.json
and generate an updatedyarn.lock
file as detailed below. - If the library is a transitive dependency, delete entries in
yarn.lock
file that reference the library with the security update . - Run
yarn install
to generate the new content inyarn.lock
. Verify the library versions satisfy the security update. - Run
yarn dedupe
to remove duplicated content from theyarn.lock
file. - Create a PR with the modified
yarn.lock
file. - Extra points for updating our direct dependencies (
package.json
files) that introduce the transitive dependency to their most recent versions 🥇 😃
Compliance timeframe for addressing security alerts can be found in Terra Vulnerability Management Plan.
Prod deploy time has been chosen as 11am Eastern Time
. See this PR for start of DST and this PR for end of DST.
Terra UI is a Google App Engine application. Every time CircleCI deploys Terra UI, it creates a new version. App Engine allows a maximum of 210 versions of any app. Therefore, we must periodically clean up old versions.
In prod, there is an automated script to handle the cleanup. In non-prod (dev, alpha, staging, and perf) there is a manual script which should be run instead:
- Ensure you have
jq
installed beforehand - Ensure that you are logged in to the correct
gcloud
account that has app engine deletion permission -
git clone
the Terra UI repository or download the script from https://github.com/DataBiosphere/terra-ui/tree/dev/scripts - Run the
delete-old-app-engine-versions.sh
script and specify the environment to delete, which will prompt you to confirm deletion of old app engine resources in that environment
You're done! Thank you for your service!
Terra UI Wiki.
- Getting Started
- Contributor Guide
- Intro to UI Development
- Troubleshooting Build Failures
- Editor Configuration
- BEEs
- Pull Requests
- How to Find a PR Site
- Feature Flags
- Mixpanel
- Cobranding and White-Label Sites
- Using Terra UI packages in other projects