profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/carolynvs/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Carolyn Van Slyck carolynvs @Azure Chicago, IL carolynvanslyck.com Professional Yak Shaver

carolynvs/az-cli 5

unofficial helper binary for uploading to azure storage

carolynvs/assign-myself 1

GitHub app that lets contributors assign themselves to issues

carolynvs/bamboo-gitversion 1

GitVersion for Bamboo

carolynvs/2016-talks 0

Slides and Links to slides for 2016 talks

carolynvs/2016.writespeakcode.com 0

2016 Conference Website

carolynvs/2017-talks 0

Slides and links for 2017 talks

carolynvs/2019-talks 0

Slides and links for 2019 talks

carolynvs/aad-pod-identity 0

Assign Azure Active Directory Identities to kubernetes applications.

carolynvs/acs-engine 0

Azure Container Service Engine - provision and deploy container orchestrators on Azure: Kubernetes, DC/OS, and Docker Swarm.

push eventcarolynvs/porter

Carolyn Van Slyck

commit sha 2e5ab7c2a2a6c1f04bbbec4ad6ed0411aac97297

Typo

view details

push time in 14 hours

push eventcarolynvs/porter

Carolyn Van Slyck

commit sha cf2657890190d77d32372fa81029950bcd2e7e5d

Use codeblock around --atomic

view details

push time in 14 hours

PR opened getporter/porter

Blog about the helm3 v0.1.14 mixin release

Explain the latest release and why Porter moved to it immediately.

+48 -0

0 comment

1 changed file

pr created time in 14 hours

push eventcarolynvs/porter

Carolyn Van Slyck

commit sha 91647e06130879847136ff6ab6acedc3190496d7

Blog about the helm3 v0.1.14 mixin release Explain the latest release and why Porter moved to it immediately. Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>

view details

push time in 14 hours

create barnchcarolynvs/porter

branch : helm3-updates

created branch time in 14 hours

delete branch carolynvs/porter

delete branch : typos

delete time in 14 hours

push eventgetporter/porter

Carolyn Van Slyck

commit sha e2caa5bfa216c9e245aa1934b7b4181e760d1ab5

Fix website typos Address the typos identified in https://github.com/getporter/porter/issues/1767#issuecomment-925483889 Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>

view details

Carolyn Van Slyck

commit sha 7d7d60586eec61a2e7eb1e97d18d75602ffe728a

Merge pull request #1776 from carolynvs/typos Fix website typos

view details

push time in 14 hours

PR merged getporter/porter

Reviewers
Fix website typos

What does this change

Address the typos identified in

https://github.com/getporter/porter/issues/1767#issuecomment-925483889

What issue does it fix

Part of #1767

Notes for the reviewer

Sorry, I'm awful at spotting typos myself. Thanks everyone for the collective effort to eradicate my oopsies!

Checklist

  • [ ] Unit Tests
  • [x] Documentation
  • [ ] Schema (porter.yaml)
+6 -6

0 comment

3 changed files

carolynvs

pr closed time in 14 hours

pull request commentgetporter/porter

Lock down file permissions

/azp run porter-integration

carolynvs

comment created time in 14 hours

Pull request review commentgetporter/porter

Lock down file permissions

+// +build integration

I made this an integration test because it hits the filesystem and relies on build having been run first.

carolynvs

comment created time in 14 hours

PullRequestReviewEvent

push eventcarolynvs/porter

Carolyn Van Slyck

commit sha a4e0aece9633de3f7ce9e06d08d838f25190bb93

Lock down file permissions Only give permission to the current user, never to the group or other. Only mixins/plugins and scripts should be executable. When porter accesses files, it checks that there aren't any credentials/parameteres/claims with wide-open permissions. This check is skipped on Windows because files by default are not shared, and Go can't accurately report file permissions on Windows. $ porter --debug list --debug-plugins Resolved porter binary from /usr/local/bin/porter to /Users/carolynvs/.porter/porter Resolved storage plugin to storage.porter.filesystem /Users/carolynvs/.porter/porter plugin run storage.porter.filesystem PORTER HOME: /Users/carolynvs/.porter Checking file permissions in /Users/carolynvs/.porter Error: could not list installations: could not read storage schema document: incorrect file permissions on /Users/carolynvs/.porter/config.toml, it should be 600. Correct it manually or by running porter storage fix-permissions. $ porter storage fix-permissions Resetting file permissions in /Users/carolynvs/.porter... $ porter --debug list --debug-plugins Resolved porter binary from /usr/local/bin/porter to /Users/carolynvs/.porter/porter Resolved storage plugin to storage.porter.filesystem /Users/carolynvs/.porter/porter plugin run storage.porter.filesystem Checking file permissions in /Users/carolynvs/.porter NAME CREATED MODIFIED LAST ACTION LAST STATUS bug 2021-09-20 2021-09-20 install succeeded porter-hello 2021-09-20 2021-09-20 install failed whalegap 2021-09-14 2021-09-14 install succeeded mas-whales 2021-09-14 2021-09-14 install failed mybuns 2021-08-10 2021-08-10 install succeeded spike-helm3-mysql 2021-08-02 2021-08-02 install succeeded helm3-mysql 2021-07-29 2021-07-29 install succeeded hello-keyvault 2021-07-27 2021-07-27 install succeeded helm-mysql 2021-06-30 2021-06-30 install succeeded tmp 2021-06-30 2021-06-30 upgrade succeeded credentials-tutorial 2021-06-29 2021-06-29 install succeeded porterops 2021-06-10 2021-06-10 install succeeded kubernetes 2020-08-24 2020-08-24 install succeeded HELLO 2020-08-17 2020-08-17 uninstall succeeded Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>

