Skip to content

Add service token authentication mechanism#75

Open
abs51295 wants to merge 1 commit into
fabric8-analytics:masterfrom
abs51295:add_auth
Open

Add service token authentication mechanism#75
abs51295 wants to merge 1 commit into
fabric8-analytics:masterfrom
abs51295:add_auth

Conversation

@abs51295

@abs51295 abs51295 commented Jul 6, 2018

Copy link
Copy Markdown
Contributor

No description provided.

@abs51295 abs51295 added the enhancement New feature or request label Jul 6, 2018
@abs51295 abs51295 self-assigned this Jul 6, 2018
@abs51295 abs51295 force-pushed the add_auth branch 3 times, most recently from 1b64779 to ccb0de6 Compare July 6, 2018 09:53
@centos-ci

centos-ci commented Jul 6, 2018

Copy link
Copy Markdown
Collaborator

@abs51295 Your image is available in the registry: docker pull registry.devshift.net/fabric8-analytics/f8a-gemini-server:SNAPSHOT-PR-75

@centos-ci

centos-ci commented Jul 6, 2018

Copy link
Copy Markdown
Collaborator

@abs51295 Your image is available in the registry: docker pull registry.devshift.net/fabric8-analytics/f8a-gemini-server:SNAPSHOT-PR-75

@centos-ci

centos-ci commented Jul 6, 2018

Copy link
Copy Markdown
Collaborator

@abs51295 Your image is available in the registry: docker pull registry.devshift.net/fabric8-analytics/f8a-gemini-server:SNAPSHOT-PR-75

@centos-ci

centos-ci commented Jul 6, 2018

Copy link
Copy Markdown
Collaborator

@abs51295 Your image is available in the registry: docker pull registry.devshift.net/fabric8-analytics/f8a-gemini-server:SNAPSHOT-PR-75

@centos-ci

centos-ci commented Jul 6, 2018

Copy link
Copy Markdown
Collaborator

@abs51295 Your image is available in the registry: docker pull registry.devshift.net/fabric8-analytics/f8a-gemini-server:SNAPSHOT-PR-75

samuzzal-choudhury
samuzzal-choudhury approved these changes Jul 26, 2018
View reviewed changes

@samuzzal-choudhury samuzzal-choudhury left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, but the PR has to be rebased to remove conflicts before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@samuzzal-choudhury samuzzal-choudhury samuzzal-choudhury approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@abs51295 abs51295

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@abs51295 @centos-ci @samuzzal-choudhury