Refactor Development Tooling (Tilt) #57
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jorgecuesta
added
documentation
11 tasks
bryanchriswhite
approved these changes
Jul 4, 2025
Collaborator
There was a problem hiding this comment.
Amazing work @jorgecuesta - thank you! ❤️🔥 🚀 🙌
As discussed, merged when ready - this is wonderful. 😎
Collaborator
Author
|
@Alann27 I need you open a PR with your changes, so this can be point to main after yours is merged. |
Collaborator
|
|
Collaborator
|
Could you |
Contributor
@jorgecuesta here: #58 |
Collaborator
|
Can we move "commons" and "static server" out of this section? I find that they take up visual space but aren't relevant to the developer.
|
…ha or localnet easily. WIP: 1. add docker build scripting to generate the images manually if needed. 2. update README.md
…handling - Removed `api_endpoint` configurations from multiple `cm.yaml` overlays (alpha, beta, mainnet, localnet) as they are no longer required. - Updated Tiltfile dependencies by introducing the `resource_deps` property across multiple components (`commons`, `postgresql`, `pgadmin`, `indexer`, `query`, and `static-server`). - Added `only_db` feature in `Tiltfile` for developers working solely with databases. - Centralized network constants (`mainnet`, `testnet_alpha`, `testnet_beta`, `localnet`) and default credentials in `cons.Tiltfile` for reuse across the project. - Added support for `uuid-ossp` extension in PostgreSQL values for extended UUID functionality. - Refactored file paths in the main `Tiltfile` to leverage the `base_path` property for better flexibility and maintainability.
The commit removes obsolete Kubernetes and Docker configuration files like indexer.dockerfile, various YAML files for Kubernetes resources, and PostgreSQL configurations. These files are no longer relevant to the current project setup. - Deleted `indexer.dockerfile`, previously used for Docker image creation, as it is no longer required. - Removed Kubernetes deployment files like `configmap.yaml`, `db-secrets.yaml`, `indexer-deployment.yaml`, and more, as the project no longer operates in a Kubernetes environment. - Deleted PostgreSQL Helm values file (postgresql-values.yaml) and related setup scripts, which are no longer being referenced.
- Centralized reusable constants and defaults (e.g., `defaultEmail`, `defaultPassword`, `mainnet`, `testnet_beta`) into `cons.Tiltfile` to reduce hardcoding and improve maintainability. - Updated the main `Tiltfile` and others (`pgadmin`, `indexer`, `query`) to load values from `cons.Tiltfile`. - Enhanced Port Forward configurations for `indexer`, `pgadmin`, and `query` to include specific labels and metadata (e.g., links to documentation and tools). - Improved `pgadmin` setup by integrating fallback defaults for email and password, with warnings for clarity. - Streamlined configuration across Tiltfiles to improve code readability and consistency.
- Corrected the relative path for `cons.Tiltfile` in the main `Tiltfile` to fix a loading issue. - Updated `tools_labels` to replace `pocketdex-db` with `pocketdex-tools` for consistency. - Adjusted `pgadmin` port forward from port `50` to `80` for proper web interface accessibility. - Replaced `apps_labels` with `tools_labels` for better alignment in `deployment` configurations.
Collaborator
Author
|
@bryanchriswhite everything done:
You need to update also Let me know to merge this asap. Any non-critical will need to be delayed since there are top priorities like the rewards/mint/burn: pokt-network/poktroll#1642 |
Collaborator
|
@jorgecuesta I pushed a couple of small commits, but otherwise LGTM - let's goooo! 🙌 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
.envfileIssue
Current tooling was very confusing and was also not working well with
poktrollType of change
Select one or more:
Sanity Checklist