view details

push time in 14 hours

pull request commentgetporter/porter

Lock down file permissions

Oops sorry I accidentally flipped this out of draft but I'm still fixing stuff!

carolynvs

comment created time in 14 hours

pull request commentgetporter/porter

Use the helm3 mixin everywhere

/azp run porter-integration

carolynvs

comment created time in 14 hours

push eventcarolynvs/porter

Carolyn Van Slyck

commit sha 7bc58f1af4175cf51d267b14c4fb0d8669e495ce

Lock down file permissions Only give permission to the current user, never to the group or other. Only mixins/plugins and scripts should be executable. $ porter --debug list --debug-plugins Resolved porter binary from /usr/local/bin/porter to /Users/carolynvs/.porter/porter Resolved storage plugin to storage.porter.filesystem /Users/carolynvs/.porter/porter plugin run storage.porter.filesystem PORTER HOME: /Users/carolynvs/.porter Checking file permissions in /Users/carolynvs/.porter Error: could not list installations: could not read storage schema document: incorrect file permissions on /Users/carolynvs/.porter/config.toml, it should be 600. Correct it manually or by running porter storage fix-permissions. $ porter storage fix-permissions Resetting file permissions in /Users/carolynvs/.porter... $ porter --debug list --debug-plugins Resolved porter binary from /usr/local/bin/porter to /Users/carolynvs/.porter/porter Resolved storage plugin to storage.porter.filesystem /Users/carolynvs/.porter/porter plugin run storage.porter.filesystem Checking file permissions in /Users/carolynvs/.porter NAME CREATED MODIFIED LAST ACTION LAST STATUS bug 2021-09-20 2021-09-20 install succeeded porter-hello 2021-09-20 2021-09-20 install failed whalegap 2021-09-14 2021-09-14 install succeeded mas-whales 2021-09-14 2021-09-14 install failed mybuns 2021-08-10 2021-08-10 install succeeded spike-helm3-mysql 2021-08-02 2021-08-02 install succeeded helm3-mysql 2021-07-29 2021-07-29 install succeeded hello-keyvault 2021-07-27 2021-07-27 install succeeded helm-mysql 2021-06-30 2021-06-30 install succeeded tmp 2021-06-30 2021-06-30 upgrade succeeded credentials-tutorial 2021-06-29 2021-06-29 install succeeded porterops 2021-06-10 2021-06-10 install succeeded kubernetes 2020-08-24 2020-08-24 install succeeded HELLO 2020-08-17 2020-08-17 uninstall succeeded Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>

view details

push time in 14 hours

PR opened getporter/porter

Fix website typos

What does this change

Address the typos identified in

https://github.com/getporter/porter/issues/1767#issuecomment-925483889

What issue does it fix

Part of #1767

Notes for the reviewer

Sorry, I'm awful at spotting typos myself. Thanks everyone for the collective effort to eradicate my oopsies!

Checklist

  • [ ] Unit Tests
  • [x] Documentation
  • [ ] Schema (porter.yaml)
+6 -6

0 comment

3 changed files

pr created time in 15 hours

create barnchcarolynvs/porter

branch : typos

created branch time in 15 hours

issue openedMChorfa/porter-helm3

Support the --timeout and --debug flags

Now that we specify --atomic by default, it may be a good idea to allow people to change the --timeout flag as well. When --atomic is set, helm waits for the deployment to be successful (until --timeout is reached) and then rollsback if needed. Some people may need to change --timeout either because they don't want to wait forever (the default timeout is like 5m) to realize the deployment failed, or because they want to wait even longer.

Along those same lines, when --atomic fails, it can be really hard to tell what was wrong! When --debug is set on the helm call, it will print out which deployment wasn't ready that caused helm to revert. Exposing --debug as well would help people figure out what's wrong.

created time in 15 hours

push eventcarolynvs/porter

Carolyn Van Slyck

commit sha 1fc226ea53a2b1ddec2b474a59946fd8cf5f9f07

