Skip to content

[v13] Allow sudoer files to be created without host users#32404

Merged
r0mant merged 1 commit intobranch/v13from
lxea/separated-sudoers-v13
Sep 26, 2023
Merged

[v13] Allow sudoer files to be created without host users#32404
r0mant merged 1 commit intobranch/v13from
lxea/separated-sudoers-v13

Conversation

@lxea
Copy link
Copy Markdown
Contributor

@lxea lxea commented Sep 22, 2023

backport #31793

@programmerq
Copy link
Copy Markdown
Contributor

I tried compiling this locally to see if it fixed an issue that a customer had raised, and I got the same error that CI is failing.

# github.com/aquasecurity/libbpfgo
In file included from /go/pkg/mod/github.com/aquasecurity/libbpfgo@v0.4.5-libbpf-1.0.1/libbpfgo.h:16,
                 from /go/pkg/mod/github.com/aquasecurity/libbpfgo@v0.4.5-libbpf-1.0.1/libbpfgo.go:5:
/usr/libbpf-1.0.1/include/bpf/libbpf.h:70:54: warning: ‘enum bpf_link_type’ declared inside parameter list will not be visible outside of this definition or declaration
   70 | LIBBPF_API const char *libbpf_bpf_link_type_str(enum bpf_link_type t);
      |                                                      ^~~~~~~~~~~~~
# github.com/gravitational/teleport/lib/services
lib/services/access_checker.go:900:27: type func(a "github.com/gravitational/teleport/api/types".Role, b "github.com/gravitational/teleport/api/types".Role) int of func(a types.Role, b types.Role) int {…} does not match inferred type func(a "github.com/gravitational/teleport/api/types".Role, b "github.com/gravitational/teleport/api/types".Role) bool for func(a E, b E) bool

@lxea lxea force-pushed the lxea/separated-sudoers-v13 branch 2 times, most recently from fb6609d to e0dad65 Compare September 26, 2023 09:40
Dont sort when fetching HostUsers

Dont cleanup at session end

Resolve comments

Resolve comments, switch to a notimplemented host sudoers
@lxea lxea force-pushed the lxea/separated-sudoers-v13 branch from e0dad65 to c943f25 Compare September 26, 2023 09:56
@r0mant r0mant added this pull request to the merge queue Sep 26, 2023
Merged via the queue into branch/v13 with commit 86db462 Sep 26, 2023
@r0mant r0mant deleted the lxea/separated-sudoers-v13 branch September 26, 2023 12:41
@fheinecke fheinecke mentioned this pull request Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants