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.

Sign up for GitHub

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

Jump to bottom

[DO NOT MERGE][POC] register an oauth application with OC-ID #7894

Closed
wants to merge 21 commits into from
Closed

[DO NOT MERGE][POC] register an oauth application with OC-ID #7894

wants to merge 21 commits into from

Conversation

RajeshPaul38
Copy link
Collaborator

🔩 Description: What code changed, and why?

[POC] register an oauth application with OC-ID

⛓️ Related Resources

👍 Definition of Done

👟 How to Build and Test the Change

✅ Checklist

All PRs must tick these:

With occasional exceptions, all PRs from Progress employees must tick these:

  • Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
  • Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
  • Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
  • Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
  • Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

  • Tests added/updated? (all new code needs new tests)
  • Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.

📷 Screenshots, if applicable

ayush-billore and others added 21 commits May 2, 2023 16:38
@ayush-billore
add ocid to .bldr.toml
4e04932 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
add ocid component
e17dfb3 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
add configuration related changes related to ocid
0933a7a 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
add ocid api config
641b1be 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
update deployment api config
f7b8cf4 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
add ocid config details to automate-deployment component
a90f908 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
add ocid in the pg_ident.conf
07c37ab 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
add config for ocid
f3b728d 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
add ocid specific nginx config
5b82a19 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
add config files in ocid component
176c32a 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
refactored run hook
fcac4b4 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
remove unwanted code from plan.sh of ocid
c0a75ca 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
remove unwanted files
efaf3ca 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
remove ocid related changes from ctl.go
b4d789f 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
- move database url variable to default.toml
d0e8fb6 
- move database related task from plan.sh to run hook

Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
- remove unnecessary code from package.meta
8a2f538 
- updated permission for schema.rb and /tmp as these files will be u from run hook
- add autogenerated gen.go

Signed-off-by: ayush-billore <ayush.billore@progress.com>
@ayush-billore
Chef 2588/add OCID component in automate (#7861)
196a267 
* add ocid to .bldr.toml

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add ocid component

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add configuration related changes related to ocid

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add ocid api config

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* update deployment api config

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add ocid config details to automate-deployment component

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add ocid in the pg_ident.conf

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add config for ocid

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add ocid specific nginx config

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add config files in ocid component

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* refactored run hook

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* remove unwanted code from plan.sh of ocid

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* remove unwanted files

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* remove ocid related changes from ctl.go

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* - move database url variable to default.toml
- move database related task from plan.sh to run hook

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* - remove unnecessary code from package.meta
- updated permission for schema.rb and /tmp as these files will be u from run hook
- add autogenerated gen.go

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* add a comment for tmp folder and make them into a single line

Signed-off-by: ayush-billore <ayush.billore@progress.com>

* Fix health check for oc-id component (#7869)

Signed-off-by: ayush-billore <ayush.billore@progress.com>

---------

Signed-off-by: ayush-billore <ayush.billore@progress.com>
@RajeshPaul38
Dynamically set webui key from erchef to ocid (#7878)
e9d1728 
Signed-off-by: RajeshPaul38 <rajesh.paul@progress.com>
@ayush-billore
update value for WriteMetadata to false in spec.go
78ab424 
Signed-off-by: ayush-billore <ayush.billore@progress.com>
@RajeshPaul38
[POC] register an oauth application with OC-ID
8447cb7 
Signed-off-by: Rajesh Paul <rajesh.paul@progress.com>
@RajeshPaul38
merged feature branch in reconfigure implementation branch
24677d8 
Signed-off-by: RajeshPaul38 <rajesh.paul@progress.com>
@RajeshPaul38 RajeshPaul38 marked this pull request as draft May 16, 2023 12:24
@sonarqube-for-infrastructure-prod
Copy link

SonarQube Quality Gate

Quality Gate failed

Failed condition 7.1% 7.1% Coverage on New Code (is less than 80%)

See analysis details on SonarQube

@RajeshPaul38 RajeshPaul38 changed the title Rajeshpaul38/implement reconfigure command/chef-2896 [POC] register an oauth application with OC-ID May 16, 2023
@RajeshPaul38 RajeshPaul38 changed the title [POC] register an oauth application with OC-ID [DO NOT MERGE][POC] register an oauth application with OC-ID May 16, 2023
@ayush-billore ayush-billore force-pushed the chef-1493/supermarket-and-chef-manage-integration-with-automate-HA branch from fae6b90 to 488cd93 Compare May 30, 2023 09:02
@RajeshPaul38
Copy link
Collaborator Author

POC is now adopted in our code in PR: #7909

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RajeshPaul38 @ayush-billore