Skip to content
This repository was archived by the owner on Mar 23, 2019. It is now read-only.

Disable "Terminate All" button when no EC2 resources available #128

Closed
helfi92 opened this issue Sep 21, 2018 · 32 comments
Closed

Disable "Terminate All" button when no EC2 resources available #128

helfi92 opened this issue Sep 21, 2018 · 32 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@helfi92
Copy link
Contributor

helfi92 commented Sep 21, 2018

screen shot 2018-09-21 at 3 35 27 pm

Steps to reproduce:

@helfi92 helfi92 added enhancement New feature or request good first issue Good for newcomers labels Sep 21, 2018
@ariellegordon
Copy link
Contributor

/claim

@ariellegordon
Copy link
Contributor

This is my first bug for Mozilla! If possible, would love assistance on this issue. Thank you.

@helfi92
Copy link
Contributor Author

helfi92 commented Sep 24, 2018

Hi arielle! The first step is to have your local development setup by going over the README. Once that's done, you'll want add some logic that would disable the button when the list of ec2 resources is empty. Here's the button:

<Button
disabled={actionLoading}
requiresAuth
onClick={this.handleTerminateAllInstances}
variant="fab"
className={classes.terminateButton}>
<DeleteEmptyIcon />
</Button>
. Hope that helps :)

@helfi92
Copy link
Contributor Author

helfi92 commented Sep 25, 2018

Please let me know if I can help in any way :)

@aimenbatool
Copy link
Contributor

I logged in using dummy credentials. And I am getting this error. How can I fix this error now? Are there any working credentials?
What am I suppose to do here?
aws provisioner ami test taskcluster

@helfi92
Copy link
Contributor Author

helfi92 commented Sep 26, 2018

  1. Click Logout
  2. Get Taskcluster credentials by doing https://github.com/taskcluster/taskcluster-web-server#taskcluster-credentials. Once you do that in your terminal, you will see something like
export TASKCLUSTER_CLIENT_ID='mozilla-auth0/ad|Mozilla-LDAP|haali/cli'
export TASKCLUSTER_ACCESS_TOKEN='...'
  1. Head over to localhost:5080 and login via manual credentials (not github)
  2. Paste your Client ID and Access Token. In my case it will be mozilla-auth0/ad|Mozilla-LDAP|haali/cli and ....
  3. Once you're logged in, navigate to the EC2 resources tab

Let me know if you that works or whether a different error is shown. Hope that helps.

@helfi92
Copy link
Contributor Author

helfi92 commented Sep 26, 2018

@aimenbatool This bug is assigned to @ariellegordon. Let me know if you can't find another bug in the repo, I will find you one :)

@aimenbatool
Copy link
Contributor

Okay, No problem.
Can you please update me on issue #127 ?
How to reproduce this error?

@ariellegordon
Copy link
Contributor

Hi guys - just configuring my environment and installing yarn. I will let you know if I need any help with the bug. Hoping to get it fixed by this weekend. Thank you @helfi92 !

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 5, 2018

Hi @ariellegordon. Do you still plan on tackling this issue?

@ariellegordon
Copy link
Contributor

Hi @helfi92. Thanks for checking in. So sorry about the delay, and thanks for your patience. It took me a while to set up the environment for the very first time, but I now have it running and will make a PR soon. Thanks.

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 5, 2018

Glad to hear. Thank you for the update :)

@ariellegordon
Copy link
Contributor

ariellegordon commented Oct 5, 2018

@helfi92 Hi, can you help with the Login Credentials and the JSON certificate? I ran the CLI script but I'm not seeing any kind of export variables. Thank you.

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 5, 2018

@ariellegordon Have a look at the discussion in #121 (comment). Let me know if you're able to successfully login after reading it :)

@ariellegordon
Copy link
Contributor

Hi @helfi92 -- thank you so much for your help. I followed those instructions but I am still not able to see TASKCLUSTER_CLIENT_ID or TASKCLUSTER_ACCESS_TOKEN after I log in using the CLI. I've attached a screenshot of what I do see.

screen shot 2018-10-05 at 4 32 38 pm

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 5, 2018

@ariellegordon Once you click on "Create Client", the terminal will show TASKCLUSTER_CLIENT_ID or TASKCLUSTER_ACCESS_TOKEN.

@ariellegordon
Copy link
Contributor

@helfi92 Thank you so much. Do you happen to know how to find the JSON certificate?

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 5, 2018 via email

@ariellegordon
Copy link
Contributor

ariellegordon commented Oct 5, 2018

Sorry, sign in appears to be disabled if I leave it empty? @helfi92
screen shot 2018-10-05 at 4 40 27 pm

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 5, 2018 via email

@ariellegordon
Copy link
Contributor

@helfi92 refreshing worked -thank you so much for your help!

@ariellegordon
Copy link
Contributor

ariellegordon commented Oct 6, 2018

@helfi92 I keep getting restricted access on all AWS Provisioner Worker Type screens, and refreshing the page / removing and rebuilding the node modules does not seem to change anything. Any help would be very greatly appreciated.
screen shot 2018-10-06 at 3 05 58 pm

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 6, 2018

I just gave you the required to scopes to view worker types. Try refreshing the page and see if that works.

@ariellegordon
Copy link
Contributor

It works, thank you!

@ariellegordon
Copy link
Contributor

@helfi92 Hi! I have a working disabled button feature but I'm having trouble pushing to the branch. I am denied permission to push to taskcluster-web at origin/branch-name. Is there another way to push or make a PR?

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 6, 2018

Does git push origin <branch-name> not work? A screenshot of what you've been trying to do would be helpful.

@ariellegordon
Copy link
Contributor

ariellegordon commented Oct 6, 2018

Yes, exactly. My branch name is "working". Screenshot attached. Thank you.
screen shot 2018-10-06 at 4 48 50 pm

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 6, 2018

You're trying to push to the wrong remote. You'll have to create a fork of taskcluster-web and then push to that fork.

Click the fork button to create a clone of that repo under your account:

screen shot 2018-10-06 at 4 58 17 pm

Once that's done, I think you can do something like:

git push <url> working

where <url> can be copied from https://github.com/ariellegordon/taskcluster-web (after you have forked the repo)

screen shot 2018-10-06 at 5 00 18 pm

@ariellegordon
Copy link
Contributor

ariellegordon commented Oct 6, 2018

Ah of course. I'm sorry, a very silly mistake. I've pushed to a branch of my forked repo and made a PR --
https://github.com/ariellegordon/taskcluster-web/tree/working-branch
The only minor change is that I had to add a .prettierrc.json and as such needed to add to the .gitignore. But obviously this can be removed if/when the branch is merged.
Thank you so much for your help @helfi92.

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 11, 2018

Fixed in 50e5197.

@helfi92 helfi92 closed this as completed Oct 11, 2018
@ariellegordon
Copy link
Contributor

@helfi92 Thank you for all of your help throughout this issue! It is so greatly appreciated.

@helfi92
Copy link
Contributor Author

helfi92 commented Oct 12, 2018

My pleasure!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants