Skip to content

Conversation

@russellb
Copy link
Collaborator

@russellb russellb commented Dec 3, 2018

m1.medium was created with 10240 MB of RAM. Change this to 8192,
which matches what is being used on AWS (m4.large).

openstack flavor create --ram 1024 --disk 10 --vcpu 2 --public tiny
openstack flavor create --ram 10240 --disk 20 --vcpu 2 --public m1.medium
openstack flavor create --ram 8192 --disk 20 --vcpu 2 --public m1.medium
openstack network create --external --provider-physical-network datacentre --provider-network-type flat public
Copy link
Collaborator

Choose a reason for hiding this comment

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

The commit message says 4096 but the change is 8192?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oops. I guess I ammended the commit message but forgot to include the actual change.

I started with 8192, as that matches the AWS flavor in use, but that wasn't actually small enough to work in my test env.

A better solution here is making flavor a deployment choice, and i've got an installer patch in progress to add that. I still think 4096 is probably a kinder default though.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

m1.medium was created with 10240 MB of RAM.  Change this to 4096
to be more dev env friendly.

AWS uses a flavor with 8 GB by default (m4.large), but the libvirt
provider is using 2 and 3 GB VMs, so 4 GB should be enough.

We may just need to make flavor selection easier.
Copy link
Collaborator

@hardys hardys left a comment

Choose a reason for hiding this comment

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

lgtm, I've been testing locally with as little as 2G and coreos will boot so 4G should be a reasonable default I think

@russellb
Copy link
Collaborator Author

russellb commented Dec 5, 2018

related, added flavor selection support here: openshift/installer#788

@tomassedovic
Copy link
Collaborator

Once we've got the flavor selection merged, could we create multiple flavours here and default to the 8GB one? I'm sympathetic to the low-mem users, but for a default I'd like to stick with what AWS has.

@hardys
Copy link
Collaborator

hardys commented Dec 6, 2018

Once we've got the flavor selection merged, could we create multiple flavours here and default to the 8GB one? I'm sympathetic to the low-mem users, but for a default I'd like to stick with what AWS has.

In that case should we just make the number a variable, defaulted to $whatever, then folks can set whatever they want in their CONFIG file?

@russellb
Copy link
Collaborator Author

russellb commented Dec 6, 2018 via email

fi
if ! openstack flavor show m1.medium; then
openstack flavor create --ram 10240 --disk 20 --vcpu 2 --public m1.medium
openstack flavor create --ram 4096 --disk 20 --vcpu 2 --public m1.medium
Copy link
Collaborator

Choose a reason for hiding this comment

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

whaaaaaaaaaaaaaaaaaaaaat? Don't you like wasting resources? 😄

@flaper87 flaper87 merged commit 9ee3e5d into imain:master Dec 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants