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

Stackdriver Export to Datadog v2 #43

Merged
merged 1 commit into from
Dec 12, 2019

Conversation

smbreslow
Copy link

new file:   examples/datadog-sink/main.tf
new file:   examples/datadog-sink/outputs.tf
new file:   examples/datadog-sink/screenshots/Screen Shot 2019-12-09 at 4.44.11 PM.png
new file:   examples/datadog-sink/terraform.tfvars.sample
new file:   examples/datadog-sink/variables.tf
new file:   examples/datadog-sink/versions.tf
modified:   modules/pubsub/README.md
modified:   modules/pubsub/main.tf
modified:   modules/pubsub/outputs.tf
modified:   modules/pubsub/variables.tf
modified:   modules/pubsub/versions.tf

I tried to rebase and squash but it was becoming tedious and there were enough commits that I just reset my fork back to the upstream master and added the files I needed from a backup. Certainly not best practice, but I'm tired of fighting with git and hopefully this works.

@smbreslow
Copy link
Author

smbreslow commented Dec 12, 2019

@morgante Do you have some time this afternoon to chat? I'm not sure why this is failing (still no logs access) and my test setup seems irreparably tanked.

UPDATE: I have log access, but it looks like the tests are failing on the prepare step with an IAM error.

@morgante
Copy link
Contributor

Sure, we can look at it together this afternoon.

	new file:   examples/datadog-sink/main.tf
	new file:   examples/datadog-sink/outputs.tf
	new file:   examples/datadog-sink/screenshots/Screen Shot 2019-12-09 at 4.44.11 PM.png
	new file:   examples/datadog-sink/terraform.tfvars.sample
	new file:   examples/datadog-sink/variables.tf
	new file:   examples/datadog-sink/versions.tf
	modified:   modules/pubsub/README.md
	modified:   modules/pubsub/main.tf
	modified:   modules/pubsub/outputs.tf
	modified:   modules/pubsub/variables.tf
	modified:   modules/pubsub/versions.tf
@morgante morgante merged commit 4164865 into terraform-google-modules:master Dec 12, 2019
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.

2 participants