Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Autonaming of SNS fifo topics #1546

Closed
woeps opened this issue Jun 25, 2021 · 0 comments · Fixed by #1549
Closed

Autonaming of SNS fifo topics #1546

woeps opened this issue Jun 25, 2021 · 0 comments · Fixed by #1549
Assignees
Labels
kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Milestone

Comments

@woeps
Copy link

woeps commented Jun 25, 2021

The name of a SNS fifo topic has to end with .fifo. Currently, a hash is appended to the topic name provided via args. (no matter if fifo or not) Trying to create a fifo topic yields the error invalid topic name.

The code example of the Pulumi SNS docs demonstrate this exact erroneous behaviour.

related issue

The same kind of issue has already been reported and resolved for SQS fifo queues (#967). The provider's autonaming implementation for SQS fifo queues calculates names like this: <name>-<hash>.fifo, if fifo argument is true.
Whereas a SNS fifo topic's name is not handled differently than a regular SNS topic in the implementation.

possible workarounds

  • overwrite the resource name via opts (could result in other issues - especially if trying to deploy the same code multiple times to different environments or similar)
  • use stack transformation to rename all topics accordingly

Steps to reproduce

  1. create a SNS fifo topic: code taken from the Pulumi SNS docs:
import * as pulumi from "@pulumi/pulumi";
import * as aws from "@pulumi/aws";

const userUpdates = new aws.sns.Topic("user_updates", {
    contentBasedDeduplication: true,
    fifoTopic: true,
});
  1. deploy Pulumi application

Expected:
SNS fifo topic created, named user_updates-<hash>.fifo

Actual:
SNS fifo topic created, named user_updates-<hash> - note the missing fifo

@woeps woeps added the kind/bug Some behavior is incorrect or out of spec label Jun 25, 2021
@stack72 stack72 self-assigned this Jun 25, 2021
@stack72 stack72 modified the milestones: 0.59, 0.58 Jun 25, 2021
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants