From be5e3d1d1d9374c4587a0464d908aa6f570eec35 Mon Sep 17 00:00:00 2001 From: Ben Zhang Date: Sat, 17 Aug 2024 15:45:00 -0700 Subject: [PATCH] Add documentation about WATcloud guidelines (#2943) This PR documents a set of guidelines we want WATcloud members to follow. --- .../watcloud/development-manual.mdx | 29 ++++ .../community-docs/watcloud/guidelines.mdx | 163 ++++++++++++++++++ .../community-docs/watcloud/observability.mdx | 112 ++++++++++++ pages/get-involved/join.mdx | 12 +- 4 files changed, 313 insertions(+), 3 deletions(-) create mode 100644 pages/docs/community-docs/watcloud/guidelines.mdx create mode 100644 pages/docs/community-docs/watcloud/observability.mdx diff --git a/pages/docs/community-docs/watcloud/development-manual.mdx b/pages/docs/community-docs/watcloud/development-manual.mdx index 5523cbd..c5aef3d 100644 --- a/pages/docs/community-docs/watcloud/development-manual.mdx +++ b/pages/docs/community-docs/watcloud/development-manual.mdx @@ -21,6 +21,28 @@ We use the following terminology - **Provisioning**: The process of setting up a service. For example, setting up a VM. - **Provisioner**: A tool that is used to provision a service. This can be low-level tools like Ansible or Terraform, or high-level tools like our GitHub provisioner and our Google Workspace provisioner. +## General Guidelines + +- Read and understand the [WATcloud Guidelines](./guidelines) before starting development. +- Pull-request early and often. We have a lot of safe guards and automation in place to help you follow best practices. + When CI is run, look out for automated comments on and pull requests against your PR! +- When writing commit messages and pull requests, start with a title that describes the change in imperative mood (e.g. "Add", "Fix", "Update")[^commit-message-convention], + followed by more information in the body of the message. Use [linking keywords][linking-keywords] like `Resolves #` to automatically close issues. + For example: + + ```markdown + Create status-page Sentry project + + This commit introduces a new Sentry project for the [status page][sp]. + + Resolves #123 + + [sp]: https://github.com/WATonomous/status + ``` + +[^commit-message-convention]: Derived from https://github.com/joelparkerhenderson/git-commit-message/blob/d5bcb65e263217bfe47d898c69f9c6c0dfd6d413/README.md +[linking-keywords]: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue + ## Getting Started Many provisioners require access to the cluster network. @@ -353,3 +375,10 @@ Now, we can run the provisioner as usual. Terraform will use the local provider #### References - https://discuss.hashicorp.com/t/development-overrides-for-providers-under-development/18888/2 - https://developer.hashicorp.com/terraform/cli/config/config-file#development-overrides-for-provider-developers + +{ +// Separate footnotes from the main content +} +import { Separator } from "@/components/ui/separator" + + diff --git a/pages/docs/community-docs/watcloud/guidelines.mdx b/pages/docs/community-docs/watcloud/guidelines.mdx new file mode 100644 index 0000000..3dd8b34 --- /dev/null +++ b/pages/docs/community-docs/watcloud/guidelines.mdx @@ -0,0 +1,163 @@ +# WATcloud Guidelines + +WATcloud is a student-run organization. +The guaranteed turnover of team members as they graduate poses unique challenges in terms of +cluster maintenance and making progress. +Because of this, we've developed a set of guidelines to ensure that the knowledge of the team is +passed down effectively and that the quality of service is maintained. + +This document outlines guidelines that every WATcloud team member should follow, +to maximize the learning of team members and to ensure the highest quality of service for users. + +We divide our guidelines into two categories: technical—to address the maintainability of our systems, +and organizational—to encourage team members to maintain a fast-paced and productive environment amidst +internal and external bureaucratic challenges. + +import { Steps } from 'nextra/components' + +## Technical Guidelines + +The central tenet of our technical guidelines is that everything we do must be trivially maintainable. +This means that with little guidance, a resourceful team member[^resourceful] should be able to understand +and maintain the systems we build and deploy, even if the original author is no longer with the team. + +The following are a set of guidelines to strive for in our technical work: + +[^resourceful]: A resourceful team member is one who is willing to learn and is able to find the information they need to solve a problem. + Most of the time, this boils down to being good at Googling. + + +### Simplify everything + +Complexity is the enemy of maintainability. +We should strive to make everything as simple as possible. +For example, we follow the principle of having a single source of truth[^ssot] by having a single [status page](https://status.watonomous.ca/) +that displays the status of all our services, and a single [provisioner interface](./development-manual#getting-started) for provisioning +different types of resources. + +[^ssot]: Single source of truth (SSOT) is a principle that states that every piece of data should be stored in only one place. + This reduces the likelihood of data inconsistencies and makes it easier to maintain the data. + See [Wikipedia](https://en.wikipedia.org/wiki/Single_source_of_truth) for more information. + +### Only host services that are necessary and easy to maintain + +If we host something, it is inevitable that it will go down once in a while[^best-code-is-no-code]. +We should only host services if the value they provide is worth the maintenance burden. + +Some common red flags that a service is not worth hosting are: +- Static websites (use GitHub Pages) +- Services that have hosted alternatives (commercial services usually have free plans for open-source projects/non-profit organizations, e.g. Sentry, Elastic) +- Services that have unnecessary runtime complexity (e.g. services that call an API to retrieve information, when the information can be bundled with the service during building/deployment) + +[^best-code-is-no-code]: As the saying goes, "The best code is no code at all." + +### Version-control everything + +All code, configuration, and documentation should be version-controlled. +Achieving 100% infrastructure-as-code[^iac] is impossible because we work with physical hardware, but we should strive for it as much as possible. +In cases where manual changes are necessary, they should be documented extensively. + +[^iac]: IaC, [1](https://www.redhat.com/en/topics/automation/what-is-infrastructure-as-code-iac), [2](https://developer.hashicorp.com/terraform/tutorials/aws-get-started/infrastructure-as-code) + +### Make sure everything is reproducible + +If a service goes down, we should be able to bring it back up with minimal effort. +This means that all dependencies should be documented (either as code or as documentation) and that the deployment process should be reasonably automated. + +### Healthcheck everything + +WATcloud has a lot of [observability](./observability) infrastructure in place to detect issues before they become problems. +Every service should be monitored, and alerts should be set up to notify us when something goes wrong. + +### Communicate accurately + +Communication, especially technical, should be accurate, as it can be very misleading if it is incorrect. +For example: +- `MB` (megabytes--$10^6$ bytes or $2^{20}$ bytes, depending on the context) is not the same as `Mb` (megabits--$10^6$ bits or $2^{20}$ bits, depending on the context) or `mb` (even more ambiguous, could be anything). +- Use `MiB` instead of `MB` whenever you know for sure that you are referring to the IEC[^iec] mebibytes ($2^{20}$ bytes) instead of the SI[^si] megabytes ($10^6$ bytes). + +[^iec]: The International Electrotechnical Commission (IEC) defines binary prefixes "kibi", "mebi", "gibi", etc., to refer to powers of 2. +[^si]: The International System of Units (SI) defines prefixes "kilo", "mega", "giga", etc., to refer to powers of 10. However, in computing, these prefixes are often used to refer to powers of 2. + +Brand/organization names should also be capitalized correctly. For example: +- Use `WATcloud` or `watcloud` instead of `WATCloud`, `WatCloud`, or `Watcloud`. +- Use `WATonomous` or `watonomous` instead of `Watonomous`. + + + +## Organizational Guidelines + +Student life is busy, and it can be difficult to balance the demands of school/work, the team, and personal life. +The following are a set of guidelines to help team members maintain a fast-paced and productive environment: + + + + +### Iterate quickly + +Most of WATcloud's projects go through many iterations before they are sufficiently polished for production. +Embrace the concept of rapid prototyping. +Start with a minimum viable product (MVP) and iterate quickly and often based on feedback (from users, team members, or your future self). +This approach allows us to quickly identify what works and what doesn't, and to pivot as necessary. + +### Proactively firefight + +The term "fire" is a metaphor for an urgent issue that needs to be resolved immediately. +The term "firefighting" refers to the act of resolving these issues. +Firefighting is an unavoidable part of maintaining services like ones that WATcloud provides. +Some common causes of fires are: +- Power outages +- Hardware failures +- Software bugs + +During development, follow the [technical guidelines](#technical-guidelines) to reduce the likelihood of fires. +When something goes wrong, it's important to act quickly to minimize the impact on users. +After issues are resolved, we should conduct a post-mortem to understand and document what went wrong and how we can prevent it from happening again. + +### Minimize bureaucracy + +Bureaucracy is the enemy of productivity. +At WATcloud, we strive to minimize bureaucracy as much as possible. +This means that we should avoid unnecessary hierarchies and hiding of information. + +Sometimes, external bureaucracy is unavoidable. +For example, we have to follow the university's process for purchasing using the team's funds. +In these cases, we should try to be as efficient as possible to achieve our goals. +For example, oftentimes, we need to experiment with different products to find the best ones for our needs, +and potentially return the products that don't meet our requirements. +Instead of asking the university to purchase on our behalf, we can purchase the products ourselves and get reimbursed +for the products that we decide to keep. + +### Be scrappy + +Sometimes, we'll need to work within constraints that are not ideal. +In these cases, it's helpful to adopt a "hacker mentality" and find creative solutions to problems. +For example, since the beginning, a major constraint for WATcloud has been the budget. +Historically, instead of getting set-and-forget hardware that is expensive, +we worked very hard to minimize costs by opting for consumer-grade hardware and +using lots of healthchecks to ensure that the hardware is running as expected. + +The same applies when your work depends on other people's work. +If you need a work-in-progress feature to be completed before you can start your work, +find a way to work on something else in the meantime. +It's almost always possible to divide work into smaller pieces that can be worked on independently. +Being blocked on someone else's work shouldn't be an excuse to stop working. + +### Move fast and break things + +At WATcloud, we embrace the motto "Move fast and break things" to encourage risk-taking and experimentation. +Making mistakes is a natural part of the learning process, and as long as we also move fast to fix what we break, +we can turn those mistakes into opportunities for improvement. +This philosophy complements our [technical guidelines](#technical-guidelines), which prioritize trivially maintainable systems. + +With comprehensive observability, a dedication to infrastructure-as-code, and a commitment to minimize and document manual changes, +we are equipped to quickly identify and resolve issues as they arise. + + + +{ +// Separate footnotes from the main content +} +import { Separator } from "@/components/ui/separator" + + diff --git a/pages/docs/community-docs/watcloud/observability.mdx b/pages/docs/community-docs/watcloud/observability.mdx new file mode 100644 index 0000000..7ea0f20 --- /dev/null +++ b/pages/docs/community-docs/watcloud/observability.mdx @@ -0,0 +1,112 @@ +# Observability + +Observability is the ability to understand the internal state of a system based on its external outputs. +The term originated in control theory and has since been adopted by the software industry to describe the same concept in the context of software systems[^observability-software]. + +[^observability-software]: See this [Wikipedia article](https://en.wikipedia.org/wiki/Observability_(software)) for more information. + +At WATcloud, we have a number of tools to help us understand the state of our systems, detect issues, and optimize processes. + +## Healthchecks + +Healthchecks are periodic checks that verify that a service is running as expected. +Each healthcheck typically outputs a simple `up` or `down` status. +This section describes the healthcheck infrastructure at WATcloud. + +### Quick Links +- [Status Page][status-page] +- [Healthchecks.io][healthchecks-io] +- [Sentry Crons][sentry-crons] +- [Alertmanager][alertmanager] + +[status-page]: https://status.watonomous.ca/ +[healthchecks-io]: https://healthchecks.io/ +[sentry-crons]: https://watonomous.sentry.io/crons +[alertmanager]: https://prometheus.watonomous.teleport.sh/alerts + +### Status Page + +The [status page][status-page] is a collection of healthchecks from various sources. +The goal of the status page is to provide a single source of truth for the status of all our services. +We regularly use the status page as a troubleshooting tool to quickly identify the source of an issue. + +### Healthchecks.io + +[Healthchecks.io][healthchecks-io] is a [dead man's switch (DMS)](https://en.wikipedia.org/wiki/Dead_man%27s_switch) service that accepts periodic pings from services. +When a service fails to ping the service within a specified time frame, Healthchecks.io marks the service as down. +It can be configured to send alerts to various channels. Currently, we receive alerts on Discord. + +### Sentry Crons + +Similar to Healthchecks.io, [Sentry Crons][sentry-crons] is a DMS service. +We receive alerts on Discord when a service fails to ping Sentry Crons within a specified time frame. + +### Alertmanager + +[Alertmanager][alertmanager] is a component of the Prometheus monitoring system. +It uses [metrics](#metrics) collected by Prometheus to send alerts to various channels. +Currently, we receive alerts on Discord. + +## Metrics + +Metrics are quantitative measurements of a system's status and performance. +This section describes the metrics infrastructure at WATcloud. + +### Quick Links +- [Prometheus][prometheus] + +[prometheus]: https://prometheus.watonomous.teleport.sh/ + +### Prometheus + +[Prometheus][prometheus] is an open-source monitoring and alerting toolkit. +It collects metrics from various sources and stores them in a time-series database. +We use Prometheus to monitor the health of our systems and to set up alerts (via [Alertmanager](#alertmanager)) for potential issues. + +## Logs + +Logs are records of events that happen in a system. +Examples of logs include Linux system logs and Kubernetes container logs. +This section describes the logging infrastructure at WATcloud. + +### Quick Links +- [Elastic Cloud][elastic-cloud] + +[elastic-cloud]: https://wato-elastic-cloud-deployment.kb.us-east4.gcp.elastic-cloud.com:9243/ + +### Elastic Cloud + +[Elastic Cloud][elastic-cloud] is a managed Elasticsearch service. +It is used to store logs from various sources, including Kubernetes clusters and Linux servers. + +## Error Tracking + +Error tracking is the practice of recording and monitoring errors that occur in a system. +This section describes the error tracking infrastructure at WATcloud. + +### Quick Links +- [Sentry][sentry] + +[sentry]: https://watonomous.sentry.io/ + +### Sentry + +[Sentry][sentry] is an open-source error tracking tool. +It captures and aggregates errors from various sources, including web applications and backend services. +Use cases for Sentry at WATcloud include error monitoring for websites, APIs, and CI pipelines. + +## Tracing + +Tracing is the practice of recording the life cycle of an object. +An example of tracing is recording the different stages of a CI pipeline (e.g. start/finish times of each job stage, stages passed/failed). + +Currently, WATcloud does not have a tracing system in place. +You can following along [this internal issue](https://github.com/WATonomous/infra-config/issues/1795) for updates on the status of our tracing infrastructure. + +{ +// Separate footnotes from the main content +} + +import { Separator } from "@/components/ui/separator" + + diff --git a/pages/get-involved/join.mdx b/pages/get-involved/join.mdx index a3e2de7..77a873f 100644 --- a/pages/get-involved/join.mdx +++ b/pages/get-involved/join.mdx @@ -16,13 +16,19 @@ - Web Development - Robots taking over the world -Does any of the above sound familiar? Are you interested in bringing powerful compute to the masses? Do you want to work with compute infrastructure similar to those used by the world's most well-known companies[^tesla]? If so, we'd love to have you onboard! +Does any of the above sound familiar? Are you interested in bringing powerful compute to the masses? +Do you want to work with compute infrastructure similar to those used by the world's most well-known companies[^tesla]? +If so, we'd love to have you onboard! -[^tesla]: Our compute infrastructure is errily similar to the dev farm used by the Tesla Autopilot team 😱 +[^tesla]: Our compute infrastructure is errily similar to the dev farm used by the Tesla Autopilot team 😱. ## Who we're looking for -WATcloud is not like a course project, where you can do some work, get a grade, and then forget about it. We provide a service that is always up. We have users that depend on us. We have a responsibility to keep our service running, and to keep our users happy. We're looking for people who are passionate about what they do, and who are willing to put in the effort to build and quickly iterate on projects until every aspectis fully automated, reliable, observable, and trivially maintainable[^e2e-maintainable]. +WATcloud is not like a course project, where you can do some work, get a grade, and then forget about it. +We provide a service that is always up. We have users that depend on us. +We have a responsibility to keep our service running, and to keep our users happy. +We're looking for people who are passionate about what they do, and who are willing to put in the effort to build and quickly iterate on projects until every aspectis fully automated, reliable, observable, and trivially maintainable[^e2e-maintainable]. +Please take a look at our [guidelines](/docs/community-docs/watcloud/guidelines) to get a sense of what we expect from our team members. [^e2e-maintainable]: A project is trivially maintainable if it can be maintained by someone who has never seen the project before, and who has no prior knowledge of the project's internals beyond a high-level overview of its purpose. Most of the time, this involves building something that we can take down and rebuild from scratch by running a single command.