This message was deleted.
# generals
sparse-intern-71089
03/25/2022, 2:25 PMThis message was deleted.
s
stale-vase-87890
03/25/2022, 2:33 PMThe Github app has to be assigned to each repo or the entire org. It also has to be able to detect when a pulumi up/preview is ran as part of an action that is ran on a PR
a
adorable-gpu-98268
03/25/2022, 2:33 PMIt’s assigned to the org
s
stale-vase-87890
03/25/2022, 2:35 PMHow is the workflow calling pulumi preview?
a
adorable-gpu-98268
03/25/2022, 2:37 PMIt’s calling
pulumi preview --stack foos
stale-vase-87890
03/25/2022, 2:38 PMI am thinking the plugin isn't going to be able to detect that because it is all happening insisde a container
stale-vase-87890
03/25/2022, 2:38 PMWhy not use the pulumi action?
a
adorable-gpu-98268
03/25/2022, 2:38 PMIt’s setup using a system for reproducible builds
adorable-gpu-98268
03/25/2022, 2:39 PMSo all dependencies are explicitly injected
adorable-gpu-98268
03/25/2022, 2:39 PM• source (only what is needed for pulumi)
• build artefacts to deploy
• environment variables & secrets
adorable-gpu-98268
03/25/2022, 2:39 PMI’m not going to change that
adorable-gpu-98268
03/25/2022, 2:40 PMThis is the way to go for build systems and pulumi should adapt. In the meantime I’m asking if there is a way that I can have an explicitly configured integration
adorable-gpu-98268
03/25/2022, 2:43 PMLooking at the pulumi code, I find functions like this:
adorable-gpu-98268
03/25/2022, 2:44 PMThis detects the environment variables of a github actions run, do you have a list what I have to set/forward to get the integration to work?
w
witty-candle-66007
03/25/2022, 2:50 PMDo you have PULUMI_ACCESS_TOKEN configured as an environment variable? It uses the Pulumi service to get the update information.
a
adorable-gpu-98268
03/25/2022, 2:51 PMYes, that works fine, I get the updates in the saas
adorable-gpu-98268
03/25/2022, 2:51 PMThe state management needs that anyway
adorable-gpu-98268
03/25/2022, 2:53 PMBut in the saas backend I see the notice:
adorable-gpu-98268
03/25/2022, 2:55 PMI’ll try setting the variables following the code in https://github.com/pulumi/pulumi/tree/v3.27.0/sdk/go/common/util/ciutil
Though I think that really should be documented.
w
witty-candle-66007
03/25/2022, 2:59 PMI think getting those Github variables in the environment is necessary since that’s how things are mapped between Pulumi and SaaS.
I agree this should be documented better. Can I suggest you opening an issue here: https://github.com/pulumi/service-requests
(If you get it working.)
If you don’t get it working open an issue about github app support for a container-based approach as you are using here:
https://github.com/pulumi/pulumi/issues
a
adorable-gpu-98268
03/25/2022, 3:03 PMI had opened an issue some time ago, but the discussion back then was around pulumi needing access to the
.git5 Views