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

Content Alignment Followup #35

Merged
merged 2 commits into from
Apr 29, 2024

Conversation

ale8k
Copy link
Contributor

@ale8k ale8k commented Apr 25, 2024

Content alignment checks done so far:

  • JIMM is a single pane into infrastructure (already there)
  • Identity based access control to juju models/controllers
  • Transparently proxies Juju clients to models (haven't added this as upcoming work will change it)
  • Abstraction of Juju resources irrespective of cloud provider or Juju controller (clear already)
  • Single identity, backed by SSO, that works irrespective of which cloud/controller a model lives on (clear already)
  • Focal point for Juju ops automation (clear already)
  • SREs, I think this is clear by its usage case
  • Developers, I think this is clears too no need for more docs on this
  • CI/CD/Automation - I don't think we need additional docs for this, it's kinda obvious
  • Providing users access to models by identity and not local credentials - Already clear in our security_scope section (rebac part I think makes it clear, but it does still say candid and needs changing later)
  • Agreed, switching between Jimmctl and Juju is confusing.
    Non-public charms make deployment difficult. Obtaining dependencies (binaries, charms, snaps, etc.) is hard. - I can't do anything about this really...

@ale8k ale8k requested review from alesstimec and babakks April 25, 2024 12:57
@ale8k ale8k changed the title wip Content Alignment Followup Apr 25, 2024
Copy link
Contributor

@kian99 kian99 left a comment

Choose a reason for hiding this comment

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

lgtm, just a question on one of the points in the description

Transparently proxies Juju clients to models (haven't added this as upcoming work will change it)

What upcoming work is going to change this?

@ale8k ale8k merged commit fbee4a2 into canonical:v3 Apr 29, 2024
1 check passed
@kian99
Copy link
Contributor

kian99 commented Apr 29, 2024

I recommend we get this reviewed by more people, as I'm not sure myself on the finer details of the content-alignment .

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.

2 participants