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

tetragon: Add LoadProgramOpts function #2489

Merged
merged 3 commits into from
Jun 4, 2024
Merged

tetragon: Add LoadProgramOpts function #2489

merged 3 commits into from
Jun 4, 2024

Conversation

olsajiri
Copy link
Contributor

@olsajiri olsajiri commented May 30, 2024

Adding LoadProgramOpts function, so external users can use
the loading extended with program.LoadOpts.

Also making loadOpts global as program.LoadOpts.

Signed-off-by: Jiri Olsa [email protected]

@olsajiri olsajiri added the release-note/minor This PR introduces a minor user-visible change label May 30, 2024
@olsajiri olsajiri changed the title Pr/olsajiri/loadopts tetragon: Add LoadProgramOpts function May 31, 2024
Moving customInstall fields to loadOpts object. It's more direct
and it will ease up things when we will make the loadOpts global
in following change.

Signed-off-by: Jiri Olsa <[email protected]>
Adding LoadProgramOpts function, so external users can use
the loading extended with program.LoadOpts.

Also making loadOpts global as program.LoadOpts.

Signed-off-by: Jiri Olsa <[email protected]>
With new LoadProgramOpts function it's no longer needed.

Signed-off-by: Jiri Olsa <[email protected]>
@olsajiri olsajiri marked this pull request as ready for review June 3, 2024 11:08
@olsajiri olsajiri requested a review from a team as a code owner June 3, 2024 11:08
secPrefix string
type tailCall struct {
name string
prefix string
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️

@olsajiri olsajiri merged commit d507441 into main Jun 4, 2024
40 checks passed
@olsajiri olsajiri deleted the pr/olsajiri/loadopts branch June 4, 2024 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/minor This PR introduces a minor user-visible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants