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

Adds Pulumi ESC Module #22

Merged
merged 13 commits into from
Jun 21, 2024
Merged

Adds Pulumi ESC Module #22

merged 13 commits into from
Jun 21, 2024

Conversation

desteves
Copy link

@desteves desteves commented May 10, 2024

Adds a module covering Pulumi ESC 101

Fixes #18

@desteves desteves marked this pull request as ready for review May 13, 2024 19:05
@desteves desteves requested a review from jkodroff May 13, 2024 19:05
@desteves desteves mentioned this pull request May 17, 2024
module-10-intro-to-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-intro-to-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-intro-to-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-intro-to-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved

---

## Module 10 - Intro to Pulumi ESC - Consumers
Copy link
Member

Choose a reason for hiding this comment

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

Give examples here (on additional slides if necessary).

module-10-intro-to-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-intro-to-pulumi-esc/exercises/README.md Outdated Show resolved Hide resolved
module-10-intro-to-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
- Imports `env_parent` values
- Adds an `aws:region` value to be consumed by a Pulumi program**

** Hint: take a look at the ESC `pulumiConfig` syntax
Copy link
Member

Choose a reason for hiding this comment

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

Maybe make a Pulumi program that verifies the correctness of the environment config.

Copy link
Author

Choose a reason for hiding this comment

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

This is part of the Read exercise.

Copy link
Author

Choose a reason for hiding this comment

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

While working on this solution, I think I ran into a bug, filed pulumi/pulumi-pulumiservice#319

module-10-intro-to-pulumi-esc/exercises/README.md Outdated Show resolved Hide resolved
@ericrudder
Copy link
Member

ericrudder commented May 29, 2024 via email

@desteves desteves requested a review from jkodroff June 7, 2024 17:01
module-10-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved

---

## Module 10 - Pulumi ESC - Advanced Features
Copy link
Member

Choose a reason for hiding this comment

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

These aren't advanced IMO - they're foundational pieces of functionality in the tool. Suggest breaking each one of these out into their own slide if not covered later in the presentation.

Copy link
Author

Choose a reason for hiding this comment

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

How about calling them capabilities?

Copy link
Member

Choose a reason for hiding this comment

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

Or just "features". My objection was that they are not advanced per-se.

module-10-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
module-10-pulumi-esc/slides/slides.md Outdated Show resolved Hide resolved
@@ -0,0 +1,58 @@
# Module 10 - Pulumi ESC - Exercises - 🎯 Goal
Copy link
Member

Choose a reason for hiding this comment

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

The exercises need to show:

  • AWS OIDC
  • AWS Secrets Manager as a source
  • config values
  • environment inheritance

and then pass those things into a Pulumi program.

@desteves
Copy link
Author

Related Docs issue

@desteves desteves requested a review from jkodroff June 18, 2024 14:50
@desteves desteves changed the title Adds Pulumi ESC Module 10 Adds Pulumi ESC Module Jun 18, 2024
Copy link
Member

Choose a reason for hiding this comment

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

Extra committed file.

Copy link
Author

Choose a reason for hiding this comment

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

Oops, removing...

@desteves desteves merged commit 4237792 into main Jun 21, 2024
1 check passed
@desteves desteves deleted the esc branch June 21, 2024 15:48
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.

Module: ESC
3 participants