Coder Social home page Coder Social logo

Comments (18)

gingerwizard avatar gingerwizard commented on September 24, 2024 3

@strootman @patcon i'll look at this tomorrow. It hasn't been on the priority list. We need to discuss as it seems to require giving access to the Elastic repo.

from ansible-elasticsearch.

pmoust avatar pmoust commented on September 24, 2024 3

On the other hand, nothing prevents us from creating a bot user as an external collaborator with access only to elastic/ansible-elasticsearch and provide public_repo to its token.
/cc @dliappis @elasticdog

from ansible-elasticsearch.

gingerwizard avatar gingerwizard commented on September 24, 2024 3

Added https://galaxy.ansible.com/elastic/elasticsearch/

from ansible-elasticsearch.

gingerwizard avatar gingerwizard commented on September 24, 2024 2

@strootman We're assessing but don't currently see any reason why not.

from ansible-elasticsearch.

bbaassssiiee avatar bbaassssiiee commented on September 24, 2024 2

Elastic, you only give read-access to your repo. The advantage of Galaxy is that we can all install a stable version of the vendor supported role instead of browsing through hundreds of homegrown roles by third parties where they don't keep up to their maintenance burden, like @strootman.

from ansible-elasticsearch.

strootman avatar strootman commented on September 24, 2024 1

For those who may have looked at the role and found it wasn't up to date on Ansible galaxy, it is now up to date with the current master.

from ansible-elasticsearch.

bbaassssiiee avatar bbaassssiiee commented on September 24, 2024 1

That is not the same thing, it is a fork, but the maintainer is not Elastic. Elastic pushed to GitHub, they can also tag the release and publish on Galaxy to do a proper job.

from ansible-elasticsearch.

mariussturm avatar mariussturm commented on September 24, 2024 1

Using this role as a dependency is also only possible when there is an official Elastic role on Galaxy. We can't use a fork for that.

from ansible-elasticsearch.

patcon avatar patcon commented on September 24, 2024

bump?

from ansible-elasticsearch.

strootman avatar strootman commented on September 24, 2024

FYI, I've forked this repo and imported to ansible galaxy;

https://galaxy.ansible.com/CyVerse-Ansible/elasticsearch/

No promises on keeping it up to date ;)

from ansible-elasticsearch.

gingerwizard avatar gingerwizard commented on September 24, 2024

When i tried this it needed readonly access to the whole elastic repo. Given this contains commercial code i need @elasticdog @dliappis input prior to proceeding. cc @kevinkluge

from ansible-elasticsearch.

pmoust avatar pmoust commented on September 24, 2024

AFAIK Github does not provide a much needed granular control over oauth scopes per repo.
repo and public_repo provide read-only and ready-write access across the whole org as @gingerwizard mentions above. Ref: https://developer.github.com/v3/oauth/#scopes

from ansible-elasticsearch.

elasticdog avatar elasticdog commented on September 24, 2024

I spoke with @gingerwizard and agree that it makes sense to have a dedicated machine user to provide access for this use case.

from ansible-elasticsearch.

bbaassssiiee avatar bbaassssiiee commented on September 24, 2024

🦄

from ansible-elasticsearch.

mariussturm avatar mariussturm commented on September 24, 2024

The Galaxy app only requests access to public repositories, permissions look like:

  • Access public repositories
  • Read org and team membership
  • Access user email addresses (read-only)

from ansible-elasticsearch.

jesusch avatar jesusch commented on September 24, 2024

any news on this?

from ansible-elasticsearch.

strootman avatar strootman commented on September 24, 2024

@jesusch I've been pushing a fork of this repo to ansible galaxy, and I just updated it.
https://galaxy.ansible.com/CyVerse-Ansible/elasticsearch/

from ansible-elasticsearch.

strootman avatar strootman commented on September 24, 2024

@bbaassssiiee @mariussturm then don't use it.
I put it up there just in case it fit someone's needs. It fits mine.
I also don't keep up on it, and only updated it today since I saw that @jesusch posted a *bump* on this thread.

I would prefer that Elastic support it officially, but they don't yet.

I'm honestly glad to see so many people actively monitoring this issue.
I hope it illustrates that there is a real customer need for having this role published to ansible galaxy.

Particularly for its use as a dependency, as @mariussturm suggests 👍

from ansible-elasticsearch.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.