Skip to content

Conversation

eamonnmoloney-nuuday
Copy link

Added the port testing module. This installs a simple server that dashboards can poll to confirm port openings on the internal firewall

@eamonnmoloney-nuuday eamonnmoloney-nuuday requested a review from a team July 15, 2020 08:33
@anarsen
Copy link
Contributor

anarsen commented Jul 15, 2020

Will give this a look asap.

chart_version = var.chart_version
release_name = "port-tester"
repository = "https://harbor.aws.c.dk/chartrepo/shared-platforms"
create_namespace = true
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be removed as it's replaced by kubernetes_namespace.this

Suggested change
create_namespace = true

@anarsen anarsen force-pushed the add-port-testing-module branch from fac299f to 9532e6d Compare August 5, 2020 05:19
@anarsen
Copy link
Contributor

anarsen commented Aug 5, 2020

Seems like the shared-platforms repository on Harbor is private. Instead of messing with Harbor authentication from EKS, I suggest we move this Docker image to ECR instead.

@eamonnmoloney Let's sit down when you're back and get you set up with access to ECR.

@anarsen anarsen marked this pull request as draft August 5, 2020 05:25
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.

4 participants