Skip to content

Conversation

@emmanuelmathot
Copy link
Contributor

This branch contains a polished, production-ready version of the data pipeline tools with significant improvements to maintainability and documentation.

Key Changes

Infrastructure & Deployment

  • Improved GitHub Actions workflow with docker metadata action
  • Enhanced Docker build with proper tagging strategy (SHA, branch, PR, tags)
  • Removed RabbitMQ dependency in favor of HTTP webhooks

Operator Tools

  • Reorganized submission tools into dedicated operator-tools/ directory
  • Comprehensive documentation for staging (devseed-staging) and production (devseed) environments
  • Removed deprecated RabbitMQ scripts, streamlined to webhook-only approach

Documentation

  • Clear separation of concerns: application code vs deployment configuration
  • Links to platform-deploy for infrastructure details
  • Updated Argo Workflows URLs to correct endpoints
  • Environment-specific instructions for both staging and production

All tools have been tested and documentation updated against the current staging data pipeline infrastructure deployed

- Implement tests for converting S3 URLs to HTTPS and vice versa.
- Add tests for rewriting asset hrefs in STAC items.
- Include tests for adding alternate S3 assets to STAC items.
- Cover edge cases such as handling of query parameters, empty path components, and Unicode characters.
- Ensure compatibility with old S3 formats and various bucket naming conventions.
```

This makes the webhook endpoint available at `http://localhost:12000/samples`.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same info is in the operator readme, could we just replace l61-l69 by

See [operator-tools/README.md](operator-tools/README.md#port-forwarding) for webhook port forwarding setup.

@lhoupert lhoupert merged commit 0283833 into main Nov 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants