FAQ | This is a LIVE service | Changelog

Skip to content
Snippets Groups Projects

Add initial (trivial) pubsub module and faas module that supports pubsub

Merged E. Kirk requested to merge ek599-3-faas-and-pub-sub-terraform-module-boilerplate into main
2 unresolved threads

Relates #3 (closed)

Follow up MRs or split tickets for alerting and working dead letter queue

Edited by E. Kirk

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Robin Goodall
  • Robin Goodall
  • Robin Goodall
  • This probably needs rebasing after !5 (merged) was merged.

  • E. Kirk added 1 commit

    added 1 commit

    • 28ef1fb2 - chore: remove duplicate and conflicting logan settings/cmds

    Compare with previous version

  • E. Kirk added 1 commit

    added 1 commit

    • 7cdb677d - fix: move logan file again and simplify container definition

    Compare with previous version

  • E. Kirk added 1 commit

    added 1 commit

    • e7138753 - fix: re-add missing docker FROM

    Compare with previous version

  • README doesn't explain how to use the terraform added here. Is the project using FaaS supposed to copy the modules or somehow remotely reference them?

  • 1 1 [tool.poetry]
    2 2 name = "ucam-faas"
    3 3 version = "0.1.3"
  • 1 resource "google_pubsub_topic" "main" {
    2 name = var.topic_name
    3 }
  • Robin Goodall mentioned in merge request !8 (merged)

    mentioned in merge request !8 (merged)

  • Think we just need the following then this is done:

    • chopping pubsub module (for now) as it is single resource
    • upversioning
    • CHANGELOG.md
  • E. Kirk added 2 commits

    added 2 commits

    • 85000c58 - chore: remove pubsub module
    • 9cf94c27 - chore: add changelog and version for new release

    Compare with previous version

  • Robin Goodall approved this merge request

    approved this merge request

  • E. Kirk requested review from @rjg21

    requested review from @rjg21

  • E. Kirk enabled an automatic merge when the pipeline for 9cf94c27 succeeds

    enabled an automatic merge when the pipeline for 9cf94c27 succeeds

  • E. Kirk canceled the automatic merge

    canceled the automatic merge

  • E. Kirk enabled an automatic merge when the pipeline for 9cf94c27 succeeds

    enabled an automatic merge when the pipeline for 9cf94c27 succeeds

  • merged

  • E. Kirk mentioned in commit b30a8f58

    mentioned in commit b30a8f58

  • Please register or sign in to reply
    Loading