Fix deploy of wordpress example bundle With the helm3 mixin using --atomic, helm is waiting for the deployment to be considered complete. However when installing on kind, the default service type of loadbalancer will never get an ip address assigned. I've changed the service type to NodePort so that the helm release completes successfully and doesn't just timeout waiting for an ip that is never coming. Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>

view details

push time in 15 hours

issue commentgetporter/porter

v1 website bugs

Thanks @joshuabezaleel!

carolynvs

comment created time in 16 hours

issue commentgetporter/porter

Rename "table" format to plaintext

It's all yours, @joshuabezaleel!

carolynvs

comment created time in 16 hours

delete branch carolynvs/porter

delete branch : remove-debug-statement2

delete time in 16 hours

push eventgetporter/porter

Carolyn Van Slyck

commit sha c3b7717227253e611ec8571a38ebf396bdf6cdfe

Remove debug print statement This was a debug print statement that I meant to just add while fixing something locally. It shouldn't be printed every time you run a bundle! Signed-off-by: Carolyn Van Slyck <me@carolynvanslyck.com>

view details

Carolyn Van Slyck

commit sha 6cffc84becaab0b260b720862ad39cf0b1f29e39

Merge pull request #1775 from carolynvs/remove-debug-statement2 Remove debug print statement

view details

push time in 16 hours

PR merged getporter/porter

Reviewers
Remove debug print statement

Backport #1747 to v0.38

+0 -2

1 comment

1 changed file

carolynvs

pr closed time in 16 hours

pull request commentgetporter/porter

Remove debug print statement

Important work was done here today. Well done, folks! 👏

carolynvs

comment created time in 16 hours

Pull request review commentgetporter/porter

Add installation instruction to porter explain command

 func (p *Porter) printDependenciesExplainTable(bun *PrintableBundle) error { 		} 	return printer.PrintTable(p.Out, bun.Dependencies, printDependencyRow, "Alias", "Reference") }++func (p *Porter) printInstallationInstructionBlock(bun *PrintableBundle, bundleReference string) error {+	fmt.Fprint(p.Out, "To install this bundle run the following command, passing --param KEY=VALUE for any parameters you want to customize:\n")++	var bundleReferenceFlag string+	if bundleReference != "" {+		bundleReferenceFlag += " --reference " + bundleReference+	}++	// Generate any predefined credentials first+	for _, credential := range bun.Credentials {+		fmt.Fprintf(p.Out, "porter credentials generate %s%s\n", credential.Name, bundleReferenceFlag)+	}++	// Bundle installation instruction+	var requiredParameterFlags string+	for _, parameter := range bun.Parameters {+		if parameter.Required {+			requiredParameterFlags += parameter.Name + "=TODO "+		}+	}++	if requiredParameterFlags != "" {+		requiredParameterFlags += " --param "+	}++	var credentialFlags string

This can be a conditional "if there are creds then include a single -c mycreds flag", otherwise don't print anything.

joshuabezaleel

comment created time in 16 hours

Pull request review commentgetporter/porter

Add installation instruction to porter explain command

 func (p *Porter) printBundleExplainTable(bun *PrintableBundle) error { 	p.printOutputsExplainBlock(bun) 	p.printActionsExplainBlock(bun) 	p.printDependenciesExplainBlock(bun)+	p.printInstallationInstructionBlock(bun, bundleReference)

This is really minor but would you please move this line under 331 where we print the tools included in the bundle? That way the output looks like this

$ porter explain -r somebundle
// a lot of information about that bundle
// the tools in the bundle
// BIG FINISH: How to install the bundle
joshuabezaleel

comment created time in 16 hours

Pull request review commentgetporter/porter

Add installation instruction to porter explain command

 func (p *Porter) printDependenciesExplainTable(bun *PrintableBundle) error { 		} 	return printer.PrintTable(p.Out, bun.Dependencies, printDependencyRow, "Alias", "Reference") }++func (p *Porter) printInstallationInstructionBlock(bun *PrintableBundle, bundleReference string) error {+	fmt.Fprint(p.Out, "To install this bundle run the following command, passing --param KEY=VALUE for any parameters you want to customize:\n")++	var bundleReferenceFlag string+	if bundleReference != "" {+		bundleReferenceFlag += " --reference " + bundleReference+	}++	// Generate any predefined credentials first+	for _, credential := range bun.Credentials {

We don't need to iterate for printing out how to generate credentials because when you run generate it will make a credential set that includes all of the credentials required by the bundle. So it can be a single statement that we only print out if the length of credentials is > 0.

The final command that we will tell them to run will be:

porter credentials generate mycreds -r REFERENCE

Normally if you leave off the name of the credential set, it will default to the name of the bundle. We could try to mimic that but I just checked and ... there's actually a potential bug in that code. So let's avoid that and just use a hard coded "mycreds" for the name in this command and in porter install. 😅

joshuabezaleel

comment created time in 16 hours

PullRequestReviewEvent
PullRequestReviewEvent