Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Provide the ability to create a new cli application registration #236

Merged
merged 8 commits into from
Nov 2, 2020

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Oct 29, 2020

Summary of the Pull Request

What is this about?

PR Checklist

  • Applies to work item: Provide the ability to create a new cli application registration #147
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

Comment on lines 436 to 445

# todo: update the internal docs (https://dev.azure.com/msresearch/OneFuzz/_git/OneFuzz?path=%2FREADME.md&_a=preview&version=GBinternal-information) with

# Mandatory role assigment
# if your org require that you prevent guest access by requiring a role assigment.
# The default CLI will not be able to access the service.
# you will need to create a new cli registration by executing the following
# ```
# python registration.py create_cli_registration INSTANCENAME [--registration_name REGISTRATIONNAME]
# ```
Copy link
Contributor Author

@chkeita chkeita Oct 29, 2020

Choose a reason for hiding this comment

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

self reminder to update the documentation before the merge

Copy link
Contributor Author

@chkeita chkeita Oct 30, 2020

Choose a reason for hiding this comment

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

will update the docs once this PR is approved
https://dev.azure.com/msresearch/OneFuzz/_git/OneFuzz?path=%2FREADME.md&version=GBinternal-information

Guest account access

If you need prevent guest access by requiring a role assigment, the default CLI will not be able to access the service.
You will need to create a new cli registration by executing the following.

python registration.py create_cli_registration INSTANCENAME [--registration_name REGISTRATIONNAME]

Then use the provided credentials in your onefuzz config

onefuzz config --client_id CLIENTID

@bmc-msft
Copy link
Contributor

Performing integration tests

@bmc-msft bmc-msft linked an issue Oct 30, 2020 that may be closed by this pull request
Copy link
Contributor

@bmc-msft bmc-msft left a comment

Choose a reason for hiding this comment

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

Currently fails due to:

running deployment
Traceback (most recent call last):
  File "deploy.py", line 63, in <module>
    from registration import (
ImportError: cannot import name 'update_pool_registration' from 'registration' (/tmp/tmpitt7aqre/registration.py)
Resource group 'pr-check-3ff769f69f004a0780e6d3a3cbc7253c' could not be found.
Traceback (most recent call last):
  File "/home/bcaswell/projects/onefuzz/tools/check-pr", line 254, in <module>
    main()
  File "/home/bcaswell/projects/onefuzz/tools/check-pr", line 247, in main
    d.run(merge_on_success=args.merge_on_success)
  File "/home/bcaswell/projects/onefuzz/tools/check-pr", line 217, in run
    self.deploy(dist)
  File "/home/bcaswell/projects/onefuzz/tools/check-pr", line 168, in deploy
    subprocess.check_call(cmd, shell=True)
  File "/usr/lib/python3.8/subprocess.py", line 364, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command './deploy-venv/bin/python deploy.py uksouth pr-check-3ff769f69f004a0780e6d3a3cbc7253c pr-check-3ff769f69f004a0780e6d3a3cbc7253c cicd' returned non-zero exit status 1.

@bmc-msft
Copy link
Contributor

bmc-msft commented Nov 2, 2020

Integration testing in progress (currently waiting on fresh build)

@bmc-msft bmc-msft merged commit e026e50 into microsoft:main Nov 2, 2020
@chkeita chkeita deleted the chkeita/147 branch March 31, 2021 21:52
@ghost ghost locked as resolved and limited conversation to collaborators May 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants