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

Merge releases/v2 into releases/v1 #1142

Merged
merged 52 commits into from
Jul 13, 2022
Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jul 13, 2022

Merging 3e7e3b3 into releases/v1

Conductor for this PR is @criemen

Contains the following pull requests:

Please review the following:

  • The CHANGELOG displays the correct version and date.
  • The CHANGELOG includes all relevant, user-facing changes since the last release.
  • There are no unexpected commits being merged into the releases/v1 branch.
  • The docs team is aware of any documentation changes that need to be released.

aibaars and others added 30 commits June 27, 2022 16:34
This avoids creating a __pycache__ folder in the _actions folder, which
may cause file ownership problems on self-hosted runners
when run in a docker container.
This decorator enabled us to use the functionality of the Actions
toolcache within the runner too.
Now that we've deleted the runner we no longer need it.
Mergeback v2.1.15 refs/heads/releases/v2 into main
`toolcache.extractTar` currently falls over when `ACTIONS_TEMP` contains
a symlink, and the runner no longer exists, so it's unlikely our
customers would be running with temporary directories that contain
symlinks.
Python-Setup: run auto_install_packages.py with -B flag
This decorator enabled us to use the functionality of the Actions
toolcache within the runner too.
Now that we've deleted the runner we no longer need it.
This commit prints diagnostic messages to the Actions log when debug
logging is enabled by passing `debug: true` to `codeql-action/init` or
enabling Actions step debug logging.
The tracer is very good at preserving itself, so unsetting the tracing-specific
variables from within a process will not end tracing for children of
that process.
The way the actions process model works means that we're running inside
a process for the entire build step that was launched with the tracer
variables set, so we'll have the tracer injected into the entire build
step and its children.
If we unset the variables in end-tracing, we will get into an intermediate
state: Not all variables in there are preserved by the tracer,
but the tracer is still active.
Usually, that wouldn't be a problem, but the autobuilders called from
the finalize step will suddenly run under a half-configured tracer.

Particularly, this half-configured tracer is unable to execute the dotnet
CLI without hangs, as the environment variable that prevents hangs for
dotnet on MacOS has been unset, but the tracer is still active.

This is an issue for the the go autobuilder, that invokes
user-provided build scripts in the hope of installing dependencies.
If that build script then invokes dotnet, it will hang.

This is only of concern for the Lua tracer that now implements proper
multi-language tracing: Previously, when encountering the go autobuilder,
the tracer disabled itself entirely, thus side-stepping any hangs.
In the new, multi-language tracing world, the tracer will stay active
as long as there is at least one other language that's been set up
for tracing.
Thus, we also get hangs when invoking the dotnet CLI through the go
autobuilder.
…-debug-enabled

Print diagnostic messages when debugging mode is enabled
@github-actions github-actions bot added the Update dependencies Trigger PR workflow to update dependencies label Jul 13, 2022
@criemen criemen marked this pull request as ready for review July 13, 2022 11:29
@criemen criemen requested a review from a team as a code owner July 13, 2022 11:29
@criemen criemen enabled auto-merge July 13, 2022 11:42
@github-actions github-actions bot removed the Update dependencies Trigger PR workflow to update dependencies label Jul 13, 2022
@criemen criemen closed this Jul 13, 2022
auto-merge was automatically disabled July 13, 2022 11:54

Pull request was closed

@criemen criemen reopened this Jul 13, 2022
@criemen criemen enabled auto-merge July 13, 2022 11:54
@criemen criemen merged commit f7b6c00 into releases/v1 Jul 13, 2022
@criemen criemen deleted the update-v1.1.16-3e7e3b32 branch July 13, 2022 12:43
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.

5 participants